[Koha-bugs] [Bug 19791] patron Modification log redirects to circulation page

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

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

Josef Moravec <josef.moravec at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |josef.moravec at gmail.com

--- Comment #3 from Josef Moravec <josef.moravec at gmail.com> ---
(In reply to Abdou Yousuf from comment #2)
> (In reply to Jonathan Druart from comment #1)
> > With bug 19080 we tried to redirect to the circulation home page when the
> > borrowernumber did not exist (anymore).
> > 
> > When you click on the "modification log" tab, the "object" parameter is
> > automatically filled with the borrowernumber of the patron you are looking
> > at.
> > 
> > If you decide to empty this field or use a non-existent borrowernumber, you
> > will get redirected to the circulation home page.
> > 
> > I would suggest to:
> > - Disable the "object" input and force the value to borrowernumber when we
> > come from the circulation module and redirect to circ home page if
> > non-existent
> > - Make it editable when come from the tools module and display a "no logs"
> > message if non-existent borrowernumber is used.
> 
> I don't think that it's beneficial to automatically fill (Object field) with
> the borrower number, On the contrary I think that the (Librarian:) field
> should be automatically filled by borrower number rather than (Object
> field), And it will be more beneficial, In fact I don't know why borrower
> number should be in (Object field) and not (Librarian field) when selecting
> "modification log" tab, What are the benefits?

The tabs on patron pages are all relevant to current patron, that is the reason
why borrowernumber is in object - you want here to get information about this
patron. 

> Also, You said that you tried to redirect to the circulation home page when
> the
> borrower number did not exist (anymore), How it can be? I think it's not
> logical even though it can be done programmatically.

It could be possible - for example one librarian deletes a patron and the
second librarian wants to show some information about patron. 

> 
> Regards.

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