Preview stopped working for "content" about cockpit

[aheinze]

@pozylon Thanks, unfortunately I still can't reproduce it. Do you have any custom code that hooks into the content model save event maybe?

[Mikearaya]

additional info: this happens when setting a preview page URL for more than one collection/content

[aheinze]

when does the error occur?

[Mikearaya]

@aheinze when setting a preview URL for a collection. when it's set for only one collection it is fine but if you set the preview endpoint for more than two collections there is a problem

[aheinze]

when you create a new collection or update an existing one?

[aheinze]

Is it possible to create a video, so I can understand when and where the error happens?

[Mikearaya]

update existing one, with a preview url

[aheinze]

nope, can't reproduce

[Mikearaya]

@pozylon and I were able to identify the root cause. it is related to the behavior of array_merge

If the input arrays have the same string keys, then the later value for that key will overwrite the previous one. If, however, the arrays contain numeric keys, the later value will not overwrite the original value but will be appended.

basically, if you have multiple preview endpoints for different collections with the same key it will break the code

so while we can work around this issue for now I think it should not be implemented like that.
it is not like this on older versions

[aheinze]

basically, if you have multiple preview endpoints for different collections with the same key it will break the code

I still don't get it. Can you provide more details or context?

[pozylon]

basically, if you have multiple preview endpoints for different collections with the same key it will break the code

I still don't get it. Can you provide more details or context?

• I set the preview endpoint for collection a and name it "test" with some url
• I set the preview endpoint for collection b and name it "test" with same url as above
• I go to content overview page, hard refresh -> collections don't show up anymore (only way to fix is manually editing the collection content php files)

If i use test-a and test-b as keys it works

[aheinze]

Let me know if I can support you to investigate the issue, as I want to release a new version soon and want to make sure that this issue won't get shipped with the next version

[pozylon]

Let me know if I can support you to investigate the issue, as I want to release a new version soon and want to make sure that this issue won't get shipped with the next version

We are trying to reproduce the issue locally but without success yet. It's weird because we use the cockpit pro docker image. Trying to spot the difference...