Closed
Bug 284547
Opened 20 years ago
Closed 20 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•20 years ago
|
||
| Assignee | ||
Updated•20 years ago
|
Assignee: documentation → justdave
Status: NEW → ASSIGNED
Attachment #176101 -
Flags: review?(mkanat)
Updated•20 years ago
|
Attachment #176101 -
Flags: review?(mkanat) → review+
Updated•20 years ago
|
Flags: approval?
Target Milestone: --- → Bugzilla 2.20
Version: 2.19.1 → 2.19.2
| Assignee | ||
Comment 2•20 years ago
|
||
this patch also fixes the spelling of "functionailty" :)
| Assignee | ||
Comment 3•20 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: 20 years ago
Flags: approval? → approval+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•