Summary: | add item not respecting framework in acq | ||
---|---|---|---|
Product: | Koha | Reporter: | Paul Poulain <paul.poulain> |
Component: | Acquisitions | Assignee: | Paul Poulain <paul.poulain> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | major | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, dpavlin |
Version: | 3.4 | ||
Hardware: | All | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6125 | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
proposed patch
Signed off patch |
Description
Paul Poulain
2011-04-07 08:54:10 UTC
Created attachment 3771 [details] [review] proposed patch The ... appear now, but they don't seem to work. I was testing 952$d date acquired. The current date was not added into the field. yikes ! This did not work because the plugin dateaccessioned is wrong, not because of this patch ! 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. I propose : * to signoff this patch (katrin ?) * to file another bug to move set_to_today to the plugin itself instead of additem Created attachment 3811 [details] [review] Signed off patch This bug does not fix the plugin for dateaccessioned, but other plugins like barcode work. I will file a separate bug for the problem with dateaccessioned like Paul suggested. Pushed, please test Tested: - mandatory (item type) - plugin (barcode with AutoBarcode activated) - hiding several subfields This works nicely. Date accessioned is still broken, see bug 6125. |