- Issue created by @jsacksick
- Merge request !478Issue #3536290: Order::getPlacedTime() should cast the return value to an int. → (Merged) created by jsacksick
- 🇳🇴Norway zaporylie
Let's target both getPlacedTime and getCompletedTime. Not sure about setting the return type on the interface though - I am all for having it, but this will break things for those who override/decorate the entity class
- 🇮🇱Israel jsacksick
Actually, more in favor of increasing the scope and doing this once for all... This change is long overdue and should have been part of 3.0.0
-
jsacksick →
committed fa73d2ae on 3.x
Issue #3536290 by jsacksick, zaporylie: Make sure getters for timestamp...
-
jsacksick →
committed fa73d2ae on 3.x
- 🇮🇱Israel jsacksick
Ok the change shouldn't be disruptive as I finally skipped return types, will go ahead and merge this.
-
jsacksick →
committed fd969106 on 3.x
Issue #3536290 followup by jsacksick: Cast more timestamp fields and...
-
jsacksick →
committed fd969106 on 3.x
- ivnish Kazakhstan
Is it fixing my issue https://www.drupal.org/project/commerce/issues/3485574 🐛 Placed time return string but should be int Active ?