The new API driven items view defaults to 20 per page. This is a bit low, it would be great if it could be configurable to a higher value per the table configuration.
Created attachment 158109 [details] [review] Bug 35191: Make entries per page configurable for items table on staff detail page The new API driven items view defaults to 20 per page. This is a bit low, it would be great if it could be configurable to a higher value per the table configuration. Test plan: 1. Apply this patch 2. restart_all (memcached restart is required) 3. Go to Administration > Table settings 4. Catalogue > holdings_table | otherholdings_table => Notice the new "Default display length" 5. Select something else than 20 6. Go to the detail page of a record with lot of records => Notice that the default value defined in the settings is used.
Created attachment 169259 [details] [review] Bug 35191: Make entries per page configurable for items table on staff detail page The new API driven items view defaults to 20 per page. This is a bit low, it would be great if it could be configurable to a higher value per the table configuration. Test plan: 1. Apply this patch 2. restart_all (memcached restart is required) 3. Go to Administration > Table settings 4. Catalogue > holdings_table | otherholdings_table => Notice the new "Default display length" 5. Select something else than 20 6. Go to the detail page of a record with lot of records => Notice that the default value defined in the settings is used. Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Created attachment 171146 [details] [review] Bug 35191: Make entries per page configurable for items table on staff detail page The new API driven items view defaults to 20 per page. This is a bit low, it would be great if it could be configurable to a higher value per the table configuration. Test plan: 1. Apply this patch 2. restart_all (memcached restart is required) 3. Go to Administration > Table settings 4. Catalogue > holdings_table | otherholdings_table => Notice the new "Default display length" 5. Select something else than 20 6. Go to the detail page of a record with lot of records => Notice that the default value defined in the settings is used. Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
I like small patches :)
Pushed for 24.11! Well done everyone, thank you!
*** Bug 38074 has been marked as a duplicate of this bug. ***
*** Bug 37310 has been marked as a duplicate of this bug. ***
Backported to 24.05.x for upcoming 24.05.05
Depends on Bug 33568 not in 23.11.x