To recreate: 1 - Export your default framework 2 - Import it into the ACQ framework 3 - Attempt to add to a basket from an existing record 4 - Internal server error: Control fields (generally, just tags below 010) do not have subfields, use data() at /home/vagrant/kohaclone/acqui/neworderempty.pl line 295.
Why would you want to do that? Before bug 19289 the ACQ framework was only used for items, now it is used to edit *few* fields of a bibliographic records when creating a new order. If you create ACQ from the default framework you will end up with a huge list of inputs/selects.
Yes, you don't want all, just easy way to recreate You might want 001 or 003 though
I agree 001/003 would be good! Also possible 008 if you want to have them show up correctly in publication year search (might make sense especially looking for new things).
Created attachment 86114 [details] [review] Bug 21316: Handle control fields in the ACQ frameworks Test plan: Turn UseACQFrameworkForBiblioRecords on Add 003@ to the ACQ framework Create an order from an existing record => The 003 value will be displayed! It fixes the following error: Control fields (generally, just tags below 010) do not have subfields, use data() at /home/vagrant/kohaclone/acqui/neworderempty.pl line 293.
Created attachment 91484 [details] [review] Bug 21316: Handle control fields in the ACQ frameworks Test plan: Turn UseACQFrameworkForBiblioRecords on Add 003@ to the ACQ framework Create an order from an existing record => The 003 value will be displayed! It fixes the following error: Control fields (generally, just tags below 010) do not have subfields, use data() at /home/vagrant/kohaclone/acqui/neworderempty.pl line 293. Signed-off-by: CS <christian.stelzenmueller@bsz-bw.de>
Confirmed in master, but also in 18.11.06
Created attachment 91516 [details] [review] Bug 21316: Handle control fields in the ACQ frameworks Test plan: - Turn UseACQFrameworkForBiblioRecords on - Add 003@ to the ACQ framework - Create an order from an existing record => The 003 value will be displayed! It fixes the following error: Control fields (generally, just tags below 010) do not have subfields, use data() at /home/vagrant/kohaclone/acqui/neworderempty.pl line 293. Signed-off-by: Christian Stelzenmüller <christian.stelzenmueller@bsz-bw.de> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Fixed the sign-off line, but as I helped with testing, it would be nice to get another set of QA eyes on this.
Created attachment 91613 [details] [review] Bug 21316: Handle control fields in the ACQ frameworks Test plan: - Turn UseACQFrameworkForBiblioRecords on - Add 003@ to the ACQ framework - Create an order from an existing record => The 003 value will be displayed! It fixes the following error: Control fields (generally, just tags below 010) do not have subfields, use data() at /home/vagrant/kohaclone/acqui/neworderempty.pl line 293. Signed-off-by: Christian Stelzenmüller <christian.stelzenmueller@bsz-bw.de> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.03
backported to 18.11.x for 18.11.09