summaryrefslogtreecommitdiffstats
path: root/install.php
diff options
context:
space:
mode:
authorSimon Rettberg2017-12-16 18:33:16 +0100
committerSimon Rettberg2017-12-16 18:33:16 +0100
commitcd55ead3e2810e209b726faca12fa749f6875d0f (patch)
tree94b2f29b023e8a0c5100e1d06c974c3c04f890fd /install.php
parentMerge branch 'permission-manager' of openslx.org:openslx-ng/slx-admin into pe... (diff)
downloadslx-admin-cd55ead3e2810e209b726faca12fa749f6875d0f.tar.gz
slx-admin-cd55ead3e2810e209b726faca12fa749f6875d0f.tar.xz
slx-admin-cd55ead3e2810e209b726faca12fa749f6875d0f.zip
Fix A LOT of type problems, logic flaws, uninitialized variables etc.
Most of them were found by phpstorm, so I put in some time and went through the list, fixing quite a bunch of them.
Diffstat (limited to 'install.php')
-rw-r--r--install.php5
1 files changed, 5 insertions, 0 deletions
diff --git a/install.php b/install.php
index a5d18f0c..88eb82c2 100644
--- a/install.php
+++ b/install.php
@@ -256,6 +256,9 @@ function hasUpdateScript($module)
return is_readable($module->getDir() . '/install.inc.php');
}
+/**
+ * @param Module $module
+ */
function runUpdateScript($module)
{
require_once $module->getDir() . '/install.inc.php';
@@ -277,6 +280,7 @@ if (DIRECT_MODE) {
$new[] = $entry;
}
}
+ /* @var Module[] $new */
$modules = $new;
if (empty($modules)) {
finalResponse(UPDATE_NOOP, 'No modules with install scripts, nothing to do');
@@ -287,6 +291,7 @@ if (DIRECT_MODE) {
for ($i = 0; $i < $count; ++$i) {
$assoc[$modules[$i]->getIdentifier()] = $modules[($i + 1) % $count];
}
+ /* @var Module[] $assoc */
if (!empty($argv[1])) {
$last = $argv[1];