https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30822
--- Comment #6 from Martin Renvoize <[email protected]> --- (In reply to Joonas Kylmälä from comment #5) > Good improvement! Just a small nit: as BatchCommitItems is a public function > it should take either skip_record_index as a parameter or document clearly > that it doesn't do indexing and it is left to the caller. Good Catch.. I'm wondering if the third option is to rename the function to be 'private'.. it appears to only be called inside this module and isn't exported either. Certainly some POD around it would also be sensible though. -- 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/
