Currently "revision" is not defined in the entity_property_info definition of the commerce_order. This prevents it to be used in contribs like rules. Besides this the default rules in commerce should perhaps also have the data set "revision" to TRUE to make sure revisions are handled in rules.

In my use case I have the issue that an order is updated to order state pending through rules. But instead of creating a new revision, it overrides the previous revision. Preventing me to see the correct revision flow of an order when supporting my customer.

CommentFileSizeAuthor
#6 2882816-6.revision_property_info.patch1.06 KBrszrama
#2 make_revision_available_in_rules-2882816-1.patch605 bytesjefuri

Comments

jefuri created an issue. See original summary.

jefuri’s picture

jefuri commented
Status: Active » Needs review
StatusFileSize
new make_revision_available_in_rules-2882816-1.patch605 bytes
jefuri’s picture

jefuri commented
Issue summary: View changes
upunkt’s picture

upunkt commented

Thanks a lot. Had the exact problem.

czigor’s picture

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

Simple and awesome, thanks! (It also works.)

rszrama’s picture

rszrama commented
Assigned: jefuri » Unassigned
StatusFileSize
new 2882816-6.revision_property_info.patch1.06 KB

Made a slight tweak to fix the copy / paste of "node" instead of order. : )

  • rszrama committed 14b3f5d on 7.x-1.x authored by jefuri 
    Issue #2882816 by jefuri, rszrama: define the revision property of an...
rszrama’s picture

rszrama commented
Status: Reviewed & tested by the community » Fixed

And committed.

Status: Fixed » Closed (fixed)

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