amarkal / amarkal-ui

components/composite/controller.php

Spacing +22 added lines, -22 removed lines

@@ -25,16 +25,16 @@ 
      * The __set magic method is overridden here to apply value & name changes to 
      * child components.
      */
-    public function __set( $name, $value )
+    public function __set($name, $value)
     {
         parent::__set($name, $value);
         
-        if( 'value' === $name )
+        if ('value' === $name)
         {
             $this->set_value($value);
         }
 
-        if( 'name' === $name )
+        if ('name' === $name)
         {
             $this->set_name($value);
         }
@@ -46,9 +46,9 @@ 
      * @param array $value
      * @return void
      */
-    public function set_value( array $value )
+    public function set_value(array $value)
     {
-        foreach($value as $n => $v)
+        foreach ($value as $n => $v)
         {
             $component = $this->get_component($n);
             $component->value = $v;
@@ -61,9 +61,9 @@ 
      * @param string $name
      * @return void
      */
-    public function set_name( $name )
+    public function set_name($name)
     {
-        foreach($this->components as $c)
+        foreach ($this->components as $c)
         {
             $c->name_template = str_replace('{{parent_name}}', $this->get_name(), $c->composite_name_template);
         }
@@ -91,7 +91,7 @@ 
      */
     public function required_arguments()
     {
-        return array('name','components','template');
+        return array('name', 'components', 'template');
     }
     
     /**
@@ -109,7 +109,7 @@ 
      */
     public function parse_template()
     {
-        return preg_replace_callback('/\{\{([a-zA-Z\d-_]+)\}\}/', function($a){
+        return preg_replace_callback('/\{\{([a-zA-Z\d-_]+)\}\}/', function($a) {
             $component = $this->get_component($a[1]);
             return $component->render();
         }, $this->model['template']);
@@ -124,7 +124,7 @@ 
      */
     public function get_name()
     {
-        if('' !== $this->parent_name)
+        if ('' !== $this->parent_name)
         {
             return "{$this->parent_name}[{$this->name}]";
         }
@@ -133,9 +133,9 @@ 
 
     public function filter($v)
     {
-        foreach($this->components as $component)
+        foreach ($this->components as $component)
         {
-            if($component instanceof FilterableComponentInterface &&
+            if ($component instanceof FilterableComponentInterface &&
                \is_callable($component->filter))
             {
                 $n = $component->name;
@@ -146,15 +146,15 @@ 
         return $v;
     }
 
-    public function validation($v,&$e)
+    public function validation($v, &$e)
     {
-        foreach($this->components as $component)
+        foreach ($this->components as $component)
         {
-            if($component instanceof ValidatableComponentInterface &&
+            if ($component instanceof ValidatableComponentInterface &&
                \is_callable($component->validation))
             {
                 $n = $component->name;
-                if(!$component->validation($v[$n],$e))
+                if (!$component->validation($v[$n], $e))
                 {
                     return false;
                 }
@@ -168,7 +168,7 @@ 
      */
     protected function on_created()
     {
-        foreach( $this->model['components'] as $args )
+        foreach ($this->model['components'] as $args)
         {
             $this->components[$args['name']] = $this->create_component($args);
         }
@@ -179,16 +179,16 @@ 
      * @param type $args
      * @return type
      */
-    private function create_component( $args )
+    private function create_component($args)
     {
         $type = $args['type'];
         
-        if('composite' === $type)
+        if ('composite' === $type)
         {
             $args['parent_name'] = $this->get_name();
         }
         
-        $c = \Amarkal\UI\ComponentFactory::create( $type, $args );
+        $c = \Amarkal\UI\ComponentFactory::create($type, $args);
         
         // Apply the composite name template
         $c->name_template = str_replace('{{parent_name}}', $this->get_name(), $c->composite_name_template);
@@ -203,9 +203,9 @@ 
      * @return UI\AbstractComponent
      * @throws \RuntimeException If there's no child component corresponding to the given name
      */
-    private function get_component( $name )
+    private function get_component($name)
     {
-        if(!array_key_exists($name, $this->components))
+        if (!array_key_exists($name, $this->components))
         {
             throw new \RuntimeException("Composite sub-component not found with name $name");
         }

components/toggle/controller.php

Spacing +5 added lines, -5 removed lines

@@ -24,13 +24,13 @@ 
             'multi'         => false,
             'data'          => array(),
             'default'       => array(),
-            'filter'        => array( $this, 'filter' )
+            'filter'        => array($this, 'filter')
         );
     }
     
     public function required_arguments()
     {
-        return array('name','data');
+        return array('name', 'data');
     }
     
     public function get_template_path() 
@@ -49,16 +49,16 @@ 
      */
     public function filter($v)
     {
-        if($this->multi && !\is_array($v)) 
+        if ($this->multi && !\is_array($v)) 
         {
             return \explode(',', $v);
         }
         return $v;
     }
 
-    protected function is_selected( $value )
+    protected function is_selected($value)
     {
-        if($this->multi) 
+        if ($this->multi) 
         {
             return \in_array($value, $this->value);
         }

functions.php

Spacing +14 added lines, -14 removed lines

@@ -12,15 +12,15 @@ 
  */
 
 // Prevent direct file access
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'AMARKAL_UI' ) ) return false;
-define( 'AMARKAL_UI', true );
+if (defined('AMARKAL_UI')) return false;
+define('AMARKAL_UI', true);
 
-if(!function_exists('amarkal_ui_render'))
+if (!function_exists('amarkal_ui_render'))
 {
     /**
      * Render a UI component.
@@ -30,14 +30,14 @@ 
      * @param array $props The component's properties
      * @return string The rendered HTML
      */
-    function amarkal_ui_render( $type, array $props = array() )
+    function amarkal_ui_render($type, array $props = array())
     {
-        $component = Amarkal\UI\ComponentFactory::create( $type, $props );
+        $component = Amarkal\UI\ComponentFactory::create($type, $props);
         return $component->render();
     }
 }
 
-if(!function_exists('amarkal_ui_register_component'))
+if (!function_exists('amarkal_ui_register_component'))
 {
     /**
      * Register a custom UI component. The registered component's class should
@@ -47,13 +47,13 @@ 
      * to one of the core component's type, it will override the core component.
      * @param string $class_name The component's class name.
      */
-    function amarkal_ui_register_component( $type, $class_name )
+    function amarkal_ui_register_component($type, $class_name)
     {
-        Amarkal\UI\ComponentFactory::register( $type, $class_name );
+        Amarkal\UI\ComponentFactory::register($type, $class_name);
     }
 }
 
-if(!function_exists('amarkal_ui_register_scripts'))
+if (!function_exists('amarkal_ui_register_scripts'))
 {
     /**
      * Register Amarkal UI styles & scripts. These scripts are later enqueued by
@@ -61,9 +61,9 @@ 
      */
     function amarkal_ui_register_scripts()
     {
-        \wp_register_script('amarkal-ui',Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/js/dist/amarkal-ui.min.js'),array('jquery'),false,true);
-        \wp_register_script('ace-editor','https://cdnjs.cloudflare.com/ajax/libs/ace/1.2.8/ace.js',array('jquery'),false,true);
-        \wp_register_style('amarkal-ui',Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/css/dist/amarkal-ui.min.css'),array());
+        \wp_register_script('amarkal-ui', Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/js/dist/amarkal-ui.min.js'), array('jquery'), false, true);
+        \wp_register_script('ace-editor', 'https://cdnjs.cloudflare.com/ajax/libs/ace/1.2.8/ace.js', array('jquery'), false, true);
+        \wp_register_style('amarkal-ui', Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/css/dist/amarkal-ui.min.css'), array());
     }
-    \add_action('admin_init','amarkal_ui_register_scripts');
+    \add_action('admin_init', 'amarkal_ui_register_scripts');
 }
\ No newline at end of file

components/code/controller.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     
     public function required_arguments()
     {
-        return array('name','language');
+        return array('name', 'language');
     }
     
     public function get_template_path() 

components/progressbar/controller.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     
     public function required_arguments()
     {
-        return array('value','min','max');
+        return array('value', 'min', 'max');
     }
     
     public function get_template_path() 

ComponentList.php

Spacing +12 added lines, -12 removed lines

@@ -19,7 +19,7 @@ 
      *
      * @param array $components
      */
-    public function __construct( array $components = array() )
+    public function __construct(array $components = array())
     {
         $this->add_components($components);
     }
@@ -30,12 +30,12 @@ 
      * @param string $name
      * @return AbstractComponent
      */
-    public function get_by_name( $name )
+    public function get_by_name($name)
     {
         $filtered = $this->filter(function($c) use ($name) {
             return $c instanceof ValueComponentInterface && $c->name === $name;
         });
-        if(count($filtered) < 1)
+        if (count($filtered) < 1)
         {
             throw new \RuntimeException("No component with the name <b>$name</b> could be found.");
         }
@@ -48,7 +48,7 @@ 
      * @param string $type
      * @return AbstractComponent[]
      */
-    public function get_by_type( $type )
+    public function get_by_type($type)
     {
         return $this->filter(function($c) use ($type) {
             return $c->component_type === $type;
@@ -107,9 +107,9 @@ 
      * @param AbstractComponent[] $components
      * @return void
      */
-    public function add_components( array $components )
+    public function add_components(array $components)
     {
-        foreach( $components as $component )
+        foreach ($components as $component)
         {
             $this->add_component($component);
         }
@@ -121,9 +121,9 @@ 
      * @param AbstractComponent[] $args
      * @return void
      */
-    public function add_component( array $args )
+    public function add_component(array $args)
     {
-        if(!array_key_exists('type', $args))
+        if (!array_key_exists('type', $args))
         {
             throw new \RuntimeException("Component configuration arrays must have a <b>type</b> argument");
         }
@@ -140,16 +140,16 @@ 
      * @param AbstractComponent $comp
      * @return void
      */
-    private function verify_name_uniqueness( AbstractComponent $comp )
+    private function verify_name_uniqueness(AbstractComponent $comp)
     {
-        if(!($comp instanceof ValueComponentInterface))
+        if (!($comp instanceof ValueComponentInterface))
         {
             return;
         }
 
         foreach ($this->components as $c) 
         {
-            if($c->name === $comp->name)
+            if ($c->name === $comp->name)
             {
                 throw new \RuntimeException("Duplicate component name detected for the name <b>{$c->name}</b>.");
             }
@@ -162,7 +162,7 @@ 
      * @param function $callable
      * @return array
      */
-    private function filter( $callable )
+    private function filter($callable)
     {
         // array_values is needed in order to reindex the array after filtering it
         return array_values(array_filter($this->components, $callable));

Form.php

Spacing +23 added lines, -23 removed lines

@@ -59,9 +59,9 @@ 
      * 
      * @param ComponentList $component_list
      */
-    public function __construct( ComponentList $component_list = null )
+    public function __construct(ComponentList $component_list = null)
     {
-        if(null === $component_list)
+        if (null === $component_list)
         {
             $component_list = new ComponentList();
         }
@@ -85,16 +85,16 @@ 
      * 
      * @return array The updated values array.
      */
-    public function update( array $new_instance = array(), array $old_instance = array() )
+    public function update(array $new_instance = array(), array $old_instance = array())
     {
-        $this->old_instance   = array_merge($this->get_defaults(),$old_instance);
-        $this->new_instance   = array_merge($this->old_instance,$new_instance);
+        $this->old_instance   = array_merge($this->get_defaults(), $old_instance);
+        $this->new_instance   = array_merge($this->old_instance, $new_instance);
         $this->final_instance = $this->new_instance;
         
-        foreach ( $this->component_list->get_value_components() as $component ) 
+        foreach ($this->component_list->get_value_components() as $component) 
         {
             // Update individual fields, as well as the composite parent field.
-            $this->update_component( $component );
+            $this->update_component($component);
         }
         
         return $this->final_instance;
@@ -118,9 +118,9 @@ 
      * @param array $components
      * @return void
      */
-    public function reset_components( array $components )
+    public function reset_components(array $components)
     {
-        foreach( $components as $component )
+        foreach ($components as $component)
         {
             $this->reset_component($component);
         }
@@ -133,7 +133,7 @@ 
      * @param AbstractComponent $component
      * @return void
      */
-    public function reset_component( $component )
+    public function reset_component($component)
     {
         $component->value = $component->default;
         $this->final_instance[$component->name] = $component->default;
@@ -168,21 +168,21 @@ 
      * 
      * @param ValueComponentInterface $component The component to validate.
      */
-    private function update_component( ValueComponentInterface $component )
+    private function update_component(ValueComponentInterface $component)
     {
         $component->value = $this->final_instance[$component->name];
         
         // Skip if this field is disabled
-        if( $this->is_disabled($component) )
+        if ($this->is_disabled($component))
         {
             return;
         }
 
         // Apply user-defined filter
-        $this->filter( $component );
+        $this->filter($component);
         
         // Validate value
-        $this->validate( $component );
+        $this->validate($component);
     }
     
     /**
@@ -191,7 +191,7 @@ 
      * @param UI\AbstractComponent $component
      * @return boolean
      */
-    private function is_disabled( $component )
+    private function is_disabled($component)
     {
         return 
             $component instanceof DisableableComponentInterface &&
@@ -206,13 +206,13 @@ 
      * 
      * @param UI\AbstractComponent $component
      */
-    private function filter( $component )
+    private function filter($component)
     {
-        if( $component instanceof FilterableComponentInterface )
+        if ($component instanceof FilterableComponentInterface)
         {
             $filter = $component->filter;
 
-            if( is_callable( $filter ) ) 
+            if (is_callable($filter)) 
             {
                 $component->value = \call_user_func_array($filter, array($this->final_instance[$component->name]));
                 $this->final_instance[$component->name] = $component->value;
@@ -228,22 +228,22 @@ 
      * 
      * @param UI\AbstractComponent $component The component to validate.
      */
-    private function validate( $component )
+    private function validate($component)
     {
-        if( !($component instanceof ValidatableComponentInterface) ) return;
+        if (!($component instanceof ValidatableComponentInterface)) return;
         
         $name     = $component->name;
         $validate = $component->validation;
         
         $component->validity = $component::VALID;
         
-        if(is_callable($validate))
+        if (is_callable($validate))
         {
             $error = '';
             $valid = \call_user_func_array($validate, array($this->final_instance[$name], &$error));
             
             // Invalid input, use old instance or default value
-            if ( true !== $valid ) 
+            if (true !== $valid) 
             {
                 $this->errors[$name]         = $error ? $error : ValidatableComponentInterface::DEFAULT_MESSAGE;
                 $component->value            = $this->old_instance[$name];
@@ -262,7 +262,7 @@ 
     {
         $defaults = array();
         
-        foreach( $this->component_list->get_value_components() as $component )
+        foreach ($this->component_list->get_value_components() as $component)
         {
             $defaults[$component->name] = $component->default;
         }