diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2009-09-30 17:32:27 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2009-09-30 17:32:27 +0200 |
commit | 8780b44bf82d56ee7eab1322f58cba2034807a3c (patch) | |
tree | 3af37dacad43a2111a8aedcae28b21a2f75bbcd0 /src/Gui | |
parent | 61a85ed6dc697abf71178f9de1cfd851a09f59ee (diff) | |
download | abrt-8780b44bf82d56ee7eab1322f58cba2034807a3c.tar.gz abrt-8780b44bf82d56ee7eab1322f58cba2034807a3c.tar.xz abrt-8780b44bf82d56ee7eab1322f58cba2034807a3c.zip |
abrt-gui: remove pending_jobs member, it is no longer necessary
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'src/Gui')
-rw-r--r-- | src/Gui/CCDBusBackend.py | 21 |
1 files changed, 4 insertions, 17 deletions
diff --git a/src/Gui/CCDBusBackend.py b/src/Gui/CCDBusBackend.py index 32310203..ef8d82e9 100644 --- a/src/Gui/CCDBusBackend.py +++ b/src/Gui/CCDBusBackend.py @@ -21,7 +21,6 @@ class DBusManager(gobject.GObject): # and later with policyKit bus = None def __init__(self): - self.pending_jobs = [] session = None # binds the dbus to glib mainloop DBusGMainLoop(set_as_default=True) @@ -81,7 +80,6 @@ class DBusManager(gobject.GObject): self.bus.add_signal_receiver(self.warning_cb, "Warning", dbus_interface=CC_IFACE) # watch for job-done signals self.bus.add_signal_receiver(self.jobdone_cb, "JobDone", dbus_interface=CC_IFACE) - self.bus.add_signal_receiver(self.jobstarted_cb, "JobStarted", dbus_interface=CC_IFACE) # disconnect callback def disconnected(self, *args): @@ -111,15 +109,11 @@ class DBusManager(gobject.GObject): def update_cb(self, message, job_id=0): print "Update >>%s<< for job: %s" % (message, job_id) - # FIXME: use dest instead of 0 once we implement it in daemon - if job_id == 0 or job_id in self.pending_jobs: - self.emit("update", message, job_id) + self.emit("update", message, job_id) def warning_cb(self, message, job_id=0): print "Warning >>%s<< for job: %s" % (message, job_id) - # FIXME: use dest instead of 0 once we implement it in daemon - if job_id == 0 or job_id in self.pending_jobs: - self.emit("warning", message) + self.emit("warning", message) # Seems to be not needed at all. Not only that, it is actively harmful # when abrtd is autostarted by dbus-daemon: connect_to_daemon() would install @@ -137,7 +131,6 @@ class DBusManager(gobject.GObject): #self.proxy = None self.emit("daemon-state-changed", "down") - def connect_to_daemon(self): if not self.bus: self.bus = dbus.SystemBus() @@ -154,15 +147,9 @@ class DBusManager(gobject.GObject): else: raise Exception(_("Please check if abrt daemon is running.")) - def addJob(self, job_id): - self.pending_jobs.append(job_id) - - def jobstarted_cb(self, dest, job_id): - # the job belongs to this client - print "Started our job: %s" % job_id - self.addJob(job_id) - def jobdone_cb(self, dest, uuid): + # TODO: check that it is indeed OUR job: + # remember uuid in getReport and compare here print "Our job for UUID %s is done." % uuid dump = self.cc.GetJobResult(uuid) if dump: |