| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.32.0 |
| Request #10867 | Changes to "file a bug report" link in the manual | ||
|---|---|---|---|
| Submitted: | 2007-04-28 08:43 UTC | ||
| From: | wiesemann | Assigned: | wiesemann |
| Status: | Closed | Package: | pearweb (version 1.14.0) |
| PHP Version: | Irrelevant | OS: | |
| Roadmaps: | (Not assigned) | ||
[2007-04-28 08:43 UTC] wiesemann
(Mark Wiesemann)
Description:
------------
The "file a bug report" at the bottom of every page in the manual points to .../bugs/report.php?package=Documentation.
But we're having now the but type "Documentation Problem". Therefore, some magic to detect the package name would be good, so that the bug could be added to the right package with this type.
For example, "package.html.html-quickform.html-quickform-element.freeze.php" indicates that this manual page talks about a package (first part) and that it talks about HTML_QuickForm (third part, replace '-' by '_'), while "developers.recommendations.php" doesn't start with "developers" and could still link to report.php?package=Documentation (<= would be reported to pear-doc list, which would be the right place).
Comments
[2007-04-28 17:51 UTC] cellog
(Greg Beaver)
this may be impossible, as the package name is
case-sensitive in the bug tracker, and it is lower case in
the documentation ID.
[2007-04-28 19:21 UTC] wiesemann
(Mark Wiesemann)
> this may be impossible, as the package name is
> case-sensitive in the bug tracker, and it is
> lower case in the documentation ID.
But I guess that we won't have two packages like HTML_QuickForm and HTML_Quickform (note the "F"), or pearweb and Pearweb (note the "P").
A simple and short query against the package table with something like "SELECT name FROM packages WHERE name = LOWER($idFromDocURL)" (table/column name and MySQL function name might be wrong, didn't look them up) should give us the right package name in the right case.
[2007-05-01 20:11 UTC] wiesemann
(Mark Wiesemann)
To avoid duplicate work: A patch is almost ready.
(And the trick with the comparison with the names in the package table is already in the code, but isn't used due to a typo. [c.p. my comment on Request #5350, my patch will also fix the typo])
[2007-05-04 04:18 UTC] davidc
(David Coallier)
This bug has been fixed in CVS.
If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.