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

BUG: Updating node properties of type reference or references can't be published #4151

Closed
1 task done
ahaeslich opened this issue Mar 31, 2023 · 6 comments
Closed
1 task done
Assignees

Comments

@ahaeslich
Copy link
Member

ahaeslich commented Mar 31, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Changing the value of a node property of type reference or references will update the value in the user workspace but the publish button will not be "activated". The Workspace module will also not show any changes on the updated node. Therefor the change will be shown in the user workspace but cannot be published on it's own.

Changing e.g. a text node property afterwards and publishing all changes will also publish the changes on the reference node property.

Expected Behavior

Changing the value of a node property of type reference or references should lead to an "active" publish button.

Steps To Reproduce

  1. Define a node property of type reference or references
  2. Edit the property in the inspector panel and apply the changes.

Example YAML

MyFancy.Site:Document.BugHunter:
  ui:
    inspector:
      groups:
        debug:
          label: 'Debugging'

  properties:
    textProp:
      type: string
      ui:
        label: 'Text prop'
        inspector:
          group: 'debug'

    referencesProp:
      type: references
      ui:
        label: 'References prop'
        reloadPageIfChanged: true
        inspector:
          group: 'debug'
          editorOptions:
            nodeTypes: ['Neos.Neos:Document']

In comparison with changing e.g. a node property of type string, the payload of the API call for references to the backend contains an empty array on feedbacks.0.payload.publishableNodes. Additionally looking into the database the table cr_default_p_neos_change will only indicate the changes on a text node property.

Environment

- Flow: 8.3
- Neos: 9.0
- PHP: 8.1

Anything else?

I'm not so sure if this should be fixed here or in https://github.com/neos/neos-ui

@skurfuerst skurfuerst self-assigned this Apr 2, 2023
@skurfuerst
Copy link
Member

I will take care of this :)

@bwaidelich
Copy link
Member

The same is true when the node type was changed.
I assume that there are just some when* cases missing from the \Neos\Neos\PendingChangesProjection\ChangeProjection

@ahaeslich
Copy link
Member Author

Not quite sure if this is related: if you change a date property in the inspector the publish butten will be activated but the value in the inspector field will be reset to the published value instead of the workspace value. And the backend preview will show the new date.

image

PS: Tested on the current v9 branch.

@ahaeslich
Copy link
Member Author

Moved it to "In Progress" again as we mentioned other cases here. I don't want them to go missing.

@dlubitz
Copy link
Contributor

dlubitz commented Apr 14, 2023

The same is true when the node type was changed.

I put that into a separate issue:
#4191

@dlubitz
Copy link
Contributor

dlubitz commented Apr 14, 2023

I also moved the datetime issue into a separate issue:
#4192

So we can use this ticket for the reference issues only? @ahaeslich

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

4 participants