When importing batches of bib records if you choose a framework and no matter what you choose it imports as default. Your choice is ignored.
Created attachment 38319 [details] [review] Bug 11064: Hide the framework choice when importing biblio This choice is ignored, it's preferable to hide it. Test plan: 0/ Before to apply the patch, confirm the issue 1/ Stage some records for import and on the detail of the batch (/tools/manage-marc-import.pl?import_batch_id=XXX), confirm the framework dropdown list does not appear anymore.
This patch does not fix the problem but hide the framework choice. It will prevent to pretend it works!
Created attachment 39052 [details] [review] [SIGNED-OFF] Bug 11064: Hide the framework choice when importing biblio This choice is ignored, it's preferable to hide it. Test plan: 0/ Before to apply the patch, confirm the issue 1/ Stage some records for import and on the detail of the batch (/tools/manage-marc-import.pl?import_batch_id=XXX), confirm the framework dropdown list does not appear anymore. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Woks as described, no more dropdown. As stated, is not a fix, only hide the option. No koha-qa errors
Is this related to http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10906?
(In reply to David Roberts from comment #4) > Is this related to > http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10906? In a way I guess (hard to say for sure, because original bug report is a bit vaque, and over 1.5 years old). It is somehow complicated.. There are 2 possible cases when importing biblio records: 1) some of the biblios in the given import batch do match some existing local record[s], and our choice is to replace such local records contents with imported ones if they match 2) some of the biblios in the import batch do not match any existing record[s] in the local catalogue, or we wish to import them 'as new' even if there are some possible matches found for them Before Bug 10906 fix, for cases #1, framework choice in import batch commit form was being ignored no matter what (but note that this particular setting is worded as 'Add _new_ bibliographic records into this framework', so it should not apply for records whitch are not imported 'as new' - ?) and also the previousy set framework in the existing recods being replaced by imported ones was getting lost in the process After Bug 10906, for cases #1, previously set framework in the existing records is not getting lost any longer, but framework choosen in the import batch commit form (if any) is still being ignored for the records which are not imported "as new". But I guess it probably should behave like this = this is an expected behaviour for such cases? Now, for cases #2 (biblios imported "as new" for whatever reason - there are no matches, or we don't wish to replace any existing records even if there is a match), framework choosen in the import batch commit form _should_ apply. But it seems to me that it _does_ apply and is not being ignored (at least not for records imported "as new") - ? Note: I tested it only in 3.18 and in ~2 months old master (and it appears to work fine / as expected), but not in the current clean master - perhaps it got somehow broken in more recent versions.
(In reply to Jonathan Druart from comment #1) > Created attachment 38319 [details] [review] [review] > Bug 11064: Hide the framework choice when importing biblio > > This choice is ignored, it's preferable to hide it. > > Test plan: > 0/ Before to apply the patch, confirm the issue I'm unable to replicate such issue - framework choice seems to work for me as expected, and it doesn't appear to be ignored (at least not always / not in the cases when it shouldn't be ignored). Can you provide some more details regarding the circumstances when this issue still occurs?
(In reply to Jacek Ablewicz from comment #6) > (In reply to Jonathan Druart from comment #1) > > Created attachment 38319 [details] [review] [review] [review] > > Bug 11064: Hide the framework choice when importing biblio > > > > This choice is ignored, it's preferable to hide it. > > > > Test plan: > > 0/ Before to apply the patch, confirm the issue > > I'm unable to replicate such issue - framework choice seems to work for me > as expected, and it doesn't appear to be ignored (at least not always / not > in the cases when it shouldn't be ignored). Can you provide some more > details regarding the circumstances when this issue still occurs? Hum, weird. I don't remember what I tested, but I am pretty sure something was wrong. I have just retested: FW1 : 300$a in tab 1 FW2 : ignore 300$a Create a record with 300$a = "test", export it, import it using the FW2. On editing the imported record, it used FW2 and 300$a is not displayed. If I select FW1, 300$a does not exist. This seems to be the expected behavior. Bernardo, how did you confirm the issue?
Moving this to "In dicussion" Can someone please test this again and document the results please? Jonathan, I don't understand your last comment - can you try and explain?
All seems to work as expected and I suspect that the original report was a duplicate of bug 10906. Please reopen and detail if I am wrong. Sorry for the confusion. *** This bug has been marked as a duplicate of bug 10906 ***