Closed Bug 679096 Opened 14 years ago Closed 14 years ago

Replace FILTER url_quote by FILTER uri

Categories

(Testopia :: Integrations, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

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.
Same here on Bugzilla 4.2 release
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.
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.
Peter: would you agree to attach a patch with these changes? :)
Attachment #612828 - Flags: review?
Attachment #612829 - Flags: review?
Attachment #612830 - Flags: review?
Attachment #612833 - Flags: review?
Attachment #612831 - 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?
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)
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 on attachment 613195 [details] [diff] [review] url_quote to uri singular patch Thanks for the patch. r=LpSolit
Attachment #613195 - Flags: review?(LpSolit) → review+
Attached file fake (obsolete) —
(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?
Attachment #613199 - Attachment is obsolete: true
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.

Attachment

General

Creator:
Created:
Updated:
Size: