michaeluno / admin-page-framework

development/cli/AdminPageFramework_InclusionClassFilesHeader.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * If accessed from a console, include the registry class to laod 'AdminPageFramework_Registry_Base'.
  */
 if ( php_sapi_name() === 'cli' ) {
-    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ) . '/admin-page-framework.php';
+    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ).'/admin-page-framework.php';
     if ( file_exists( $_sFrameworkFilePath ) ) {
         include_once( $_sFrameworkFilePath );
     }

development/cli/AdminPageFramework_BeautifiedVersionHeader.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * If accessed from a console, include the registry class to laod 'AdminPageFramework_Registry_Base'.
  */
 if ( php_sapi_name() === 'cli' ) {
-    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ) . '/admin-page-framework.php';
+    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ).'/admin-page-framework.php';
     if ( file_exists( $_sFrameworkFilePath ) ) {
         include_once( $_sFrameworkFilePath );
     }

development/factory/admin_page/_model/delegate/validaor/AdminPageFramework_Model__FormSubmission__Validator__ResetConfirm.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
                                 
-        if ( ! $this->_shouldProceed( $oFactory, $aSubmits ) ) {
+        if ( !$this->_shouldProceed( $oFactory, $aSubmits ) ) {
             return;
         }
         
@@ -46,7 +46,7 @@ 
         );            
         
         // Go to the catch clause.
