Unverified Commit f8b13ab5 authored by alexpott's avatar alexpott

Issue #2937844 by andypost, ravi.shankar, Spokje, mrinalini9, Deepak Goyal,...

Issue #2937844 by andypost, ravi.shankar, Spokje, mrinalini9, Deepak Goyal, RoSk0, daffie, borisson_, xjm, tstoeckler: [8.9/9.0 backport] Fix 'Squiz.PHP.NonExecutableCode' coding standard
parent 44143438
......@@ -506,8 +506,6 @@ private function readLine() {
$this->errors[] = new FormattableMarkup('The translation stream %uri ended unexpectedly at line %line.', $log_vars);
return FALSE;
}
return;
}
/**
......
......@@ -176,7 +176,6 @@ protected function openBrowser($url, SymfonyStyle $io) {
$php = "<?php sleep(2); passthru(\"$cmd\"); ?>";
$process = new PhpProcess($php);
$process->start();
return;
}
/**
......
......@@ -49,9 +49,7 @@ public function rowCount() {
/**
* {@inheritdoc}
*/
public function setFetchMode($mode, $a1 = NULL, $a2 = []) {
return;
}
public function setFetchMode($mode, $a1 = NULL, $a2 = []) {}
/**
* {@inheritdoc}
......
......@@ -232,9 +232,7 @@ public function render() {
/**
* Gets the title of the page.
*/
protected function getTitle() {
return;
}
protected function getTitle() {}
/**
* Ensures that a destination is present on the given URL.
......
......@@ -29,5 +29,4 @@ function _config_import_test_config_import_steps_alter(&$context, ConfigImporter
$config_importer->logError('_config_import_test_config_import_steps_alter ConfigImporter error');
}
$context['finished'] = 1;
return;
}
......@@ -60,8 +60,6 @@ public function isFile($path) {
return FALSE;
}
public function chmodJailed($path, $mode, $recursive) {
return;
}
public function chmodJailed($path, $mode, $recursive) {}
}
......@@ -48,7 +48,6 @@ protected function prepareEnvironment() {
*/
protected function setUpSettings() {
// This form will never be reached.
return;
}
/**
......@@ -56,7 +55,6 @@ protected function setUpSettings() {
*/
protected function setUpSite() {
// This form will never be reached.
return;
}
/**
......
......@@ -21,7 +21,6 @@ class InstallerExistingConfigNoConfigTest extends InstallerExistingConfigTestBas
*/
protected function setUpSite() {
// There are errors therefore there is nothing to do here.
return;
}
/**
......
......@@ -22,7 +22,7 @@ protected function prepareEnvironment() {
* {@inheritdoc}
*/
public function setUpSite() {
return;
// There are are errors. Therefore, there is nothing to do here.
}
/**
......
......@@ -32,7 +32,6 @@ function config_profile_with_hook_install_install() {
*/
protected function setUpSettings() {
// There are errors therefore there is nothing to do here.
return;
}
/**
......@@ -40,7 +39,6 @@ protected function setUpSettings() {
*/
protected function setUpSite() {
// There are errors therefore there is nothing to do here.
return;
}
/**
......
......@@ -46,7 +46,6 @@ function testing_config_install_multilingual_install() {
protected function setUpProfile() {
// This is the form we are testing so wait until the test method to do
// assertions.
return;
}
/**
......@@ -54,7 +53,6 @@ protected function setUpProfile() {
*/
protected function setUpRequirementsProblem() {
// This form will never be reached.
return;
}
/**
......@@ -62,7 +60,6 @@ protected function setUpRequirementsProblem() {
*/
protected function setUpSettings() {
// This form will never be reached.
return;
}
/**
......@@ -70,7 +67,6 @@ protected function setUpSettings() {
*/
protected function setUpSite() {
// This form will never be reached.
return;
}
/**
......
......@@ -42,7 +42,6 @@ protected function setUpSettings() {
protected function setUpSite() {
// This step will not occur because there is an error.
return;
}
/**
......
......@@ -67,9 +67,7 @@ public function databaseType() {
/**
* {@inheritdoc}
*/
public function createDatabase($database) {
return;
}
public function createDatabase($database) {}
/**
* {@inheritdoc}
......
......@@ -49,9 +49,7 @@ public function databaseType() {
/**
* {@inheritdoc}
*/
public function createDatabase($database) {
return;
}
public function createDatabase($database) {}
/**
* {@inheritdoc}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment