Skip to content
Snippets Groups Projects
Commit 7886fa92 authored by Ted Bowman's avatar Ted Bowman
Browse files

Issue #3227588 by tedbow: Remove duplicate calls to validation in updater now...

Issue #3227588 by tedbow: Remove duplicate calls to validation in updater now that caller should validate
parent c64f7e04
No related branches found
No related tags found
1 merge request!8Issue #3227588: Remove duplicate calls to validation in updater now that caller should validate
......@@ -90,15 +90,6 @@ class BatchProcessor {
}
}
/**
* Calls the updater's validateStaged() method.
*
* @see \Drupal\automatic_updates\Updater::validateStaged()
*/
public static function validateStaged() {
static::getUpdater()->validateStaged();
}
/**
* Finishes the batch job.
*
......
......@@ -160,13 +160,8 @@ class Updater {
*
* @return string
* A key for this stage update process.
*
* @throws \Drupal\automatic_updates\Exception\UpdateException
* Thrown if any readiness checkers return results with an error. Warnings
* from readiness checker will not stop an update.
*/
public function begin(): string {
$this->dispatchUpdateEvent(AutomaticUpdatesEvents::PRE_START);
$stage_key = $this->createActiveStage();
$this->beginner->begin(static::getActiveDirectory(), static::getStageDirectory(), NULL, 120, $this->getExclusions());
return $stage_key;
......@@ -241,7 +236,6 @@ class Updater {
* Commits the current update.
*/
public function commit(): void {
$this->dispatchUpdateEvent(AutomaticUpdatesEvents::PRE_COMMIT);
$this->committer->commit($this->getStageDirectory(), static::getActiveDirectory());
}
......@@ -313,13 +307,6 @@ class Updater {
return $value;
}
/**
* Validates that an update was performed as expected.
*/
public function validateStaged():void {
$this->dispatchUpdateEvent(AutomaticUpdatesEvents::PRE_COMMIT);
}
/**
* Dispatches an update event.
*
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment