Closed
Bug 284547
Opened 19 years ago
Closed 19 years ago
We have unescaped >'s inside C<> tags in our pod docs
Categories
(Bugzilla :: Documentation, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: justdave, Assigned: justdave)
Details
Attachments
(1 file)
1.92 KB,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
dave@g4tower [0:17 bugzilla 10] tcsh> egrep -rn 'C<.+->.+>' * Bugzilla/User.pm:1158:obtained by C<values(%{$user->groups})>.) Bugzilla/User.pm:1172:obtained by C<values(%{$user->bless_groups})>.) Bugzilla.pm:244:Note that all C<Bugzilla> functionailty is method based; use C<Bugzilla->dbh> Bugzilla.pm:291:Essentially, causes calls to C<Bugzilla->user> to return C<undef>. This has the These show up in the rendered pod as (using the Bugzilla.pm line 291 example) : Essentially, causes calls to Bugzilla-user> to return undef. with "Bugzilla-" being monospaced, and "user>" being the normal font. This happens because the > inside the C<> tag isn't escaped. Patch to come shortly.
Assignee | ||
Comment 1•19 years ago
|
||
Assignee | ||
Updated•19 years ago
|
Assignee: documentation → justdave
Status: NEW → ASSIGNED
Attachment #176101 -
Flags: review?(mkanat)
Updated•19 years ago
|
Attachment #176101 -
Flags: review?(mkanat) → review+
Updated•19 years ago
|
Flags: approval?
Target Milestone: --- → Bugzilla 2.20
Version: 2.19.1 → 2.19.2
Assignee | ||
Comment 2•19 years ago
|
||
this patch also fixes the spelling of "functionailty" :)
Assignee | ||
Comment 3•19 years ago
|
||
Checking in Bugzilla.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla.pm,v <-- Bugzilla.pm new revision: 1.17; previous revision: 1.16 done Checking in Bugzilla/User.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v <-- User.pm new revision: 1.42; previous revision: 1.41 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Flags: approval? → approval+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•