@@ -19,13 +19,13 @@ discard block |
||
19 | 19 | |
20 | 20 | private static function send(Entitizer\Entity\User $user, string $view, string $subject, string $link) : bool { |
21 | 21 | |
22 | - $message = View::get((Auth::isAdmin() ? 'Blocks/Auth/Mail/' : 'Blocks/Profile/Auth/Mail/') . $view); |
|
22 | + $message = View::get((Auth::isAdmin() ? 'Blocks/Auth/Mail/' : 'Blocks/Profile/Auth/Mail/').$view); |
|
23 | 23 | |
24 | 24 | $message->name = $user->name; $message->link = $link; $message->copyright = Date::getYear(); |
25 | 25 | |
26 | 26 | $to = $user->email; $sender = Settings::get('site_title'); $reply_to = Settings::get('system_email'); |
27 | 27 | |
28 | - $from = ((false !== ($host = parse_url(Settings::get('system_url'), PHP_URL_HOST))) ? ('noreply@' . $host) : ''); |
|
28 | + $from = ((false !== ($host = parse_url(Settings::get('system_url'), PHP_URL_HOST))) ? ('noreply@'.$host) : ''); |
|
29 | 29 | |
30 | 30 | # ------------------------ |
31 | 31 | |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | |
39 | 39 | public static function sendPasswordMessage(Entitizer\Entity\User $user, string $code) : bool { |
40 | 40 | |
41 | - $link = (Settings::get('system_url') . (Auth::isAdmin() ? '/admin' : '/profile') . '/recover?code=' . $code); |
|
41 | + $link = (Settings::get('system_url').(Auth::isAdmin() ? '/admin' : '/profile').'/recover?code='.$code); |
|
42 | 42 | |
43 | 43 | return self::send($user, 'Reset', Language::get('MAIL_SUBJECT_RESET'), $link); |
44 | 44 | } |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | |
50 | 50 | public static function sendRegistrationMessage(Entitizer\Entity\User $user) : bool { |
51 | 51 | |
52 | - $link = (Settings::get('system_url') . (Auth::isAdmin() ? '/admin' : '/profile')); |
|
52 | + $link = (Settings::get('system_url').(Auth::isAdmin() ? '/admin' : '/profile')); |
|
53 | 53 | |
54 | 54 | return self::send($user, 'Register', Language::get('MAIL_SUBJECT_REGISTER'), $link); |
55 | 55 | } |
@@ -87,7 +87,7 @@ |
||
87 | 87 | |
88 | 88 | public static function isInitial() : bool { |
89 | 89 | |
90 | - DB::send("SHOW TABLE STATUS WHERE name LIKE '" . TABLE_USERS . "'"); |
|
90 | + DB::send("SHOW TABLE STATUS WHERE name LIKE '".TABLE_USERS."'"); |
|
91 | 91 | |
92 | 92 | if (!(DB::getLast() && (null !== ($data = DB::getLast()->getRow())))) return false; |
93 | 93 |
@@ -31,11 +31,15 @@ discard block |
||
31 | 31 | |
32 | 32 | # Check session code |
33 | 33 | |
34 | - if (!is_string($code = Session::get('code'))) return false; |
|
34 | + if (!is_string($code = Session::get('code'))) { |
|
35 | + return false; |
|
36 | + } |
|
35 | 37 | |
36 | 38 | # Authorize |
37 | 39 | |
38 | - if (false === ($result = Auth\Utils\Connector\Session::authorize($code, self::$admin))) return false; |
|
40 | + if (false === ($result = Auth\Utils\Connector\Session::authorize($code, self::$admin))) { |
|
41 | + return false; |
|
42 | + } |
|
39 | 43 | |
40 | 44 | # Update auth |
41 | 45 | |
@@ -62,7 +66,9 @@ discard block |
||
62 | 66 | |
63 | 67 | public static function logout() : bool { |
64 | 68 | |
65 | - if ((null === self::$user) || (0 === self::$user->id)) return false; |
|
69 | + if ((null === self::$user) || (0 === self::$user->id)) { |
|
70 | + return false; |
|
71 | + } |
|
66 | 72 | |
67 | 73 | # Remove auth entry from db |
68 | 74 | |
@@ -89,7 +95,9 @@ discard block |
||
89 | 95 | |
90 | 96 | DB::send("SHOW TABLE STATUS WHERE name LIKE '" . TABLE_USERS . "'"); |
91 | 97 | |
92 | - if (!(DB::getLast() && (null !== ($data = DB::getLast()->getRow())))) return false; |
|
98 | + if (!(DB::getLast() && (null !== ($data = DB::getLast()->getRow())))) { |
|
99 | + return false; |
|
100 | + } |
|
93 | 101 | |
94 | 102 | # ------------------------ |
95 | 103 |
@@ -44,13 +44,15 @@ discard block |
||
44 | 44 | |
45 | 45 | # Validate values |
46 | 46 | |
47 | - if (false === ($password_new = Validate::userPassword($password_new))) |
|
48 | - |
|
47 | + if (false === ($password_new = Validate::userPassword($password_new))) { |
|
48 | + |
|
49 | 49 | return ['password_new', 'USER_ERROR_PASSWORD_NEW_INVALID']; |
50 | + } |
|
50 | 51 | |
51 | - if (0 !== strcmp($password_new, $password_retype)) |
|
52 | - |
|
52 | + if (0 !== strcmp($password_new, $password_retype)) { |
|
53 | + |
|
53 | 54 | return ['password_retype', 'USER_ERROR_PASSWORD_MISMATCH']; |
55 | + } |
|
54 | 56 | |
55 | 57 | # Encode password |
56 | 58 | |
@@ -60,7 +62,9 @@ discard block |
||
60 | 62 | |
61 | 63 | $data = ['auth_key' => $auth_key, 'password' => $password]; |
62 | 64 | |
63 | - if (!$this->user->edit($data)) return 'USER_ERROR_AUTH_RECOVER'; |
|
65 | + if (!$this->user->edit($data)) { |
|
66 | + return 'USER_ERROR_AUTH_RECOVER'; |
|
67 | + } |
|
64 | 68 | |
65 | 69 | # Remove secret |
66 | 70 |
@@ -56,7 +56,7 @@ |
||
56 | 56 | |
57 | 57 | if ((!$this->user->init($$init_by, $init_by)) || (Auth::isAdmin() && ($this->user->rank < RANK_ADMINISTRATOR))) { |
58 | 58 | |
59 | - return ['name_email', ('USER_ERROR_' . strtoupper($init_by) .'_INCORRECT')]; |
|
59 | + return ['name_email', ('USER_ERROR_'.strtoupper($init_by).'_INCORRECT')]; |
|
60 | 60 | } |
61 | 61 | |
62 | 62 | # Check access |
@@ -46,9 +46,13 @@ discard block |
||
46 | 46 | |
47 | 47 | if ((false === ($name = Validate::userName($name_email))) && |
48 | 48 | |
49 | - (false === ($email = Validate::userEmail($name_email)))) return ['name_email', 'USER_ERROR_NAME_INVALID']; |
|
49 | + (false === ($email = Validate::userEmail($name_email)))) { |
|
50 | + return ['name_email', 'USER_ERROR_NAME_INVALID']; |
|
51 | + } |
|
50 | 52 | |
51 | - if (false === Security::checkCaptcha($captcha)) return ['captcha', 'USER_ERROR_CAPTCHA_INCORRECT']; |
|
53 | + if (false === Security::checkCaptcha($captcha)) { |
|
54 | + return ['captcha', 'USER_ERROR_CAPTCHA_INCORRECT']; |
|
55 | + } |
|
52 | 56 | |
53 | 57 | # Init user |
54 | 58 | |
@@ -61,7 +65,9 @@ discard block |
||
61 | 65 | |
62 | 66 | # Check access |
63 | 67 | |
64 | - if (!Auth::isAdmin() && ($this->user->rank === RANK_GUEST)) return 'USER_ERROR_ACCESS'; |
|
68 | + if (!Auth::isAdmin() && ($this->user->rank === RANK_GUEST)) { |
|
69 | + return 'USER_ERROR_ACCESS'; |
|
70 | + } |
|
65 | 71 | |
66 | 72 | # Create secret |
67 | 73 | |
@@ -71,7 +77,9 @@ discard block |
||
71 | 77 | |
72 | 78 | $data = ['id' => $this->user->id, 'code' => $code, 'ip' => $ip, 'time' => $time]; |
73 | 79 | |
74 | - if (!$secret->create($data)) return 'USER_ERROR_AUTH_RESET'; |
|
80 | + if (!$secret->create($data)) { |
|
81 | + return 'USER_ERROR_AUTH_RESET'; |
|
82 | + } |
|
75 | 83 | |
76 | 84 | # Send mail |
77 | 85 |
@@ -56,7 +56,7 @@ |
||
56 | 56 | |
57 | 57 | if ((!$this->user->init($$init_by, $init_by)) || (Auth::isAdmin() && ($this->user->rank < RANK_ADMINISTRATOR))) { |
58 | 58 | |
59 | - return ['name_email', ('USER_ERROR_' . strtoupper($init_by) .'_INCORRECT')]; |
|
59 | + return ['name_email', ('USER_ERROR_'.strtoupper($init_by).'_INCORRECT')]; |
|
60 | 60 | } |
61 | 61 | |
62 | 62 | # Check password |
@@ -46,9 +46,13 @@ discard block |
||
46 | 46 | |
47 | 47 | if ((false === ($name = Validate::userName($name_email))) && |
48 | 48 | |
49 | - (false === ($email = Validate::userEmail($name_email)))) return ['name_email', 'USER_ERROR_NAME_INVALID']; |
|
49 | + (false === ($email = Validate::userEmail($name_email)))) { |
|
50 | + return ['name_email', 'USER_ERROR_NAME_INVALID']; |
|
51 | + } |
|
50 | 52 | |
51 | - if (false === ($password = Validate::userPassword($password))) return ['password', 'USER_ERROR_PASSWORD_INVALID']; |
|
53 | + if (false === ($password = Validate::userPassword($password))) { |
|
54 | + return ['password', 'USER_ERROR_PASSWORD_INVALID']; |
|
55 | + } |
|
52 | 56 | |
53 | 57 | # Init user |
54 | 58 | |
@@ -63,11 +67,15 @@ discard block |
||
63 | 67 | |
64 | 68 | $password = Str::encode($this->user->auth_key, $password); |
65 | 69 | |
66 | - if (0 !== strcmp($this->user->password, $password)) return ['password', 'USER_ERROR_PASSWORD_INCORRECT']; |
|
70 | + if (0 !== strcmp($this->user->password, $password)) { |
|
71 | + return ['password', 'USER_ERROR_PASSWORD_INCORRECT']; |
|
72 | + } |
|
67 | 73 | |
68 | 74 | # Check access |
69 | 75 | |
70 | - if (!Auth::isAdmin() && ($this->user->rank === RANK_GUEST)) return 'USER_ERROR_ACCESS'; |
|
76 | + if (!Auth::isAdmin() && ($this->user->rank === RANK_GUEST)) { |
|
77 | + return 'USER_ERROR_ACCESS'; |
|
78 | + } |
|
71 | 79 | |
72 | 80 | # Create session |
73 | 81 | |
@@ -77,7 +85,9 @@ discard block |
||
77 | 85 | |
78 | 86 | $data = ['id' => $this->user->id, 'code' => $code, 'ip' => $ip, 'time' => $time]; |
79 | 87 | |
80 | - if (!$session->create($data)) return 'USER_ERROR_AUTH_LOGIN'; |
|
88 | + if (!$session->create($data)) { |
|
89 | + return 'USER_ERROR_AUTH_LOGIN'; |
|
90 | + } |
|
81 | 91 | |
82 | 92 | # Set session variable |
83 | 93 |
@@ -44,27 +44,45 @@ discard block |
||
44 | 44 | |
45 | 45 | # Validate values |
46 | 46 | |
47 | - if (false === ($name = Validate::userName($name))) return ['name', 'USER_ERROR_NAME_INVALID']; |
|
47 | + if (false === ($name = Validate::userName($name))) { |
|
48 | + return ['name', 'USER_ERROR_NAME_INVALID']; |
|
49 | + } |
|
48 | 50 | |
49 | - if (false === ($password = Validate::userPassword($password))) return ['password', 'USER_ERROR_PASSWORD_INVALID']; |
|
51 | + if (false === ($password = Validate::userPassword($password))) { |
|
52 | + return ['password', 'USER_ERROR_PASSWORD_INVALID']; |
|
53 | + } |
|
50 | 54 | |
51 | - if (false === ($email = Validate::userEmail($email))) return ['email', 'USER_ERROR_EMAIL_INVALID']; |
|
55 | + if (false === ($email = Validate::userEmail($email))) { |
|
56 | + return ['email', 'USER_ERROR_EMAIL_INVALID']; |
|
57 | + } |
|
52 | 58 | |
53 | - if (0 !== strcmp($password, $password_retype)) return ['password_retype', 'USER_ERROR_PASSWORD_MISMATCH']; |
|
59 | + if (0 !== strcmp($password, $password_retype)) { |
|
60 | + return ['password_retype', 'USER_ERROR_PASSWORD_MISMATCH']; |
|
61 | + } |
|
54 | 62 | |
55 | - if (false === Security::checkCaptcha($captcha)) return ['captcha', 'USER_ERROR_CAPTCHA_INCORRECT']; |
|
63 | + if (false === Security::checkCaptcha($captcha)) { |
|
64 | + return ['captcha', 'USER_ERROR_CAPTCHA_INCORRECT']; |
|
65 | + } |
|
56 | 66 | |
57 | 67 | # Check name exists |
58 | 68 | |
59 | - if (false === ($check_name = $this->user->check($name, 'name'))) return 'USER_ERROR_AUTH_REGISTER'; |
|
69 | + if (false === ($check_name = $this->user->check($name, 'name'))) { |
|
70 | + return 'USER_ERROR_AUTH_REGISTER'; |
|
71 | + } |
|
60 | 72 | |
61 | - if ($check_name === 1) return ['name', 'USER_ERROR_NAME_DUPLICATE']; |
|
73 | + if ($check_name === 1) { |
|
74 | + return ['name', 'USER_ERROR_NAME_DUPLICATE']; |
|
75 | + } |
|
62 | 76 | |
63 | 77 | # Check email exists |
64 | 78 | |
65 | - if (false === ($check_email = $this->user->check($email, 'email'))) return 'USER_ERROR_AUTH_REGISTER'; |
|
79 | + if (false === ($check_email = $this->user->check($email, 'email'))) { |
|
80 | + return 'USER_ERROR_AUTH_REGISTER'; |
|
81 | + } |
|
66 | 82 | |
67 | - if ($check_email === 1) return ['email', 'USER_ERROR_EMAIL_DUPLICATE']; |
|
83 | + if ($check_email === 1) { |
|
84 | + return ['email', 'USER_ERROR_EMAIL_DUPLICATE']; |
|
85 | + } |
|
68 | 86 | |
69 | 87 | # Encode password |
70 | 88 | |
@@ -87,7 +105,9 @@ discard block |
||
87 | 105 | $data['time_registered'] = REQUEST_TIME; |
88 | 106 | $data['time_logged'] = REQUEST_TIME; |
89 | 107 | |
90 | - if (!$this->user->create($data)) return 'USER_ERROR_AUTH_REGISTER'; |
|
108 | + if (!$this->user->create($data)) { |
|
109 | + return 'USER_ERROR_AUTH_REGISTER'; |
|
110 | + } |
|
91 | 111 | |
92 | 112 | # Send mail |
93 | 113 |
@@ -12,7 +12,7 @@ |
||
12 | 12 | |
13 | 13 | protected function init() { |
14 | 14 | |
15 | - $this->config->addParam('active', '', function (bool $active) { |
|
15 | + $this->config->addParam('active', '', function(bool $active) { |
|
16 | 16 | |
17 | 17 | return ($active ? "ent.active = 1" : ''); |
18 | 18 | }); |
@@ -38,7 +38,9 @@ |
||
38 | 38 | |
39 | 39 | # Set remove button |
40 | 40 | |
41 | - if ($user->id === Auth::get('id')) $view->getBlock('remove')->class = 'disabled'; |
|
41 | + if ($user->id === Auth::get('id')) { |
|
42 | + $view->getBlock('remove')->class = 'disabled'; |
|
43 | + } |
|
42 | 44 | } |
43 | 45 | } |
44 | 46 | } |
@@ -78,11 +78,11 @@ discard block |
||
78 | 78 | |
79 | 79 | # Set link |
80 | 80 | |
81 | - $link = (INSTALL_PATH . static::$link . '/'); |
|
81 | + $link = (INSTALL_PATH.static::$link.'/'); |
|
82 | 82 | |
83 | - if (static::$nesting) $contents->link = ($link . ($this->create ? 'create' : 'edit') . '?id=' . $this->parent->id); |
|
83 | + if (static::$nesting) $contents->link = ($link.($this->create ? 'create' : 'edit').'?id='.$this->parent->id); |
|
84 | 84 | |
85 | - else $contents->link = ($link . ($this->create ? 'create' : ('edit?id=' . $this->entity->id))); |
|
85 | + else $contents->link = ($link.($this->create ? 'create' : ('edit?id='.$this->entity->id))); |
|
86 | 86 | |
87 | 87 | # Process selector block |
88 | 88 | |
@@ -143,7 +143,7 @@ discard block |
||
143 | 143 | |
144 | 144 | $this->entity = Entitizer::get(static::$table, (!$this->create ? $id : 0)); |
145 | 145 | |
146 | - if (!$this->create && (0 === $this->entity->id)) return Request::redirect(INSTALL_PATH . static::$link); |
|
146 | + if (!$this->create && (0 === $this->entity->id)) return Request::redirect(INSTALL_PATH.static::$link); |
|
147 | 147 | |
148 | 148 | # Create parent entity |
149 | 149 | |
@@ -163,7 +163,7 @@ discard block |
||
163 | 163 | |
164 | 164 | if ($this->create && (0 !== $this->parent->id)) $this->entity->move($this->parent->id); |
165 | 165 | |
166 | - Request::redirect(INSTALL_PATH . static::$link . '/edit?id=' . $this->entity->id . '&submitted'); |
|
166 | + Request::redirect(INSTALL_PATH.static::$link.'/edit?id='.$this->entity->id.'&submitted'); |
|
167 | 167 | } |
168 | 168 | |
169 | 169 | # Display success message |
@@ -56,7 +56,9 @@ discard block |
||
56 | 56 | |
57 | 57 | private function processSelector(Template\Block $selector) { |
58 | 58 | |
59 | - if ($this->create) return $selector->disable(); |
|
59 | + if ($this->create) { |
|
60 | + return $selector->disable(); |
|
61 | + } |
|
60 | 62 | |
61 | 63 | $selector->parent_id = $this->entity->parent_id; |
62 | 64 | |
@@ -81,25 +83,33 @@ discard block |
||
81 | 83 | |
82 | 84 | # Set path / title |
83 | 85 | |
84 | - if (static::$nesting) $contents->path = $this->path; |
|
85 | - |
|
86 | - else $contents->title = ($this->create ? Language::get(static::$naming_new) : $this->entity->get(static::$naming)); |
|
86 | + if (static::$nesting) { |
|
87 | + $contents->path = $this->path; |
|
88 | + } else { |
|
89 | + $contents->title = ($this->create ? Language::get(static::$naming_new) : $this->entity->get(static::$naming)); |
|
90 | + } |
|
87 | 91 | |
88 | 92 | # Process parent block |
89 | 93 | |
90 | - if (static::$nesting) $this->processParent($contents->getBlock('parent')); |
|
94 | + if (static::$nesting) { |
|
95 | + $this->processParent($contents->getBlock('parent')); |
|
96 | + } |
|
91 | 97 | |
92 | 98 | # Set link |
93 | 99 | |
94 | 100 | $link = (INSTALL_PATH . static::$link . '/'); |
95 | 101 | |
96 | - if (static::$nesting) $contents->link = ($link . ($this->create ? 'create' : 'edit') . '?id=' . $this->parent->id); |
|
97 | - |
|
98 | - else $contents->link = ($link . ($this->create ? 'create' : ('edit?id=' . $this->entity->id))); |
|
102 | + if (static::$nesting) { |
|
103 | + $contents->link = ($link . ($this->create ? 'create' : 'edit') . '?id=' . $this->parent->id); |
|
104 | + } else { |
|
105 | + $contents->link = ($link . ($this->create ? 'create' : ('edit?id=' . $this->entity->id))); |
|
106 | + } |
|
99 | 107 | |
100 | 108 | # Process selector block |
101 | 109 | |
102 | - if (static::$nesting) $this->processSelector($contents->getBlock('selector')); |
|
110 | + if (static::$nesting) { |
|
111 | + $this->processSelector($contents->getBlock('selector')); |
|
112 | + } |
|
103 | 113 | |
104 | 114 | # Implement form |
105 | 115 | |
@@ -134,11 +144,15 @@ discard block |
||
134 | 144 | |
135 | 145 | $parent_id = Number::forceInt(Request::post('parent_id')); |
136 | 146 | |
137 | - if (!$this->entity->move($parent_id)) return $ajax->setError(Language::get(static::$message_error_move)); |
|
147 | + if (!$this->entity->move($parent_id)) { |
|
148 | + return $ajax->setError(Language::get(static::$message_error_move)); |
|
149 | + } |
|
138 | 150 | |
139 | 151 | } else if (Request::post('action') === 'remove') { |
140 | 152 | |
141 | - if (!$this->entity->remove()) return $ajax->setError(Language::get(static::$message_error_remove)); |
|
153 | + if (!$this->entity->remove()) { |
|
154 | + return $ajax->setError(Language::get(static::$message_error_remove)); |
|
155 | + } |
|
142 | 156 | } |
143 | 157 | |
144 | 158 | # ------------------------ |
@@ -154,7 +168,9 @@ discard block |
||
154 | 168 | |
155 | 169 | protected function handle(bool $ajax = false) { |
156 | 170 | |
157 | - if (!$this->create && $ajax) return $this->handleAjax(); |
|
171 | + if (!$this->create && $ajax) { |
|
172 | + return $this->handleAjax(); |
|
173 | + } |
|
158 | 174 | |
159 | 175 | # Create entity |
160 | 176 | |
@@ -162,7 +178,9 @@ discard block |
||
162 | 178 | |
163 | 179 | $this->entity = Entitizer::get(static::$table, (!$this->create ? $id : 0)); |
164 | 180 | |
165 | - if (!$this->create && (0 === $this->entity->id)) return Request::redirect(INSTALL_PATH . static::$link); |
|
181 | + if (!$this->create && (0 === $this->entity->id)) { |
|
182 | + return Request::redirect(INSTALL_PATH . static::$link); |
|
183 | + } |
|
166 | 184 | |
167 | 185 | # Create parent entity |
168 | 186 | |
@@ -170,7 +188,9 @@ discard block |
||
170 | 188 | |
171 | 189 | # Get path |
172 | 190 | |
173 | - if (false !== ($path = $this->parent->getPath())) $this->path = $path; |
|
191 | + if (false !== ($path = $this->parent->getPath())) { |
|
192 | + $this->path = $path; |
|
193 | + } |
|
174 | 194 | |
175 | 195 | # Create form |
176 | 196 | |
@@ -180,7 +200,9 @@ discard block |
||
180 | 200 | |
181 | 201 | if ($this->form->handle(new static::$controller_class($this->entity), true)) { |
182 | 202 | |
183 | - if ($this->create && (0 !== $this->parent->id)) $this->entity->move($this->parent->id); |
|
203 | + if ($this->create && (0 !== $this->parent->id)) { |
|
204 | + $this->entity->move($this->parent->id); |
|
205 | + } |
|
184 | 206 | |
185 | 207 | Request::redirect(INSTALL_PATH . static::$link . '/edit?id=' . $this->entity->id . '&submitted'); |
186 | 208 | } |