While I was adding a date to an order by rules I found that the function :entity_property_verbatim_date_get() was not handeling the $data string right, it expects to be an array, and in my case it is an object.

So I changed that, and now it is working like I expected. Not sure if this is a bug or that it works as designed. But if it's a bug here's a patch.

CommentFileSizeAuthor
#4 entity.entity-property-wrapper.1697654-4.patch851 bytesdasjo
#2 date_entity_array_to_object.patch642 bytesrbosscher
date_entity_array_to_object.patch884 bytesrbosscher

Comments

Status: Needs review » Needs work

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

rbosscher’s picture

rbosscher commented
StatusFileSize
new date_entity_array_to_object.patch642 bytes

Oeps, sorry, wrong file locations.
Another attempt...

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented
Title: Date values from rules not properly converted » entity_property_verbatim_date_get should work on objects instead of arrays
Status: Needs work » Needs review
dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented
Title: entity_property_verbatim_date_get should work on objects instead of arrays » entity_property_verbatim_date_get should work both on objects and arrays
StatusFileSize
new entity.entity-property-wrapper.1697654-4.patch851 bytes

the attached patch allows for both objects and arrays to be used with entity_property_verbatim_date_get

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Fixed

Yep, we need to support both. Committed, thanks.

Status: Fixed » Closed (fixed)

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