[Koha-bugs] [Bug 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 --- Comment #8 from David Cook --- I also like how this patch removes the useless "note" parameter from the constructor/new method (bug 38499). I've wanted that done for a while. I'm going to update the title of this bug to specify the note profile. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 --- Comment #7 from David Cook --- Sounds like a reasonable change to me, but it does seem like we need some clarification. David: I don't the original description mentioned description lists per se? But I do see the patch says it adds "unordered list, ordered list and list item html tags", but it only adds "ul" and "li" - not "ol". Martin: Did you intend to add "ol" as well? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 David Nind changed: What|Removed |Added CC||[email protected] Text to go in the||This adds unordered lists release notes||to the HTML tags that are ||allowed in notes fields ||(for example, course ||reserves staff and public ||notes). --- Comment #6 from David Nind --- I've signed off, as things work as per the test plan. However, I noticed that the description for the bug says it also allows ordered lists () and description list (, , and ). Only unordered lists work for me - looking at the patch, only ul and li were added. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 David Nind changed: What|Removed |Added Attachment #182985|0 |1 is obsolete|| --- Comment #5 from David Nind --- Created attachment 183014 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183014&action=edit Bug 40079: Add ul/ol and li to allowed tags in notes profile This patch add the unordered list, ordered list and list item html tags to the list of allowed tags in C4::Scrubber. Test plan: 1) Add a course reserve and include the following in the public note field: First item Second item 2) View the course details page and note "First item Second item" all appears on the same line (the ul and li tags have been stripped) 3) View the course reserves page and note the same issue in the public notes field of the table display 4) Apply this patch 5) Repeat 2 and 3 and note you now have a bullet pointed list displayed in both places 6) The display of the note on the details page may look mis-aligned.. that will be dealt with on it's own bug Signed-off-by: David Nind -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 David Nind 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 [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 --- Comment #4 from Phil Ringnalda --- *** Bug 40078 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 [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 --- Comment #3 from Phil Ringnalda --- *** Bug 40077 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 [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Martin Renvoize (ashimema) changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=38499 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Martin Renvoize (ashimema) changed: What|Removed |Added CC||[email protected] --- Comment #2 from Martin Renvoize (ashimema) --- *** Bug 38499 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 [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Martin Renvoize (ashimema) changed: What|Removed |Added Depends on||36328 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36328 [Bug 36328] C4::Scrubber should allow more HTML tags -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Martin Renvoize (ashimema) changed: What|Removed |Added Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Martin Renvoize (ashimema) changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 --- Comment #1 from Martin Renvoize (ashimema) --- Created attachment 182985 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=182985&action=edit Bug 40079: Add ul/ol and li to allowed tags in notes profile This patch add the unordered list, ordered list and list item html tags to the list of allowed tags in C4::Scrubber. Test plan: 1) Add a course reserve and include the following in the public note field: First item Second item 2) View the course details page and note "First item Second item" all appears on the same line (the ul and li tags have been stripped) 3) View the course reserves page and note the same issue in the public notes field of the table display 4) Apply this patch 5) Repeat 2 and 3 and note you now have a bullet pointed list displayed in both places 6) The display of the note on the details page may look mis-aligned.. that will be dealt with on it's own bug -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Mirjam Vantieghem changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=23978 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Mirjam Vantieghem changed: What|Removed |Added CC||[email protected] See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=22501 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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 40079] C4::Scrubber should allow for list (ul/ol and li) HTML tags
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40079 Martin Renvoize (ashimema) changed: What|Removed |Added Assignee|[email protected] |[email protected] |ity.org |o.uk -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list [email protected] 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/
