Application Indicator doesn't process menu changes (e.g. checkboxes)

Bug #524308 reported by Nicolò Chieffo
54
This bug affects 7 people
Affects Status Importance Assigned to Milestone
DBus Menu
Fix Released
Undecided
Unassigned
libdbusmenu (Ubuntu)
Fix Released
Medium
Ted Gould

Bug Description

When you toggle a checkbox in an Application Indicator menu, the status of that CheckBoxMenuItem isn't updated, even though the toggle signal is sent.
This seems to be caused by a broader bug, namely that menu changes aren't processed by Indicator Application.

Related branches

Revision history for this message
Nicolò Chieffo (yelo3) wrote :
Revision history for this message
Ara Pulido (ara) wrote :

I can confirm this, but, for me, it is the other way round, it is always showing "Mute All". This is a regression, it was working correctly in 0.1.1-0ubuntu1

Changed in indicator-sound (Ubuntu):
status: New → Confirmed
Revision history for this message
Omer Akram (om26er) wrote :

This is not an indicator sound bug. becasue when you are playing something in rhythmbox its play button is also not ticked

Revision history for this message
Sune Keller (sune-keller) wrote :

@Omer Akram
Do you think it's a libappindicator or libindicate bug then? In that case we should mark it as affecting those packages.

Revision history for this message
Conor Curran (cjcurran) wrote :

Hmm I cannot reproduce this. It sounds like it could have something to do with the dbus communication. I will hopefully get to test this this afternoon with a machine that has no extra dx team ppa installs.

Revision history for this message
Nicolò Chieffo (yelo3) wrote : Re: [Bug 524308] Re: unmute does not change back to mute

I don't understand why rhythmbox play button and the indicator-sound
mute menu entry might be related...

Revision history for this message
Omer Akram (om26er) wrote : Re: unmute does not change back to mute

@Nicolò Chieffo because it all started together and also the same case is with transmission so this is not a indicator-sound bug. changing it to libindicate.

summary: - unmute does not change back to mute
+ indicator applet does not reflect any change for any application
Omer Akram (om26er)
affects: indicator-sound (Ubuntu) → libindicate (Ubuntu)
Changed in libindicate (Ubuntu):
importance: Undecided → Medium
status: Confirmed → Triaged
Revision history for this message
Omer Akram (om26er) wrote : Re: indicator applet does not reflect any change for any application

I think this needs to be fixed for Alpha-3 else we are gonna receive many duplicates as people test Alpha-3 and they are gonna report against the app they face this with.

Revision history for this message
Conor Curran (cjcurran) wrote :

Okay will try to reproduce and fix tmrw.
My installation does not do this....

Revision history for this message
Vish (vish) wrote :

This is not a bug , in the sense , the current menu in indicator-sound is done according to the specs described in the wiki > https://wiki.ubuntu.com/SoundMenu#Menu%20structure

Changing bug to right package.

@Mpt , does the menu item need to toggle mute/unmute ?

affects: libindicate (Ubuntu) → indicator-sound (Ubuntu)
Changed in indicator-sound (Ubuntu):
status: Triaged → Incomplete
Revision history for this message
Vish (vish) wrote :
Changed in indicator-sound (Ubuntu):
status: Incomplete → Triaged
Revision history for this message
Vish (vish) wrote :

err.. Pretend nothing happened :/ [i guess I'm a bit sleep deprived.]

affects: indicator-sound (Ubuntu) → libindicate (Ubuntu)
Revision history for this message
Sense Egbert Hofstede (sense) wrote :

Am moving this to indicator-application, which is where it belongs.
I would have set the status to Triaged since I'm opening an upstream task, but that's not necessary since it is already Triaged. Please only use the status Triaged when all required triaging steps have been fulfilled. Thanks!

affects: libindicate (Ubuntu) → indicator-application (Ubuntu)
summary: - indicator applet does not reflect any change for any application
+ Application Indicator doesn't process menu changes (e.g. checkboxes)
description: updated
Revision history for this message
Conor Curran (cjcurran) wrote : Re: [Bug 524308] Re: indicator applet does not reflect any change for any application

Hi Sense,

I have yet to proper confirm this bug. Please let this with me and I
will hopefully get to this on an alternative machine with a fresh iso
install tmrw. I cannot reproduce this on my dev machine.

Conor

On 23/02/10 15:52, Sense Hofstede wrote:
> Am moving this to indicator-application, which is where it belongs.
> I would have set the status to Triaged since I'm opening an upstream task, but that's not necessary since it is already Triaged. Please only use the status Triaged when all required triaging steps have been fulfilled. Thanks!
>
> ** Package changed: libindicate (Ubuntu) => indicator-application
> (Ubuntu)
>
> ** Also affects: indicator-application
> Importance: Undecided
> Status: New
>
> ** Summary changed:
>
> - indicator applet does not reflect any change for any application
> + Application Indicator doesn't process menu changes (e.g. checkboxes)
>
> ** Description changed:
>
> - Binary package hint: indicator-sound
> -
> - I'm stuck with the menu containing always "unmute", even if the audio is
> - not mute. Anyway the problem reguards only text, because the action is
> - correctly taken.
> -
> - ProblemType: Bug
> - Architecture: amd64
> - Date: Fri Feb 19 10:33:12 2010
> - DistroRelease: Ubuntu 10.04
> - Package: indicator-sound 0.1.1-0ubuntu2
> - ProcVersionSignature: Ubuntu 2.6.32-13.18-generic
> - SourcePackage: indicator-sound
> - Uname: Linux 2.6.32-13-generic x86_64
> + When you toggle a checkbox in an Application Indicator menu, the status of that CheckBoxMenuItem isn't updated, even though the toggle signal is sent.
> + This seems to be caused by a broader bug, namely that menu changes aren't processed by Indicator Application.
>
>

--
Conor Curran
Desktop Architect Sound Engineer
Desktop Experience team
Canonical Ltd
Email: <email address hidden>
web: http://www.canonical.com

Revision history for this message
Omer Akram (om26er) wrote :

@qense ok This was marked triaged because the upstream developer is aware of the bug report is there any other thing necessary for a bug to be marked triaged?

Revision history for this message
Dave Gilbert (ubuntu-treblig) wrote :

Conor: Given I can easily repeat it, is there any useful debug I can gather, or if you want to build a package filled with a pile of debug I'll happily give it a go.

Dave

Revision history for this message
Conor Curran (cjcurran) wrote :

Hi all,

Finally got my hands on a machine where I could reproduce this. The sound service on the backend is doing what it should i.e. sending appropriate text property updates over DBus to the dbusmenu. The bug seeems to lie somewhere in the libdbusmenuitem. I will talk to Ted as soon as is possible later on today.

I think though this bug might belong in libindicator as opposed to libappindicator since the indicator-sound like the latter is built on top of the former and not on top of the latter. In less confusing speak maybe the bug belongs to the former => libindicator
:)
Conor

