[PATCH 2 of 3] bugzilla: extract optional hours from commit message and update bug time
Jim Hague
jim.hague at acm.org
Thu Mar 1 09:30:53 CST 2012
# HG changeset patch
# User Jim Hague <jim.hague at acm.org>
# Date 1330615537 0
# Node ID 85e64967582f5868d476c7a6964adcdbbac19bf9
# Parent ec5308b63154f8ae16fab87258655ee8886b0553
bugzilla: extract optional hours from commit message and update bug time
If an 'hours' group is present in the bug matching regular expression,
and that group contains a float value, update the bug working time in
Bugzilla with those hours.
The patch adds a key 'hours' to the bug state dictionary, and adds
support for the key to the XMLRPC and XMLRPC/email access methods.
The MySQL access method is not supported; a warning is given.
diff -r ec5308b63154 -r 85e64967582f hgext/bugzilla.py
--- a/hgext/bugzilla.py Thu Mar 01 14:51:40 2012 +0000
+++ b/hgext/bugzilla.py Thu Mar 01 15:25:37 2012 +0000
@@ -12,7 +12,8 @@
that refer to bugs by Bugzilla ID are seen. The comment is formatted using
the Mercurial template mechanism.
-The hook does not change bug status.
+The bug references can optionally include an update for Bugzilla of the
+hours spent working on the bug.
Three basic modes of access to Bugzilla are provided:
@@ -32,7 +33,7 @@
MySQL user and password to have full access rights to the Bugzilla
database. For these reasons this access mode is now considered
deprecated, and will not be updated for new Bugzilla versions going
-forward.
+forward. Only adding comments is supported in this access mode.
Access via XMLRPC needs a Bugzilla username and password to be specified
in the configuration. Comments are added under that username. Since the
@@ -63,10 +64,15 @@
bugzilla.regexp
Regular expression to match bug IDs in changeset commit message.
- Must contain one "()" group. The default expression matches ``Bug
- 1234``, ``Bug no. 1234``, ``Bug number 1234``, ``Bugs 1234,5678``,
- ``Bug 1234 and 5678`` and variations thereof. Matching is case
- insensitive.
+ It must contain one "()" named group ``<ids>`` containing the bug
+ IDs separated by non-digit characters. It may also contain
+ a named group ``<hours>`` with a floating-point number giving the
+ hours worked on the bug. If no named groups are present, the first
+ "()" group is assumed to contain the bug IDs, and work time is not
+ updated. The default expression matches ``Bug 1234``, ``Bug no. 1234``,
+ ``Bug number 1234``, ``Bugs 1234,5678``, ``Bug 1234 and 5678`` and
+ variations thereof, followed by an hours number prefixed by ``h`` or
+ ``hours``, e.g. ``hours 1.5``. Matching is case insensitive.
bugzilla.style
The style file to use when formatting comments.
@@ -276,8 +282,10 @@
# Methods to be implemented by access classes.
#
# 'bugs' is a dict keyed on bug id, where values are a dict holding
- # updates to bug state. Currently no states are recognised, but this
- # will change soon.
+ # updates to bug state. Recognised dict keys are:
+ #
+ # 'hours': Value, float containing work hours to be updated.
+
def filter_real_bug_ids(self, bugs):
'''remove bug IDs that do not exist in Bugzilla from bugs.'''
pass
@@ -449,6 +457,9 @@
Try adding comment as committer of changeset, otherwise as
default bugzilla user.'''
+ if len(newstate) > 0:
+ self.ui.warn(_("Bugzilla/MySQL cannot update bug state\n"))
+
(user, userid) = self.get_bugzilla_user(committer)
now = time.strftime('%Y-%m-%d %H:%M:%S')
self.run('''insert into longdescs
@@ -611,6 +622,8 @@
def updatebug(self, bugid, newstate, text, committer):
args = dict(id=bugid, comment=text)
+ if 'hours' in newstate:
+ args['work_time'] = newstate['hours']
self.bzproxy.Bug.add_comment(args)
class bzxmlrpcemail(bzxmlrpc):
@@ -618,8 +631,11 @@
Advantages of sending updates via email:
1. Comments can be added as any user, not just logged in user.
- 2. Bug statuses and other fields not accessible via XMLRPC can
- be updated. This is not currently used.
+ 2. Bug statuses or other fields not accessible via XMLRPC can
+ potentially be updated.
+
+ Currently all status updates recognised can be done via XMLRPC, so
+ item 1 is the only actual advantage.
"""
# The email interface changes subtly between 3.4 and 3.6. In 3.4,
@@ -680,7 +696,10 @@
sendmail(user, bzemail, msg.as_string())
def updatebug(self, bugid, newstate, text, committer):
- self.send_bug_modify_email(bugid, [], text, committer)
+ cmds = []
+ if 'hours' in newstate:
+ cmds.append(self.makecommandline("work_time", newstate['hours']))
+ self.send_bug_modify_email(bugid, cmds, text, committer)
class bugzilla(object):
# supported versions of bugzilla. different versions have
@@ -694,7 +713,8 @@
}
_default_bug_re = (r'bugs?\s*,?\s*(?:#|nos?\.?|num(?:ber)?s?)?\s*'
- r'((?:\d+\s*(?:,?\s*(?:and)?)?\s*)+)')
+ r'(?P<ids>(?:\d+\s*(?:,?\s*(?:and)?)?\s*)+)'
+ r'\.?\s*(?:h(?:ours?)?\s*(?P<hours>\d*(?:\.\d+)?))?')
_bz = None
@@ -735,16 +755,32 @@
re.IGNORECASE)
bugzilla._split_re = re.compile(r'\D+')
start = 0
+ hours = 0.0
bugs = {}
while True:
+ bugattribs = {}
m = bugzilla._bug_re.search(ctx.description(), start)
if not m:
break
start = m.end()
- for id in bugzilla._split_re.split(m.group(1)):
+ try:
+ ids = m.group('ids')
+ except IndexError:
+ ids = m.group(1)
+ try:
+ hours = float(m.group('hours'))
+ bugattribs['hours'] = hours
+ except IndexError:
+ pass
+ except TypeError:
+ pass
+ except ValueError:
+ self.ui.status(_("%s: invalid hours\n") % m.group('hours'))
+
+ for id in bugzilla._split_re.split(ids):
if not id:
continue
- bugs[int(id)] = {}
+ bugs[int(id)] = bugattribs
if bugs:
self.filter_real_bug_ids(bugs)
if bugs:
More information about the Mercurial-devel
mailing list