http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |REOPENED CC| |fridolyn.somers@biblibre.co | |m Resolution|DUPLICATE |--- -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=5524 -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #4 from Owen Leonard <oleonard@myacpl.org> --- Testing after applying the patch: - Viewing private lists, navigate to second page of lists - Click 'delete' link next to a list name for an EMPTY list. - Deletion successful. Success. - Viewing private lists, navigate to second page of lists - Click 'delete' link next to a list name for an list WITH CONTENTS. - Deletion successful but WITHOUT WARNING (I guess it's this way in master too). Success. - Viewing private lists, navigate to second page of lists. - Click an EMPTY list to view its contents. - Click the "delete list" button from the list contents view. - No error, list deleted. Success. - Viewing private lists, navigate to second page of lists. - Click a list WITH CONTENTS to view its contents. - Click the "delete list" button from the list contents view. - No error. Redirected to the first page of private lists with no link to the second page of lists. Navigating back to private lists and then to the second page of lists shows that the list was not deleted. Fail. - Viewing public lists, navigate to second page of lists - Click 'delete' link next to a list name for an EMPTY list. - No error. Redirected to the first page of public lists with no link to the second page of lists. Navigating back to public lists and then to the second page of lists shows that the list WAS deleted. Success* - Viewing public lists, navigate to second page of lists - Click 'delete' link next to a list name for list WITH CONTENTS. - No error. Redirected to the first page of public lists with no link to the second page of lists. Navigating back to public lists and then to the second page of lists shows that the list WAS NOT deleted. Fail. - Viewing public lists, navigate to second page of lists. - Click an EMPTY list to view its contents. - Click the "delete list" button from the list contents view. - No error. Redirected to the first page of public lists with no link to the second page of lists. Navigating back to public lists and then to the second page of lists shows that the list WAS deleted. Success* - Viewing public lists, navigate to second page of lists. - Click an list WITH CONTENTS to view its contents. - Click the "delete list" button from the list contents view. - No error. Redirected to the first page of public lists with no link to the second page of lists. Navigating back to public lists and then to the second page of lists shows that the list WAS NOT deleted. Fail. * The problem of being redirected to the first page of lists with no pagination link to more lists is one which I thought we had a separate bug report for, but I can't find it. In terms of the list being deleted these tests are successful, but in terms of offering the user a working process they are a failure. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |RESOLVED CC| |jonathan.druart@biblibre.co | |m Resolution|--- |FIXED --- Comment #6 from Jonathan Druart <jonathan.druart@biblibre.com> --- Works for me on master. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |REOPENED Resolution|FIXED |--- --- Comment #7 from Jonathan Druart <jonathan.druart@biblibre.com> --- Problem in master and 3.14.x (at least) -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl Assignee|jonathan.druart@biblibre.co |m.de.rooy@rijksmuseum.nl |m | --- Comment #8 from Jonathan Druart <jonathan.druart@biblibre.com> --- Marcel, Could you have a look at this bug please? I am a little bit lost in the VirtualShelves code. I don't understand why the DelShelf is done as far in the code. We shouldn't need to call GetShelves to delete a shelf. Please tell me if you don't have the time and I will try to provide a patch. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #10 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- The (horrible) code in VirtualShelves/Page.pm needs refactoring. I would like to do that when converting it to DBIx in one pass. I try to reproduce your problem on master: I create 22 private lists. List 21 has one item, list 22 has no items. I delete list 21, have to confirm and it is gone. I delete list 22; no confirm button because it is empty. It is gone too. OK. Reading back: If I click on list 21 and want to delete there. It says: list number unrecognized. Cool ;) Looking further.. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #11 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 31883 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31883&action=edit Bug 6279: Cannot delete lists from second page of lists in the OPAC The code in Page.pm still needs more attention, but this may serve as a quick fix for the problem; it will at least improve the situation. If you delete a list from its individual page (viewshelf), you already see the items (if any). So we will assume that after you said Yes to the message box, you want to delete it. There will not be a confirmation button anymore. Note that after you deleted the list, the intended message is not shown and the pagination bar is gone. I leave those for the future refactoring. Test plan: Delete a list from its own viewshelf page in OPAC and staff. Note that OPAC will delete it rightaway now. Note that staff still has the confirmation button. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #13 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Owen Leonard from comment #12)

(In reply to Owen Leonard from comment #4)

...

Testing after applying the patch:

I once again followed this test plan.

...

- Viewing public lists, navigate to second page of lists. - Click an list WITH CONTENTS to view its contents. - Click the "delete list" button from the list contents view. - No error. Redirected to the first page of public lists with no link to the second page of lists. Navigating back to public lists and then to the second page of lists shows that the list WAS NOT deleted.

Fail.

This was the only failure among those tests, other than the continuing problem of the pagination links disappearing.

I'm not sure whether to sign off on this because it is an improvement or ask that this one case be fixed as well.

Do you mean that the list was not deleted in this case? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #15 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Jonathan Druart from comment #14)

Marcel, There are several occurrences of "DEL-" in the template. Your patch only changes one.

Something else, I didn't try but, if you remove the 4 lines in the pm file, are you sure we won't introduce a regression? I am under the impression that this test was there to verify the user has the right (i.e he is the owner/creator of the list) to delete the list?

I am working on it. Please wait.. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #18 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Owen Leonard from comment #12)

...

- Viewing public lists, navigate to second page of lists. - Click an list WITH CONTENTS to view its contents. - Click the "delete list" button from the list contents view. - No error. Redirected to the first page of public lists with no link to the second page of lists. Navigating back to public lists and then to the second page of lists shows that the list WAS NOT deleted.

Fail.

This was the only failure among those tests, other than the continuing problem of the pagination links disappearing.

I'm not sure whether to sign off on this because it is an improvement or ask that this one case be fixed as well.

Thanks for testing, Owen. I recall that deleting a list on the second page (from the all shelves view) brought me back to the first page without pagination. So the confirm button never appeared and I could not delete the list. But deleting it from its own shelf page (with the items) was successfull with the first patch. The second patch should resolve some further issues with pagination. In order to achieve the best result in not too much time, I sacrificed the confirmation buttons. They were at least debatable since you already confirm a delete. Hopefully this can be signed off now. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #20 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 31894 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31894&action=edit Bug 6279: Follow-up for pagination and confirm This follow-up patch restores the pagination and removes the Confirm buttons. This is not really needed since you already confirmed a javascript messagebox. The code needs refactoring. This is a quick fix. Test plan: Delete lists from the second page (from list view and detail view) in both OPAC and staff. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #31893|0 |1 is obsolete| | --- Comment #22 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 31898 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31898&action=edit Bug 6279: Cannot delete lists from second page of lists in the OPAC The code in Page.pm still needs more attention, but this may serve as a quick fix for the problem; it will at least improve the situation. If you delete a list from its individual page (viewshelf), you already see the items (if any). So we will assume that after you said Yes to the message box, you want to delete it. There will not be a confirmation button anymore. Note that after you deleted the list, the intended message is not shown and the pagination bar is gone. I leave those for the future refactoring. Test plan: Delete a list from its own viewshelf page in OPAC and staff. Note that OPAC will delete it rightaway now. Note that staff still has the confirmation button. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #24 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Nice job, works fine -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|m.de.rooy@rijksmuseum.nl |gmcharlt@gmail.com --- Comment #26 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Owen Leonard from comment #25)

Testing this step in the OPAC:

...

- Viewing private [and private] lists, navigate to second page of lists - Click 'delete' link next to a list name for an list WITH CONTENTS.

Clicking 'delete' give me a JavaScript prompt but doesn't redirect back to the list with a confirmation button to delete the contents of the list as it did before. This means that if JavaScript is turned off a list with contents can be deleted without any warning at all. This seems like a regression to me.

OK. I am not sure if running Koha without JavaScript these days will be very progressive, but this sounds valid. At this point, I would go for refactoring instead of spending more time on the current code. So I obsolete both patches. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #31899|0 |1 is obsolete| | -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 --- Comment #28 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to M. de Rooy from comment #26)

At this point, I would go for refactoring instead of spending more time on the current code. So I obsolete both patches.

It's a major bug and still exists on master. I would like to see this bug fixed quickly. And the quicker way is to push these patches, don't you think? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=14544 --- Comment #30 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Will be fixed by 14544. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6279 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #32 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Fixed in 3.22 by bug 14544. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.