@@ -29,9 +29,13 @@ discard block |
||
| 29 | 29 | |
| 30 | 30 | # Check name exists |
| 31 | 31 | |
| 32 | - if (false === ($check_name = $this->variable->check('name', $name))) return 'VARIABLE_ERROR_MODIFY'; |
|
| 32 | + if (false === ($check_name = $this->variable->check('name', $name))) { |
|
| 33 | + return 'VARIABLE_ERROR_MODIFY'; |
|
| 34 | + } |
|
| 33 | 35 | |
| 34 | - if ($check_name === 1) return 'VARIABLE_ERROR_NAME_DUPLICATE'; |
|
| 36 | + if ($check_name === 1) { |
|
| 37 | + return 'VARIABLE_ERROR_NAME_DUPLICATE'; |
|
| 38 | + } |
|
| 35 | 39 | |
| 36 | 40 | # Modify variable |
| 37 | 41 | |
@@ -43,7 +47,9 @@ discard block |
||
| 43 | 47 | |
| 44 | 48 | $modifier = ((0 === $this->variable->id) ? 'create' : 'edit'); |
| 45 | 49 | |
| 46 | - if (!$this->variable->$modifier($data)) return 'VARIABLE_ERROR_MODIFY'; |
|
| 50 | + if (!$this->variable->$modifier($data)) { |
|
| 51 | + return 'VARIABLE_ERROR_MODIFY'; |
|
| 52 | + } |
|
| 47 | 53 | |
| 48 | 54 | # ------------------------ |
| 49 | 55 | |
@@ -35,9 +35,13 @@ discard block |
||
| 35 | 35 | |
| 36 | 36 | # Check name exists |
| 37 | 37 | |
| 38 | - if (false === ($check_name = $this->page->check('hash', $hash))) return 'PAGE_ERROR_MODIFY'; |
|
| 38 | + if (false === ($check_name = $this->page->check('hash', $hash))) { |
|
| 39 | + return 'PAGE_ERROR_MODIFY'; |
|
| 40 | + } |
|
| 39 | 41 | |
| 40 | - if ($check_name === 1) return 'PAGE_ERROR_NAME_DUPLICATE'; |
|
| 42 | + if ($check_name === 1) { |
|
| 43 | + return 'PAGE_ERROR_NAME_DUPLICATE'; |
|
| 44 | + } |
|
| 41 | 45 | |
| 42 | 46 | # Modify page |
| 43 | 47 | |
@@ -57,7 +61,9 @@ discard block |
||
| 57 | 61 | |
| 58 | 62 | $modifier = ((0 === $this->page->id) ? 'create' : 'edit'); |
| 59 | 63 | |
| 60 | - if (!$this->page->$modifier($data)) return 'PAGE_ERROR_MODIFY'; |
|
| 64 | + if (!$this->page->$modifier($data)) { |
|
| 65 | + return 'PAGE_ERROR_MODIFY'; |
|
| 66 | + } |
|
| 61 | 67 | |
| 62 | 68 | # ------------------------ |
| 63 | 69 | |
@@ -2,7 +2,8 @@ |
||
| 2 | 2 | |
| 3 | 3 | namespace Modules\Entitizer\Controller { |
| 4 | 4 | |
| 5 | - use Modules\Entitizer, Arr; |
|
| 5 | + use Modules\Entitizer; |
|
| 6 | + use Arr; |
|
| 6 | 7 | |
| 7 | 8 | class Page { |
| 8 | 9 | |
@@ -39,7 +39,9 @@ |
||
| 39 | 39 | |
| 40 | 40 | $modifier = ((0 === $this->menuitem->id) ? 'create' : 'edit'); |
| 41 | 41 | |
| 42 | - if (!$this->menuitem->$modifier($data)) return 'MENUITEM_ERROR_MODIFY'; |
|
| 42 | + if (!$this->menuitem->$modifier($data)) { |
|
| 43 | + return 'MENUITEM_ERROR_MODIFY'; |
|
| 44 | + } |
|
| 43 | 45 | |
| 44 | 46 | # ------------------------ |
| 45 | 47 | |
@@ -29,9 +29,13 @@ discard block |
||
| 29 | 29 | |
| 30 | 30 | # Check name exists |
| 31 | 31 | |
| 32 | - if (false === ($check_name = $this->widget->check('name', $name))) return 'WIDGET_ERROR_MODIFY'; |
|
| 32 | + if (false === ($check_name = $this->widget->check('name', $name))) { |
|
| 33 | + return 'WIDGET_ERROR_MODIFY'; |
|
| 34 | + } |
|
| 33 | 35 | |
| 34 | - if ($check_name === 1) return 'WIDGET_ERROR_NAME_DUPLICATE'; |
|
| 36 | + if ($check_name === 1) { |
|
| 37 | + return 'WIDGET_ERROR_NAME_DUPLICATE'; |
|
| 38 | + } |
|
| 35 | 39 | |
| 36 | 40 | # Modify widget |
| 37 | 41 | |
@@ -44,7 +48,9 @@ discard block |
||
| 44 | 48 | |
| 45 | 49 | $modifier = ((0 === $this->widget->id) ? 'create' : 'edit'); |
| 46 | 50 | |
| 47 | - if (!$this->widget->$modifier($data)) return 'WIDGET_ERROR_MODIFY'; |
|
| 51 | + if (!$this->widget->$modifier($data)) { |
|
| 52 | + return 'WIDGET_ERROR_MODIFY'; |
|
| 53 | + } |
|
| 48 | 54 | |
| 49 | 55 | # ------------------------ |
| 50 | 56 | |
@@ -28,7 +28,9 @@ |
||
| 28 | 28 | |
| 29 | 29 | protected function processEntity(Template\Asset\Block $contents) { |
| 30 | 30 | |
| 31 | - if ($this->create) return $contents->block('info')->disable(); |
|
| 31 | + if ($this->create) { |
|
| 32 | + return $contents->block('info')->disable(); |
|
| 33 | + } |
|
| 32 | 34 | |
| 33 | 35 | $contents->block('info')->time_registered = Date::get(DATE_FORMAT_DATETIME, $this->entity->time_registered); |
| 34 | 36 | |
@@ -29,7 +29,9 @@ |
||
| 29 | 29 | |
| 30 | 30 | public static function get(string $type, int $id = 0) { |
| 31 | 31 | |
| 32 | - if (!isset(self::$classes[$type])) throw new Exception\General(self::$error_message); |
|
| 32 | + if (!isset(self::$classes[$type])) { |
|
| 33 | + throw new Exception\General(self::$error_message); |
|
| 34 | + } |
|
| 33 | 35 | |
| 34 | 36 | $cached = isset(self::$cache[$type][$id]); |
| 35 | 37 | |
@@ -14,13 +14,13 @@ |
||
| 14 | 14 | |
| 15 | 15 | # Add params |
| 16 | 16 | |
| 17 | - $this->addTextual ('name', true, 255, false, true, true); |
|
| 18 | - $this->addTextual ('title', true, 255, false, true, false); |
|
| 19 | - $this->addTextual ('value', true, 255, false, false, false); |
|
| 17 | + $this->addTextual('name', true, 255, false, true, true); |
|
| 18 | + $this->addTextual('title', true, 255, false, true, false); |
|
| 19 | + $this->addTextual('value', true, 255, false, false, false); |
|
| 20 | 20 | |
| 21 | 21 | # Add orderers |
| 22 | 22 | |
| 23 | - $this->addOrderer ('title'); |
|
| 23 | + $this->addOrderer('title'); |
|
| 24 | 24 | } |
| 25 | 25 | } |
| 26 | 26 | } |
@@ -14,23 +14,23 @@ |
||
| 14 | 14 | |
| 15 | 15 | # Add params |
| 16 | 16 | |
| 17 | - $this->addInteger ('parent_id', false, 10, 0, true, false); |
|
| 18 | - $this->addInteger ('visibility', true, 1, VISIBILITY_DRAFT, true, false); |
|
| 19 | - $this->addInteger ('access', true, 1, ACCESS_PUBLIC, true, false); |
|
| 20 | - $this->addTextual ('hash', true, 40, true, true, true); |
|
| 21 | - $this->addTextual ('name', true, 255, false, true, false); |
|
| 22 | - $this->addTextual ('title', true, 255, false, true, false); |
|
| 23 | - $this->addTextual ('contents', false, 0, false, false, false); |
|
| 24 | - $this->addTextual ('description', false, 0, false, false, false); |
|
| 25 | - $this->addTextual ('keywords', false, 0, false, false, false); |
|
| 26 | - $this->addBoolean ('robots_index', true, false); |
|
| 27 | - $this->addBoolean ('robots_follow', true, false); |
|
| 28 | - $this->addInteger ('time_created', false, 10, 0, true, false); |
|
| 29 | - $this->addInteger ('time_modified', false, 10, 0, true, false); |
|
| 17 | + $this->addInteger('parent_id', false, 10, 0, true, false); |
|
| 18 | + $this->addInteger('visibility', true, 1, VISIBILITY_DRAFT, true, false); |
|
| 19 | + $this->addInteger('access', true, 1, ACCESS_PUBLIC, true, false); |
|
| 20 | + $this->addTextual('hash', true, 40, true, true, true); |
|
| 21 | + $this->addTextual('name', true, 255, false, true, false); |
|
| 22 | + $this->addTextual('title', true, 255, false, true, false); |
|
| 23 | + $this->addTextual('contents', false, 0, false, false, false); |
|
| 24 | + $this->addTextual('description', false, 0, false, false, false); |
|
| 25 | + $this->addTextual('keywords', false, 0, false, false, false); |
|
| 26 | + $this->addBoolean('robots_index', true, false); |
|
| 27 | + $this->addBoolean('robots_follow', true, false); |
|
| 28 | + $this->addInteger('time_created', false, 10, 0, true, false); |
|
| 29 | + $this->addInteger('time_modified', false, 10, 0, true, false); |
|
| 30 | 30 | |
| 31 | 31 | # Add orderers |
| 32 | 32 | |
| 33 | - $this->addOrderer ('title'); |
|
| 33 | + $this->addOrderer('title'); |
|
| 34 | 34 | } |
| 35 | 35 | } |
| 36 | 36 | } |
@@ -14,9 +14,9 @@ |
||
| 14 | 14 | |
| 15 | 15 | # Add params |
| 16 | 16 | |
| 17 | - $this->addTextual ('code', true, 40, true, true, true); |
|
| 18 | - $this->addTextual ('ip', true, 255, false, true, false); |
|
| 19 | - $this->addInteger ('time', false, 10, 0, true, false); |
|
| 17 | + $this->addTextual('code', true, 40, true, true, true); |
|
| 18 | + $this->addTextual('ip', true, 255, false, true, false); |
|
| 19 | + $this->addInteger('time', false, 10, 0, true, false); |
|
| 20 | 20 | } |
| 21 | 21 | } |
| 22 | 22 | } |