@@ -14,8 +14,8 @@ |
||
| 14 | 14 | { |
| 15 | 15 | foreach( $command->shortcodes as $shortcode ) { |
| 16 | 16 | $shortcodeClass = glsr( Helper::class )->buildClassName( $shortcode.'-shortcode', 'Shortcodes' ); |
| 17 | - if( !class_exists( $shortcodeClass )) { |
|
| 18 | - glsr_log()->error( sprintf( 'Class missing (%s)', $shortcodeClass )); |
|
| 17 | + if( !class_exists( $shortcodeClass ) ) { |
|
| 18 | + glsr_log()->error( sprintf( 'Class missing (%s)', $shortcodeClass ) ); |
|
| 19 | 19 | continue; |
| 20 | 20 | } |
| 21 | 21 | add_shortcode( $shortcode, [glsr( $shortcodeClass ), 'buildShortcode'] ); |
@@ -16,8 +16,8 @@ |
||
| 16 | 16 | global $wp_widget_factory; |
| 17 | 17 | foreach( $command->widgets as $key => $values ) { |
| 18 | 18 | $widgetClass = glsr( Helper::class )->buildClassName( $key.'-widget', 'Widgets' ); |
| 19 | - if( !class_exists( $widgetClass )) { |
|
| 20 | - glsr_log()->error( sprintf( 'Class missing (%s)', $widgetClass )); |
|
| 19 | + if( !class_exists( $widgetClass ) ) { |
|
| 20 | + glsr_log()->error( sprintf( 'Class missing (%s)', $widgetClass ) ); |
|
| 21 | 21 | continue; |
| 22 | 22 | } |
| 23 | 23 | // Here we bypass register_widget() in order to pass our custom values to the widget |
@@ -11,10 +11,10 @@ |
||
| 11 | 11 | */ |
| 12 | 12 | public function handle( Command $command ) |
| 13 | 13 | { |
| 14 | - if( in_array( $command->postType, get_post_types( ['_builtin' => true] )))return; |
|
| 14 | + if( in_array( $command->postType, get_post_types( ['_builtin' => true] ) ) )return; |
|
| 15 | 15 | register_post_type( $command->postType, $command->args ); |
| 16 | 16 | glsr()->postTypeColumns = wp_parse_args( glsr()->postTypeColumns, [ |
| 17 | 17 | $command->postType => $command->columns, |
| 18 | - ]); |
|
| 18 | + ] ); |
|
| 19 | 19 | } |
| 20 | 20 | } |
@@ -12,10 +12,10 @@ |
||
| 12 | 12 | */ |
| 13 | 13 | public function handle( Command $command ) |
| 14 | 14 | { |
| 15 | - if( !get_post( $command->id )) { |
|
| 15 | + if( !get_post( $command->id ) ) { |
|
| 16 | 16 | return false; |
| 17 | 17 | } |
| 18 | - if( is_null( $command->pinned )) { |
|
| 18 | + if( is_null( $command->pinned ) ) { |
|
| 19 | 19 | $meta = get_post_meta( $command->id, 'pinned', true ); |
| 20 | 20 | $command->pinned = !wp_validate_boolean( $meta ); |
| 21 | 21 | } |
@@ -20,7 +20,7 @@ discard block |
||
| 20 | 20 | public function registerPostType() |
| 21 | 21 | { |
| 22 | 22 | if( !glsr()->hasPermission() )return; |
| 23 | - $command = new RegisterPostType([ |
|
| 23 | + $command = new RegisterPostType( [ |
|
| 24 | 24 | 'capabilities'=> ['create_posts' => 'create_'.Application::POST_TYPE], |
| 25 | 25 | 'columns' => [ |
| 26 | 26 | 'title' => '', |
@@ -38,7 +38,7 @@ discard block |
||
| 38 | 38 | 'plural' => __( 'Reviews', 'site-reviews' ), |
| 39 | 39 | 'post_type' => Application::POST_TYPE, |
| 40 | 40 | 'single' => __( 'Review', 'site-reviews' ), |
| 41 | - ]); |
|
| 41 | + ] ); |
|
| 42 | 42 | $this->execute( $command ); |
| 43 | 43 | } |
| 44 | 44 | |
@@ -48,11 +48,11 @@ discard block |
||
| 48 | 48 | */ |
| 49 | 49 | public function registerShortcodes() |
| 50 | 50 | { |
| 51 | - $command = new RegisterShortcodes([ |
|
| 51 | + $command = new RegisterShortcodes( [ |
|
| 52 | 52 | 'site_reviews', |
| 53 | 53 | 'site_reviews_form', |
| 54 | 54 | 'site_reviews_summary', |
| 55 | - ]); |
|
| 55 | + ] ); |
|
| 56 | 56 | $this->execute( $command ); |
| 57 | 57 | } |
| 58 | 58 | |
@@ -62,13 +62,13 @@ discard block |
||
| 62 | 62 | */ |
| 63 | 63 | public function registerTaxonomy() |
| 64 | 64 | { |
| 65 | - $command = new RegisterTaxonomy([ |
|
| 65 | + $command = new RegisterTaxonomy( [ |
|
| 66 | 66 | 'hierarchical' => true, |
| 67 | 67 | 'meta_box_cb' => 'glsr_categories_meta_box', |
| 68 | 68 | 'public' => false, |
| 69 | 69 | 'show_admin_column' => true, |
| 70 | 70 | 'show_ui' => true, |
| 71 | - ]); |
|
| 71 | + ] ); |
|
| 72 | 72 | $this->execute( $command ); |
| 73 | 73 | } |
| 74 | 74 | |
@@ -78,7 +78,7 @@ discard block |
||
| 78 | 78 | */ |
| 79 | 79 | public function registerWidgets() |
| 80 | 80 | { |
| 81 | - $command = new RegisterWidgets([ |
|
| 81 | + $command = new RegisterWidgets( [ |
|
| 82 | 82 | 'site-reviews' => [ |
| 83 | 83 | 'class' => 'glsr-widget glsr-widget-site-reviews', |
| 84 | 84 | 'description' => __( 'Your site’s most recent reviews.', 'site-reviews' ), |
@@ -94,7 +94,7 @@ discard block |
||
| 94 | 94 | 'description' => __( 'A summary of your site’s reviews.', 'site-reviews' ), |
| 95 | 95 | 'title' => __( 'Summary of Reviews', 'site-reviews' ), |
| 96 | 96 | ], |
| 97 | - ]); |
|
| 97 | + ] ); |
|
| 98 | 98 | $this->execute( $command ); |
| 99 | 99 | } |
| 100 | 100 | |
@@ -37,7 +37,7 @@ discard block |
||
| 37 | 37 | */ |
| 38 | 38 | public static function load() |
| 39 | 39 | { |
| 40 | - if( empty( static::$instance )) { |
|
| 40 | + if( empty(static::$instance) ) { |
|
| 41 | 41 | static::$instance = new static; |
| 42 | 42 | } |
| 43 | 43 | return static::$instance; |
@@ -49,14 +49,14 @@ discard block |
||
| 49 | 49 | */ |
| 50 | 50 | public function __get( $property ) |
| 51 | 51 | { |
| 52 | - if( property_exists( $this, $property ) && !in_array( $property, static::PROTECTED_PROPERTIES )) { |
|
| 52 | + if( property_exists( $this, $property ) && !in_array( $property, static::PROTECTED_PROPERTIES ) ) { |
|
| 53 | 53 | return $this->$property; |
| 54 | 54 | } |
| 55 | - $constant = 'static::'.strtoupper( $this->make( Helper::class )->snakeCase( $property )); |
|
| 56 | - if( defined( $constant )) { |
|
| 55 | + $constant = 'static::'.strtoupper( $this->make( Helper::class )->snakeCase( $property ) ); |
|
| 56 | + if( defined( $constant ) ) { |
|
| 57 | 57 | return constant( $constant ); |
| 58 | 58 | } |
| 59 | - return isset( $this->storage[$property] ) |
|
| 59 | + return isset($this->storage[$property]) |
|
| 60 | 60 | ? $this->storage[$property] |
| 61 | 61 | : null; |
| 62 | 62 | } |
@@ -68,14 +68,14 @@ discard block |
||
| 68 | 68 | */ |
| 69 | 69 | public function __set( $property, $value ) |
| 70 | 70 | { |
| 71 | - if( !property_exists( $this, $property ) || in_array( $property, static::PROTECTED_PROPERTIES )) { |
|
| 71 | + if( !property_exists( $this, $property ) || in_array( $property, static::PROTECTED_PROPERTIES ) ) { |
|
| 72 | 72 | $this->storage[$property] = $value; |
| 73 | 73 | } |
| 74 | - else if( !isset( $this->$property )) { |
|
| 74 | + else if( !isset($this->$property) ) { |
|
| 75 | 75 | $this->$property = $value; |
| 76 | 76 | } |
| 77 | 77 | else { |
| 78 | - throw new Exception( sprintf( 'The "%s" property cannot be changed once set.', $property )); |
|
| 78 | + throw new Exception( sprintf( 'The "%s" property cannot be changed once set.', $property ) ); |
|
| 79 | 79 | } |
| 80 | 80 | } |
| 81 | 81 | |
@@ -97,16 +97,16 @@ discard block |
||
| 97 | 97 | */ |
| 98 | 98 | public function make( $abstract ) |
| 99 | 99 | { |
| 100 | - if( !isset( $this->services[$abstract] )) { |
|
| 100 | + if( !isset($this->services[$abstract]) ) { |
|
| 101 | 101 | $abstract = $this->addNamespace( $abstract ); |
| 102 | 102 | } |
| 103 | - if( isset( $this->services[$abstract] )) { |
|
| 103 | + if( isset($this->services[$abstract]) ) { |
|
| 104 | 104 | $abstract = $this->services[$abstract]; |
| 105 | 105 | } |
| 106 | - if( is_callable( $abstract )) { |
|
| 106 | + if( is_callable( $abstract ) ) { |
|
| 107 | 107 | return call_user_func_array( $abstract, [$this] ); |
| 108 | 108 | } |
| 109 | - if( is_object( $abstract )) { |
|
| 109 | + if( is_object( $abstract ) ) { |
|
| 110 | 110 | return $abstract; |
| 111 | 111 | } |
| 112 | 112 | return $this->resolve( $abstract ); |
@@ -120,7 +120,7 @@ discard block |
||
| 120 | 120 | */ |
| 121 | 121 | public function singleton( $alias, $binding ) |
| 122 | 122 | { |
| 123 | - $this->bind( $alias, $this->make( $binding )); |
|
| 123 | + $this->bind( $alias, $this->make( $binding ) ); |
|
| 124 | 124 | } |
| 125 | 125 | |
| 126 | 126 | /** |
@@ -130,7 +130,7 @@ discard block |
||
| 130 | 130 | */ |
| 131 | 131 | protected function addNamespace( $abstract ) |
| 132 | 132 | { |
| 133 | - if( strpos( $abstract, __NAMESPACE__ ) === false && !class_exists( $abstract )) { |
|
| 133 | + if( strpos( $abstract, __NAMESPACE__ ) === false && !class_exists( $abstract ) ) { |
|
| 134 | 134 | $abstract = __NAMESPACE__.'\\'.$abstract; |
| 135 | 135 | } |
| 136 | 136 | return $abstract; |
@@ -152,7 +152,7 @@ discard block |
||
| 152 | 152 | throw new Exception( 'Target ['.$concrete.'] is not instantiable.' ); |
| 153 | 153 | } |
| 154 | 154 | $constructor = $reflector->getConstructor(); |
| 155 | - if( empty( $constructor )) { |
|
| 155 | + if( empty($constructor) ) { |
|
| 156 | 156 | return new $concrete; |
| 157 | 157 | } |
| 158 | 158 | return $reflector->newInstanceArgs( |
@@ -82,7 +82,7 @@ |
||
| 82 | 82 | { |
| 83 | 83 | $this->render()->div( glsr( Notice::class )->get(), [ |
| 84 | 84 | 'id' => 'glsr-notices', |
| 85 | - ]); |
|
| 85 | + ] ); |
|
| 86 | 86 | } |
| 87 | 87 | |
| 88 | 88 | /** |
@@ -130,8 +130,8 @@ |
||
| 130 | 130 | public function validateRequired( $attribute, $value ) |
| 131 | 131 | { |
| 132 | 132 | return is_null( $value ) |
| 133 | - || ( is_string( $value ) && trim( $value ) === '' ) |
|
| 134 | - || ( is_array( $value ) && count( $value ) < 1 ) |
|
| 133 | + || (is_string( $value ) && trim( $value ) === '') |
|
| 134 | + || (is_array( $value ) && count( $value ) < 1) |
|
| 135 | 135 | ? false |
| 136 | 136 | : true; |
| 137 | 137 | } |
@@ -15,10 +15,10 @@ discard block |
||
| 15 | 15 | */ |
| 16 | 16 | public function add( $type, $message, array $args = [] ) |
| 17 | 17 | { |
| 18 | - if( empty( array_filter( [$message, $type] )))return; |
|
| 18 | + if( empty(array_filter( [$message, $type] )) )return; |
|
| 19 | 19 | $args['message'] = $message; |
| 20 | 20 | $args['type'] = $type; |
| 21 | - add_settings_error( Application::ID, '', json_encode( $this->normalize( $args ))); |
|
| 21 | + add_settings_error( Application::ID, '', json_encode( $this->normalize( $args ) ) ); |
|
| 22 | 22 | } |
| 23 | 23 | |
| 24 | 24 | /** |
@@ -54,11 +54,11 @@ discard block |
||
| 54 | 54 | public function get() |
| 55 | 55 | { |
| 56 | 56 | $notices = array_map( 'unserialize', |
| 57 | - array_unique( array_map( 'serialize', get_settings_errors( Application::ID ))) |
|
| 57 | + array_unique( array_map( 'serialize', get_settings_errors( Application::ID ) ) ) |
|
| 58 | 58 | ); |
| 59 | - if( empty( $notices ))return; |
|
| 59 | + if( empty($notices) )return; |
|
| 60 | 60 | return array_reduce( $notices, function( $carry, $notice ) { |
| 61 | - return $carry.$this->buildNotice( json_decode( $notice['message'], true )); |
|
| 61 | + return $carry.$this->buildNotice( json_decode( $notice['message'], true ) ); |
|
| 62 | 62 | }); |
| 63 | 63 | } |
| 64 | 64 | |
@@ -79,7 +79,7 @@ discard block |
||
| 79 | 79 | } |
| 80 | 80 | return glsr( Builder::class )->div( $messages, [ |
| 81 | 81 | 'class' => $class, |
| 82 | - ]); |
|
| 82 | + ] ); |
|
| 83 | 83 | } |
| 84 | 84 | |
| 85 | 85 | /** |
@@ -94,13 +94,13 @@ discard block |
||
| 94 | 94 | 'type' => '', |
| 95 | 95 | ]; |
| 96 | 96 | $args = shortcode_atts( $defaults, $args ); |
| 97 | - if( !in_array( $args['type'], ['error', 'warning', 'success'] )) { |
|
| 97 | + if( !in_array( $args['type'], ['error', 'warning', 'success'] ) ) { |
|
| 98 | 98 | $args['type'] = 'success'; |
| 99 | 99 | } |
| 100 | 100 | $args['messages'] = is_wp_error( $args['message'] ) |
| 101 | 101 | ? (array)$args['message']->get_error_message() |
| 102 | 102 | : (array)$args['message']; |
| 103 | - unset( $args['message'] ); |
|
| 103 | + unset($args['message']); |
|
| 104 | 104 | return $args; |
| 105 | 105 | } |
| 106 | 106 | } |