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

Inconsistent conversion in CreationDialogPropertiesCreationHandler #3719

Closed
kdambekalns opened this issue Feb 16, 2024 · 0 comments · Fixed by #3720
Closed

Inconsistent conversion in CreationDialogPropertiesCreationHandler #3719

kdambekalns opened this issue Feb 16, 2024 · 0 comments · Fixed by #3720
Assignees
Labels
8.3 Backend / Backend-API Bug Label to mark the change as bugfix

Comments

@kdambekalns
Copy link
Member

Description

The CreationDialogPropertiesCreationHandler does property value conversion from "raw" to "typed" on it's own, and in a different way compared to the NodeConverter from the Content Repository or even the NodePropertyConversionService from the UI itself.

Steps to Reproduce

  1. Create a nodetype with a datetime property
  2. Create a node using the creation dialog
  3. Create such a node without the creation dialog
  4. Compare the stored dattime values in the database

Expected behavior

The same timezone handling is applied, dates are stored in the server's default timezone.

Actual behavior

Dates are stored in the users' timezone, as sent by the browser.

Affected Versions

UI: 8.3 (and earlier)

@kdambekalns kdambekalns self-assigned this Feb 16, 2024
mhsdesign added a commit that referenced this issue Feb 17, 2024
mhsdesign added a commit that referenced this issue Feb 17, 2024
mhsdesign added a commit that referenced this issue Feb 17, 2024
mhsdesign added a commit that referenced this issue Feb 28, 2024
@grebaldi grebaldi added Bug Label to mark the change as bugfix Backend / Backend-API 8.3 labels Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.3 Backend / Backend-API Bug Label to mark the change as bugfix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants