Bug 39423: Fix column checkboxes in batch item modification hide incorrect columns
In batch item modification, unchecking any column hides the previous column.
This is caused by new HTML in table header :
<span class="dt-column-title"></span><span class="dt-column-order"></span>
This patch fixed by adding ".dt-column-title" to function guess_nb_cols().
Test plan :
1) Go to batch item modification
2) In "Show/hide columns:" click on "Home library" column
=> Check correct column is hidden
3) Go to another batch item modification
=> Check column "Home library" is hidden (from cookie storage)
Signed-off-by: Emmi Takkinen <emmi.takkinen@koha-suomi.fi>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 7c501cc381)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
This commit is contained in:
parent
82ac526335
commit
ee3bf78fd4
|
|
@ -5,7 +5,7 @@ date.setTime(date.getTime() + 365 * 24 * 60 * 60 * 1000);
|
|||
|
||||
function guess_nb_cols() {
|
||||
// This is a bit ugly, we are trying to know if there are checkboxes in the first column of the table
|
||||
if ($("#itemst tr:first th:first").html() == "") {
|
||||
if ($("#itemst tr:first th:first .dt-column-title").html() == "") {
|
||||
// First header is empty, it's a checkbox
|
||||
return 3;
|
||||
} else {
|
||||
|
|
|
|||
Loading…
Reference in New Issue