In the code for addorderiso2709.pl we only use the fields form MFTO if MIFTO is not populated. This is not clear when setting things up
Created attachment 154913 [details] [review] Bug 34644: Add clarifying text to sysprefs to indicate that MarcFieldsToOrder is a fallback to MarcItemFieldsToOrder Clarify that fields will fall back to MarcFieldsToOrder from MarcItemFieldsToOrder.
Created attachment 154914 [details] [review] Bug 34644: Improve readability of sysprefs
Created attachment 154916 [details] [review] Bug 34644: replacementprice is support by MarcFieldsToOrder but is not listed
Created attachment 154929 [details] [review] Bug 34644: Add clarifying text to sysprefs to indicate that MarcFieldsToOrder is a fallback to MarcItemFieldsToOrder Clarify that fields will fall back to MarcFieldsToOrder from MarcItemFieldsToOrder. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 154930 [details] [review] Bug 34644: Improve readability of sysprefs Signed-off-by: David Nind <david@davidnind.com>
Created attachment 154931 [details] [review] Bug 34644: replacementprice is support by MarcFieldsToOrder but is not listed Signed-off-by: David Nind <david@davidnind.com>
Created attachment 154932 [details] [review] Bug 34644: (follow-up) Additional edits to description Linked system preference in description to cross reference, and and removed the spaces in the bracketed text. Signed-off-by: David Nind <david@davidnind.com>
Testing notes: 1. Compared before and after text for the system preference. 2. I added a follow-up - linked system preference in the description for MarcItemFieldsToOrder (the same as we do for other preferences), and removed the spaces in the bracketed text. A nice clarification!
Moving to blocked since depends on a NSO.
Dependency is unlocked.
Blocked again :(