Revision history for this message
Sense Egbert Hofstede (sense) wrote :

Conor: I'm not sure if it really is libindicator that is to blame here since applications using libappindicator are also affected. You said this issue is probably caused in libdbusmenu, so I would mark the bug against that package instead.

Revision history for this message
Conor Curran (cjcurran) wrote :

Hi Sense,

Yes correct libdbusmenu is where it belongs.

affects: indicator-application (Ubuntu) → libdbusmenu (Ubuntu)
Changed in libdbusmenu (Ubuntu):
assignee: nobody → Ted Gould (ted)
status: Triaged → Confirmed
Revision history for this message
Conor Curran (cjcurran) wrote :

Okay this cannot be reproduced on my dev machine with this

libdbusmenu-glib1:
  Installed: 0.3.1+r70+201002100418
  Candidate: 0.3.1+r70+201002100418
  Version table:
 *** 0.3.1+r70+201002100418 0
        100 /var/lib/dpkg/status
     0.2.5+r76+201002192157 0
        500 https://private-ppa.launchpad.net lucid/main Packages
     0.2.5-0ubuntu1 0
        500 http://archive.ubuntu.com lucid/main Packages
ronoc@ronoc-work-laptop:~$ apt-cache policy libdbusmenu-glib-dev
libdbusmenu-glib-dev:
  Installed: 0.3.1+r70+201002100418
  Candidate: 0.3.1+r70+201002100418
  Version table:
 *** 0.3.1+r70+201002100418 0
        100 /var/lib/dpkg/status
     0.2.5+r76+201002192157 0
        500 https://private-ppa.launchpad.net lucid/main Packages
     0.2.5-0ubuntu1 0
        500 http://archive.ubuntu.com lucid/main Packages

While it can be reproduced on my test laptop (fresh alpha 3 install => currently present in lucid)

libdbusmenu-glib1:
  Installed: 0.2.5-0ubuntu1
  Candidate: 0.2.5-0ubuntu1
  Version table:
 *** 0.2.5-0ubuntu1 0
        500 http://gb.archive.ubuntu.com lucid/main Packages
        100 /var/lib/dpkg/status

Revision history for this message
Conor Curran (cjcurran) wrote :

Dbus trace from test machine:

signal sender=:1.147 -> dest=(null destination) serial=47 path=/org/ayatana/indicator/sound/service; interface=org.ayatana.indicator.sound; member=SinkMuteUpdate
   boolean true
signal sender=:1.147 -> dest=(null destination) serial=48 path=/org/ayatana/indicator/sound/menu; interface=org.ayatana.dbusmenu; member=ItemPropertyUpdated
   int32 2
   string "label"
   variant string "Unmute"
method return sender=:1.147 -> dest=:1.148 reply_serial=35
method call sender=:1.148 -> dest=:1.147 serial=36 path=/org/ayatana/indicator/sound/menu; interface=org.ayatana.dbusmenu; member=Event
   int32 2
   string "clicked"
   variant int32 0
   uint32 27944840
signal sender=:1.147 -> dest=(null destination) serial=50 path=/org/ayatana/indicator/sound/service; interface=org.ayatana.indicator.sound; member=SinkVolumeUpdate
   double 52.0538
signal sender=:1.147 -> dest=(null destination) serial=51 path=/org/ayatana/indicator/sound/menu; interface=org.ayatana.dbusmenu; member=ItemPropertyUpdated
   int32 2
   string "label"
   variant string "Mute All"
method return sender=:1.147 -> dest=:1.148 reply_serial=36

Ted Gould (ted)
affects: indicator-application → dbusmenu
Changed in dbusmenu:
status: New → Fix Committed
milestone: none → 0.2.6
Ted Gould (ted)
Changed in dbusmenu:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libdbusmenu - 0.2.6-0ubuntu1

---------------
libdbusmenu (0.2.6-0ubuntu1) lucid; urgency=low

  * Upstream release 0.2.6
    * Fix testing to make it more reliable
    * Remove delayed property queueing as it was causing properties to not
      get updated. (lp: #524308)
 -- Ted Gould <email address hidden> Thu, 25 Feb 2010 10:23:49 -0600

Changed in libdbusmenu (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.