Bug 1326 – RSS Plugin adds itself to screensaver menu even if disabled

Bug 1326 - RSS Plugin adds itself to screensaver menu even if disabled

Summary:

RSS Plugin adds itself to screensaver menu even if disabled

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Plugin
Version:	6.0.1
Platform:	All All

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Dave Cohen

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-04-06 16:30 UTC by KDF
Modified:	2008-08-18 10:54 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description KDF 2005-04-06 16:30:12 UTC

regardless of enabled or disabled, the plugins handler needs to call addMenu()
for each plugin to determine what menu it uses, in order to add or remove as
needed.  In the routine, the rss plugin adds a screensaver menu item ignoring
its enabled/disabled status.  This causes a problem when fixing bug1167 to
remove items when disabled becuase there is no way to link this back to the
plugin itself.

Comment 1 Blackketter Dean 2005-04-20 16:01:50 UTC

Dave, can you take a look at this?  Should be easy to kill.

Comment 2 KDF 2005-04-20 16:13:26 UTC

I believe change 2954 took care of this, for 6.1 at least.

Comment 3 Dave Cohen 2005-04-20 17:08:28 UTC

Fixed by r2954.  Neglected to resolve the bug.