Closed
Bug 679096
Opened 14 years ago
Closed 14 years ago
Replace FILTER url_quote by FILTER uri
Categories
(Testopia :: Integrations, defect)
Testopia
Integrations
Tracking
(Not tracked)
RESOLVED
FIXED
2.5
People
(Reporter: justin, Assigned: florijn.peter)
Details
Attachments
(1 file, 13 obsolete files)
|
39.40 KB,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux i686) AppleWebKit/535.1 (KHTML, like Gecko) Chrome/13.0.782.112 Safari/535.1
Steps to reproduce:
I have a test environment with Bugzilla 4.1.3 in place. I copied over the Testopia files from the latest revision(rev 648) and manually applied the patch for 4.0 as some of the Bugzilla files that it patches have changed. I then ran checksetup.pl.
Actual results:
When accessing the Bugzilla/Testopia the following page is presented.
"
Bugzilla has suffered an internal error. Please save this page and send it to root@bugzilla with details of what you were doing at the time this message appeared.
URL: http://###.###.###.###/
Template->process() failed twice.
First error: undef error - url_quote: filter not found
Second error: undef error - url_quote: filter not found
"
Expected results:
The Bugzilla main page should have been presented, containing the Testopia links on it.
I have encountered the same issue today with same error message.
Testopia was working with older version, but when i tried to install testopia
for Bugzilla 4.1.3 i'm encountering the same issue as above.
| Assignee | ||
Comment 2•14 years ago
|
||
Same here on Bugzilla 4.2 release
| Assignee | ||
Comment 3•14 years ago
|
||
Template->process() failed twice.
First error: undef error - url_quote: filter not found
Second error: undef error - url_quote: filter not found
Seems url_quote is deprecated
Found this in Bugzilla message 398701 with fix
find . *-type f -name "*.tmpl" -exec replace 'FILTER url_quote' 'FILTER uri' -- {} \;*
Executed this from the Testopia extensions directory and this updated a number of template files:
./template/en/default/hook/bug/create/create-end.html.tmpl converted
./template/en/default/hook/bug/edit-after_custom_fields.html.tmpl converted
./template/en/default/hook/bug/process/results-links.html.tmpl converted
./template/en/default/hook/global/common-links-link-row.html.tmpl converted
./template/en/default/hook/global/useful-links-end.html.tmpl converted
./template/en/default/testopia/reports/build-coverage.html.tmpl converted
./template/en/default/testopia/reports/completion.html.tmpl converted
./template/en/default/testopia/reports/execution.html.tmpl converted
./template/en/default/testopia/reports/priority-breakdown.html.tmpl converted
./template/en/default/testopia/reports/report-table.html.tmpl converted
./template/en/default/testopia/reports/report.html.tmpl converted
./template/en/default/testopia/reports/status.html.tmpl converted
./template/en/default/testopia/tag/list.html.tmpl converted
I now get the index page.
Comment 4•14 years ago
|
||
I can confirm this solution at least gives me a working index page on on Windows, Bugzilla 4.2. I changed the files mentioned manually using notepad in windows (note, your changes case sensitive). Thanks for sharing.
Comment 5•14 years ago
|
||
Peter: would you agree to attach a patch with these changes? :)
| Assignee | ||
Comment 6•14 years ago
|
||
Attachment #612828 -
Flags: review?
| Assignee | ||
Comment 7•14 years ago
|
||
Attachment #612829 -
Flags: review?
| Assignee | ||
Comment 8•14 years ago
|
||
Attachment #612830 -
Flags: review?
| Assignee | ||
Comment 9•14 years ago
|
||
| Assignee | ||
Comment 10•14 years ago
|
||
Attachment #612833 -
Flags: review?
| Assignee | ||
Updated•14 years ago
|
Attachment #612831 -
Flags: review?
| Assignee | ||
Comment 11•14 years ago
|
||
Attachment #612834 -
Flags: review?
| Assignee | ||
Comment 12•14 years ago
|
||
Attachment #612835 -
Flags: review?
| Assignee | ||
Comment 13•14 years ago
|
||
Attachment #612836 -
Flags: review?
| Assignee | ||
Comment 14•14 years ago
|
||
Attachment #612838 -
Flags: review?
| Assignee | ||
Comment 15•14 years ago
|
||
Attachment #612839 -
Flags: review?
| Assignee | ||
Comment 16•14 years ago
|
||
Attachment #612840 -
Flags: review?
| Assignee | ||
Comment 17•14 years ago
|
||
Attachment #612841 -
Flags: review?
Comment 18•14 years ago
|
||
This is Peters patches bound together for easy committing.
LpSolit, would you be able review this so we can make some progress.
Thanks
Attachment #613195 -
Flags: review?(LpSolit)
Comment 19•14 years ago
|
||
Peter, next time, attach a single patch with all related changes in it, as rojanu did. Much easier to review and to commit. :)
Assignee: gregaryh → florijn.peter
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 20•14 years ago
|
||
Comment on attachment 613195 [details] [diff] [review]
url_quote to uri singular patch
Thanks for the patch. r=LpSolit
Attachment #613195 -
Flags: review?(LpSolit) → review+
Comment 21•14 years ago
|
||
(fake attachment to obsolete all patches at once; ignore this)
Attachment #612828 -
Attachment is obsolete: true
Attachment #612829 -
Attachment is obsolete: true
Attachment #612830 -
Attachment is obsolete: true
Attachment #612831 -
Attachment is obsolete: true
Attachment #612833 -
Attachment is obsolete: true
Attachment #612834 -
Attachment is obsolete: true
Attachment #612835 -
Attachment is obsolete: true
Attachment #612836 -
Attachment is obsolete: true
Attachment #612838 -
Attachment is obsolete: true
Attachment #612839 -
Attachment is obsolete: true
Attachment #612840 -
Attachment is obsolete: true
Attachment #612841 -
Attachment is obsolete: true
Attachment #612828 -
Flags: review?
Attachment #612829 -
Flags: review?
Attachment #612830 -
Flags: review?
Attachment #612831 -
Flags: review?
Attachment #612833 -
Flags: review?
Attachment #612834 -
Flags: review?
Attachment #612835 -
Flags: review?
Attachment #612836 -
Flags: review?
Attachment #612838 -
Flags: review?
Attachment #612839 -
Flags: review?
Attachment #612840 -
Flags: review?
Attachment #612841 -
Flags: review?
Updated•14 years ago
|
Attachment #613199 -
Attachment is obsolete: true
Comment 22•14 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/extensions/testopia/trunk/
modified extensions/Testopia/template/en/default/hook/bug/edit-after_custom_fields.html.tmpl
modified extensions/Testopia/template/en/default/hook/bug/create/create-end.html.tmpl
modified extensions/Testopia/template/en/default/hook/bug/process/results-links.html.tmpl
modified extensions/Testopia/template/en/default/hook/global/common-links-link-row.html.tmpl
modified extensions/Testopia/template/en/default/hook/global/useful-links-end.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/build-coverage.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/completion.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/execution.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/priority-breakdown.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/report-table.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/report.html.tmpl
modified extensions/Testopia/template/en/default/testopia/reports/status.html.tmpl
modified extensions/Testopia/template/en/default/testopia/tag/list.html.tmpl
Committed revision 649.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Summary: Bugzilla 4.1 series integration → Replace FILTER url_quote by FILTER uri
Target Milestone: --- → 2.5
You need to log in
before you can comment on or make changes to this bug.
Description
•