Passed
Push — master ( 35a5e3...2ddbf5 )
by Georgi
03:24
created
src/System/Logo/LogoSettings.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 			$logo->setThumbnailSrc(asset('storage/' . self::alias() . '/' . self::$defaultLogo));
49 49
 		});
50 50
 
51
-		$logo->onUpload(function ($files) use ($form, $logo) {
51
+		$logo->onUpload(function($files) use ($form, $logo) {
52 52
 			if ($files === 'error')	return $form->error('logo', __('Error uploading image'));
53 53
 			
54 54
 			$tmpPath = self::alias() . '/tmp/' . $files['name'];
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
 		});
60 60
 					
61 61
 		$form->onSubmit(function($form) {
62
-			if ($name = $form->model['custom_logo']? $form->model['logo']: null) {
62
+			if ($name = $form->model['custom_logo'] ? $form->model['logo'] : null) {
63 63
 				$storage = $this->storage();
64 64
 				$from = self::alias() . '/tmp/' . $name;
65 65
 				$to = self::alias() . '/' . $name;				
Please login to merge, or discard this patch.
src/System/ModuleAdministration.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -28,8 +28,8 @@  discard block
 block discarded – undo
28 28
 	{
29 29
 		$modules = ModuleManager::getAll();
30 30
 		
31
-		return $modules->filter(function ($subModuleClass) use ($modules) {
32
-			return ! $modules->map(function($moduleClass){
31
+		return $modules->filter(function($subModuleClass) use ($modules) {
32
+			return !$modules->map(function($moduleClass) {
33 33
 				return $moduleClass::namespace();
34 34
 			})->contains($subModuleClass::parentNamespace());
35 35
 		})->sort();
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 
61 61
 			$section->add($label, 'title')->setStyle('float', 'right');
62 62
 			
63
-			$submodules = ModuleManager::getAll()->filter(function ($subModuleClass) use ($moduleClass) {
63
+			$submodules = ModuleManager::getAll()->filter(function($subModuleClass) use ($moduleClass) {
64 64
 				return $subModuleClass::isSubModuleOf($moduleClass);
65 65
 			});
66 66
 			
@@ -72,11 +72,11 @@  discard block
 block discarded – undo
72 72
 	
73 73
 	public function formatModuleInfo($moduleClass)
74 74
 	{
75
-		$moduleInfo = (array) ($moduleClass::info()?: __(' No details provided by author'));
75
+		$moduleInfo = (array) ($moduleClass::info() ?: __(' No details provided by author'));
76 76
 		
77 77
 		$ret = [];
78 78
 		foreach ($moduleInfo as $label => $text) {
79
-			$ret[] = (is_string($label)? "<strong>$label</strong>: ": '') . $text;
79
+			$ret[] = (is_string($label) ? "<strong>$label</strong>: " : '') . $text;
80 80
 		}
81 81
 		
82 82
 		return implode('<br>', $ret);
@@ -109,12 +109,12 @@  discard block
 block discarded – undo
109 109
 					
110 110
 					$form = $view->add(new Form());
111 111
 					foreach ($recommended as $childModule) {
112
-						if (! ModuleManager::isAvailable($childModule)) continue;
112
+						if (!ModuleManager::isAvailable($childModule)) continue;
113 113
 						
114 114
 						$form->addField($childModule::alias(), ['CheckBox', 'caption' => $childModule::label()]);
115 115
 					}
116 116
 					
117
-					$form->onSubmit(function ($form) use ($moduleClass) {
117
+					$form->onSubmit(function($form) use ($moduleClass) {
118 118
 						ob_start();
119 119
 						ModuleManager::install($moduleClass, array_keys($form->getValues(), true));
120 120
 						
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 				}
124 124
 				
125 125
 				$view->add(['Button', __('Install'), 'primary'])->on('click', [
126
-						isset($form)? $form->submit(): $installCallback
126
+						isset($form) ? $form->submit() : $installCallback
127 127
 				]);
128 128
 			});
129 129
 			
Please login to merge, or discard this patch.
Braces   +7 added lines, -4 removed lines patch added patch discarded remove patch
@@ -51,8 +51,7 @@  discard block
 block discarded – undo
51 51
 				$this->addUninstallButton($section, $moduleClass);
52 52
 				
53 53
 // 				$this->addReinstallButton($section, $moduleClass);
54
-			}
55
-			else {
54
+			} else {
56 55
 				$label = ['Label', __('Available'), 'yellow'];
57 56
 				
58 57
 				$this->addInstallButton($section, $moduleClass);
@@ -64,7 +63,9 @@  discard block
 block discarded – undo
64 63
 				return $subModuleClass::isSubModuleOf($moduleClass);
65 64
 			});
66 65
 			
67
-			if ($submodules->isEmpty()) continue;
66
+			if ($submodules->isEmpty()) {
67
+				continue;
68
+			}
68 69
 			
69 70
 			$this->addAccordion($section, $submodules);
70 71
 		}
@@ -109,7 +110,9 @@  discard block
 block discarded – undo
109 110
 					
110 111
 					$form = $view->add(new Form());
111 112
 					foreach ($recommended as $childModule) {
112
-						if (! ModuleManager::isAvailable($childModule)) continue;
113
+						if (! ModuleManager::isAvailable($childModule)) {
114
+							continue;
115
+						}
113 116
 						
114 117
 						$form->addField($childModule::alias(), ['CheckBox', 'caption' => $childModule::label()]);
115 118
 					}
Please login to merge, or discard this patch.
src/System/Integration/Modules/ModuleManager.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 
21 21
 	public static function isInstalled($classOrAlias)
22 22
 	{
23
-		return self::getClass($classOrAlias, true)? 1: 0;
23
+		return self::getClass($classOrAlias, true) ? 1 : 0;
24 24
 	}
25 25
 	
26 26
 	public static function isAvailable($classOrAlias)
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 	 * @return string;
36 36
 	 */
37 37
 	public static function getClass($classOrAlias, $installedOnly = false) {
38
-		$modules = $installedOnly? self::getInstalled(): self::getAll();
38
+		$modules = $installedOnly ? self::getInstalled() : self::getAll();
39 39
 		
40 40
 		if (collect($modules)->contains($classOrAlias)) return $classOrAlias;
41 41
 		
@@ -67,9 +67,9 @@  discard block
 block discarded – undo
67 67
 	 */
68 68
 	public static function getAll()
69 69
 	{
70
-		return self::getCached('epesi-modules-available', function () {
70
+		return self::getCached('epesi-modules-available', function() {
71 71
 			$modules = collect();
72
-			foreach (array_merge(config('epesi.modules', []), self::packageManifest()->modules()?: []) as $namespace => $path) {
72
+			foreach (array_merge(config('epesi.modules', []), self::packageManifest()->modules() ?: []) as $namespace => $path) {
73 73
 				foreach (self::discoverModuleClasses($namespace, $path) as $moduleClass) {
74 74
 					$modules->add(['alias' => $moduleClass::alias(), 'class' => $moduleClass]);
75 75
 				}
@@ -82,14 +82,14 @@  discard block
 block discarded – undo
82 82
 	protected static function discoverModuleClasses($namespace, $basePath)
83 83
 	{
84 84
 		$ret = collect();
85
-		foreach (glob($basePath . '/*', GLOB_ONLYDIR|GLOB_NOSORT) as $path) {
85
+		foreach (glob($basePath . '/*', GLOB_ONLYDIR | GLOB_NOSORT) as $path) {
86 86
 			$moduleNamespace = trim($namespace, '\\') . '\\' . basename($path);
87 87
 			
88 88
 			$ret = $ret->merge(self::discoverModuleClasses($moduleNamespace, $path));
89 89
 			
90 90
 			$moduleClass = $moduleNamespace . '\\' . basename($path) . 'Core';
91 91
 			
92
-			if (! is_a($moduleClass, ModuleCore::class, true)) continue;
92
+			if (!is_a($moduleClass, ModuleCore::class, true)) continue;
93 93
 			
94 94
 			$ret->add($moduleClass);
95 95
 		}
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 	 */
107 107
 	protected static function getCached($key, \Closure $default)
108 108
 	{
109
-		if (! Cache::has($key)) {
109
+		if (!Cache::has($key)) {
110 110
 			Cache::forever($key, $default());
111 111
 		}
112 112
 
@@ -144,12 +144,12 @@  discard block
 block discarded – undo
144 144
 	 */
145 145
 	public static function collect($method, $args = [])
146 146
 	{
147
-		$args = is_array($args)? $args: [$args];
147
+		$args = is_array($args) ? $args : [$args];
148 148
 		
149 149
 		$installedModules = self::getInstalled();
150 150
 		
151 151
 		// if epesi is not installed fake having the system module to enable its functionality
152
-		if (! $installedModules->contains(\Epesi\Core\System\SystemCore::class)) {
152
+		if (!$installedModules->contains(\Epesi\Core\System\SystemCore::class)) {
153 153
 			$installedModules = collect([
154 154
 				'system' => \Epesi\Core\System\SystemCore::class
155 155
 			]);
@@ -157,9 +157,9 @@  discard block
 block discarded – undo
157 157
 		
158 158
 		$ret = [];
159 159
 		foreach ($installedModules as $module) {
160
-			if (! $list = $module::$method(...$args)) continue;
160
+			if (!$list = $module::$method(...$args)) continue;
161 161
 			
162
-			$ret = array_merge($ret, is_array($list)? $list: [$list]);
162
+			$ret = array_merge($ret, is_array($list) ? $list : [$list]);
163 163
 		}
164 164
 		
165 165
 		return $ret;
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
 			return true;
179 179
 		}
180 180
 		
181
-		if (! $moduleClass = self::getClass($classOrAlias)) {			
181
+		if (!$moduleClass = self::getClass($classOrAlias)) {			
182 182
 			throw new \Exception('Module "' . $classOrAlias . '" could not be identified');
183 183
 		}
184 184
 		
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
 		]);
209 209
 		
210 210
 		if ($installRecommended) {
211
-			$installRecommended = is_array($installRecommended)? $installRecommended: $module->recommended();
211
+			$installRecommended = is_array($installRecommended) ? $installRecommended : $module->recommended();
212 212
 			
213 213
 			foreach ($installRecommended as $recommendedModule) {
214 214
 				try {
@@ -241,10 +241,10 @@  discard block
 block discarded – undo
241 241
 			$parentModule = array_shift($unsatisfiedDependencies);
242 242
 				
243 243
 			if (self::$processing[$parentModule]?? false) {
244
-				throw new Exception('Cross dependency: '. $parentModule);
244
+				throw new Exception('Cross dependency: ' . $parentModule);
245 245
 			}
246 246
 				
247
-			if (! self::isAvailable($parentModule)) {
247
+			if (!self::isAvailable($parentModule)) {
248 248
 				throw new Exception('Module not found: "' . $parentModule . '"');
249 249
 			}
250 250
 	
@@ -297,13 +297,13 @@  discard block
 block discarded – undo
297 297
 	
298 298
 	public static function uninstall($classOrAlias)
299 299
 	{
300
-		if (! self::isInstalled($classOrAlias)) {
300
+		if (!self::isInstalled($classOrAlias)) {
301 301
 			print ('Module "' . $classOrAlias . '" is not installed!');
302 302
 			
303 303
 			return true;
304 304
 		}
305 305
 		
306
-		if (! $moduleClass = self::getClass($classOrAlias)) {
306
+		if (!$moduleClass = self::getClass($classOrAlias)) {
307 307
 			throw new \Exception('Module "' . $classOrAlias . '" could not be identified');
308 308
 		}
309 309
 		
Please login to merge, or discard this patch.
src/Controllers/SystemController.php 1 patch
Indentation   +43 added lines, -43 removed lines patch added patch discarded remove patch
@@ -11,66 +11,66 @@
 block discarded – undo
11 11
 
12 12
 class SystemController extends Controller
13 13
 {
14
-    public function index()
15
-    {
16
-    	return SystemCore::isInstalled()? redirect('home'): redirect('install');
17
-    }
14
+	public function index()
15
+	{
16
+		return SystemCore::isInstalled()? redirect('home'): redirect('install');
17
+	}
18 18
     
19
-    public function install(Epesi $epesi)
20
-    {
21
-    	// make sure the installation information is fresh
22
-    	ModuleManager::clearCache();
19
+	public function install(Epesi $epesi)
20
+	{
21
+		// make sure the installation information is fresh
22
+		ModuleManager::clearCache();
23 23
     	
24
-    	if (SystemCore::isInstalled()) return redirect('home');
24
+		if (SystemCore::isInstalled()) return redirect('home');
25 25
     	
26
-    	$epesi->title = __(':epesi > Installation', ['epesi' => config('epesi.app.title')]);
26
+		$epesi->title = __(':epesi > Installation', ['epesi' => config('epesi.app.title')]);
27 27
     	
28
-    	$epesi->initLayout('Centered');
28
+		$epesi->initLayout('Centered');
29 29
     	
30
-    	$epesi->layout->set('logo', url('logo'));
31
-    	$epesi->layout->template->setHTML('copyright', config('epesi.app.copyright'));
30
+		$epesi->layout->set('logo', url('logo'));
31
+		$epesi->layout->template->setHTML('copyright', config('epesi.app.copyright'));
32 32
     	
33
-    	$epesi->add(new \Epesi\Core\System\SystemInstallWizard());
33
+		$epesi->add(new \Epesi\Core\System\SystemInstallWizard());
34 34
     	
35
-    	return $epesi->response();
36
-    }
35
+		return $epesi->response();
36
+	}
37 37
     
38
-    public function home()
39
-    {
40
-    	return redirect(SystemCore::isInstalled()? \Epesi\Core\HomePage\HomePageSettings::getUserHomePagePath(): 'install');
41
-    }
38
+	public function home()
39
+	{
40
+		return redirect(SystemCore::isInstalled()? \Epesi\Core\HomePage\HomePageSettings::getUserHomePagePath(): 'install');
41
+	}
42 42
     
43
-    public function logo()
44
-    { 
45
-    	$logoFile = \Epesi\Core\System\Logo\LogoSettings::getLogoFile();
43
+	public function logo()
44
+	{ 
45
+		$logoFile = \Epesi\Core\System\Logo\LogoSettings::getLogoFile();
46 46
 
47
-    	return response(File::get($logoFile), 200, ['Content-type' => File::mimeType($logoFile)])->setMaxAge(604800)->setPublic();
48
-    }
47
+		return response(File::get($logoFile), 200, ['Content-type' => File::mimeType($logoFile)])->setMaxAge(604800)->setPublic();
48
+	}
49 49
     
50
-    public function view(Epesi $epesi, $module, $method = 'body', $args = [])
51
-    {
52
-    	$epesi->initLayout(new LayoutView());
50
+	public function view(Epesi $epesi, $module, $method = 'body', $args = [])
51
+	{
52
+		$epesi->initLayout(new LayoutView());
53 53
     	
54
-    	$alias = explode(':', $module);
54
+		$alias = explode(':', $module);
55 55
     	
56
-    	$moduleAlias = $alias[0];
57
-    	$viewAlias = $alias[1]?? null;
56
+		$moduleAlias = $alias[0];
57
+		$viewAlias = $alias[1]?? null;
58 58
     	
59
-    	$view = null;
60
-    	if ($module = ModuleManager::getClass($moduleAlias, true)) {
61
-    		$viewClass = $module::view($viewAlias);
59
+		$view = null;
60
+		if ($module = ModuleManager::getClass($moduleAlias, true)) {
61
+			$viewClass = $module::view($viewAlias);
62 62
 
63
-    		if (class_exists($viewClass)) {
64
-    			$view = new $viewClass();
65
-    		}
66
-    	}
63
+			if (class_exists($viewClass)) {
64
+				$view = new $viewClass();
65
+			}
66
+		}
67 67
 
68
-    	if (! $view) abort(404);
68
+		if (! $view) abort(404);
69 69
     	
70
-    	$epesi->add($view)->displayModuleContent($method, $args);
70
+		$epesi->add($view)->displayModuleContent($method, $args);
71 71
     	
72
-    	$epesi->setLocation($view->location());
72
+		$epesi->setLocation($view->location());
73 73
     	
74
-    	return $epesi->response();
75
-    }
74
+		return $epesi->response();
75
+	}
76 76
 }
Please login to merge, or discard this patch.