http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8218 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Passed QA --- Comment #35 from Jonathan Druart <jonathan.druart@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.