[Koha-bugs] [Bug 8218] Add a script to sanitize entity where "&" character was wrongly replaced with "& "

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8218

Jonathan Druart <jonathan.druart at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Passed QA

--- Comment #35 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to Tomás Cohen Arazi from comment #33)
> I'm willing to push the feature, because I like it, and it doesn't break
> people's funcionality, and hence marked as rel_3_18_candidate. This means
> you are not catch by feature freeze.
> 
> Can you please work a bit on the fact that the feature is about ampersands
> only?
> I propose
> 
> - keeping the SanitizeRecord name general.
> - renaming _entity_clean as _clean_ampersand
> - renaming the script to sanitize_records.pl
> - add a --fix-ampersand switch (the only one FOR NOW, enabled by default) so
> it is obvious what the script does.
> - make POD and usage reflect this changes.
> 
> I mark it FAILED-QA until the authors answer this proposal.

I agree, all done in the last patch, back to passed QA.

-- 
You are receiving this mail because:
You are watching all bug changes.