https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15636

Marc Véron <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #5 from Marc Véron <[email protected]> ---
(In reply to Blou from comment #4)
> Hello Cate,
> We'll need your help on that.  You see, we cannot reproduce the situation
> without hacking the code.  It arose because we're adding an extra column in
> there for ebooks, in a custom development.  It causes a jqury message to pop
> up in the UI because of the imbalance in column count.
> 
> The origin of it is commit 6d32d66f, by bug #7720.  You can see in there
> that it's missing the javascript part for the test.
> 
> So, not sure how to test it.  Except that by looking at the code (and 7720's
> code), it's "obvious".  Just need a way to translate it into a scenario...

Blou,

Maybe you could add a separate patch with the "hack" that is necessary for
testing (marked with something like FOR TESTING ONLY - DO NOT PUSH)

The scenario would then be something like:
- apply the "Hack" patch
- verify the bug
- apply the patch with the jQuery fix
- verify that the error is gone
- remove both patches and apply the jQuery fix patch only
- verify that nothing is broken
- sign-off the patch with the jQuery fix

Marc

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to