See #996842: Provide framework to allow modules to update their transaction (D7).

This patch claims insert/delete/ownership gain/loss transactions so that they show nice descriptions and so on..

CommentFileSizeAuthor
#4 userpoints_nc_convert.patch1.68 KBberdir
userpoints_nc_convert.patch1.68 KBberdir

Comments

BenK’s picture

BenK commented

Subscribing

BenK’s picture

BenK commented

Does this need to be re-rolled or should I test as is?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Needs work

There seems to be no patch, strange.

Will upload a new one tomorrow.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs work » Needs review
StatusFileSize
new userpoints_nc_convert.patch1.68 KB

Status: Needs review » Needs work

The last submitted patch, userpoints_nc_convert.patch, failed testing.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs work » Needs review

#4: userpoints_nc_convert.patch queued for re-testing.

BenK’s picture

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

I tested this with legacy data and it seemed to work great!

--Ben

BenK’s picture

BenK commented

I guess that this is ready to be committed! ;-)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Postponed

No point in committing it before the base issue that provides the modul to handle the claiming is done. Setting to postponed...