Bug 34109: Check mandatory item fields when creating during recipt
authorNick Clemens <nick@bywatersolutions.com>
Fri, 4 Aug 2023 14:13:25 +0000 (14:13 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Wed, 6 Sep 2023 12:57:02 +0000 (09:57 -0300)
commiteaab32eb0ee100752bee8822a367144157c69c51
tree8346ca5a5012839a2b20a8f976b2ec701624ffcb
parent021d083a9a2f5690cb0e92fa9c8b5fa3da122f06
Bug 34109: Check mandatory item fields when creating during recipt

This patch adds a check of mandatory item fields before saving during order receipt.
We check both when adding the item initially, and when submitting the order, as it will use
the values from the open form.

To test:
* Go to administration > frameworks > ACQ > MARC structure > 952
* Set callnumber (o) or barcode (p) to mandatory
* In acquisition:
* Create a basket with "create items on order" and "is standing"
  "standing" helps as you can do multiple receives testing different things,
  but is optional
* Add an order line
* Receive shipment
* Verify the mandatory fields show in the item form
* Set itemtype = Music
* Save with the fields blank
* Item row is added, no warning
* Edit the item, change the item type to blank
* Click 'Confirm' for the order without closing the item edit form
* Verify the item was added with itemtype Books (on sample data) and empty mandatory field
* Apply patch
* Repeat above, however, you should not be able to submit the item or order until mandatory fields are filled

Signed-off-by: Sam Lau <samalau@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt
koha-tmpl/intranet-tmpl/prog/js/additem.js