Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #3445775: validateEntityFields is not correctly transforming date #3892

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zanvidmar
Copy link
Contributor

Problem

validateEntityFields() is not correctly transforming date field values for datetime type "date" fields (it works as designed for "datetime" type only). To fix the issue correct date format is applied for date only fields.

Solution

Issue tracker

3445775 (PROD-29315)

Theme issue tracker

N/A

How to test

Create behat test step, that uses validateEntityFields method and validate date only field.
Created entity will have empty date field as an result of wrong date format applied.
With this fix applied date field will be correctly transformed.

Screenshots

N/A

Release notes

Improve tests.

Change Record

N/A

Translations

N/A

@zanvidmar zanvidmar added type: bug Fixes a bug in Open Social status: needs review This pull request is waiting for a requested review prio: medium type: test Adds or improves test coverage without other changes team: iata labels May 7, 2024
@zanvidmar zanvidmar added this to the 13.0.0-alpha2 milestone May 7, 2024
@zanvidmar zanvidmar marked this pull request as ready for review May 7, 2024 16:50
@zanvidmar zanvidmar added type: test Adds or improves test coverage without other changes and removed type: test Adds or improves test coverage without other changes type: bug Fixes a bug in Open Social labels May 7, 2024
…te field values.

validateEntityFields() is not correctly transforming date field values for
datetime type "date" fields (it works as designed for "datetime" type only).
To fix the issue correct date format is applied for date only fields.
@zanvidmar zanvidmar force-pushed the issue/PROD-29315-behat-test-date-validation branch from 87b36e8 to 918186f Compare May 9, 2024 06:46
@open-social-tugboat
Copy link

Tugboat has finished building the preview for this pull request!

Link:

Dashboard:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio: medium status: needs review This pull request is waiting for a requested review team: iata type: test Adds or improves test coverage without other changes
3 participants