http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8307

--- Comment #19 from M. de Rooy <[email protected]> ---
QA Comment:
Tested this under AcqCreateItem=ordering. And verified that it did not change
the items with AcqCreateItem=receiving.
It works (I only struggled with booksellerid 952e that was overwritten later
on). [We use AcqCreateItem=cataloging and have a different use on 952e.
Probably, this field should not be in the marc 952 tag, as it is now.]
No complaints from qa-tools.

But I do have my doubts over this feature.
First, AcqItemSetSubfieldsWhenReceived only works for AcqCreateItem=ordering.
This should be noted more clearly in the pref description.
What about setting an item value for AcqCreateItem=receiving or cataloging btw?

Second, it looks like you want to update some status in the acquisition process
and you implemented it in a perhaps somewhat unusual way that makes completely
different applications possible. I am afraid that it could make code
maintenance more difficult in the long run.

Added a small followup for the split statement.

Setting the status to Passed QA. I am not blocking this patch, but leave the
final comment to RM.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://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/

Reply via email to