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 Reload content in raw content mode is broken #5402

Open
thetechnick opened this issue Dec 13, 2024 · 5 comments
Open

BUG Reload content in raw content mode is broken #5402

thetechnick opened this issue Dec 13, 2024 · 5 comments

Comments

@thetechnick
Copy link

Hey Neos team,
I am checking out Neos for a project and am trying to understand the demo/example site.

Seems like going through the docker flow:
https://docs.neos.io/guide/installation-development-setup/docker-and-docker-compose-setup

Will result in a demo page where you cannot create new Text nodes directly within the Main content area, although they are used in a few places without being contained within a column content wrapper.

Trying to create a new Text node within a column content will result in an error:

Neos\Neos\Fusion\Helper\ArrayHelper::sortByPropertyPath(): Argument neos/Neos.Demo#1 ($set) must be of type array, null given - Check the logs for details

The container logs just show accepted connections, so I am not quite sure where Neos is logging to for the detailed error description.

@thetechnick
Copy link
Author

Found the logs :)
neos-exception.txt

@mhsdesign
Copy link
Member

Thanks for attaching the log. It seems youre using the newer-ish raw content mode of Neos in the Ui rather than the 'normal' edit mode.

And this line:

[email protected] = ${Neos.Array.sortByPropertyPath(value, 'ui.inspector.position')}

Causes an error as node.nodeType.properties seems to be NULL.
I had this error on my todo stash - at least for the upcoming Neos version 9 which is still in beta.

Or are you using Neos 8 and its also a problem there?

@thetechnick
Copy link
Author

I am a total Neos Noob, I just followed the docker example and ended up with: UI version: 8.3.11
Checking out Neos 9 is next on the todo list.
Good to hear a fix is already on it's way!

(Also: dang you guys are fast responding - here and in slack)

@dlubitz
Copy link
Contributor

dlubitz commented Dec 13, 2024

I had a look into this and it is related somehow to OutOfBandRendering. A full page reload works fine.

For some reason, during OutOfBandRendering there is no node in the context, which causes issues in different places.

  1. [email protected] = ${Neos.Array.sortByPropertyPath(value, 'ui.inspector.position')}

Tested with Neos 8.3 latest dev

@mhsdesign mhsdesign transferred this issue from neos/Neos.Demo Dec 13, 2024
@mhsdesign mhsdesign changed the title Text nodes broken. BUG Reload content in raw content mode is broken Dec 13, 2024
@mhsdesign
Copy link
Member

Thanks @dlubitz in that case there are two bugs :D one for Neos 8.3 (that i didnt know) and another for 9.0 :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants