Merge lp:~ara/gwibber/apport_bug_384552 into lp:gwibber/1.2

Proposed by Greg Grossmeier
Status: Rejected
Rejected by: Ken VanDine
Proposed branch: lp:~ara/gwibber/apport_bug_384552
Merge into: lp:gwibber/1.2
Diff against target: None lines
To merge this branch: bzr merge lp:~ara/gwibber/apport_bug_384552
Reviewer Review Type Date Requested Status
Ken VanDine Disapprove
Review via email: mp+10266@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Sorry this took so long to get reviewed, the client has been re-written. The ubuntu package now includes an apport hook.

Thanks for your submission.

review: Disapprove

Unmerged revisions

350. By Ara Pulido

Added ubuntu-bug link to Report A Problem menu. Fixes LP:#384552

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'gwibber/client.py'
2--- gwibber/client.py 2009-07-12 20:56:59 +0000
3+++ gwibber/client.py 2009-07-13 09:42:48 +0000
4@@ -5,7 +5,7 @@
5
6 """
7
8-import time, os, threading, logging, mx.DateTime, hashlib
9+import time, os, threading, logging, mx.DateTime, hashlib, subprocess
10 from . import table
11 import gtk, gtk.glade, gobject, functools, traceback
12 import microblog
13@@ -742,6 +742,10 @@
14 key, mod = gtk.accelerator_parse("Escape")
15 accelGroup.connect_group(key, mod, gtk.ACCEL_VISIBLE, self.on_cancel_reply)
16
17+ def report_a_problem():
18+ if subprocess.call(['ubuntu-bug', '-p', 'gwibber'], stdout=subprocess.PIPE) != 0:
19+ gintegration.load_url("https://bugs.launchpad.net/gwibber/+filebug")
20+
21 def create_action(name, accel, stock, fn, parent = menuGwibber):
22 mi = gtk.Action("gwibber%s" % name, "%s" % name, None, stock)
23 gtk.accel_map_add_entry("<Gwibber>/%s" % name, *gtk.accelerator_parse(accel))
24@@ -773,7 +777,7 @@
25 actTranslate = gtk.Action("gwibberTranslate", _("Translate This Application..."), None, None)
26 actTranslate.connect("activate", lambda *a: gintegration.load_url("https://translations.launchpad.net/gwibber"))
27 actReportProblem = gtk.Action("gwibberReportProblem", _("Report a Problem"), None, None)
28- actReportProblem.connect("activate", lambda *a: gintegration.load_url("https://bugs.launchpad.net/gwibber/+filebug"))
29+ actReportProblem.connect("activate", lambda *a: report_a_problem())
30 actAbout = gtk.Action("gwibberAbout", _("_About"), None, gtk.STOCK_ABOUT)
31 actAbout.connect("activate", self.on_about)
32

Subscribers

People subscribed via source and target branches