MozReview.attachments() doesn't create flags on new attachments

RESOLVED FIXED

Status

()

bugzilla.mozilla.org
Extensions: MozReview Integration
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mcote, Assigned: dylan)

Tracking

Production

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

2 years ago
I can successfully update an attachment and set/clear flags, but the flags are not being set when I create a new attachment.

Example:

I'm sending this:

{'bug_id': 1, 'api_key': u'XXXX', 'attachments': [{'comment': u'Review commit: http://172.16.130.136:64784/r/22/diff/#index_header\nSee other reviews: http://172.16.130.136:64784/r/22/', 'file_name': 'reviewboard-22-url.txt', 'summary': u'MozReview Request: Bug 1 - Add foo. r?jrandom', 'flags': [{'status': '?', 'requestee': u'jrandom@example.org', 'name': 'review', 'new': True}], 'content_type': 'text/x-review-board-request', 'data': u'http://172.16.130.136:64784/r/22/diff/#index_header'}]}

And receiving this:

{'attachments_created': {'12': {'description': 'MozReview Request: Bug 1 - Add foo. r?jrandom', 'creator': 'admin@example.com', 'is_obsolete': 0, 'is_patch': 0, 'creation_time': <DateTime '20160304T01:43:56' at 7fa31fae7b48>, 'summary': 'MozReview Request: Bug 1 - Add foo. r?jrandom', 'bug_id': 1, 'flags': [], 'last_change_time': <DateTime '20160304T01:43:56' at 7fa31fae7bd8>, 'content_type': 'text/x-review-board-request', 'file_name': 'reviewboard-22-url.txt', 'attacher': 'admin@example.com', 'data': <xmlrpclib.Binary instance at 0x7fa31fae7b00>, 'id': 12, 'is_private': 0, 'size': 51}}, 'attachments_modified': {}}

Note that the response has an empty value for 'flags'.
(Assignee)

Comment 1

2 years ago
Created attachment 8726528 [details] [diff] [review]
1253483_1.patch
Attachment #8726528 - Flags: review?(dkl)
Comment on attachment 8726528 [details] [diff] [review]
1253483_1.patch

Review of attachment 8726528 [details] [diff] [review]:
-----------------------------------------------------------------

Testing of adding attachments and flags worked fine. When updating the flag status of the attachment I just created I got the following error:

Can't call method "value" without a package or object reference at /loader/0x3c73b60/Bugzilla/Extension/MozReview/WebService.pm line 181.
 at /loader/0x3c73b60/Bugzilla/Extension/MozReview/WebService.pm line 181.

Changing extensions/MozReview/lib/WebService.pm line 101 fixed the problem for me. Take a look. Not sure how I would have missed it in the earlier review.

    my %attachments_modified = map { (ref $_->{id} ? $_->{id}->value : $_->{id}) => $_ } @modified;

::: Bugzilla/WebService/Util.pm
@@ -57,4 @@
>      # Copy the user provided $flags as we may call extract_flags more than
>      # once when editing multiple bugs or attachments.
>      my $flags_copy = dclone($flags);
> -

nit: unnecessary change for this patch

::: extensions/MozReview/lib/WebService.pm
@@ +155,4 @@
>                  isprivate   => $attachment->{is_private},
>              });
>  
> +            

nit: remove extra newline (whitespace)
Attachment #8726528 - Flags: review?(dkl) → review-
(Assignee)

Comment 3

2 years ago
Created attachment 8727843 [details] [diff] [review]
1253483_2.patch
Attachment #8726528 - Attachment is obsolete: true
Attachment #8727843 - Flags: review?(dkl)
Comment on attachment 8727843 [details] [diff] [review]
1253483_2.patch

Review of attachment 8727843 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #8727843 - Flags: review?(dkl) → review+
(Assignee)

Comment 5

2 years ago
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   30143b3..6ec9ecf  master -> master
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.