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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BenK’s picture

BenK CreditAttribution: BenK commented

Subscribing

BenK’s picture

BenK CreditAttribution: BenK commented

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

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir 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
CreditAttribution: Berdir commented
Status: Needs work » Needs review
FileSize
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
CreditAttribution: Berdir commented
Status: Needs work » Needs review

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

BenK’s picture

BenK CreditAttribution: 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 CreditAttribution: BenK commented

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

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir 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...