@@ -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::getDataset(TABLE_PAGES, $data); |
|
38 | + while (null !== ($data = DB::getLast()->getRow())) { |
|
39 | + yield Entitizer::getDataset(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 |
@@ -34,7 +34,9 @@ discard block |
||
34 | 34 | |
35 | 35 | public function addParam(string $name, $default, callable $validator = null) : Dataset { |
36 | 36 | |
37 | - if ('' === $name) return $this; |
|
37 | + if ('' === $name) { |
|
38 | + return $this; |
|
39 | + } |
|
38 | 40 | |
39 | 41 | $this->data[$name] = $default; $this->defaults[$name] = $default; |
40 | 42 | |
@@ -53,7 +55,9 @@ discard block |
||
53 | 55 | |
54 | 56 | public function addParams(array $data) : Dataset { |
55 | 57 | |
56 | - foreach ($data as $name => $value) if (is_scalar($name)) $this->addParam($name, $value); |
|
58 | + foreach ($data as $name => $value) { |
|
59 | + if (is_scalar($name)) $this->addParam($name, $value); |
|
60 | + } |
|
57 | 61 | |
58 | 62 | return $this; |
59 | 63 | } |
@@ -66,16 +70,16 @@ discard block |
||
66 | 70 | |
67 | 71 | public function set(string $name, $value) { |
68 | 72 | |
69 | - if (!isset($this->validators[$name])) return null; |
|
73 | + if (!isset($this->validators[$name])) { |
|
74 | + return null; |
|
75 | + } |
|
70 | 76 | |
71 | 77 | try { |
72 | 78 | |
73 | - if (null === ($value = $this->validators[$name]($value))) return false; |
|
74 | - |
|
75 | - else { $this->data[$name] = $value; return true; } |
|
76 | - } |
|
77 | - |
|
78 | - catch (TypeError $e) { return false; } |
|
79 | + if (null === ($value = $this->validators[$name]($value))) { |
|
80 | + return false; |
|
81 | + } else { $this->data[$name] = $value; return true; } |
|
82 | + } catch (TypeError $e) { return false; } |
|
79 | 83 | } |
80 | 84 | |
81 | 85 | /** |
@@ -90,7 +94,9 @@ discard block |
||
90 | 94 | |
91 | 95 | foreach ($data as $name => $value) { |
92 | 96 | |
93 | - if (null !== ($value = $this->set($name, $value))) $setted[$name] = $value; |
|
97 | + if (null !== ($value = $this->set($name, $value))) { |
|
98 | + $setted[$name] = $value; |
|
99 | + } |
|
94 | 100 | } |
95 | 101 | |
96 | 102 | # ------------------------ |
@@ -106,16 +112,18 @@ discard block |
||
106 | 112 | |
107 | 113 | public function cast(string $name, $value) { |
108 | 114 | |
109 | - if (!isset($this->validators[$name])) return null; |
|
115 | + if (!isset($this->validators[$name])) { |
|
116 | + return null; |
|
117 | + } |
|
110 | 118 | |
111 | 119 | try { |
112 | 120 | |
113 | - if (null !== ($value = $this->validators[$name]($value))) return $value; |
|
114 | - |
|
115 | - else return $this->data[$name]; |
|
116 | - } |
|
117 | - |
|
118 | - catch (TypeError $e) { return $this->data[$name]; } |
|
121 | + if (null !== ($value = $this->validators[$name]($value))) { |
|
122 | + return $value; |
|
123 | + } else { |
|
124 | + return $this->data[$name]; |
|
125 | + } |
|
126 | + } catch (TypeError $e) { return $this->data[$name]; } |
|
119 | 127 | } |
120 | 128 | |
121 | 129 | /** |
@@ -130,7 +138,9 @@ discard block |
||
130 | 138 | |
131 | 139 | foreach ($data as $name => $value) { |
132 | 140 | |
133 | - if (null !== ($value = $this->cast($name, $value))) $casted[$name] = $value; |
|
141 | + if (null !== ($value = $this->cast($name, $value))) { |
|
142 | + $casted[$name] = $value; |
|
143 | + } |
|
134 | 144 | } |
135 | 145 | |
136 | 146 | # ------------------------ |
@@ -117,7 +117,7 @@ |
||
117 | 117 | |
118 | 118 | $block = clone (self::$block ?? (self::$block = Template::createBlock( |
119 | 119 | |
120 | - '<$name${ for:attributes } $name$="$value$"{ / for:attributes }>' . |
|
120 | + '<$name${ for:attributes } $name$="$value$"{ / for:attributes }>'. |
|
121 | 121 | |
122 | 122 | '{ block:contents / }$contents${ ! block:closing }</$name$>{ / block:closing }' |
123 | 123 | ))); |
@@ -58,7 +58,9 @@ discard block |
||
58 | 58 | |
59 | 59 | public function setAttributes(array $attributes) : Tag { |
60 | 60 | |
61 | - foreach ($attributes as $name => $value) if (is_scalar($value)) $this->setAttribute($name, $value); |
|
61 | + foreach ($attributes as $name => $value) { |
|
62 | + if (is_scalar($value)) $this->setAttribute($name, $value); |
|
63 | + } |
|
62 | 64 | |
63 | 65 | return $this; |
64 | 66 | } |
@@ -71,7 +73,9 @@ discard block |
||
71 | 73 | |
72 | 74 | public function setContents($contents) : Tag { |
73 | 75 | |
74 | - if (Template::isBlock($contents) || is_scalar($contents) || is_null($contents)) $this->contents = $contents; |
|
76 | + if (Template::isBlock($contents) || is_scalar($contents) || is_null($contents)) { |
|
77 | + $this->contents = $contents; |
|
78 | + } |
|
75 | 79 | |
76 | 80 | return $this; |
77 | 81 | } |
@@ -28,7 +28,9 @@ discard block |
||
28 | 28 | |
29 | 29 | public function __clone() { |
30 | 30 | |
31 | - foreach ($this->items as $name => $item) $this->items[$name] = clone $item; |
|
31 | + foreach ($this->items as $name => $item) { |
|
32 | + $this->items[$name] = clone $item; |
|
33 | + } |
|
32 | 34 | } |
33 | 35 | |
34 | 36 | /** |
@@ -52,7 +54,9 @@ discard block |
||
52 | 54 | |
53 | 55 | public function addItems(array $items) : Group { |
54 | 56 | |
55 | - foreach ($items as $item) if ($item instanceof Block) $this->addItem($item); |
|
57 | + foreach ($items as $item) { |
|
58 | + if ($item instanceof Block) $this->addItem($item); |
|
59 | + } |
|
56 | 60 | |
57 | 61 | return $this; |
58 | 62 | } |
@@ -111,7 +115,9 @@ discard block |
||
111 | 115 | |
112 | 116 | # Process items |
113 | 117 | |
114 | - foreach ($this->items as $block) $contents .= $block->getContents(); |
|
118 | + foreach ($this->items as $block) { |
|
119 | + $contents .= $block->getContents(); |
|
120 | + } |
|
115 | 121 | |
116 | 122 | # ------------------------ |
117 | 123 |
@@ -29,13 +29,19 @@ discard block |
||
29 | 29 | |
30 | 30 | # Validate name |
31 | 31 | |
32 | - if (false === ($name = Validate::templateComponentName($name))) return ['name', 'WIDGET_ERROR_NAME_INVALID']; |
|
32 | + if (false === ($name = Validate::templateComponentName($name))) { |
|
33 | + return ['name', 'WIDGET_ERROR_NAME_INVALID']; |
|
34 | + } |
|
33 | 35 | |
34 | 36 | # Check name exists |
35 | 37 | |
36 | - if (false === ($check_name = $this->widget->check($name, 'name'))) return 'WIDGET_ERROR_MODIFY'; |
|
38 | + if (false === ($check_name = $this->widget->check($name, 'name'))) { |
|
39 | + return 'WIDGET_ERROR_MODIFY'; |
|
40 | + } |
|
37 | 41 | |
38 | - if ($check_name === 1) return ['name', 'WIDGET_ERROR_NAME_DUPLICATE']; |
|
42 | + if ($check_name === 1) { |
|
43 | + return ['name', 'WIDGET_ERROR_NAME_DUPLICATE']; |
|
44 | + } |
|
39 | 45 | |
40 | 46 | # Modify widget |
41 | 47 | |
@@ -48,7 +54,9 @@ discard block |
||
48 | 54 | |
49 | 55 | $modifier = ((0 === $this->widget->id) ? 'create' : 'edit'); |
50 | 56 | |
51 | - if (!$this->widget->$modifier($data)) return 'WIDGET_ERROR_MODIFY'; |
|
57 | + if (!$this->widget->$modifier($data)) { |
|
58 | + return 'WIDGET_ERROR_MODIFY'; |
|
59 | + } |
|
52 | 60 | |
53 | 61 | # ------------------------ |
54 | 62 |
@@ -67,7 +67,7 @@ |
||
67 | 67 | |
68 | 68 | if (false === ($handler = self::parseString($route['handler'], REGEX_MAP_ITEM_HANDLER))) return; |
69 | 69 | |
70 | - self::$routes['/' . implode('/', $path)] = ('Addons\\' . $name . '\\' . implode('\\', $handler)); |
|
70 | + self::$routes['/'.implode('/', $path)] = ('Addons\\'.$name.'\\'.implode('\\', $handler)); |
|
71 | 71 | } |
72 | 72 | |
73 | 73 | # Parse item |
@@ -52,7 +52,9 @@ discard block |
||
52 | 52 | |
53 | 53 | $parts = preg_split('/\//', $string, 0, PREG_SPLIT_NO_EMPTY); |
54 | 54 | |
55 | - foreach ($parts as $name) if (!preg_match($regex, $name)) return false; |
|
55 | + foreach ($parts as $name) { |
|
56 | + if (!preg_match($regex, $name)) return false; |
|
57 | + } |
|
56 | 58 | |
57 | 59 | # ------------------------ |
58 | 60 | |
@@ -63,9 +65,13 @@ discard block |
||
63 | 65 | |
64 | 66 | private static function parseRoute(string $name, array $route) { |
65 | 67 | |
66 | - if (false === ($path = self::parseString($route['path'], REGEX_MAP_ITEM_PATH))) return; |
|
68 | + if (false === ($path = self::parseString($route['path'], REGEX_MAP_ITEM_PATH))) { |
|
69 | + return; |
|
70 | + } |
|
67 | 71 | |
68 | - if (false === ($handler = self::parseString($route['handler'], REGEX_MAP_ITEM_HANDLER))) return; |
|
72 | + if (false === ($handler = self::parseString($route['handler'], REGEX_MAP_ITEM_HANDLER))) { |
|
73 | + return; |
|
74 | + } |
|
69 | 75 | |
70 | 76 | self::$routes['/' . implode('/', $path)] = ('Addons\\' . $name . '\\' . implode('\\', $handler)); |
71 | 77 | } |
@@ -74,14 +80,18 @@ discard block |
||
74 | 80 | |
75 | 81 | private static function parseItem(array $item) { |
76 | 82 | |
77 | - foreach ($item['routes'] as $route) self::parseRoute($item['name'], $route); |
|
83 | + foreach ($item['routes'] as $route) { |
|
84 | + self::parseRoute($item['name'], $route); |
|
85 | + } |
|
78 | 86 | } |
79 | 87 | |
80 | 88 | # Autoloader |
81 | 89 | |
82 | 90 | public static function __autoload() { |
83 | 91 | |
84 | - foreach (Extend\Addons::items() as $item) self::parseItem($item); |
|
92 | + foreach (Extend\Addons::items() as $item) { |
|
93 | + self::parseItem($item); |
|
94 | + } |
|
85 | 95 | } |
86 | 96 | |
87 | 97 | # Get handler by url |