When you try to update a node, even one created by mail-handler itself, i get this error.

"This content has been modified by another user, changes cannot be saved."

CommentFileSizeAuthor
#5 mailhandler-update-error.patch789 bytesz.stolar
#4 mailhandler-448592-update-node.patch995 bytesa_c_m
#2 mailhandler-448592-update-node.patch1.65 KBa_c_m

Comments

z.stolar’s picture

z.stolar commented
Status: Active » Postponed (maintainer needs more info)

I wasn't able to reproduce this bug.
Can you supply more details for recreating the bug?

a_c_m’s picture

a_c_m commented
Status: Postponed (maintainer needs more info) » Active
StatusFileSize
new mailhandler-448592-update-node.patch1.65 KB

i'm in #drupal now if you want to talk.

The bug is quite simple to create.

Create a node using mailhandler, go to the site, note its NID (e.g. 123), then send an update email e.g.

*Subject*
Updated Title

*Body*
nid: 123
pass: mypass

When it tries to process this, it will correctly try to update the node - but will fail with the above message.

The attached patch fixes it.

z.stolar’s picture

z.stolar commented
Status: Active » Needs work

I'll test and commit quickly, but you need to clean your patch. It includes debugging in it (BTW, I suggest devel module for debugging)

a_c_m’s picture

a_c_m commented
StatusFileSize
new mailhandler-448592-update-node.patch995 bytes

better?

z.stolar’s picture

z.stolar commented
Status: Needs work » Needs review
StatusFileSize
new mailhandler-update-error.patch789 bytes

Please test the attached patch, for updated and new content.

a_c_m’s picture

a_c_m commented
Status: Needs review » Reviewed & tested by the community

Works for me, there is a minor typo in the patch "teh" instead of "the" - but other than that, perfect!

z.stolar’s picture

z.stolar commented
Status: Reviewed & tested by the community » Fixed

commited

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.