[Bug 38499] New: Remove unused "note" argument to HTML::Scrubber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Bug ID: 38499 Summary: Remove unused "note" argument to HTML::Scrubber Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: dcook@prosentient.com.au QA Contact: testopia@bugs.koha-community.org It looks like bug 36328 accidentally added a "note" parameter to HTML::Scrubber in C4::Scrubber. Looking at the HTML::Scrubber documentation, there is no "note" parameter. It looks like a bit of confusion because "comment" is both a HTML::Scrubber parameter and a C4::Scrubber type. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 David Cook <dcook@prosentient.com.au> 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. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@openfifth.c | |o.uk See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=40079 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 --- Comment #1 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- I inlined a fix for this in bug 40079 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |martin.renvoize@openfifth.c |ity.org |o.uk -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |DUPLICATE --- Comment #2 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- *** This bug has been marked as a duplicate of bug 40079 *** -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 --- Comment #3 from David Cook <dcook@prosentient.com.au> --- (In reply to Martin Renvoize (ashimema) from comment #1)
I inlined a fix for this in bug 40079
You're the best :3 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Bug 38499 depends on bug 36328, which changed state. Bug 36328 Summary: C4::Scrubber should allow more HTML tags https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36328 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldoldoldstable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Bug 38499 depends on bug 36328, which changed state. Bug 36328 Summary: C4::Scrubber should allow more HTML tags https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36328 What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |Needs documenting Resolution|FIXED |--- -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38499 Bug 38499 depends on bug 36328, which changed state. Bug 36328 Summary: C4::Scrubber should allow more HTML tags https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36328 What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org