-        $_oException = new Exception( 'aReturn' );  // the property name to return from the catch clasue.
+        $_oException = new Exception( 'aReturn' ); // the property name to return from the catch clasue.
         $_oException->aReturn = $this->_confirmSubmitButtonAction( 
             $this->getElement( $aSubmitInformation, 'input_name' ), 
             $this->getElement( $aSubmitInformation, 'section_id' ), 

development/factory/_common/form/field_type/AdminPageFramework_FieldType_size.php

Doc Comments +2 added lines

@@ -165,6 +165,7 @@ 
              * Returns the HTML output of the number input part.
              * 
              * @since       3.5.3
+             * @param boolean $bMultiLabels
              * @return      string      The number input output.
              */
             private function _getNumberInputPart( array $aField, array $aBaseAttributes, $isKey, $bMultiLabels ) {
@@ -217,6 +218,7 @@ 
              * Returns the HTML output of the unit select input part.
              * 
              * @since       3.5.3
+             * @param boolean $bMultiLabels
              * @return      string      The unit select input output.
              */
             private function _getUnitSelectInput( array $aField, array $aBaseAttributes, $isKey, $bMultiLabels ) {

Spacing +29 added lines, -29 removed lines

@@ -65,7 +65,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'size', );
+    public $aFieldTypeSlugs = array( 'size',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -74,7 +74,7 @@ 
      */
     protected $aDefaultKeys = array(
         'is_multiple'           => false, // indicates whether the select tag alloes multiple selections.
-        'units'                 => null,  // do not define units here since this will be merged with the user defined field array.
+        'units'                 => null, // do not define units here since this will be merged with the user defined field array.
         'attributes'            => array(
             'size'      => array(
                 'min'           => null,
@@ -102,7 +102,7 @@ 
      */
     protected $aDefaultUnits = array(
         'px'    => 'px', // pixel
-        '%'     => '%',  // percentage
+        '%'     => '%', // percentage
         'em'    => 'em', // font size
         'ex'    => 'ex', // font height
         'in'    => 'in', // inch
@@ -162,8 +162,8 @@ 
         );
 
         $_aOutput = array();
-        foreach( ( array ) $aField[ 'label' ] as $_isKey => $_sLabel ) {
-            $_aOutput[] = $this->_getFieldOutputByLabel( 
+        foreach ( ( array ) $aField[ 'label' ] as $_isKey => $_sLabel ) {
+            $_aOutput[ ] = $this->_getFieldOutputByLabel( 
                 $_isKey, 
                 $_sLabel,
                 $aField
@@ -186,8 +186,8 @@ 
 
             $_aBaseAttributes   = $_bMultiLabels
                 ? array(
-                        'name'  => $aField[ 'attributes' ][ 'name' ] . "[{$isKey}]",
-                        'id'    => $aField[ 'attributes' ][ 'id' ] . "_{$isKey}",
+                        'name'  => $aField[ 'attributes' ][ 'name' ]."[{$isKey}]",
+                        'id'    => $aField[ 'attributes' ][ 'id' ]."_{$isKey}",
                         'value' => $aField[ 'value' ],
                     ) 
                     + $aField[ 'attributes' ]
@@ -199,9 +199,9 @@ 
                      
             $_aOutput = array(
                 $this->getElementByLabel( $aField[ 'before_label' ], $isKey, $aField[ 'label' ] ),
-                    "<div " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-select-label' ) . ">",
-                        $this->_getNumberInputPart( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ),  // The size (number) part
-                        $this->_getUnitSelectInput( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ),  // The unit (select) part
+                    "<div ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-select-label' ).">",
+                        $this->_getNumberInputPart( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ), // The size (number) part
+                        $this->_getUnitSelectInput( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ), // The unit (select) part
                     "</div>",
                 $this->getElementByLabel( $aField[ 'after_label' ], $isKey, $aField[ 'label' ] )
             );
@@ -218,7 +218,7 @@ 
             private function _getNumberInputPart( array $aField, array $aBaseAttributes, $isKey, $bMultiLabels ) {
                 
                 // Size and Size Label
-                $_aSizeAttributes       = $this->_getSizeAttributes( 
+                $_aSizeAttributes = $this->_getSizeAttributes( 
                     $aField, 
                     $aBaseAttributes,
                     $bMultiLabels
@@ -226,32 +226,32 @@ 
                         : ''
                 );
 
-                $_aSizeLabelAttributes  = array(
+                $_aSizeLabelAttributes = array(
                     'for'   => $_aSizeAttributes[ 'id' ],
                     'class' => $_aSizeAttributes[ 'disabled' ] 
                         ? 'disabled' 
                         : null,
                 );                  
                 
-                $_sLabel                = $this->getElementByLabel( 
+                $_sLabel = $this->getElementByLabel( 
                     $aField[ 'label' ], 
                     $isKey, 
                     $aField[ 'label' ]
                 );
-                return "<label " . $this->getAttributes( $_aSizeLabelAttributes ) . ">"
+                return "<label ".$this->getAttributes( $_aSizeLabelAttributes ).">"
                     . $this->getElement( 
                         $aField, 
                         $bMultiLabels
                             ? array( 'before_label', $isKey, 'size' ) 
                             : array( 'before_label', 'size' ) 
                     )
-                    . ( $aField[ 'label' ] && ! $aField[ 'repeatable' ]
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                 . $_sLabel 
                             . "</span>"
                         : "" 
                     )
-                    . "<input " . $this->getAttributes( $_aSizeAttributes ) . " />" 
+                    . "<input ".$this->getAttributes( $_aSizeAttributes )." />" 
                     . $this->getElement( 
                         $aField, 
                         $bMultiLabels
@@ -294,7 +294,7 @@ 
                     )
                     : $aField[ 'units' ];
                 
-                return "<label " . $this->getAttributes( 
+                return "<label ".$this->getAttributes( 
                         array(
                             'for'       => $_aUnitAttributes[ 'id' ],
                             'class'     => $_aUnitAttributes[ 'disabled' ] 
@@ -326,9 +326,9 @@ 
                  * @return      array       an unit attribute array
                  * @internal
                  */
-                private function _getUnitAttributes( array $aField, array $aBaseAttributes, $isLabelKey='' ) {
+                private function _getUnitAttributes( array $aField, array $aBaseAttributes, $isLabelKey = '' ) {
                     
-                    $_bIsMultiple    = $aField[ 'is_multiple' ] 
+                    $_bIsMultiple = $aField[ 'is_multiple' ] 
                         ? true 
                         : $this->getElement( 
                             $aField,
@@ -340,13 +340,13 @@ 
               
                     $_aSelectAttributes = array(
                         'type'      => 'select',
-                        'id'        => $aField[ 'input_id' ] . ( '' === $isLabelKey ? '' : '_' . $isLabelKey ) . '_' . 'unit',
+                        'id'        => $aField[ 'input_id' ].( '' === $isLabelKey ? '' : '_'.$isLabelKey ).'_'.'unit',
                         'multiple'  => $_bIsMultiple 
                             ? 'multiple' 
                             : null,
                         'name'      => $_bIsMultiple 
-                            ? "{$aField['_input_name']}" . ( '' === $isLabelKey ? '' : '[' . $isLabelKey . ']' ) . "[unit][]" 
-                            : "{$aField['_input_name']}" . ( '' === $isLabelKey ? '' : '[' . $isLabelKey . ']' ) . "[unit]",
+                            ? "{$aField[ '_input_name' ]}".( '' === $isLabelKey ? '' : '['.$isLabelKey.']' )."[unit][]" 
+                            : "{$aField[ '_input_name' ]}".( '' === $isLabelKey ? '' : '['.$isLabelKey.']' )."[unit]",
                         'value'     => $this->getElement( 
                             $aField, 
                             array( 'value', 'unit' ),
@@ -358,7 +358,7 @@ 
                         '' === $isLabelKey
                             ? array( 'attributes', 'unit' )
                             : array( 'attributes', $isLabelKey, 'unit' ),
-                        $this->aDefaultKeys['attributes']['unit'] 
+                        $this->aDefaultKeys[ 'attributes' ][ 'unit' ] 
                     )
                     + $aBaseAttributes;       
                     return $_aSelectAttributes;
@@ -372,15 +372,15 @@ 
              * @return      array       an size attribute array
              * @internal
              */
-            private function _getSizeAttributes( array $aField, array $aBaseAttributes, $sLabelKey='' ) {
+            private function _getSizeAttributes( array $aField, array $aBaseAttributes, $sLabelKey = '' ) {
 
                 return array(
                         'type'  => 'number',
-                        'id'    => $aField['input_id'] . '_' . ( '' !== $sLabelKey ? $sLabelKey . '_' : '' ) . 'size',
-                        'name'  => $aField[ '_input_name' ] . ( '' !== $sLabelKey ? "[{$sLabelKey}]" : '' ) . '[size]',
+                        'id'    => $aField[ 'input_id' ].'_'.( '' !== $sLabelKey ? $sLabelKey.'_' : '' ).'size',
+                        'name'  => $aField[ '_input_name' ].( '' !== $sLabelKey ? "[{$sLabelKey}]" : '' ).'[size]',
                         'value' => $this->getElement(
-                            $aField,        // subject
-                            array( 'value', 'size' ),   // dimensional keys
+                            $aField, // subject
+                            array( 'value', 'size' ), // dimensional keys
                             ''  // default
                         ),
                     ) 

development/factory/_common/_abstract/AdminPageFramework_Factory_Router.php

Spacing +15 added lines, -15 removed lines

@@ -141,8 +141,8 @@ 
     public function __construct( $oProp ) {
 
         // Set sub-class names.
-        foreach( $this->_aSubClassPrefixes as $_sObjectVariableName => $_sPrefix ) {
-            $this->aSubClassNames[ $_sObjectVariableName ] = $_sPrefix . $this->_sStructureType;
+        foreach ( $this->_aSubClassPrefixes as $_sObjectVariableName => $_sPrefix ) {
+            $this->aSubClassNames[ $_sObjectVariableName ] = $_sPrefix.$this->_sStructureType;
         }
         $this->aSubClassNames = $this->aSubClassNames + $this->_aSubClassNames;
     
@@ -161,7 +161,7 @@ 
         // Required sub-class objects
         $this->oProp = $oProp;
             
-        if ( $this->oProp->bIsAdmin && ! $this->oProp->bIsAdminAjax ) {
+        if ( $this->oProp->bIsAdmin && !$this->oProp->bIsAdminAjax ) {
             if ( did_action( 'current_screen' ) ) {
                 $this->_replyToLoadComponents();
             } else {                
@@ -170,8 +170,8 @@ 
         }
         
         // Call the user constructor.
-        $this->start();     // defined in the controller class.
-        $this->oUtil->addAndDoAction( $this, 'start_' . $this->oProp->sClassName, $this );
+        $this->start(); // defined in the controller class.
+        $this->oUtil->addAndDoAction( $this, 'start_'.$this->oProp->sClassName, $this );
         
     }    
         
@@ -184,15 +184,15 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( !$this->_isInThePage() ) { 
             return; 
         }
                     
-        if ( ! isset( $this->oResource ) ) {
+        if ( !isset( $this->oResource ) ) {
             $this->oResource = $this->_replyTpSetAndGetInstance_oResource();
         }
         
-        if ( ! isset(  $this->oLink ) ) {
+        if ( !isset( $this->oLink ) ) {
             $this->oLink = $this->_replyTpSetAndGetInstance_oLink();         
         }
         
@@ -240,7 +240,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( !$this->_isInThePage() ) { 
             return; 
         }
 
@@ -312,7 +312,7 @@ 
         // Set and return the sub class object instance.
         if ( isset( $this->aSubClassNames[ $sPropertyName ] ) ) {
             return call_user_func( 
-                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}"  )
+                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}" )
             );
         }
         
@@ -346,7 +346,7 @@ 
          */              
         public function _replyTpSetAndGetInstance_oMsg() {
             $this->oMsg = call_user_func_array(
-                array( $this->aSubClassNames[ 'oMsg' ], 'getInstance'),
+                array( $this->aSubClassNames[ 'oMsg' ], 'getInstance' ),
                 array( $this->oProp->sTextDomain )  // parameters
             );
             return $this->oMsg;
@@ -411,7 +411,7 @@ 
      * 
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {    
+    public function __call( $sMethodName, $aArguments = null ) {    
          
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
         
@@ -441,7 +441,7 @@ 
             
             // Check if the method name does not contain a backslash.
             if ( false === strpos( $sMethodName, "\\" ) ) {
-                return $this->oUtil->getElement( $aArguments, 0 );  // the first element - the filter value
+                return $this->oUtil->getElement( $aArguments, 0 ); // the first element - the filter value
             }
                 
             // If the method name contains a backslash, the user may be using a name space. 
@@ -452,7 +452,7 @@ 
                     array( $this, $_sAutoCallbackMethodName ),
                     $aArguments
                 )
-                : $this->oUtil->getElement( $aArguments, 0 );   // the first argument
+                : $this->oUtil->getElement( $aArguments, 0 ); // the first argument
             
         }
         
@@ -462,7 +462,7 @@ 
          */
         private function _triggerUndefinedMethodWarning( $sMethodName ) {
             trigger_error(
-                AdminPageFramework_Registry::NAME . ': ' 
+                AdminPageFramework_Registry::NAME.': ' 
                     . sprintf( 
                         __( 'The method is not defined: %1$s', $this->oProp->sTextDomain ),
                         $sMethodName 

Braces +6 added lines, -6 removed lines

@@ -164,7 +164,7 @@ 
         if ( $this->oProp->bIsAdmin && ! $this->oProp->bIsAdminAjax ) {
             if ( did_action( 'current_screen' ) ) {
                 $this->_replyToLoadComponents();
-            } else {                
+            } else {
                 add_action( 'current_screen', array( $this, '_replyToLoadComponents' ) );
             }
         }
@@ -184,7 +184,7 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
                     
@@ -211,7 +211,7 @@ 
      * @since       3.1.0
      * @internal
      */ 
-    protected function _isInstantiatable() { 
+    protected function _isInstantiatable() {
         return true; 
     }
     
@@ -226,7 +226,7 @@ 
      * @todo        Change the visibility scope to `protected` as the public version of the method `isInThePage()` has been introduced to make the design consitent.
      * @internal
      */
-    public function _isInThePage() { 
+    public function _isInThePage() {
         return true; 
     }
          
@@ -240,7 +240,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
 
@@ -411,7 +411,7 @@ 
      * 
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {    
+    public function __call( $sMethodName, $aArguments=null ) {
          
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
         

development/factory/user_meta/AdminPageFramework_UserMeta.php

Spacing +6 added lines, -6 removed lines

@@ -39,16 +39,16 @@
      * @since       3.7.4       Changed the default capability value to `read`.
      * @todo        Examine the appropriate default capability level.
      */
-    public function __construct( $sCapability='read', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sCapability = 'read', $sTextDomain = 'admin-page-framework' ) {
         
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;        
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;        
         $this->oProp = new $_sProprtyClassName( 
-            $this,                  // the caller object
-            get_class( $this ),     // the caller class name    
-            $sCapability,           // the capability level
-            $sTextDomain,           // the text domain
+            $this, // the caller object
+            get_class( $this ), // the caller class name    
+            $sCapability, // the capability level
+            $sTextDomain, // the text domain
             $this->_sStructureType  // the structure type
         );     
         

development/factory/_common/form/field_type/AdminPageFramework_FieldType_media.php

Spacing +19 added lines, -19 removed lines

@@ -21,7 +21,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'media', );
+    public $aFieldTypeSlugs = array( 'media',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -53,7 +53,7 @@ 
         return 
             $this->_getScript_MediaUploader(
                 "admin_page_framework"
-            ) . PHP_EOL
+            ).PHP_EOL
             . $this->_getScript_RegisterCallbacks();
     }    
     
@@ -148,7 +148,7 @@ 
             $_sInsertFromURL         = esc_js( $this->oMsg->get( 'insert_from_url' ) );
             
             // If the WordPress version is 3.4.x or below
-            if ( ! function_exists( 'wp_enqueue_media' ) ) {
+            if ( !function_exists( 'wp_enqueue_media' ) ) {
                 return <<<JAVASCRIPTS
                     /**
                      * Bind/rebinds the thickbox script the given selector element.
@@ -388,10 +388,10 @@ 
             // Do not include the escaping character (backslash) in the heredoc variable declaration 
             // because the minifier script will parse it and the <<<JAVASCRIPTS and JAVASCRIPTS; parts are converted to double quotes (")
             // which causes the PHP syntax error.
-            $_sButtonHTML       = '"' . $this->_getUploaderButtonHTML_Media( $sInputID, $aButtonAttributes, $bExternalSource ) . '"';
-            $_sRpeatable        = $this->getAOrB( ! empty( $abRepeatable ), 'true', 'false' );
+            $_sButtonHTML       = '"'.$this->_getUploaderButtonHTML_Media( $sInputID, $aButtonAttributes, $bExternalSource ).'"';
+            $_sRpeatable        = $this->getAOrB( !empty( $abRepeatable ), 'true', 'false' );
             $_sExternalSource   = $this->getAOrB( $bExternalSource, 'true', 'false' );
-            $_sScript                = <<<JAVASCRIPTS
+            $_sScript = <<<JAVASCRIPTS
 if ( jQuery( 'a#select_media_{$sInputID}' ).length == 0 ) {
     jQuery( 'input#{$sInputID}' ).after( $_sButtonHTML );
 }
@@ -404,7 +404,7 @@ 
                     . '/* <![CDATA[ */'
                     . $_sScript 
                     . '/* ]]> */'
-                . "</script>". PHP_EOL;
+                . "</script>".PHP_EOL;
 
         }
             /**
@@ -414,19 +414,19 @@ 
              */
             private function _getUploaderButtonHTML_Media( $sInputID, array $aButtonAttributes, $bExternalSource ) {
                     
-                $_bIsLabelSet = isset( $aButtonAttributes['data-label'] ) && $aButtonAttributes['data-label'];
+                $_bIsLabelSet = isset( $aButtonAttributes[ 'data-label' ] ) && $aButtonAttributes[ 'data-label' ];
                 $_aAttributes = $this->_getFormattedUploadButtonAttributes_Media( 
                     $sInputID, 
                     $aButtonAttributes, 
                     $_bIsLabelSet, 
                     $bExternalSource 
                 );
-                return "<a " . $this->getAttributes( $_aAttributes ) . ">"
+                return "<a ".$this->getAttributes( $_aAttributes ).">"
                         . $this->getAOrB( 
                             $_bIsLabelSet,
-                            $_aAttributes['data-label'],
+                            $_aAttributes[ 'data-label' ],
                             $this->getAOrB(
-                                strrpos( $_aAttributes['class'], 'dashicons' ),
+                                strrpos( $_aAttributes[ 'class' ], 'dashicons' ),
                                 '',
                                 $this->oMsg->get( 'select_file' )
                             )
@@ -441,26 +441,26 @@ 
                  */
                 private function _getFormattedUploadButtonAttributes_Media( $sInputID, array $aButtonAttributes, $_bIsLabelSet, $bExternalSource ) {
                    
-                    $_aAttributes           = array(
+                    $_aAttributes = array(
                             'id'        => "select_media_{$sInputID}",
                             'href'      => '#',            
-                            'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ),    //  ? 1 : 0,
-                            'data-enable_external_source'   => ( string ) ( bool ) $bExternalSource,    //  ? 1 : 0, 
+                            'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ), //  ? 1 : 0,
+                            'data-enable_external_source'   => ( string ) ( bool ) $bExternalSource, //  ? 1 : 0, 
                         ) 
                         + $aButtonAttributes
                         + array(
                             'title'     => $_bIsLabelSet 
-                                ? $aButtonAttributes['data-label'] 
+                                ? $aButtonAttributes[ 'data-label' ] 
                                 : $this->oMsg->get( 'select_file' ),
                             'data-label' => null,
                         );
-                    $_aAttributes['class']  = $this->getClassAttribute( 
+                    $_aAttributes[ 'class' ] = $this->getClassAttribute( 
                         'select_media button button-small ',
                         $this->getAOrB(
-                            trim( $aButtonAttributes['class'] ),
-                            $aButtonAttributes['class'],
+                            trim( $aButtonAttributes[ 'class' ] ),
+                            $aButtonAttributes[ 'class' ],
                             $this->getAOrB( 
-                                ! $_bIsLabelSet && version_compare( $GLOBALS['wp_version'], '3.8', '>=' ),
+                                !$_bIsLabelSet && version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' ),
                                 'dashicons dashicons-portfolio',
                                 ''
                             )

admin-page-framework-loader.php

Braces +1 added lines, -1 removed lines

@@ -190,7 +190,7 @@
      * @return      void
      */ 
     static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {  
+        if ( ! is_admin() ) {
             return; 
         }
         self::$_aAdminNotices[] = array(

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.7';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.7'; // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
     const NAME           = 'Admin Page Framework - Loader'; // the name is not 'Admin Page Framework' because warning messages gets confusing.
-    const SHORTNAME      = 'Admin Page Framework';  // used for a menu title etc.
+    const SHORTNAME      = 'Admin Page Framework'; // used for a menu title etc.
     const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
     const URI            = 'http://admin-page-framework.michaeluno.jp/';
     const AUTHOR         = 'miunosoft (Michael Uno)';
@@ -55,14 +55,14 @@ 
      * @remark      This is also accessed from `uninstall.php` so do not remove.
      * @remark      Do not exceed 8 characters as a transient name allows 45 characters or less ( 40 for site transients ) so that md5 (32 characters) can be added.
      */
-    const TRANSIENT_PREFIX         = 'APFL_';
+    const TRANSIENT_PREFIX = 'APFL_';
 
     /**
      * The hook slug used for the prefix of action and filter hook names.
      * 
      * @remark      The ending underscore is not necessary.
      */
-    const HOOK_SLUG                = 'admin_page_framework_loader';
+    const HOOK_SLUG = 'admin_page_framework_loader';
 
     /**
      * The text domain slug and its path.
@@ -116,7 +116,7 @@ 
      */
     static public $aAdminPages = array(
         // key => 'page slug'
-        'about'     => 'apfl_about',        // the welcome page
+        'about'     => 'apfl_about', // the welcome page
         'addon'     => 'apfl_addons',
         'tool'      => 'apfl_tools',
         'help'      => 'apfl_contact',
@@ -154,12 +154,12 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public static function getPluginURL( $sRelativePath='' ) {
+    public static function getPluginURL( $sRelativePath = '' ) {
         if ( isset( self::$_sPluginURLCache ) ) {
-            return self::$_sPluginURLCache . $sRelativePath;
+            return self::$_sPluginURLCache.$sRelativePath;
         }
         self::$_sPluginURLCache = trailingslashit( plugins_url( '', self::$sFilePath ) );
-        return self::$_sPluginURLCache . $sRelativePath;
+        return self::$_sPluginURLCache.$sRelativePath;
     }
         /**
          * @since       3.7.9
@@ -189,13 +189,13 @@ 
      * @since       3.5.0
      * @return      void
      */ 
-    static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {  
+    static public function setAdminNotice( $sMessage, $sClassAttribute = 'error' ) {
+        if ( !is_admin() ) {  
             return; 
         }
-        self::$_aAdminNotices[] = array(
+        self::$_aAdminNotices[ ] = array(
             'message'           => $sMessage,
-            'class_attribute'   => trim( $sClassAttribute ) . ' notice is-dismissible',
+            'class_attribute'   => trim( $sClassAttribute ).' notice is-dismissible',
         );
         add_action( 'admin_notices', array( __CLASS__, '_replyToSetAdminNotice' ) );
     }
@@ -205,12 +205,12 @@ 
          * @return      void
          */
         static public function _replyToSetAdminNotice() {
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
-                echo "<div class='" . esc_attr( $_aAdminNotice['class_attribute'] ) . " notice is-dismissible'>"
+            foreach ( self::$_aAdminNotices as $_aAdminNotice ) {
+                echo "<div class='".esc_attr( $_aAdminNotice[ 'class_attribute' ] )." notice is-dismissible'>"
                         ."<p>"
                             . sprintf(
-                                '<strong>%1$s</strong>: ' . $_aAdminNotice['message'],
-                                self::NAME . ' ' . self::VERSION
+                                '<strong>%1$s</strong>: '.$_aAdminNotice[ 'message' ],
+                                self::NAME.' '.self::VERSION
                             )
                         . "</p>"
                     . "</div>";
@@ -222,7 +222,7 @@ 
 AdminPageFrameworkLoader_Registry::setUp( __FILE__ );
 
 // Initial checks. - Do no load if accessed directly, not exiting because the 'uninstall.php' and inclusion list generator will load this file.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( !defined( 'ABSPATH' ) ) {
     return;
 }
 if ( defined( 'DOING_UNINSTALL' ) ) {
@@ -234,8 +234,8 @@ 
 
     $_bFrameworkLoaded = class_exists( 'AdminPageFramework_Registry', false );
     if (
-        ! $_bFrameworkLoaded
-        || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
+        !$_bFrameworkLoaded
+        || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
         || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
     ) {
         AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -253,18 +253,18 @@ 
 add_action( 'admin_init', 'AdminPageFrameworkLoader_Warning' );
 
 // Include the library file - the development version will be available if you cloned the GitHub repository.
-$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/development/admin-page-framework.php';
+$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/development/admin-page-framework.php';
 $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
 $_bLoadDevelopmentVersion = $_bDebugMode && file_exists( $_sDevelopmentVersionPath );
 include(
     $_bLoadDevelopmentVersion
         ? $_sDevelopmentVersionPath
-        : AdminPageFrameworkLoader_Registry::$sDirPath . '/library/apf/admin-page-framework.php'
+        : AdminPageFrameworkLoader_Registry::$sDirPath.'/library/apf/admin-page-framework.php'
 );
 
 // Include the framework loader plugin components.
 include( AdminPageFramework_Registry::$aClassFiles[ 'AdminPageFramework_PluginBootstrap' ] );
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
 new AdminPageFrameworkLoader_Bootstrap(
     AdminPageFrameworkLoader_Registry::$sFilePath,
     AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Router.php

Spacing +5 added lines, -5 removed lines

@@ -28,7 +28,7 @@ 
                 
         parent::__construct( $oProp );
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
         
@@ -39,7 +39,7 @@ 
         );
         
         add_action( 
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
             array( $this, '_replyToSetUpHooks' )
         );
         
@@ -59,11 +59,11 @@ 
             return true;
         }    
         
-        if ( ! in_array( $this->oProp->sPageNow, array( 'edit-tags.php', 'term.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'edit-tags.php', 'term.php' ) ) ) {
             return false;
         }
         
-        if ( isset( $_GET[ 'taxonomy' ] ) && ! in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {
+        if ( isset( $_GET[ 'taxonomy' ] ) && !in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {
             return false;
         }        
         
@@ -80,7 +80,7 @@ 
      */
     public function _replyToSetUpHooks( $oFactory ) {
         
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
+        foreach ( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
             
             // Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not.
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );

Braces +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
      */
     public function _replyToSetUpHooks( $oFactory ) {
         
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
+        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {
             
             // Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not.
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );