Bug #59795: Publishing throws exception - TYPO3.Neos - TYPO3 Forge

Base Distribution TYPO3.Form TYPO3.Imagine TYPO3.Media TYPO3.Neos TYPO3.Neos.NodeTypes TYPO3.Party TYPO3.Setup TYPO3.SiteKickstarter TYPO3.Twitter.Bootstrap TYPO3.TYPO3CR TYPO3.TypoScript Coordination Group Website and Communication

Bug #59795

Publishing throws exception

Added by Carsten Bleicker about 1 year ago. Updated 10 months ago.

Status:

Closed

Start date:

2014-06-22

Priority:

Should have

Due date:

Assigned To:

% Done:

Category:

Target version:

Description

Simply delete a Textelement and hit Publish (NOT Publish All).
See: https://www.dropbox.com/s/swkisnmqp4f815y/CommunicationErrorOnPublish.mov
See: Exception attached.

20140622111815a21038.txt (4.3 kB) Carsten Bleicker, 2014-06-22 13:23

History

#1 Updated by Carsten Bleicker about 1 year ago

As far as i know this should be mapped into an array holding mapped nodes, right?
@param array<\TYPO3\TYPO3CR\Domain\Model\NodeInterface> $nodes
Packages/Application/TYPO3.Neos/Classes/TYPO3/Neos/Service/Controller/WorkspaceController.php:98

But its holding an array with the contextPath.

#2 Updated by Carsten Bleicker about 1 year ago

Tried to use
@param ArrayCollection<\TYPO3\TYPO3CR\Domain\Model\NodeInterface> $nodes

With this the publishing works.
Problem is, that the \TYPO3\TYPO3CR\Service\PublishingServiceInterface::publishNodes expects an array and not a collection.
Its annotated as @api wich means, changing this will be a breaking change, i think.

So what should i do? Patching all the interface stuff to collection or
changing the array converter to reflect the target type of <\TYPO3\TYPO3CR\Domain\Model\NodeInterface>?

#3 Updated by Carsten Bleicker about 1 year ago

See my comment on:
[[https://review.typo3.org/#/c/19567/]]

#4 Updated by Christopher Hlubek about 1 year ago

We have to check if this only occurs if content dimensions are enabled, I could not yet reproduce it with the default demo site (without content dimensions). Will try to do that soon.