[Koha-bugs] [Bug 13417] New permission to allow staff to remove public lists

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13417

--- Comment #12 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to M. de Rooy from comment #9)
> (In reply to Jonathan Druart from comment #0)
> > Currently a public list can only be deleted by its owner.
> > This means lists can exist infinitely.
> 
> Please look for HandleDelBorrower.
> When you delete a patron, you delete his lists including the public lists.
> Please see bug 11889. It was opened to discuss/find a solution where you do
> not want to delete all his public lists.

Ha yes, I forgot that!
But we don't want to remove public lists.
So I could submit a patch to remove private lists only, do you agree with that?

(In reply to M. de Rooy from comment #10)
> I would favor a permission for allowing staff to manage lists that they do
> not own. It was discussed on older reports already.

Code from VirtualShelves is really not easy to read/modify, I preferred not to
change it to much.

> But the solution of this patch is imo little bit too quick.
> [1] The change of sub ShelfPossibleAction is not very refined and may have
> side-effects.

I don't think it will introduce side-effects. The change is quite trivial.

> [2] You allow deletion of public lists now, but what if you want to manage a
> private or shared list from another user? Would it not be better to show a
> third tab in staff for Other lists (or even another form?) that allows you
> to search lists, delete, edit, change ownership, etc.?

Discussion appended on a lot of bug reports, and no patch has been pushed yet.
I propose a quick and easy solution to answer a big and real problem for
librarians. Even if it is not perfect.

I would not like to add a new tab here, I already have submitted a big change
for this page (see bug 13419).

-- 
You are receiving this mail because:
You are watching all bug changes.