333040 - Cannot search based on "Attachment Creator"

Status: RESOLVED FIXED

(Bugzilla :: Query/Bug List, enhancement)

Description

[Frédéric Buclin]

I would like to be able to use boolean charts to look for bugs where:
"Attachment creator" "is" [e-mail]

Comment 1

[Frédéric Buclin]

Attachment: patch, v1

It's the first time I hack Search.pm, so don't trust this patch too much. :)

Comment 2

[Frédéric Buclin]

Attachment: patch, v1.1

For consistency with the UI of the attachment table, let's call it "attachment creator" instead of "attachment submitter".

Comment 3

[Frédéric Buclin]

joel, did you have time to test this patch?

Comment 4

[Joel Peshkin]

I did not, but I did have time to inspect it and the logic is correct.  So, if someone wants to r+ it based on coding standards and testing, they can be reassured by my insepction results.

Comment 5

[Joel Peshkin]

Of course, you could always do this....  
https://bugzilla.mozilla.org/query.cgi?query_format=advanced&short_desc_type=allwordssubstr&short_desc=&product=Bugzilla&long_desc_type=substring&long_desc=&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&resolution=DUPLICATE&resolution=---&emailassigned_to1=1&emailtype1=exact&email1=&emailassigned_to2=1&emailreporter2=1&emailqa_contact2=1&emailtype2=exact&email2=&bugidtype=include&bug_id=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=&field0-0-0=attachments.thedata&type0-0-0=changedby&value0-0-0=LpSolit%40gmail.com&field0-1-0=bug_id&type0-1-0=greaterthan&value0-1-0=330000

Which works today :-)

Comment 6

[Frédéric Buclin]

But this query doesn't only find attachments I posted, but also those I changed I guess. ;)

Comment 7

[Joel Peshkin]

(In reply to comment #6)
> But this query doesn't only find attachments I posted, but also those I changed
> I guess. ;)
> 

True, but you can't edit the attachment data after posting it.

Comment 8

[Max Kanat-Alexander]

Comment on attachment 237509 [details] [diff] [review]
patch, v1.1

Looks good, works fine!

Comment 9

[Frédéric Buclin]

Checking in Bugzilla/Field.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Field.pm,v  <--  Field.pm
new revision: 1.22; previous revision: 1.21
done
Checking in Bugzilla/Search.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.141; previous revision: 1.140
done