battery applet does not detect removal / insertion of battery

[Michael Frey]

The current moblin-applets battery status icon does not detect when a user removes / inserts a battery. This logic should be detected just like it does in gnome-power-manager.

[Tony Espy (awe)]

I marked this as Medium based on the fact that we may not need this for our customer. Mike, if this assumption is incorrect, please bump the priority.

[ToddBrandt (todd-e-brandt)]

Fixed in moblin-applets 0.60. In gutsy, the HAL daemon reacted to the better being removed and reinserted by triggering the PropertyModified callback for the battery device the applet is connected to. In Hardy, removing the battery apparantly removes the entire device object for the battery, and reinserting it adds it back in. Therefor the PropertyModified callback was no longer being triggered and the applet didn't detect anything when the batter device was gone. I added two callbacks for the DeviceAdded and DeviceRemoved events in the device manager namespace. These calls are triggered as expected and the applet icon now updates just as it did in gutsy.

[Luke (luke-yang)]

Hi ToddBrandt,
Thanks for your great support.
Could you please provide the modified package and source for us?
I want to pre-test it.

thanks a lot.

[ToddBrandt (todd-e-brandt)]

Yea, I just did, sorry about that. I usually update the bug first because I have to put the text in the changelog for the package. Then I got hung up on creating a better ppaprepare script that does everything for me so I don't have to tweak the changelog each time I release. It should be built and out there in about an hour.

[Luke (luke-yang)]

Dear ToodBrandt,

Thanks a lot.
I found a problem that when user boot the device without battery, the battery applet does not detect battery.

Test Procedure :
1. Boot the device connecting the AC adapter only ( without battery)
2. Wait for system booting completely.
3. Insert the battery.
4. The battery applet still can't detect the battery.

[Luke (luke-yang)]

Dear ToodBrandt,

Thank you very much,your solution is work fine.
I will deliver it to our SA team for verifcation.

[Luke (luke-yang)]

Hi Tony,

Accoding to ToodBrandt's support. this issue is fixed in the moblin-applet-61.
I have created the new patch that is base on mobloin-applets_0.55jax2 for comapl project.
could you please confirm and add it into next release image.

thanks a lot.

[Tony Espy (awe)]

I'm not sure why you created a patch... We will pull in the new version of moblin-applets for beta2 and merge our changes.

[RichardJL Wang (richardjl-wang)]

Good and like to see in Beta2.

-----Original Message-----
From: <email address hidden> [mailto:<email address hidden>] On Behalf Of
Tony Espy
Sent: Monday, May 12, 2008 12:51 PM
To: Wang. RichardJL (TPE)
Subject: [Bug 224884] Re: battery applet does not detect removal /
insertionof battery

I'm not sure why you created a patch... We will pull in the new version
of moblin-applets for beta2 and merge our changes.

--
battery applet does not detect removal / insertion of battery
https://bugs.launchpad.net/bugs/224884
You received this bug notification because you are a member of Acton,
which is subscribed to acton.

================================================================================================================================================================
This message may contain information which is private, privileged or confidential of Compal Electronics, Inc. If you are not the intended recipient of this message, please notify the sender and destroy/delete the message. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon this information, by persons or entities other than the intended recipient is prohibited.
================================================================================================================================================================

[cathyli (cathy-li)]

Verified in applets 0.61-ubuntu0, the issue is fixed, close it.