Michael Kuhn reported on the mailing list: As usual when creating a new framework I did export the standard framework in XML format, created a new framework ABC and the imported the XML file into the new framework ABC. I tried to do this in virgin installations of Koha 18.11.04 and Koha 19.05.00 but instead of importing all 345 MARC fields, only 102 were actually imported. I've verified this on master with the same result: - Pick any framwork, I used Kit - Export it using the XML format - Create a new framework, name it as you want - Import using the downloaded XML file - Compare both frameworks - Verify that the number of fields is different
Is this a regression or it never worked? Note: I exported the default framework, the import process is broken: 020 is the first non imported field and it is in the generated .xml file. CSV is ok
I am not sure, we've always used the OpenDocument format, so I have never run into this so far.
Why does this not show up on the dashboard with the other criticals?
(In reply to Katrin Fischer from comment #3) > Why does this not show up on the dashboard with the other criticals? We display the bug reports created in the last 60 days if the status is NEW, ASSIGNED, In disussion, FQA or does not apply.
(In reply to Jonathan Druart from comment #4) > (In reply to Katrin Fischer from comment #3) > > Why does this not show up on the dashboard with the other criticals? > > We display the bug reports created in the last 60 days if the status is NEW, > ASSIGNED, In disussion, FQA or does not apply. Didn't know about the last 60 days... guess the situation si worse than I thought it is!
Created attachment 94687 [details] [review] Bug 23398: Disable the XML import/export format for biblio framework It gives incomplete results and not sure it ever worked. To prevent data loss it is better to remove this option. Note that the code in C4/ImportExportFramework.pm is too complex to attach with this patch to deletion of the code related to the XML import/export. Safer to keep it for a separate bug report. Test plan: Import/export biblio frameworks and confirm that there is no more reference to the xml format.
Created attachment 95161 [details] [review] Bug 23398: Disable the XML import/export format for biblio framework It gives incomplete results and not sure it ever worked. To prevent data loss it is better to remove this option. Note that the code in C4/ImportExportFramework.pm is too complex to attach with this patch to deletion of the code related to the XML import/export. Safer to keep it for a separate bug report. Test plan: Import/export biblio frameworks and confirm that there is no more reference to the xml format. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Code scans well and does what is expected. I agree to the pragmatic decision of just removing the option at this point too. Signing off.
Created attachment 95179 [details] [review] Bug 23398: Disable the XML import/export format for biblio framework It gives incomplete results and not sure it ever worked. To prevent data loss it is better to remove this option. Note that the code in C4/ImportExportFramework.pm is too complex to attach with this patch to deletion of the code related to the XML import/export. Safer to keep it for a separate bug report. Test plan: Import/export biblio frameworks and confirm that there is no more reference to the xml format. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.05