There is no clean way to initialise the \Drupal\time_field\Time object straight from a \DateTime object. From a DX perspective it would be nice to introduce a new constructor createFromDateTime to initialise the time object from a date time.

CommentFileSizeAuthor
#2 3169679-2-create_time_from_date_time.patch1.6 KBErik Frèrejean
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Erik Frèrejean created an issue. See original summary.

Erik Frèrejean’s picture

Erik Frèrejean
Primary language Dutch
Location 🏴‍☠️
CreditAttribution: Erik Frèrejean at Dutch Open Projects commented
Assigned: Erik Frèrejean » Unassigned
Status: Active » Needs review
FileSize
3169679-2-create_time_from_date_time.patch1.6 KB
liesm’s picture

liesm CreditAttribution: liesm as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Seems like an usable change.

  • Erik Frèrejean authored cab40e3 on 2.x 
    Issue #3169679 by Erik Frèrejean, liesm: Create time from date time
BramDriesen’s picture

BramDriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
CreditAttribution: BramDriesen as a volunteer and at Sopra Steria commented
Status: Reviewed & tested by the community » Fixed

Thank you all!

BramDriesen’s picture

BramDriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
CreditAttribution: BramDriesen as a volunteer and at Sopra Steria commented
BramDriesen’s picture

BramDriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
CreditAttribution: BramDriesen as a volunteer and at Sopra Steria commented
Version: 8.x-1.x-dev » 2.x-dev

Status: Fixed » Closed (fixed)

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