Bug 16698 - Refreshing SqueezePlay's menu fails, if menu has >200 items
: Refreshing SqueezePlay's menu fails, if menu has >200 items
Status: NEW
Product: SqueezePlay
Classification: Unclassified
Component: SB Server
: 7.5.x
: PC Windows Vista
: -- major with 2 votes (vote)
: 7.6.x
Assigned To: Unassigned bug - please assign me!
http://forums.slimdevices.com/showthr...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-12-07 02:54 UTC by rach2
Modified: 2011-04-25 09:10 UTC (History)
4 users (show)

See Also:
Category: Bug


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description rach2 2010-12-07 02:54:53 UTC
Current situation:

When some item in the menu has an action with "nextWindow" specified as 'parent' or 'grandparent' (possible 'refresh', 'refreshOrigin' too), it is supposed to move to the parent (grandparent) menu after executing the action and refresh its contents. This would be done by invoking the command which created this menu.

This fails, if the menu contains more than 200 items - only items above 200 are refreshed. Items 1-200 are not. Possible reason is: the refresh function only refreshes the last chunk of data, by re-executing the last command. Because SqueezePlay seems to fetch data in chunks of 200 items (multiple commands), only items from the last chunk are refreshed.

This stops plugin developers from creating plugins with dynamic menus, which need to be refreshed.

Requested change:

Make the refresh update the complete menu, or provide another way to do this.

To consider:

Fixing this bug may cause possible performance problems in some situations. See forum discussion:

http://forums.slimdevices.com/showthread.php?t=83712
Comment 1 Joerg Schwieder 2010-12-14 04:02:19 UTC
I should add that I'm against always refreshing the whole menu since this would be a performance hit (unless most menus switch to the "noRefresh" versions of the "nextWindow" parameters.

I'd propose to add something like "parentRefreshAll" or similar to indicate that a whole menu should be refreshed.