Bug 6621

Summary: Javascript error in cloning subfields
Product: Koha Reporter: Justin @ Prosentient <prosentientsystems>
Component: CatalogingAssignee: Justin @ Prosentient <prosentientsystems>
Status: CLOSED WONTFIX QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: jcamins, koha.sekjal, m.de.rooy
Version: rel_3_2   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: makes var buttonDotOnClick a String for IE7

Description Justin @ Prosentient 2011-07-22 05:20:36 UTC
While cataloguing using IE7, the clone subfields button doesn't do anything. Delete subfields does, strangely enough. This is most likely a JavaScript error. (This problem can be worked around by switching to IE8.)
Comment 1 Justin @ Prosentient 2011-07-22 05:35:22 UTC
Created attachment 4709 [details]
makes var buttonDotOnClick a String for IE7
Comment 2 Marcel de Rooy 2011-08-04 11:07:59 UTC
(In reply to comment #1)
> Created attachment 4709 [details]
> makes var buttonDotOnClick a String for IE7

I have a question about:
var buttonDotOnClick = String(link.getAttribute('onclick'));

Where did you define link in the Clonefield function?
Why do you call String? Is it necessary to recover from a null? This may indeed be browser dependent.
I tested your patch in IE8, but have to say that there are more JS errors while doing that. Our users are running staff in Firefox for that reason.
Comment 3 Owen Leonard 2011-08-24 16:22:05 UTC
I'm marking this Failed QA because of the questions raised above. Let's re-evaluate after that.
Comment 4 Jared Camins-Esakov 2012-02-10 14:58:21 UTC
Because this problem is with 3.2 which is no longer supported, and IE7 which is several versions out of date, I don't think anyone is going to pick this up again. If you are able to reproduce the problem with a current version of Koha and would like to submit a revised patch that addresses Marcel's concerns, please reopen the bug.