[Koha-bugs] [Bug 33484] Generalise table saveState for all tables

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

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

--- Comment #29 from Emily Lamancusa <emily.lamancusa at montgomerycountymd.gov> ---
I can't reproduce the error on KTD either. Lisette, can you give more
information about how to reproduce it?

I am noticing so far a few other unexpected behaviors so far, though:

- Holds to Pull saves the client-side state regardless of the configuration on
the admin table settings page

- Holds Queue appears to be working correctly for the most part. If you disable
state saving from the admin table settings page, the table will continue to
save the client-side state for the rest of the session, but will respect the
configuration if you start a new session. (I'm not sure if that's an error or
if it's the expected behavior.)

- memberresultst and branches.pl (both of which use the kohaTable API wrapper
rather than the KohaTable wrapper) appear to be working correctly for the most
part. However, if the admin table settings hide a column by default, that
column stays hidden even when state saving is enabled. Other aspects of
client-side state saving (hiding columns that are shown by default, entries per
page) work, though.
Also, this was the case for branches.pl in master, too, so it wasn't introduced
by this patch.

I really like where this is going - but I think it's going to need more
thorough/systematic testing than what I've done so far to catch any anomalies.

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