update-notifier-kde

Merge lp:~freinhard/update-notifier-kde/respect-exitcodes into lp:update-notifier-kde

  1. respect-exitcodes
  2. Merge into trunk
Proposed by Florian Reinhard
Status: Merged
Merge reported by: Harald Sitter
Merged at revision: not available
Proposed branch: lp:~freinhard/update-notifier-kde/respect-exitcodes
Merge into: lp:update-notifier-kde
Diff against target: None lines
To merge this branch: bzr merge lp:~freinhard/update-notifier-kde/respect-exitcodes
Reviewer Review Type Date Requested Status
Harald Sitter Approve
Jonathan Riddell Pending
Review via email: mp+12028@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Harald Sitter (apachelogger) wrote :

Good to go IMHO

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'update-notifier-kde.py'
2--- update-notifier-kde.py 2009-06-17 15:57:57 +0000
3+++ update-notifier-kde.py 2009-06-18 15:37:38 +0000
4@@ -414,7 +414,6 @@
5
6 def codecsActivated(self):
7 """ user wants to install, show dialogue and install """
8- self.codecsTray.hide()
9 dialogue = KDialog()
10 dialogue.setButtons(KDialog.ButtonCode(KDialog.Ok | KDialog.Cancel))
11 dialogue.setButtonText(KDialog.Ok, i18n("Install Selected"))
12@@ -455,7 +454,9 @@
13 self.process.start()
14
15 def processFinished(self, exitCode, exitStatus):
16- KNotification.event("Restricted", i18n("You will need to restart %1 to use the new functionality", self.application), KIcon("system-software-update").pixmap(QSize(22,22)))
17+ if not exitCode:
18+ self.codecsTray.hide()
19+ KNotification.event("Restricted", i18n("You will need to restart %1 to use the new functionality", self.application), KIcon("system-software-update").pixmap(QSize(22,22)))
20
21 class DBusInterface(dbus.service.Object):
22 def __init__(self, ui):

Subscribers

People subscribed via source and target branches