Bug 27894

Summary: bugzilla-tool: Bugzilla.clear_attachment_review_flag() fails with ClientForm-0.2.10 and python-2.5
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, levin
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Patch v1 eric: review+

Description David Kilzer (:ddkilzer) 2009-07-31 13:31:05 PDT
$ ./WebKitTools/Scripts/bugzilla-tool land-patches --no-clean --no-build --no-test 27849
Fetching: https://bugs.webkit.org/show_bug.cgi?id=27849&ctype=xml
Landing 1 patch from 1 bug.
Updating working directory
Current branch master is up to date.
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
101  1215  101  1215    0     0   1578      0 --:--:-- --:--:-- --:--:--  2525
patching file WebKitTools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file WebKitTools/Scripts/bugzilla-tool
Parsing ChangeLog: WebKitTools/ChangeLog
Reading Keychain for bugs.webkit.org account and password.  Click "Allow" to continue...
Logging in as ddkilzer@webkit.org...
Clearing review flag on attachment: 33837

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/bugzilla-tool
Committed r46637
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/bugzilla-tool
r46637 = 20f0cf4533b301428fa09b392316943bedee805a (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/46637
Traceback (most recent call last):
  File "./WebKitTools/Scripts/bugzilla-tool", line 668, in <module>
    main()
  File "./WebKitTools/Scripts/bugzilla-tool", line 665, in main
    return tool.main()
  File "./WebKitTools/Scripts/bugzilla-tool", line 660, in main
    return command_object.execute(command_options, command_args, self)
  File "./WebKitTools/Scripts/bugzilla-tool", line 349, in execute
    self.land_patches(bug_id, bugs_to_patches[bug_id], options, tool)
  File "./WebKitTools/Scripts/bugzilla-tool", line 323, in land_patches
    tool.bugs.clear_attachment_review_flag(patch['id'], comment_text)
  File "/Volumes/Data/WebKit.git/WebKitTools/Scripts/modules/bugzilla.py", line 366, in clear_attachment_review_flag
    self.browser['comment'] = comment_text
  File "/Library/Python/2.5/site-packages/ClientForm-0.2.10-py2.5.egg/ClientForm.py", line 2895, in __setitem__
  File "/Library/Python/2.5/site-packages/ClientForm-0.2.10-py2.5.egg/ClientForm.py", line 3222, in find_control
  File "/Library/Python/2.5/site-packages/ClientForm-0.2.10-py2.5.egg/ClientForm.py", line 3304, in _find_control
ClientForm.AmbiguityError: more than one control matching name 'comment'
Comment 1 David Kilzer (:ddkilzer) 2009-07-31 13:45:09 PDT
Created attachment 33901 [details]
Patch v1
Comment 2 Eric Seidel (no email) 2009-07-31 13:50:09 PDT
Comment on attachment 33901 [details]
Patch v1

LGTM.  Are there other places we need to worry about?  We have several places we set the comment...  r=me to change the rest too.
Comment 3 Eric Seidel (no email) 2009-07-31 13:51:06 PDT
This was actually caused by bugzilla 3.0 iirc.  I swear I fixed one of these before. :)
Comment 4 David Kilzer (:ddkilzer) 2009-07-31 13:56:23 PDT
(In reply to comment #2)
> (From update of attachment 33901 [details])
> LGTM.  Are there other places we need to worry about?  We have several places
> we set the comment...  r=me to change the rest too.

These were the only two on the attachments.cgi?action=edit pages that I saw.
Comment 5 Eric Seidel (no email) 2009-07-31 14:00:57 PDT
Comment on attachment 33901 [details]
Patch v1

I only saw one in your change, but I assume I must have fixed the other one before (or someone did) and forgot to fix this one! eep! :)  Thanks again.
Comment 6 David Kilzer (:ddkilzer) 2009-07-31 14:03:53 PDT
$ git svn dcommit
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/modules/bugzilla.py
Committed r46640

http://trac.webkit.org/changeset/46640