Bug 298893 - apper-sentinel never times out after an update that needs no reboot/logout
Summary: apper-sentinel never times out after an update that needs no reboot/logout
Status: RESOLVED FIXED
Alias: None
Product: apper
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Daniel Nicoletti
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-27 08:40 UTC by Martin Kho
Modified: 2013-01-09 18:22 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 0.7.2


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Kho 2012-04-27 08:40:46 UTC
After a RefreshAndUpdate or Update check (issued by ApperD) apper-sentinel times out after some time. This, however, is not the case when updates are succesfully installed and there was no need to log out or reboot. apper-sentinel stays active forever, which blocks any future cache-refreshes (see UpdateIcon.cpp->refresh).

Reproducible: Always

Steps to Reproduce:
1. downgrade a package that needs no logout/reboot
2. wait till apper shows the 'update' for the downgraded package
3. update the package (hope it will be succesfull
4. watch apper-sentinel never times out
Actual Results:  
apper-sentinel never time out, blocking cache-refreshes

Expected Results:  
apper-sentinel time out, cache-refreshes are possible again

It looks like apper-sentinel should watch (the results of) the transaction. In ApperSentinel.cpp I see:
    // connect the watch transaction coming from the updater icon to our watcher
//     connect(m_updateIcon, SIGNAL(watchTransaction(const QString &, bool)),
//             m_transWatcher, SLOT(watchTransaction(const QString &, bool)));

Is this what can do the trick?
Comment 1 Martin Kho 2012-04-27 10:18:43 UTC
Hi,

I forgot step 5. close apper :-)

Martin Kho
Comment 2 Daniel Nicoletti 2013-01-09 17:16:54 UTC
Do you still have this issue iirc some problems around this where fixed in Apper 0.7.2, please try with that version.
Comment 3 Martin Kho 2013-01-09 18:22:36 UTC
Hi Daniel,

Sorry, I overlooked this report. This issue is indeed solved. Please close this report.

Thanks,

Martin Kho