Merge branch 'master' of github.com:akaunting/akaunting
This commit is contained in:
@ -22,7 +22,7 @@ class Modules extends Controller
|
||||
return $s;
|
||||
})->pluck('value', 'key');
|
||||
|
||||
$module = Module::get($alias);
|
||||
$module = Module::findByAlias($alias);
|
||||
|
||||
return view('settings.modules.edit', compact('setting', 'module'));
|
||||
}
|
||||
|
@ -373,8 +373,13 @@ trait Modules
|
||||
|
||||
$response = $this->getRemote($url, 'GET', ['timeout' => 30, 'referer' => true]);
|
||||
|
||||
// Exception
|
||||
if ($response instanceof RequestException) {
|
||||
return false;
|
||||
}
|
||||
|
||||
// Bad response
|
||||
if ($response->getStatusCode() != 200) {
|
||||
if (!$response || ($response->getStatusCode() != 200)) {
|
||||
return false;
|
||||
}
|
||||
|
||||
|
@ -70,7 +70,7 @@ class Updater
|
||||
}
|
||||
} else {
|
||||
// Get module instance
|
||||
$module = Module::get($alias);
|
||||
$module = Module::findByAlias($alias);
|
||||
$model = Model::where('alias', $alias)->first();
|
||||
|
||||
// Move all files/folders from temp path
|
||||
@ -114,7 +114,7 @@ class Updater
|
||||
return false;
|
||||
}
|
||||
|
||||
if ($response->getStatusCode() == 200) {
|
||||
if ($response && ($response->getStatusCode() == 200)) {
|
||||
$file = $response->getBody()->getContents();
|
||||
}
|
||||
|
||||
|
@ -99,7 +99,7 @@ class Versions
|
||||
}
|
||||
|
||||
// Bad response
|
||||
if ($response->getStatusCode() != 200) {
|
||||
if (!$response || ($response->getStatusCode() != 200)) {
|
||||
return $latest;
|
||||
}
|
||||
|
||||
@ -115,4 +115,4 @@ class Versions
|
||||
|
||||
return $latest;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user