It does not work because the plugin dateaccessioned is wrong. dateaccessioned relies on a javascript sub (set_to_today) that is defined in additem.pl. For acquisition (and serials), the set_to_today is not defined.
The plugin is still broken in current master/version 3.6
This is still a bug in current master. The plugin for accession date does not work, neither in ACQ nor in the Default framework.
Created attachment 8347 [details] [review] proposed patch #1 Repairs dateaccessioned plugin 1/ It now works with neworderempty.pl 2/ The set_to_today function was moved from additem.tt to dateaccessioned.pl (so it's available to all pages using the plugin)
Hi, I applied the patch and it works well. Tested in placing order, in receiving order and in cataloging (attach/edit items), everything is OK. Tested on sandbox. Julien
Moves set_to_today from template to script, and that seems fine. There are now some hardcoded English strings for the error messaging, though... obviously not ideal, but is it a big enough to require a followup? As an English speaker, I don't feel like I can answer this. Will someone in a non-English context please advise me on this?
I am not sure how a user can make this alerts show - if it's more for debugging than for the end user I think having them in the .pl may be fine for now and it could be improved with a follow up later on.
Alright, that sounds fair. Marking Passed QA.
Oh, indeed, that's a JS code but included in a pl file, and thus not translated... As Katrin said, it's not aimed at end user and this message was probably never displayed. Maybe it could have been removed in fact (replaced with something like "If ID exists then do something Else do nothing"). But well, could be useful one day =)
This bug will be included in the Koha 3.6.5 release.