September 2021 - Koha-bugs - lists.koha-community.org

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |ASSIGNED --- Comment #144 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- I've rebased the patches so they should apply now, however importing authority frameworks seems to be broken. I'll fix this tomorrow. Aleisha -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115388|0 |1 is obsolete| | --- Comment #143 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125390 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125390&action=e… Bug 13952: (follow-up) fixing indentation and rebasing Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115387|0 |1 is obsolete| | --- Comment #142 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125389 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125389&action=e… Bug 13952: (follow-up) Removing XML options Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115386|0 |1 is obsolete| | --- Comment #141 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125388&action=e… Bug 13952: (follow-up) Tests Run: prove -v t/db_dependent/ImportExportFramework.t and confirm all tests pass. I've also added a fix to remove the warns when exporting in ODS format. Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115385|0 |1 is obsolete| | --- Comment #140 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125387 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125387&action=e… Bug 13952: (follow-up) template filters, moving js to separate file This patch adds filters to the template file and moves the JS into a separate file. Signed-off-by: Owen Leonard <oleonard(a)myacpl.org> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115384|0 |1 is obsolete| | --- Comment #139 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125386 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125386&action=e… Bug 13952: (follow-up) Fixing export on default biblio framework Signed-off-by: Katrin Fischer <katrin.fischer.83(a)web.de> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115383|0 |1 is obsolete| | --- Comment #138 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125385 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125385&action=e… Bug 13952: (follow-up) Fixing import for default auth type Signed-off-by: Katrin Fischer <katrin.fischer.83(a)web.de> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115382|0 |1 is obsolete| | --- Comment #137 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125384 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125384&action=e… Bug 13952: (follow-up) Fixing authority type import This patch fixes the comments from Comment 29 and the import functionality. You should now be able to import an exported file without editing the file at all and the authority type code will be overwritten in the file (same behaviour as biblio frameworks). Sponsored-by: Catalyst IT Signed-off-by: Katrin Fischer <katrin.fischer.83(a)web.de> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13952] Import and export of authority types
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13952 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115381|0 |1 is obsolete| | --- Comment #136 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 125383 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125383&action=e… Bug 13952: Import and export authority types This patch amends C4::ImportExportFramework to work for authority types as well as MARC frameworks. New file: admin/import_export_authtype.pl Update: Ensuring we are passing the right column to the right tables. Update2: Making the error messages the same to be consistent with patch on Bug 15665 Update3: Fixing merge conflicts Update4: Fixing merge conflicts and removing tabs Update5: Getting rid of warns, making sure Import and Export of default authority will work Update6: Merge conflicts and making sure export of default auth type works Update7: Fixing merge conflicts and updating buttons to bootstrap3 To test: 1) Go to Admin -> Authority types 2) Confirm there are two new columns 'Export' and 'Import' in the table 3) Click 'Export' on an existing authority type and choose a file type, click 'Export' 4) Confirm that the authority type is exported as your chosen file type. Save the file 5) Create a new authority type 6) Import into your new authority type using the file you just exported 7) Confirm you are taken to auth_tag_structure.pl 8) Go back to Authority types 9) Export your new authority type. View the exported file and confirm the authtypecode has been updated to match the code you set for the new auth type 10) Click 'Import' next to any existing authority type and attempt to import a file that is not XML, CSV or ODS. Confirm that this fails and you are asked to import a file of the correct file type 11) Confirm Export and Import work for the Default authority type 12) Go to Admin -> MARC bibliographic framework 13) Confirm that the 'Export' and 'Import' functions still work here as well Sponsored-by: Catalyst IT Signed-off-by: Katrin Fischer <katrin.fischer.83(a)web.de> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Michal Denar <black23(a)gmail.com> Signed-off-by: Mazen Khallaf <mazen.i.khallaf(a)gamil.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14242] Use ISBN-field to fill out purchase suggestions (using an Ajax-request)
by bugzilla-daemon＠bugs.koha-community.org 28 Sep '21

28 Sep '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14242 --- Comment #132 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- (In reply to Victor Grousset/tuxayo from comment #128) > a. > That's odd, I'm getting the autofill even with OPACSuggestionAutoFill off. > Fixed in latest follow-up > > b. > > press the edit icon next to the field > > Where is the icon actually? Is that the Search Google Books button? A step from an old test plan - yes it is the search button, I've reworded the test plan. > > c. > Maybe an issue > > > d. > Found something else that should be related and easy to reproduce > > e. > When clearing the form, should the ISBN also be cleared? I've added a follow-up to clear the ISBN. > > > f. > I can still reproduce the issue from comment 117 I was not able to reproduce c, d or f. Can you please re-test? -- You are receiving this mail because: You are watching all bug changes.

1 0