[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Nick Clemens changed: What|Removed |Added Blocks||24654 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24654 [Bug 24654] Trailing double-quote in rda 264 subfield b on opac xslt -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Nick Clemens changed: What|Removed |Added Blocks||24605 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24605 [Bug 24605] Series link from 830 is not uri encoded -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Katrin Fischer changed: What|Removed |Added CC||k...@bywatersolutions.com --- Comment #76 from Katrin Fischer --- *** Bug 16804 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Fridolin SOMERS changed: What|Removed |Added Blocks||22085 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22085 [Bug 22085] UNIMARC default XSLT broken by Bug 14716 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Fridolin SOMERS changed: What|Removed |Added Status|Pushed to Stable|RESOLVED Resolution|--- |FIXED --- Comment #75 from Fridolin SOMERS --- I prefer not to push this change in 17.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable --- Comment #74 from Martin Renvoize --- Pushed to 18.05.x fo 18.05.06 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #73 from Nick Clemens --- Awesome work all! Pushed to master for 18.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #72 from David Cook --- Cheers for working on this Ere! I was going to suggest using XML::LibXSLT::register_function() to solve this problem, which I've done locally for years, but your comment reminded me of EXSLT! That seems like an easier way to do it. A while ago, I actually ended up using EXSLT in DSpace since Apache Xalan supports it and I needed some extension features. It's a very interesting read: http://exslt.org/. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #71 from Martin Renvoize --- Thanks for the swift followup. Looks great, works great, no regressions found. Passing QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Attachment #81054|0 |1 is obsolete|| --- Comment #69 from Martin Renvoize --- Created attachment 81522 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81522&action=edit Bug 14716: URI-encode URLs in NORMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. Signed-off-by: Magnus Enger Works as advertised. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Attachment #81305|0 |1 is obsolete|| --- Comment #70 from Martin Renvoize --- Created attachment 81523 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81523&action=edit Bug 14716: Fix more unencoded url parameters. Fixes issues found during QA and unifies the SubfieldSelect function across the utils xsl's. Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Attachment #81053|0 |1 is obsolete|| --- Comment #68 from Martin Renvoize --- Created attachment 81521 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81521&action=edit Bug 14716: URI-encode URLs in UNIMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. Signed-off-by: Julian Maurice Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Attachment #81052|0 |1 is obsolete|| --- Comment #67 from Martin Renvoize --- Created attachment 81520 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81520&action=edit Bug 14716: URI-encode URLs in MARC21 xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 David Cook changed: What|Removed |Added CC||dc...@prosentient.com.au -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #66 from Ere Maijala --- I hope I managed to fix all the remaining issues. I left the removal of extra characters in the extractControlNumber template since just encoding them is not the same as getting rid of them, but I also added encoding. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #65 from Ere Maijala --- Created attachment 81305 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81305&action=edit Bug 14716: Fix more unencoded url parameters. Fixes issues found during QA and unifies the SubfieldSelect function across the utils xsl's. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #64 from Martin Renvoize --- Great work.. as commented on irc earlier.. I found a few cases where we could be escaping but we're not yet: 1) Where we're searching the `an:` index. Unfortunately, we can't guarantee the marc field we're referring to won't contain any nasty characters. 2) rcn: - 001 isn't entirely guaranteed to be clean either 3) The 'extractControlNumber' template appears to do it's own escaping.. perhaps we could convert this to use the str:encode-uri filter you're importing - This is by no means a failure case though. That's all I can find.. nice work. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Attachment #76425|0 |1 is obsolete|| --- Comment #60 from Ere Maijala --- Created attachment 81052 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81052&action=edit Bug 14716: URI-encode URLs in MARC21 xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Attachment #80701|0 |1 is obsolete|| --- Comment #62 from Ere Maijala --- Created attachment 81054 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81054&action=edit Bug 14716: URI-encode URLs in NORMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. Signed-off-by: Magnus Enger Works as advertised. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #63 from Ere Maijala --- Now rebased. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Attachment #77471|0 |1 is obsolete|| --- Comment #61 from Ere Maijala --- Created attachment 81053 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81053&action=edit Bug 14716: URI-encode URLs in UNIMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. Signed-off-by: Julian Maurice -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Patch doesn't apply --- Comment #59 from Jonathan Druart --- First patch does not longer apply Applying: Bug 14716: URI-encode URLs in MARC21 xsl templates. Using index info to reconstruct a base tree... M koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl M koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetResults.xsl M koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl CONFLICT (content): Merge conflict in koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl Auto-merging koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetResults.xsl Auto-merging koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Magnus Enger changed: What|Removed |Added Attachment #78809|0 |1 is obsolete|| --- Comment #58 from Magnus Enger --- Created attachment 80701 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=80701&action=edit Bug 14716: URI-encode URLs in NORMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. Signed-off-by: Magnus Enger Works as advertised. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Magnus Enger changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #57 from Katrin Fischer --- Magnus, can you please test the NORMARC patch? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #56 from Ere Maijala --- Created attachment 78809 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=78809&action=edit Bug 14716: URI-encode URLs in NORMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #55 from Martin Renvoize --- Failing QA given the above comment. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #54 from Magnus Enger --- (In reply to Martin Renvoize from comment #53) > QA Comment: Does this also need corresponding NORMARC changes? If this is fixing a problem in XSLT today it would be nice to fix it for NORMARC too. We will have to use NORMARC at least until the end of the year, I think, so fixing bugs is still welcome. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #53 from Martin Renvoize --- QA Comment: Does this also need corresponding NORMARC changes? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Martin Renvoize changed: What|Removed |Added CC||mag...@libriotech.no, ||martin.renvoize@ptfs-europe ||.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Julian Maurice changed: What|Removed |Added Attachment #74228|0 |1 is obsolete|| --- Comment #52 from Julian Maurice --- Created attachment 77471 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=77471&action=edit Bug 14716: URI-encode URLs in UNIMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. Signed-off-by: Julian Maurice -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Julian Maurice changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #51 from Julian Maurice --- (In reply to Katrin Fischer from comment #49) > Signing off for MARC21 - maybe someone else could take a look for UNIMARC? > Also tested that this fixes bug 16804. I prefer this solution. I confirm that the UNIMARC patch works too. I will attach the signed off patch. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Julian Maurice changed: What|Removed |Added CC||julian.maur...@biblibre.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Katrin Fischer changed: What|Removed |Added Attachment #74804|0 |1 is obsolete|| --- Comment #50 from Katrin Fischer --- Created attachment 76425 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=76425&action=edit Bug 14716: URI-encode URLs in MARC21 xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Katrin Fischer changed: What|Removed |Added Patch complexity|--- |Medium patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #49 from Katrin Fischer --- Signing off for MARC21 - maybe someone else could take a look for UNIMARC? Also tested that this fixes bug 16804. I prefer this solution. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #48 from Katrin Fischer --- Back on testing this - continueing later. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Katrin Fischer changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=16804 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Fridolin SOMERS changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #47 from Ere Maijala --- Oops, now fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Attachment #74227|0 |1 is obsolete|| --- Comment #46 from Ere Maijala --- Created attachment 74804 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74804&action=edit Bug 14716: URI-encode URLs in MARC21 xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Katrin Fischer changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #45 from Katrin Fischer --- Staff is looking good so far, but ran into an error on the OPAC side: Error while transforming input: xmlXPathCompOpEval: function encode-uri bound to undefined prefix str xmlXPathCompiledEval: 1 objects left on the stack. runtime error: file /home/vagrant/kohaclone/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slimUtils.xsl line 273 element value-of XPath evaluation returned no result. at /home/vagrant/kohaclone/Koha/XSLT_Handler.pm line 208. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #44 from Barton Chittenden --- ... and Chrome will automagically URL encode at least *some* URLs (I'm reminded as I accidentally paste a '?' from an actual question into a bugzilla url, then have to delete '%3F' ). Again, the inspector is your friend. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #43 from Katrin Fischer --- Maybe too late yesterday, thx for the hint. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #42 from Ere Maijala --- Katrin, Firefox tends to display things in clear even if they're sent properly encoded. You can use e.g. the network tab in the developer tools to see what's actually sent. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #41 from Katrin Fischer --- Hi Ere, this is really strange: I am looking at http://localhost:8081/cgi-bin/koha/catalogue/detail.pl?biblionumber=43 from the sample data. Which is a record with Hebew in 880. The place of publication and author name appear URL-encoded in Chromium, but not in Firefox. Any idea what could go on there? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #40 from Ere Maijala --- Hi Katrin! You're right, I should have split this. I've now split this into MARC21 and UNIMARC changes and omitted NORMARC. 1. Thanks for catching the external url issue, it was definitely my bad. 2. The comma is a reserved character and should be encoded. See e.g. https://en.wikipedia.org/wiki/Percent-encoding#Types_of_URI_characters for a list of reserved characters. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #39 from Ere Maijala --- Created attachment 74228 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74228&action=edit Bug 14716: URI-encode URLs in UNIMARC xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 --- Comment #38 from Ere Maijala --- Created attachment 74227 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74227&action=edit Bug 14716: URI-encode URLs in MARC21 xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Attachment #74226|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Ere Maijala changed: What|Removed |Added Attachment #73605|0 |1 is obsolete|| --- Comment #37 from Ere Maijala --- Created attachment 74226 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74226&action=edit Bug 14716: URI-encode URLs in xsl templates. https://bugs.koha-community.org/show_bug.cgi?id=14716 Test plan: 1. Catalog a record with series name "Series & more" 2. Find the record 3. Make sure clicking the "Series & more" link works and the ampersand is properly encoded in the URL. 4. Also try the attached test records and make sure their links work. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14716] Correctly URI-encode URLs in XSLT result lists and detail pages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14716 Katrin Fischer changed: What|Removed |Added Summary|opac-detail.pl -- Series|Correctly URI-encode URLs |link fails when series |in XSLT result lists and |title ends with semicolon |detail pages |preceded by space | Status|Needs Signoff |Failed QA --- Comment #36 from Katrin Fischer --- Hi Ere, you tend to create quite big patch sets - often it's better to split things up a bit to ease testing. That will make it easier for sign-offers. For example you could have one patch for MARC21 and one for UNIMARC here. NORMARC might not be necessary, as of bug 18984. >From testing (MARC21): 1) URLs to external sources are broken at least in intranet result an detail page: Example: http://localhost:8081/cgi-bin/koha/catalogue/http%3A%2F%2Fwww.loc.gov%2Fcatdir%2Fdescription%2Frandom048%2F90032101.html 2) Shoudl the comma behind publisher not be encoded? http://localhost:8081/cgi-bin/koha/catalogue/search.pl?q=Provider:%22Vintage%20Books%2C%22 Record is from the sample data: Opus posthumous / by Wallace Stevens. Can you please check? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/