@@ -32,16 +32,18 @@ |
||
32 | 32 | |
33 | 33 | protected function processEntityParent(Template\Block $parent) { |
34 | 34 | |
35 | - if ((0 !== $this->parent->id) && $this->parent->active) $parent->getBlock('browse')->link = $this->parent->link; |
|
36 | - |
|
37 | - else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); } |
|
35 | + if ((0 !== $this->parent->id) && $this->parent->active) { |
|
36 | + $parent->getBlock('browse')->link = $this->parent->link; |
|
37 | + } else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); } |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | # Add additional parent data for specific entity |
41 | 41 | |
42 | 42 | protected function processEntity(Template\Block $contents) { |
43 | 43 | |
44 | - if (!$this->create && $this->parent->locked) $contents->getBlock('locked')->enable(); |
|
44 | + if (!$this->create && $this->parent->locked) { |
|
45 | + $contents->getBlock('locked')->enable(); |
|
46 | + } |
|
45 | 47 | } |
46 | 48 | } |
47 | 49 | } |
@@ -2,7 +2,8 @@ |
||
2 | 2 | |
3 | 3 | namespace Modules\Entitizer\Listview { |
4 | 4 | |
5 | - use Modules\Entitizer, Template; |
|
5 | + use Modules\Entitizer; |
|
6 | + use Template; |
|
6 | 7 | |
7 | 8 | class Variables extends Entitizer\Utils\Listview { |
8 | 9 |
@@ -12,14 +12,14 @@ |
||
12 | 12 | |
13 | 13 | protected function init() { |
14 | 14 | |
15 | - $this->addWorker('gravatar', function (array $data) { |
|
15 | + $this->addWorker('gravatar', function(array $data) { |
|
16 | 16 | |
17 | 17 | return md5(strtolower($data['email'])); |
18 | 18 | }); |
19 | 19 | |
20 | - $this->addWorker('full_name', function (array $data) { |
|
20 | + $this->addWorker('full_name', function(array $data) { |
|
21 | 21 | |
22 | - return trim($data['first_name'] . ' ' . $data['last_name']); |
|
22 | + return trim($data['first_name'].' '.$data['last_name']); |
|
23 | 23 | }); |
24 | 24 | } |
25 | 25 | } |
@@ -12,7 +12,9 @@ discard block |
||
12 | 12 | |
13 | 13 | protected function addWorker(string $name, callable $worker) { |
14 | 14 | |
15 | - if (isset($this->params[$name]) || isset($this->workers[$name])) return; |
|
15 | + if (isset($this->params[$name]) || isset($this->workers[$name])) { |
|
16 | + return; |
|
17 | + } |
|
16 | 18 | |
17 | 19 | $this->workers[$name] = $worker; |
18 | 20 | } |
@@ -23,7 +25,9 @@ discard block |
||
23 | 25 | |
24 | 26 | $this->params = Entitizer::definition(static::$table)->params(); |
25 | 27 | |
26 | - if (static::$nesting) $this->params['parent_id'] = $this->params['id']; |
|
28 | + if (static::$nesting) { |
|
29 | + $this->params['parent_id'] = $this->params['id']; |
|
30 | + } |
|
27 | 31 | |
28 | 32 | $this->init(); $this->reset(); |
29 | 33 | } |
@@ -34,11 +38,15 @@ discard block |
||
34 | 38 | |
35 | 39 | # Reset params |
36 | 40 | |
37 | - foreach ($this->params as $name => $param) $this->data[$name] = $param->cast(null); |
|
41 | + foreach ($this->params as $name => $param) { |
|
42 | + $this->data[$name] = $param->cast(null); |
|
43 | + } |
|
38 | 44 | |
39 | 45 | # Reset extras |
40 | 46 | |
41 | - foreach ($this->workers as $name => $worker) $this->data[$name] = $worker($this->data); |
|
47 | + foreach ($this->workers as $name => $worker) { |
|
48 | + $this->data[$name] = $worker($this->data); |
|
49 | + } |
|
42 | 50 | |
43 | 51 | # ------------------------ |
44 | 52 | |
@@ -53,12 +61,16 @@ discard block |
||
53 | 61 | |
54 | 62 | foreach ($data as $name => $value) { |
55 | 63 | |
56 | - if (isset($this->params[$name])) $this->data[$name] = $this->params[$name]->cast($value); |
|
64 | + if (isset($this->params[$name])) { |
|
65 | + $this->data[$name] = $this->params[$name]->cast($value); |
|
66 | + } |
|
57 | 67 | } |
58 | 68 | |
59 | 69 | # Update extras |
60 | 70 | |
61 | - foreach ($this->workers as $name => $worker) $this->data[$name] = $worker($this->data); |
|
71 | + foreach ($this->workers as $name => $worker) { |
|
72 | + $this->data[$name] = $worker($this->data); |
|
73 | + } |
|
62 | 74 | |
63 | 75 | # ------------------------ |
64 | 76 | |
@@ -73,7 +85,9 @@ discard block |
||
73 | 85 | |
74 | 86 | foreach ($data as $name => $value) { |
75 | 87 | |
76 | - if (isset($this->params[$name])) $cast[$name] = $this->params[$name]->cast($value); |
|
88 | + if (isset($this->params[$name])) { |
|
89 | + $cast[$name] = $this->params[$name]->cast($value); |
|
90 | + } |
|
77 | 91 | } |
78 | 92 | |
79 | 93 | # ------------------------ |
@@ -30,7 +30,7 @@ |
||
30 | 30 | |
31 | 31 | public function __construct() { |
32 | 32 | |
33 | - $system_file = (DIR_SYSTEM_DATA . 'System.json'); |
|
33 | + $system_file = (DIR_SYSTEM_DATA.'System.json'); |
|
34 | 34 | |
35 | 35 | if (false !== ($data = Explorer::json($system_file))) $this->parse($data); |
36 | 36 | } |
@@ -20,21 +20,19 @@ discard block |
||
20 | 20 | |
21 | 21 | # Connect to DB |
22 | 22 | |
23 | - try { DB::connect($server, $user, $password, $name); } |
|
24 | - |
|
25 | - catch (Exception\DBConnect $error) { return 'INSTALL_ERROR_DATABASE_CONNECT'; } |
|
26 | - |
|
27 | - catch (Exception\DBSelect $error) { return 'INSTALL_ERROR_DATABASE_SELECT'; } |
|
28 | - |
|
29 | - catch (Exception\DBCharset $error) { return 'INSTALL_ERROR_DATABASE_CHARSET'; } |
|
23 | + try { DB::connect($server, $user, $password, $name); } catch (Exception\DBConnect $error) { return 'INSTALL_ERROR_DATABASE_CONNECT'; } catch (Exception\DBSelect $error) { return 'INSTALL_ERROR_DATABASE_SELECT'; } catch (Exception\DBCharset $error) { return 'INSTALL_ERROR_DATABASE_CHARSET'; } |
|
30 | 24 | |
31 | 25 | # Create tables |
32 | 26 | |
33 | - if (!Install\Utils\Tables::create()) return 'INSTALL_ERROR_DATABASE_TABLES_CREATE'; |
|
27 | + if (!Install\Utils\Tables::create()) { |
|
28 | + return 'INSTALL_ERROR_DATABASE_TABLES_CREATE'; |
|
29 | + } |
|
34 | 30 | |
35 | 31 | # Fill tables |
36 | 32 | |
37 | - if (!Install\Utils\Tables::fill()) return 'INSTALL_ERROR_DATABASE_TABLES_FILL'; |
|
33 | + if (!Install\Utils\Tables::fill()) { |
|
34 | + return 'INSTALL_ERROR_DATABASE_TABLES_FILL'; |
|
35 | + } |
|
38 | 36 | |
39 | 37 | # Save system file |
40 | 38 | |
@@ -49,7 +47,9 @@ discard block |
||
49 | 47 | |
50 | 48 | $system_file = (DIR_SYSTEM_DATA . 'System.json'); |
51 | 49 | |
52 | - if (false === JSON::save($system_file, $system)) return 'INSTALL_ERROR_SYSTEM'; |
|
50 | + if (false === JSON::save($system_file, $system)) { |
|
51 | + return 'INSTALL_ERROR_SYSTEM'; |
|
52 | + } |
|
53 | 53 | |
54 | 54 | # ------------------------ |
55 | 55 |
@@ -24,7 +24,7 @@ |
||
24 | 24 | |
25 | 25 | # Customize captcha |
26 | 26 | |
27 | - $font = (DIR_SYSTEM_DATA . CONFIG_CAPTCHA_FONT); $size = CONFIG_CAPTCHA_FONT_SIZE; |
|
27 | + $font = (DIR_SYSTEM_DATA.CONFIG_CAPTCHA_FONT); $size = CONFIG_CAPTCHA_FONT_SIZE; |
|
28 | 28 | |
29 | 29 | $indent = CONFIG_CAPTCHA_TEXT_INDENT; $step = CONFIG_CAPTCHA_TEXT_STEP; |
30 | 30 |
@@ -44,7 +44,7 @@ |
||
44 | 44 | |
45 | 45 | # Load sitemap |
46 | 46 | |
47 | - $file_name = (DIR_SYSTEM_DATA . 'Sitemap.xml'); |
|
47 | + $file_name = (DIR_SYSTEM_DATA.'Sitemap.xml'); |
|
48 | 48 | |
49 | 49 | $modified = Explorer::getModified($file_name); |
50 | 50 |
@@ -12,7 +12,9 @@ discard block |
||
12 | 12 | |
13 | 13 | $selection = 'MAX(time_modified) as last_modified'; |
14 | 14 | |
15 | - if (!(DB::select(TABLE_PAGES, $selection) && (DB::getLast()->rows === 1))) return 0; |
|
15 | + if (!(DB::select(TABLE_PAGES, $selection) && (DB::getLast()->rows === 1))) { |
|
16 | + return 0; |
|
17 | + } |
|
16 | 18 | |
17 | 19 | # ------------------------ |
18 | 20 | |
@@ -27,11 +29,15 @@ discard block |
||
27 | 29 | |
28 | 30 | $condition = ['visibility' => VISIBILITY_PUBLISHED, 'access' => ACCESS_PUBLIC, 'locked' => 0]; |
29 | 31 | |
30 | - if (!(DB::select(TABLE_PAGES, $selection, $condition, 'slug') && DB::getLast()->status)) return; |
|
32 | + if (!(DB::select(TABLE_PAGES, $selection, $condition, 'slug') && DB::getLast()->status)) { |
|
33 | + return; |
|
34 | + } |
|
31 | 35 | |
32 | 36 | # ------------------------ |
33 | 37 | |
34 | - while (null !== ($data = DB::getLast()->getRow())) yield Entitizer::dataset(TABLE_PAGES, $data); |
|
38 | + while (null !== ($data = DB::getLast()->getRow())) { |
|
39 | + yield Entitizer::dataset(TABLE_PAGES, $data); |
|
40 | + } |
|
35 | 41 | } |
36 | 42 | |
37 | 43 | # Handle request |
@@ -50,7 +56,9 @@ discard block |
||
50 | 56 | |
51 | 57 | if ((false !== $modified) && ($modified > $this->getLastModified())) { |
52 | 58 | |
53 | - if (false !== $sitemap->load($file_name)) return $sitemap; |
|
59 | + if (false !== $sitemap->load($file_name)) { |
|
60 | + return $sitemap; |
|
61 | + } |
|
54 | 62 | } |
55 | 63 | |
56 | 64 | # Fill sitemap |
@@ -12,11 +12,19 @@ discard block |
||
12 | 12 | |
13 | 13 | private function allocateColor(array $color = null) { |
14 | 14 | |
15 | - if (null === $this->image) return false; |
|
15 | + if (null === $this->image) { |
|
16 | + return false; |
|
17 | + } |
|
16 | 18 | |
17 | - if (null === $color) $color = [0, 0, 0]; else if (count($color) !== 3) return false; |
|
19 | + if (null === $color) { |
|
20 | + $color = [0, 0, 0]; |
|
21 | + } else if (count($color) !== 3) { |
|
22 | + return false; |
|
23 | + } |
|
18 | 24 | |
19 | - foreach (array_values($color) as $key => $value) $color[$key] = Number::forceInt($value, 0, 255); |
|
25 | + foreach (array_values($color) as $key => $value) { |
|
26 | + $color[$key] = Number::forceInt($value, 0, 255); |
|
27 | + } |
|
20 | 28 | |
21 | 29 | # ------------------------ |
22 | 30 | |
@@ -27,7 +35,9 @@ discard block |
||
27 | 35 | |
28 | 36 | public function __construct(int $width, int $height, array $bg_color = null) { |
29 | 37 | |
30 | - if (false === ($image = imagecreatetruecolor($width, $height))) return; |
|
38 | + if (false === ($image = imagecreatetruecolor($width, $height))) { |
|
39 | + return; |
|
40 | + } |
|
31 | 41 | |
32 | 42 | $this->image = $image; $this->width = $width; $this->height = $height; |
33 | 43 | |
@@ -41,11 +51,17 @@ discard block |
||
41 | 51 | |
42 | 52 | public function text(string $font, int $size, int $indent, int $step, string $text, array $color = null) { |
43 | 53 | |
44 | - if (null === $this->image) return false; |
|
54 | + if (null === $this->image) { |
|
55 | + return false; |
|
56 | + } |
|
45 | 57 | |
46 | - if (!Explorer::isFile($font)) return false; |
|
58 | + if (!Explorer::isFile($font)) { |
|
59 | + return false; |
|
60 | + } |
|
47 | 61 | |
48 | - if (false === ($color = $this->allocateColor($color))) return false; |
|
62 | + if (false === ($color = $this->allocateColor($color))) { |
|
63 | + return false; |
|
64 | + } |
|
49 | 65 | |
50 | 66 | $base = (floor(($this->height - $size) / 2) + $size); |
51 | 67 | |
@@ -65,13 +81,22 @@ discard block |
||
65 | 81 | |
66 | 82 | public function noise(int $rate, array $color = null) { |
67 | 83 | |
68 | - if (null === $this->image) return false; |
|
84 | + if (null === $this->image) { |
|
85 | + return false; |
|
86 | + } |
|
69 | 87 | |
70 | - if (false === ($color = $this->allocateColor($color))) return false; |
|
88 | + if (false === ($color = $this->allocateColor($color))) { |
|
89 | + return false; |
|
90 | + } |
|
71 | 91 | |
72 | - for ($x = 0; $x < $this->width; $x++) for ($y = 0; $y < $this->height; $y++) { |
|
92 | + for ($x = 0; $x < $this->width; $x++) { |
|
93 | + for ($y = 0; |
|
94 | + } |
|
95 | + $y < $this->height; $y++) { |
|
73 | 96 | |
74 | - if (random_int(1, $rate) === 1) imagesetpixel($this->image, $x, $y, $color); |
|
97 | + if (random_int(1, $rate) === 1) { |
|
98 | + imagesetpixel($this->image, $x, $y, $color); |
|
99 | + } |
|
75 | 100 | } |
76 | 101 | |
77 | 102 | # ------------------------ |
@@ -83,9 +108,13 @@ discard block |
||
83 | 108 | |
84 | 109 | public function lines(int $count, array $color = null) { |
85 | 110 | |
86 | - if (null === $this->image) return false; |
|
111 | + if (null === $this->image) { |
|
112 | + return false; |
|
113 | + } |
|
87 | 114 | |
88 | - if (false === ($color = $this->allocateColor($color))) return false; |
|
115 | + if (false === ($color = $this->allocateColor($color))) { |
|
116 | + return false; |
|
117 | + } |
|
89 | 118 | |
90 | 119 | for ($i = 0; $i < $count; $i++) { |
91 | 120 |
@@ -12,7 +12,7 @@ |
||
12 | 12 | |
13 | 13 | public function __construct() { |
14 | 14 | |
15 | - $urlset = '<?xml version="1.0" encoding="UTF-8" ?>' . |
|
15 | + $urlset = '<?xml version="1.0" encoding="UTF-8" ?>'. |
|
16 | 16 | |
17 | 17 | '<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9" />'; |
18 | 18 |
@@ -16,14 +16,18 @@ discard block |
||
16 | 16 | |
17 | 17 | '<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9" />'; |
18 | 18 | |
19 | - if (false !== ($xml = XML::parse($urlset))) $this->xml = $xml; |
|
19 | + if (false !== ($xml = XML::parse($urlset))) { |
|
20 | + $this->xml = $xml; |
|
21 | + } |
|
20 | 22 | } |
21 | 23 | |
22 | 24 | # Load sitemap |
23 | 25 | |
24 | 26 | public function load(string $file_name) { |
25 | 27 | |
26 | - if (false === ($xml = XML::load($file_name))) return false; |
|
28 | + if (false === ($xml = XML::load($file_name))) { |
|
29 | + return false; |
|
30 | + } |
|
27 | 31 | |
28 | 32 | $this->xml = $xml; $this->loaded = true; |
29 | 33 | |
@@ -36,9 +40,13 @@ discard block |
||
36 | 40 | |
37 | 41 | public function save(string $file_name) { |
38 | 42 | |
39 | - if ((null === $this->xml) || $this->loaded) return false; |
|
43 | + if ((null === $this->xml) || $this->loaded) { |
|
44 | + return false; |
|
45 | + } |
|
40 | 46 | |
41 | - if (false === XML::save($file_name, $this->xml)) return false; |
|
47 | + if (false === XML::save($file_name, $this->xml)) { |
|
48 | + return false; |
|
49 | + } |
|
42 | 50 | |
43 | 51 | # ------------------------ |
44 | 52 | |
@@ -49,9 +57,13 @@ discard block |
||
49 | 57 | |
50 | 58 | public function add(string $loc, string $lastmod = null, string $changefreq = null, float $priority = null) { |
51 | 59 | |
52 | - if ((null === $this->xml) || $this->loaded) return false; |
|
60 | + if ((null === $this->xml) || $this->loaded) { |
|
61 | + return false; |
|
62 | + } |
|
53 | 63 | |
54 | - if (false === ($loc = Validate::url($loc))) return false; |
|
64 | + if (false === ($loc = Validate::url($loc))) { |
|
65 | + return false; |
|
66 | + } |
|
55 | 67 | |
56 | 68 | # Create url object |
57 | 69 | |
@@ -73,7 +85,9 @@ discard block |
||
73 | 85 | |
74 | 86 | # Set priority |
75 | 87 | |
76 | - if (null !== $priority) $url->addChild('priority', Number::forceFloat($priority, 0, 1, 1)); |
|
88 | + if (null !== $priority) { |
|
89 | + $url->addChild('priority', Number::forceFloat($priority, 0, 1, 1)); |
|
90 | + } |
|
77 | 91 | |
78 | 92 | # ------------------------ |
79 | 93 |
@@ -32,14 +32,18 @@ |
||
32 | 32 | |
33 | 33 | foreach (['full_name', 'city'] as $name) { |
34 | 34 | |
35 | - if ('' === ($text = Auth::user()->$name)) $contents->getBlock($name)->disable(); |
|
36 | - |
|
37 | - else $contents->getBlock($name)->text = $text; |
|
35 | + if ('' === ($text = Auth::user()->$name)) { |
|
36 | + $contents->getBlock($name)->disable(); |
|
37 | + } else { |
|
38 | + $contents->getBlock($name)->text = $text; |
|
39 | + } |
|
38 | 40 | } |
39 | 41 | |
40 | 42 | # Set country |
41 | 43 | |
42 | - if ('' === ($country = Auth::user()->country)) $contents->getBlock('country')->disable(); else { |
|
44 | + if ('' === ($country = Auth::user()->country)) { |
|
45 | + $contents->getBlock('country')->disable(); |
|
46 | + } else { |
|
43 | 47 | |
44 | 48 | $contents->getBlock('country')->code = $country; |
45 | 49 |