dspace-angular icon indicating copy to clipboard operation
dspace-angular copied to clipboard

Page size change and sorting doesn't work on some pages

Open EikLoe opened this issue 1 year ago • 2 comments

Describe the bug If you want to change the number of items on the edit-relations-lookup (edit-items -> relationships -> +Add) or change the sort settings it doesn't work. The same happens, if you try to do the same thing on a community-page in the search section.

To Reproduce

  1. Edit an entity-item
  2. Navigate to the relationship-tab and click Add + to add a new relationship.
  3. Try to change the sort order or the number of items per page

The list of found items doesn't change. (If you click Select page the page will be selected and afterwards the sorting-changes/number-of-items are applied.)

Expected behavior The settings should be applied.

EikLoe avatar May 02 '24 14:05 EikLoe

I can reproduce this on the Sandbox (https://sandbox.dspace.org). It's also very easy to reproduce as follows:

  1. Browse to any Community/Collection page
  2. On the "Search" tab (default tab), in the sidebar change the "Results per page" from 10 to 20.
  3. The entire page hangs. It become unresponsive. It doesn't appear to send a request to the REST API, but the page just shows the "loading" dots and never returns.

Flagging as high priority because it's very easy to end up with an unresponsive site.

tdonohue avatar May 02 '24 16:05 tdonohue

Pinging @artlowel and @alexandrevryghem in case this is of interest. It sounds related to #2732 (from @alexandrevryghem ) and also possibly related to #2978 (which @artlowel has claimed). It might be the same issue on both pages, or slightly different (not sure).

If you are interested, let me know or claim it for yourself.

tdonohue avatar May 02 '24 17:05 tdonohue

@alexandrevryghem : Are you still working on this one? Ideally, this would be nice to fix prior to 8.0 since it's currently possible to cause the search page to "hang indefinitely" as described here: https://github.com/DSpace/dspace-angular/issues/2996#issuecomment-2091060693

tdonohue avatar May 16 '24 16:05 tdonohue

@tdonohue: Yes I'll try to look into it tomorrow/next week

alexandrevryghem avatar May 16 '24 22:05 alexandrevryghem

@alexandrevryghem : Just a reminder this might be good to fix in time for 8.0, if possible to do so.

tdonohue avatar May 21 '24 21:05 tdonohue