A user property status info was introduced here #1031536: Add default Rules to accomplish anonymous checkout and e-mail notification to help getting the anonymous checkout email notification but this property has been already added in Entity API module in #1044332: Define the status property for the core user and comment entity so we'd need to remove our definition.

Patch attached.

CommentFileSizeAuthor
#3 1348156-remove_user_property_checkout-3.patch1.29 KBpcambra
commerce-remove_user_property_checkout-0.patch917 bytespcambra

Comments

Status: Needs review » Needs work

The last submitted patch, commerce-remove_user_property_checkout-0.patch, failed testing.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

This patch fails because testbot is using entity 7.x-1.0-beta9 (?) which doesn't include the property, but RC1 does, so it should be safe.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs work » Needs review
StatusFileSize
new 1348156-remove_user_property_checkout-3.patch1.29 KB

Let's force Entity API version

Status: Needs review » Needs work

The last submitted patch, 1348156-remove_user_property_checkout-3.patch, failed testing.

damien tournoud’s picture

damien tournoud commented
Status: Needs work » Needs review

#3: 1348156-remove_user_property_checkout-3.patch queued for re-testing.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Fixed

Merged into 7.x-1.x.

Status: Fixed » Closed (fixed)

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