Seems unlikely, but it does. Prior to the patch from bug 37947, there was the unnoticed bug 38168 - if you created a biblio in the framework SR and then edited it again, the Settings menu would say you were in the Audio Cassettes, CDs framework, but the fields and tags would say you were in Default (easy check in MARC21 is that 001 and 003 are hidden in frameworks other than Default, so if you see them in the editor you are in Default). With the patch from bug 37947, the first time you edit a newly-created record, the editor opens in Default and the Settings menu says it is in Default, then every time after that the editor opens in the correct framework and the Settings menu agrees. Steps to reproduce: 1. Cataloging - New record -> Audio Cassettes, CDs 2. Fill in the required fields and save 3. Switch to 'Normal', Edit - Edit record 4. Note that 001 and 003 are visible, and the Settings menu has a checkmark by Default 5. Click the browser Back button, then Edit - Edit record 6. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs
Actually, let's call that major. Since I was looking for it and looking for the once-only thing, I wasn't saving the record on the first opening of the editor, but in general it's not obvious that you've opened a record in a framework other than the one it should have, so you'll just save it, moving it from the framework it had into the Default framework, which can be a cause of dataloss depending on your frameworks.
Created attachment 173019 [details] [review] Bug 38211: frameworkcode must be set before read It's a stupid global variable, we MUST have this init line BEFORE we read it! Test plan: See the other bugs linked with this one, confirm the test plan for all of them still pass. Test plan for this specific bug report: 1. Cataloging - New record -> Audio Cassettes, CDs 2. Fill in the required fields and save 3. Switch to 'Normal', Edit - Edit record 4. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs 5. Click the browser Back button, then Edit - Edit record 6. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs
Created attachment 173048 [details] [review] Bug 38211: frameworkcode must be set before read It's a stupid global variable, we MUST have this init line BEFORE we read it! Test plan: See the other bugs linked with this one, confirm the test plan for all of them still pass. Test plan for this specific bug report: 1. Cataloging - New record -> Audio Cassettes, CDs 2. Fill in the required fields and save 3. Switch to 'Normal', Edit - Edit record 4. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs 5. Click the browser Back button, then Edit - Edit record 6. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Created attachment 173053 [details] [review] Bug 38211: frameworkcode must be set before read It's a stupid global variable, we MUST have this init line BEFORE we read it! Test plan: See the other bugs linked with this one, confirm the test plan for all of them still pass. Test plan for this specific bug report: 1. Cataloging - New record -> Audio Cassettes, CDs 2. Fill in the required fields and save 3. Switch to 'Normal', Edit - Edit record 4. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs 5. Click the browser Back button, then Edit - Edit record 6. Note that 001 and 003 are not visible, and the Settings menu has a checkmark by Audio Cassettes, CDs Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Oof...this script and its globals. Thanks, Jonathan! Passing QA (also tidied inline)
Thanks for the quick work here, Jonathan, Brendan and Emily!
Pushed for 24.11! Well done everyone, thank you!
Backported to 24.05.x for upcoming 24.05.05
Depends on Bug 37947 and 38076 not in 23.11.x