[Bug 4867] New: cannot clone subfields in marc editor, other clone/unclone weirdness
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Summary: cannot clone subfields in marc editor, other clone/unclone weirdness Change sponsored?: --- Product: Koha Version: rel_3_2 Platform: All OS/Version: All Status: NEW Severity: normal Priority: P5 Component: Cataloging AssignedTo: gmcharlt@gmail.com ReportedBy: cmurdock@ccfls.org QAContact: koha-bugs@lists.koha-community.org CC: ian.walls@bywatersolutions.com Estimated Hours: 0.0 Clicking the plus sign next to a repeatable subfield no longer clones the subfield. Also, clicking the minus sign will not remove the subfield initially, but if you click the plus sign, then click the minus sign, it will. Owen & wizzyrea were able to confirm the first half of that, at least. The plus & minus functions worked as recently as 3.01.00.061. There are a few changes to addbiblio.tmpl since 1/2010 that might be related. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 --- Comment #1 from Cindy Murdock Ames <cmurdock@ccfls.org> --- When I tried it in Chrome in developer mode, I got this error: Uncaught TypeError: Cannot call method 'getAttribute' of undefined Owen said he gets a javascript error: "link is undefined" -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 --- Comment #2 from Cindy Murdock Ames <cmurdock@ccfls.org> --- Owen's patch for bug 4868 fixes this. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 --- Comment #3 from Cindy Murdock Ames <cmurdock@ccfls.org> --- On further testing, the patch for 4868, makes it better, but there's still some odd behavior. You can clone the subfield, but sometimes after clicking delete it won't let you clone the subfield again. Also the delete won't always delete the subfield; it only seems to work once for me, so if you clone the subfield twice, and then try to delete them, it will only delete one but not both. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 --- Comment #4 from Fridolyn SOMERS <fridolyn.somers@gmail.com> --- Created attachment 2389 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=2389 Proposed patch add existence test on link after subfield -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Nighswonger <cnighswonger@foundations.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Hours Worked| |2 Priority|P5 |P1 CC| |cnighswonger@foundations.ed | |u Estimated Hours|0.0 |2.0 Severity|normal |major --- Comment #5 from Chris Nighswonger <cnighswonger@foundations.edu> 2010-09-14 14:38:24 UTC --- When the clone/unclone plus/minus were replaced with images the <span> tags were inadvertantly deleted causing the js to loose its ability to identify the clone/unclone links and properly udpate the indexes. The resulting incorrect indexes caused various weirdnesses to result when one attempted to clone/unclone subfields. The solution is to add classes to the two links and update the js to identify the links based on class. Thanks to Owen for his help in chasing this down. The patch is available here: http://git.koha-community.org/gitweb/?p=wip/koha-fbc.git;a=shortlog;h=refs/h... Be sure to pick up the associated tidy patch before it as well. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Nighswonger <cnighswonger@foundations.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression Priority|P1 |PATCH-Sent -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Nighswonger <cnighswonger@foundations.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Nighswonger <cnighswonger@foundations.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored AssignedTo|gmcharlt@gmail.com |cnighswonger@foundations.ed | |u -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Stefan <herrkandera@gmx.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW CC| |herrkandera@gmx.ch --- Comment #6 from Stefan <herrkandera@gmx.ch> 2010-10-16 16:55:21 UTC --- The supplied patch: http://git.koha-community.org/gitweb/?p=wip/koha-fbc.git;a=blob;f=koha-tmpl/... does not work for IE 7. When clicking on clone-subfield.png or delete-subfield.png the MARC editor site jumps to the bottom, but fields don't collapse/uncollapse. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Nighswonger <cnighswonger@foundations.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED --- Comment #7 from Chris Nighswonger <cnighswonger@foundations.edu> 2010-10-16 19:52:05 UTC --- Changing status back to assigned -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Nighswonger <cnighswonger@foundations.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |oleonard@myacpl.org --- Comment #8 from Chris Nighswonger <cnighswonger@foundations.edu> 2010-10-16 20:18:35 UTC --- (In reply to comment #6)
The supplied patch:
http://git.koha-community.org/gitweb/?p=wip/koha-fbc.git;a=blob;f=koha-tmpl/...
does not work for IE 7. When clicking on clone-subfield.png or delete-subfield.png the MARC editor site jumps to the bottom, but fields don't collapse/uncollapse.
Testing in IE 7.0.5730.13 shows that deleting subfields works as expected, but cloning subfields does not. This is probably due to some quark with IE. Since we basically state that FF is the browser supported in the staff client, I'm not sure how much of a priority this will be. However, I'm adding Owen to the cc list in case the fix is something obvious. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz Patch Status|--- |Patch Pushed --- Comment #9 from Chris Cormack <chris@bigballofwax.co.nz> 2010-12-13 03:28:29 UTC --- This is already in master -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4867 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org