[regression] datetime_timestamp widget does not use default field value

Thank you for reporting.

Moving to 11.x as the current development branch, MR should be updated also.

This will also require test coverage.

acbramley → changed the visibility of the branch temp to hidden.

acbramley → changed the visibility of the branch 3414883-datetimetimestamp-widget-does to hidden.

Recreated the MR against 11.x, please close your one @paul121

Apologies @paul121 I just saw you were fixing that branch at the same time.

paul121 → closed merge request !6185

No worries, I'll close that one. I won't have time to work on tests anytime soon.

Attaching a patch we can use in farmOS.

I'll work on some tests

Added test coverage, fails with the following without the fix:

The field "field_timestamp[0][value][date]" value is "", but "2024-01-16" expected.

As a regression this seemed higher to me.

Running the tests get the same as @acbramley does in #13

Was able to confirm the issue and that the MR fixes it.

Again as a regression believe it's good to mark now vs waiting a week.

As a regression this seemed higher to me.

Can you elaborate? Higher than what?

Again as a regression believe it's good to mark now vs waiting a week.
I do not understand this. What happens in a week?

---
Thanks for the complete issue summary!

I compared the change here to the commit for the other issue and confirmed that it doing as stating in the proposed resolution.

I tested and confirmed the problem. Then, I applied the diff and ran the test, which does fail without the fix. The test could be added to \Drupal\FunctionalTests\Datetime\TimestampTest::testWidget() and would not interfere with that test. I am in two minds about that but will leave that for now.

Leaving at RTBC

I agree with this. If we add assertions to the test method, we save an entire Drupal install, which can make a meaningful difference to gitlab ci runtimes, so when it's easy to do that, it's worth doing.

Went ahead and made the change locally, ran the test, and pushed. Assuming the whole pipeline comes green I'll go ahead and commit this, since it's only deleting three lines or so.

Committed/pushed to 11.x and cherry-picked to 10.3.x and 10.2.x, thanks!

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