Merge ~pieq/bugit/+git/qabro:fix-bug-updating into bugit:master

Proposed by Pierre Equoy
Status: Merged
Approved by: Pierre Equoy
Approved revision: 397475ab9fbe0405e2e49d5332012c0bfa729d9a
Merged at revision: 42eccbab998e86d3a82357216f1a1f564975b0a9
Proposed branch: ~pieq/bugit/+git/qabro:fix-bug-updating
Merge into: bugit:master
Diff against target: 38 lines (+10/-6)
1 file modified
qabro/bug_assistant.py (+10/-6)
Reviewer Review Type Date Requested Status
Pierre Equoy Approve
Review via email: mp+349110@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pierre Equoy (pieq) wrote :

self-approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/qabro/bug_assistant.py b/qabro/bug_assistant.py
2index 782b11a..34b03bf 100644
3--- a/qabro/bug_assistant.py
4+++ b/qabro/bug_assistant.py
5@@ -137,16 +137,15 @@ class BugAssistant:
6 print('Updating Launchpad bug report...')
7 self.bug.title = self.lp_title
8 self.bug.description = description
9- tags = self.lp_tags.split()
10- target = project
11+ self.bug.tags = self.lp_tags.split()
12 self.bug.lp_save()
13
14- if series:
15- nomination = self.bug.addNomination(target=series)
16- nomination.approve()
17-
18 # Task configuration
19 task = self.bug.bug_tasks[0]
20+ if self.bug and task.target != project:
21+ print('Updating project...')
22+ task.target = project
23+ task.lp_save()
24 print('Setting status...')
25 task.status = self.lp_status
26 print('Setting importance...')
27@@ -157,6 +156,11 @@ class BugAssistant:
28
29 task.lp_save()
30
31+ if series:
32+ print('Setting series...')
33+ nomination = self.bug.addNomination(target=series)
34+ nomination.approve()
35+
36 if ('APPORT_STAGING' in os.environ
37 or os.environ.get('APPORT_LAUNCHPAD_INSTANCE', None) == 'staging'):
38 bug_url = STAGING_WEB_ROOT+'bugs/{}'.format(self.bug.id)

Subscribers

People subscribed via source and target branches

to all changes: