The created and updated behaviour code should also define those property types as dates so that you can use the date UI and logic to set and filter dates. For example in rules or views. Currently they are just simple integers.

CommentFileSizeAuthor
#1 eck-property-type-for-dates-1512288-1.patch3.08 KBacrazyanimal

Comments

acrazyanimal’s picture

acrazyanimal commented
Status: Active » Needs review
StatusFileSize
new eck-property-type-for-dates-1512288-1.patch3.08 KB

When creating and testing the patch for this I also noted two other issues that I've included in the patch.

  1. The eck_property_behavior_getter(..) function was calling eck_property_behavior_implements on 'setter' so I changed it to 'getter'.
  2. In the eck_property_behavior_invoke_plugin_alter function's loop through the entities properties each successive behavior call would overwrite the last since it was using the original data rather then the updated 'return' data.

cheers!

acrazyanimal’s picture

acrazyanimal commented

Anyway, with the above patch the created/updated properties are now properly defined as type 'date' so the fields get formatted properly in views, rules, etc.

that0n3guy’s picture

that0n3guy commented

This seems to work for me. I had to remove my field from my view, then re-add it. It works correctly now.

fmizzell’s picture

fmizzell commented
Status: Needs review » Fixed

path applied :)

Status: Fixed » Closed (fixed)

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

  • Commit 6b67ced on 7.x-2.x, change_paths, own_permissions, 7.x-2.x-property-widgets, 7.x-3.x, entity_reference, 7.x-3.x-settings, 7.x-2.0.x, 7.x-2.1.x authored by acrazyanimal, committed by fmizzell: 
    #1512288 Date properties should be defined as date types

  • Commit 6b67ced on 7.x-2.x, change_paths, own_permissions, 7.x-2.x-property-widgets, 7.x-3.x, entity_reference, 7.x-3.x-settings, 7.x-2.0.x, 7.x-2.1.x authored by acrazyanimal, committed by fmizzell: 
    #1512288 Date properties should be defined as date types

  • Commit 6b67ced on 7.x-2.x, 7.x-3.x, 8.x authored by acrazyanimal, committed by fmizzell: 
    #1512288 Date properties should be defined as date types

  • Commit 6b67ced on 7.x-2.x, 7.x-3.x, 8.x authored by acrazyanimal, committed by fmizzell: 
    #1512288 Date properties should be defined as date types