Adding a field "edition statement" to neworderempty.pl (BibLibre MT5837)
Created attachment 7212 [details] [review] Bug 7289 Adds editionstatement field in neworderempty
Hi Jonathan, I found some problems: 1) The new mappings are only added to en and fr-FR frameworks. Could you add it to the other languages too, please? 2) I think I would use the term "Edition:" on the order form. 3) There is no database update. When I tested without adding the mapping, the edition was not saved into the record. I think we need a careful database update here, checking if people already have a mapping on the MARC field or on bilbioitems.editionstatement. 4) Edition does not show on the basket summary page. I think it would be good to have the information there too. Failing QA for 3) for now - I think we need to find a way to make it work after an update or we will end up with people reporting bugs about the new field.
Created attachment 7563 [details] [review] Bug 7289: Adds editionstatement field in neworderempty Hi Katrin, Thanks for your note. This patch adds: - the biblioitems.editionstatement mapping in all language files. - replacement by "Edition" in neworderempty - an updatedatabase version. This one checks if a mapping already exists. If yes, it does nothing, else it checks the marcflavour syspref and maps this kohafield with 205$a (unimarc) or 250$a (for others). - the edition statement value in the order list of basket.pl
Created attachment 7582 [details] [review] [SIGNED-OFF] Bug 7289: Adds editionstatement field in neworderempty Tested: Database update - existing mapping - no pre existing mapping Frameworks PROBLEM: some languages are still missing, I think only untranslated frameworks are included in this patch. Acquisition - created a new order with edition filled out - checked edition was saved to the correct field in cataloguing - modified edition and checked display in acquisition - edition also displays on basket summary page http://bugs.koha-community.org/show_bug.cgi?id=7280
Created attachment 7583 [details] [review] Bug 7289: Add biblioitems.editionstatement to missing sql files Follow-up for 7289 adding the new mapping to translated frameworks. http://bugs.koha-community.org/show_bug.cgi?id=7280
Didn't test for UNIMARC and follow-up needs sign-off. Leaving status.
Created attachment 7588 [details] [review] [SIGNED-OFF] Bug 7289: Adds editionstatement field in neworderempty Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Tested: Database update - existing mapping - no pre existing mapping Frameworks PROBLEM: some languages are still missing, I think only untranslated frameworks are included in this patch. Acquisition - created a new order with edition filled out - checked edition was saved to the correct field in cataloguing - modified edition and checked display in acquisition - edition also displays on basket summary page http://bugs.koha-community.org/show_bug.cgi?id=7280
Created attachment 7633 [details] [review] Signed-off Bug 7289 Add biblioitems.editionstatement to missing
This patch checks to see if the database field is already mapped anywhere in any framework, then applies the default values only if no mappings are found to exist. This is a slightly more cautious method than necessary (mapping ANY framework to this field prevents ALL frameworks from receiving the update), but I vastly prefer this to anything more cavalier. Marking Passed QA.
Patch pushed, 2 comments: * set DBrev to 015 * fixed a tiny git conflict in a template (edition statement display was conflicting with bug 5346 displaying a link to suggestion from order