Skip to content

Commit 7857c79

Browse files
committed
BUGFIX: Don't skip a index job if a node can't be processed
The nodes that are used to create the index jobs are fetched from the database but hidden fields are not respected. The `NodeFactory` will right now return `null` for the configured context if the node is hidden. If that occurs the index job will return true and all other nodes in that job are skipped. This fix will just skip the node that is hidden and log an information about that.
1 parent 5d1baa4 commit 7857c79

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

  • Classes/Flowpack/ElasticSearch/ContentRepositoryQueueIndexer

Classes/Flowpack/ElasticSearch/ContentRepositoryQueueIndexer/IndexingJob.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,8 @@ public function execute(QueueInterface $queue, Message $message) {
9999
]);
100100
$currentNode = $this->nodeFactory->createFromNodeData($nodeData, $context);
101101
if (!$currentNode instanceof NodeInterface) {
102-
return TRUE;
102+
$this->logger->log(sprintf('Node with identifier %s could not be processed', $node['nodeIdentifier']));
103+
continue;
103104
}
104105
$this->nodeIndexer->setIndexNamePostfix($this->indexPostfix);
105106
$this->logger->log(sprintf('Process indexing job for %s', $currentNode));

0 commit comments

Comments
 (0)