353994 - [email_in] Make the email interface able to update (process) bugs

Status: RESOLVED FIXED

(Bugzilla :: Incoming Email, enhancement)

Description

[Max Kanat-Alexander]

Okay, this is the next step of the inbound email interface--it should have the ability to update bugs, similar to process_bug.cgi.

Comment 1

[Max Kanat-Alexander]

Attachment: v1

Okay. This requires the other email_in patch, obviously.

I have tested this pretty well, and it works. The changes to process_bug are basically to avoid it ever having to output a template.

We don't have to worry about midairs, because it's impossible to midair with the email interface.

Comment 2

[Colin Ogilvie [:cso]]

Comment on attachment 239827 [details] [diff] [review]
v1

Looks good to me, tested it and it seems to work.

Again, I think at least reporting errors to the user about what has/has not worked would be nice though:

I tried @shortdesc but it's actually @short_desc and obviously nothing happened but I got no errors back again.

Locally I am getting the following:
DBI::db=HASH(0x9dcefd8)->disconnect invalidates 1 active statement handle (either destroy statement handles or call finish on them before disconnecting) at Bugzilla.pm line 392, <STDIN> line 28.

Note the DBI::db error at the bottom as seen in another bug (Bug 349210). This only occured after applying this patch, the original is fine :)

Requesting r? from ghendricks again.

Comment 3

[Greg Hendricks]

Comment on attachment 239827 [details] [diff] [review]
v1

It would be good if we include an example email message for both posting and updating a bug in the docs.

Comment 4

[Max Kanat-Alexander]

As far as the DBI::db error message goes, that's definitely a bug and we should look into it. It's probably a bug with this patch.

I agree, it would be nice to return some information about what was and wasn't processed--could you file a bug for that, Colin, and CC me?

I also agree that it would be nice to have another example there, for process_bug. ghendricks--could you file that and CC me?

Checking in email_in.pl;
/cvsroot/mozilla/webtools/bugzilla/email_in.pl,v  <--  email_in.pl
new revision: 1.2; previous revision: 1.1
done
Checking in process_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/process_bug.cgi,v  <--  process_bug.cgi
new revision: 1.344; previous revision: 1.343
done