http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8218
Jonathan Druart <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Passed QA --- Comment #35 from Jonathan Druart <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] 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/
