@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Fix Phriction link syntax a little more
Summary: This still wasn't quite right -- a link like `[[ Porcupine Facts ]]` with a space would not lookup correctly, and would render as `porcupine_facts`.
Test Plan: Verified that `[[ Porcupine Facts ]]` now works correctly.
Reviewers: chad
Reviewed By: chad
Differential Revision: https://secure.phabricator.com/D15780
+5
-2
+5
-2
src/applications/phriction/markup/PhrictionRemarkupRule.php
+5
-2
src/applications/phriction/markup/PhrictionRemarkupRule.php
···
85
85
}
86
86
87
87
$slugs = ipull($metadata, 'link');
88
+
foreach ($slugs as $key => $slug) {
89
+
$slugs[$key] = PhabricatorSlug::normalize($slug);
90
+
}
88
91
89
92
// We have to make two queries here to distinguish between
90
93
// documents the user can't see, and documents that don't
···
115
118
if (idx($existant_documents, $slug) === null) {
116
119
// The target document doesn't exist.
117
120
if ($name === null) {
118
-
$name = explode('/', trim($slug, '/'));
121
+
$name = explode('/', trim($link, '/'));
119
122
$name = end($name);
120
123
}
121
124
$class = 'phriction-link-missing';
122
125
} else if (idx($visible_documents, $slug) === null) {
123
126
// The document exists, but the user can't see it.
124
127
if ($name === null) {
125
-
$name = explode('/', trim($slug, '/'));
128
+
$name = explode('/', trim($link, '/'));
126
129
$name = end($name);
127
130
}
128
131
$class = 'phriction-link-lock';