diff --git a/src/CheckerQueriesManager.php b/src/CheckerQueriesManager.php index 82721f9..8198a3c 100644 --- a/src/CheckerQueriesManager.php +++ b/src/CheckerQueriesManager.php @@ -98,10 +98,6 @@ public function fieldGetDom($id, $type, $fieldName) { */ public function fieldSave($id, $type, $fieldName, $revisionId, $updatedText) { $updated = 0; - // This is a fix for an edge case where the $type is article. - if ($type === 'article') { - $type = 'node'; - } $entity_storage = $this->entityTypeManager->getStorage($type); // Check the node exists, update the field and save. if ($node = $entity_storage->load($id)) { diff --git a/src/Dennis/Link/Checker/Field.php b/src/Dennis/Link/Checker/Field.php index 0eb0813..e00eb46 100644 --- a/src/Dennis/Link/Checker/Field.php +++ b/src/Dennis/Link/Checker/Field.php @@ -146,7 +146,7 @@ public function save() { $updated_text = Html::serialize($this->getDOM()); return $this->checker_managers->getCheckerQueriesManager()->fieldSave( $this->getEntity()->entityId(), - $this->getEntity()->entityType(), + 'node', $this->field_name, $this->revision_id, $updated_text diff --git a/src/Dennis/Link/Checker/Processor.php b/src/Dennis/Link/Checker/Processor.php index c1a0d00..07f71d9 100644 --- a/src/Dennis/Link/Checker/Processor.php +++ b/src/Dennis/Link/Checker/Processor.php @@ -100,7 +100,6 @@ public function run() { } $end = time() + $this->timeLimit; - // Remove any old items from the queue. $this->prune(); @@ -113,9 +112,8 @@ public function run() { try { $more = $this->doNextItem(); } catch (RequestTimeoutException $e) { - // Don't try to process any more items for this run. + // Log the timeout, but keep going. $this->config->getLogger()->warning($e->getMessage()); - return FALSE; } }