@@ -21,7 +21,7 @@ |
||
21 | 21 | |
22 | 22 | public function required_arguments() |
23 | 23 | { |
24 | - return array('name','data'); |
|
24 | + return array('name', 'data'); |
|
25 | 25 | } |
26 | 26 | |
27 | 27 | public function get_template_path() |
@@ -6,7 +6,7 @@ |
||
6 | 6 | */ |
7 | 7 | |
8 | 8 | // Prevent direct file access |
9 | -defined( 'ABSPATH' ) or die( 'No script kiddies please!' ); |
|
9 | +defined('ABSPATH') or die('No script kiddies please!'); |
|
10 | 10 | |
11 | 11 | // Load module functions |
12 | 12 | require_once 'functions.php'; |
13 | 13 | \ No newline at end of file |
@@ -20,7 +20,7 @@ discard block |
||
20 | 20 | * @param array $props |
21 | 21 | * @return AbstractComponent |
22 | 22 | */ |
23 | - public static function create( $type, array $props ) |
|
23 | + public static function create($type, array $props) |
|
24 | 24 | { |
25 | 25 | try { |
26 | 26 | $component = self::create_registered_component($type, $props); |
@@ -41,9 +41,9 @@ discard block |
||
41 | 41 | * @param string $class_name |
42 | 42 | * @throws \RuntimeException |
43 | 43 | */ |
44 | - public static function register( $type, $class_name ) |
|
44 | + public static function register($type, $class_name) |
|
45 | 45 | { |
46 | - if( !in_array($type, self::$registered_components) ) |
|
46 | + if (!in_array($type, self::$registered_components)) |
|
47 | 47 | { |
48 | 48 | self::$registered_components[$type] = $class_name; |
49 | 49 | } |
@@ -56,15 +56,15 @@ discard block |
||
56 | 56 | * @param type $props |
57 | 57 | * @throws \RuntimeException |
58 | 58 | */ |
59 | - private static function create_core_component( $type, $props ) |
|
59 | + private static function create_core_component($type, $props) |
|
60 | 60 | { |
61 | 61 | $file_name = __DIR__."/components/$type/controller.php"; |
62 | 62 | $class_name = 'Amarkal\\UI\\Component_'.$type; |
63 | 63 | |
64 | 64 | // Load one of the core components |
65 | - if(!class_exists($class_name)) |
|
65 | + if (!class_exists($class_name)) |
|
66 | 66 | { |
67 | - if( file_exists( $file_name ) ) |
|
67 | + if (file_exists($file_name)) |
|
68 | 68 | { |
69 | 69 | require_once $file_name; |
70 | 70 | } |
@@ -83,9 +83,9 @@ discard block |
||
83 | 83 | * @param type $props |
84 | 84 | * @throws \RuntimeException |
85 | 85 | */ |
86 | - private static function create_registered_component( $type, $props ) |
|
86 | + private static function create_registered_component($type, $props) |
|
87 | 87 | { |
88 | - if(in_array($type, self::$registered_components)) |
|
88 | + if (in_array($type, self::$registered_components)) |
|
89 | 89 | { |
90 | 90 | $class_name = self::$registered_components[$type]; |
91 | 91 | return new $class_name($props); |
@@ -25,9 +25,9 @@ discard block |
||
25 | 25 | * @param array $model |
26 | 26 | * @param string $template_path |
27 | 27 | */ |
28 | - public function __construct( array $model = array(), $template_path = null ) |
|
28 | + public function __construct(array $model = array(), $template_path = null) |
|
29 | 29 | { |
30 | - $this->set_model( $model ); |
|
30 | + $this->set_model($model); |
|
31 | 31 | $this->template = $template_path; |
32 | 32 | } |
33 | 33 | |
@@ -38,9 +38,9 @@ discard block |
||
38 | 38 | * |
39 | 39 | * @return mixed the argument's value. |
40 | 40 | */ |
41 | - public function __get( $name ) |
|
41 | + public function __get($name) |
|
42 | 42 | { |
43 | - if( isset( $this->model[$name] ) ) |
|
43 | + if (isset($this->model[$name])) |
|
44 | 44 | { |
45 | 45 | return $this->model[$name]; |
46 | 46 | } |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | * |
55 | 55 | * @return mixed the settings' argument value. |
56 | 56 | */ |
57 | - public function __set( $name, $value ) |
|
57 | + public function __set($name, $value) |
|
58 | 58 | { |
59 | 59 | $this->model[$name] = $value; |
60 | 60 | } |
@@ -74,7 +74,7 @@ discard block |
||
74 | 74 | * |
75 | 75 | * @return array |
76 | 76 | */ |
77 | - public function set_model( $model ) |
|
77 | + public function set_model($model) |
|
78 | 78 | { |
79 | 79 | $this->model = $model; |
80 | 80 | } |
@@ -95,22 +95,22 @@ discard block |
||
95 | 95 | * @return string The rendered template. |
96 | 96 | * @throws TemplateNotFoundException Thrown if the template file cannot be found. |
97 | 97 | */ |
98 | - public function render( $echo = false ) |
|
98 | + public function render($echo = false) |
|
99 | 99 | { |
100 | 100 | $rendered_html = ''; |
101 | 101 | |
102 | - if( file_exists( $this->get_template_path() ) ) |
|
102 | + if (file_exists($this->get_template_path())) |
|
103 | 103 | { |
104 | 104 | ob_start(); |
105 | - include( $this->get_template_path() ); |
|
105 | + include($this->get_template_path()); |
|
106 | 106 | $rendered_html = ob_get_clean(); |
107 | 107 | } |
108 | 108 | else |
109 | 109 | { |
110 | - throw new \RuntimeException( "Error: cannot render HTML, template file not found at " . $this->get_template_path() ); |
|
110 | + throw new \RuntimeException("Error: cannot render HTML, template file not found at ".$this->get_template_path()); |
|
111 | 111 | } |
112 | 112 | |
113 | - if( !$echo ) |
|
113 | + if (!$echo) |
|
114 | 114 | { |
115 | 115 | return $rendered_html; |
116 | 116 | } |
@@ -6,13 +6,13 @@ |
||
6 | 6 | */ |
7 | 7 | |
8 | 8 | // Prevent direct file access |
9 | -defined( 'ABSPATH' ) or die( 'No script kiddies please!' ); |
|
9 | +defined('ABSPATH') or die('No script kiddies please!'); |
|
10 | 10 | |
11 | 11 | /** |
12 | 12 | * Load module functions. If this amarkal module has not been loaded, |
13 | 13 | * functions.php will not return false. |
14 | 14 | */ |
15 | -if(false !== (require_once 'functions.php')) |
|
15 | +if (false !== (require_once 'functions.php')) |
|
16 | 16 | { |
17 | 17 | // Load required classes if not using composer |
18 | 18 | require_once 'AbstractComponent.php'; |
@@ -27,8 +27,8 @@ discard block |
||
27 | 27 | 'required' => false, |
28 | 28 | 'readonly' => false, |
29 | 29 | 'default' => null, |
30 | - 'filter' => array( $this, 'filter' ), |
|
31 | - 'validation' => array( $this, 'validation' ) |
|
30 | + 'filter' => array($this, 'filter'), |
|
31 | + 'validation' => array($this, 'validation') |
|
32 | 32 | ); |
33 | 33 | } |
34 | 34 | |
@@ -37,19 +37,19 @@ discard block |
||
37 | 37 | return floatval($v); |
38 | 38 | } |
39 | 39 | |
40 | - public function validation($v,&$e) |
|
40 | + public function validation($v, &$e) |
|
41 | 41 | { |
42 | 42 | $max = $this->max; |
43 | 43 | $min = $this->min; |
44 | 44 | |
45 | - if(null !== $max && $v > $max) |
|
45 | + if (null !== $max && $v > $max) |
|
46 | 46 | { |
47 | - $e = sprintf( __("Value must be less than %d",'amarkal'), $max); |
|
47 | + $e = sprintf(__("Value must be less than %d", 'amarkal'), $max); |
|
48 | 48 | } |
49 | 49 | |
50 | - if(null !== $min && $v < $min) |
|
50 | + if (null !== $min && $v < $min) |
|
51 | 51 | { |
52 | - $e = sprintf( __("Value must be greater than %d",'amarkal'), $min); |
|
52 | + $e = sprintf(__("Value must be greater than %d", 'amarkal'), $min); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | return $e ? false : true; |
@@ -25,13 +25,13 @@ |
||
25 | 25 | 'min' => null, |
26 | 26 | 'max' => null, |
27 | 27 | 'step' => 1, |
28 | - 'filter' => array( $this, 'filter' ) |
|
28 | + 'filter' => array($this, 'filter') |
|
29 | 29 | ); |
30 | 30 | } |
31 | 31 | |
32 | 32 | public function required_arguments() |
33 | 33 | { |
34 | - return array('name','min','max'); |
|
34 | + return array('name', 'min', 'max'); |
|
35 | 35 | } |
36 | 36 | |
37 | 37 | public function get_template_path() |
@@ -29,7 +29,7 @@ |
||
29 | 29 | |
30 | 30 | public function required_arguments() |
31 | 31 | { |
32 | - return array('request_url','label_start'); |
|
32 | + return array('request_url', 'label_start'); |
|
33 | 33 | } |
34 | 34 | |
35 | 35 | public function get_template_path() |
@@ -19,7 +19,7 @@ discard block |
||
19 | 19 | * |
20 | 20 | * @param array $components |
21 | 21 | */ |
22 | - public function __construct( array $components = array() ) |
|
22 | + public function __construct(array $components = array()) |
|
23 | 23 | { |
24 | 24 | $this->add_components($components); |
25 | 25 | } |
@@ -30,12 +30,12 @@ discard block |
||
30 | 30 | * @param string $name |
31 | 31 | * @return AbstractComponent |
32 | 32 | */ |
33 | - public function get_by_name( $name ) |
|
33 | + public function get_by_name($name) |
|
34 | 34 | { |
35 | 35 | $filtered = $this->filter(function($c) use ($name) { |
36 | 36 | return $c instanceof ValueComponentInterface && $c->name === $name; |
37 | 37 | }); |
38 | - if(count($filtered) < 1) |
|
38 | + if (count($filtered) < 1) |
|
39 | 39 | { |
40 | 40 | throw new \RuntimeException("No component with the name <b>$name</b> could be found."); |
41 | 41 | } |
@@ -48,7 +48,7 @@ discard block |
||
48 | 48 | * @param string $type |
49 | 49 | * @return AbstractComponent[] |
50 | 50 | */ |
51 | - public function get_by_type( $type ) |
|
51 | + public function get_by_type($type) |
|
52 | 52 | { |
53 | 53 | return $this->filter(function($c) use ($type) { |
54 | 54 | return $c->component_type === $type; |
@@ -107,9 +107,9 @@ discard block |
||
107 | 107 | * @param AbstractComponent[] $components |
108 | 108 | * @return void |
109 | 109 | */ |
110 | - public function add_components( array $components ) |
|
110 | + public function add_components(array $components) |
|
111 | 111 | { |
112 | - foreach( $components as $component ) |
|
112 | + foreach ($components as $component) |
|
113 | 113 | { |
114 | 114 | $this->add_component($component); |
115 | 115 | } |
@@ -121,9 +121,9 @@ discard block |
||
121 | 121 | * @param AbstractComponent[] $args |
122 | 122 | * @return void |
123 | 123 | */ |
124 | - public function add_component( array $args ) |
|
124 | + public function add_component(array $args) |
|
125 | 125 | { |
126 | - if(!array_key_exists('type', $args)) |
|
126 | + if (!array_key_exists('type', $args)) |
|
127 | 127 | { |
128 | 128 | throw new \RuntimeException("Component configuration arrays must have a <b>type</b> argument"); |
129 | 129 | } |
@@ -140,16 +140,16 @@ discard block |
||
140 | 140 | * @param AbstractComponent $comp |
141 | 141 | * @return void |
142 | 142 | */ |
143 | - private function verify_name_uniqueness( AbstractComponent $comp ) |
|
143 | + private function verify_name_uniqueness(AbstractComponent $comp) |
|
144 | 144 | { |
145 | - if(!($comp instanceof ValueComponentInterface)) |
|
145 | + if (!($comp instanceof ValueComponentInterface)) |
|
146 | 146 | { |
147 | 147 | return; |
148 | 148 | } |
149 | 149 | |
150 | 150 | foreach ($this->components as $c) |
151 | 151 | { |
152 | - if($c->name === $comp->name) |
|
152 | + if ($c->name === $comp->name) |
|
153 | 153 | { |
154 | 154 | throw new \RuntimeException("Duplicate component name detected for the name <b>{$c->name}</b>."); |
155 | 155 | } |
@@ -162,7 +162,7 @@ discard block |
||
162 | 162 | * @param function $callable |
163 | 163 | * @return array |
164 | 164 | */ |
165 | - private function filter( $callable ) |
|
165 | + private function filter($callable) |
|
166 | 166 | { |
167 | 167 | return array_filter($this->components, $callable); |
168 | 168 | } |