michaeluno / admin-page-framework

development/factory/_common/form/input/AdminPageFramework_Input_checkbox.php

Doc Comments -2 removed lines

@@ -21,8 +21,6 @@
      * Returns the output of the input element.
      * 
      * @since       3.4.0    
-     * @param       string      $sLabel         The label text.
-     * @param       array       $aAttributes    (optional) The attribute array. If set, it will be merged with the attribute set in the constructor.
      */    
     public function get( /* $sLabel, $aAttributes=array() */ ) {
         

Indentation +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 
         // Output
         return 
-           "<{$this->aOptions[ 'input_container_tag' ]} " . $this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ) . ">"
+            "<{$this->aOptions[ 'input_container_tag' ]} " . $this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ) . ">"
                 // the unchecked value must be set prior to the checkbox input field.
                 . "<input " . $this->getAttributes( 
                     array(

Spacing +13 added lines, -13 removed lines

@@ -34,11 +34,11 @@ 
     public function get( /* $sLabel, $aAttributes=array() */ ) {
         
         // Parameters
-        $_aParams       = func_get_args() + array( 
-            0 => '',            // 1st parameter
+        $_aParams = func_get_args() + array( 
+            0 => '', // 1st parameter
             1 => array()        // 2nd parameter
         );
-        $_sLabel        = $_aParams[ 0 ];       // first parameter
+        $_sLabel        = $_aParams[ 0 ]; // first parameter
 
         // Attributes
         $_aAttributes   = $this->uniteArrays(   // second parameter
@@ -48,10 +48,10 @@ 
 
         // Output
         return 
-           "<{$this->aOptions[ 'input_container_tag' ]} " . $this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ) . ">"
+           "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
                 . $this->_getInputElements( $_aAttributes, $this->aOptions )
             . "</{$this->aOptions[ 'input_container_tag' ]}>"
-            . "<{$this->aOptions[ 'label_container_tag' ]} " . $this->getAttributes( $this->aOptions[ 'label_container_attributes' ] ) . ">"
+            . "<{$this->aOptions[ 'label_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'label_container_attributes' ] ).">"
                 . $_sLabel
             . "</{$this->aOptions[ 'label_container_tag' ]}>"
         ;
@@ -65,16 +65,16 @@ 
         private function _getInputElements( $aAttributes, $aOptions ) {
             $_sOutput = $this->aOptions[ 'save_unchecked' ]
                 // the unchecked value must be set prior to the checkbox input field.
-                ? "<input " . $this->getAttributes( 
+                ? "<input ".$this->getAttributes( 
                     array(
                         'type'      => 'hidden',
                         'class'     => $aAttributes[ 'class' ],
                         'name'      => $aAttributes[ 'name' ],
                         'value'     => '0',
                     ) 
-                ) . " />"
+                )." />"
                 : '';
-            $_sOutput .= "<input " . $this->getAttributes( $aAttributes ) . " />";
+            $_sOutput .= "<input ".$this->getAttributes( $aAttributes )." />";
             return $_sOutput;
         }
         
@@ -87,7 +87,7 @@ 
     public function getAttributesByKey( /* $sKey */ ) {
         
         // Parameters
-        $_aParams       = func_get_args() + array( 0 => '', );
+        $_aParams       = func_get_args() + array( 0 => '',);
         $_sKey          = $_aParams[ 0 ];        
         $_bIsMultiple   = '' !== $_sKey;
         
@@ -99,13 +99,13 @@ 
             // The type needs to be specified since the postytpe field type extends this class. If not set, the 'posttype' will be passed to the type attribute.
             + array(
                 'type'      => 'checkbox', 
-                'id'        => $this->aAttributes[ 'id' ] . '_' . $_sKey,
+                'id'        => $this->aAttributes[ 'id' ].'_'.$_sKey,
                 'checked'   => $this->_getCheckedAttributeValue( $_sKey ),
-                'value'     => 1,   // this must be always 1 because the key value can be zero. In that case, the value always will be false and unchecked.
+                'value'     => 1, // this must be always 1 because the key value can be zero. In that case, the value always will be false and unchecked.
                 'name'      => $_bIsMultiple 
                     ? "{$this->aAttributes[ 'name' ]}[{$_sKey}]" 
                     : $this->aAttributes[ 'name' ],
-                'data-id'   => $this->aAttributes[ 'id' ],       // referenced by the JavaScript scripts such as the revealer script.
+                'data-id'   => $this->aAttributes[ 'id' ], // referenced by the JavaScript scripts such as the revealer script.
             )
             + $this->aAttributes
             ;
@@ -122,7 +122,7 @@ 
                 : array( 'value', $_sKey );
             return $this->getElement( $this->aAttributes, $_aValueDimension )
                 ? 'checked' 
-                : null;    // to not to set, pass null. An empty value '' will still set the attribute.            
+                : null; // to not to set, pass null. An empty value '' will still set the attribute.            
         
         }
         

development/factory/_common/form/input/AdminPageFramework_Input_radio.php

Doc Comments -4 removed lines

@@ -21,8 +21,6 @@ 
      * Returns the output of the input element.
      * 
      * @since       3.4.0    
-     * @param       string      $sLabel         The label text.
-     * @param       array       $aAttributes    (optional) The attribute array. If set, it will be merged with the attribute set in the constructor.
      */    
     public function get( /* $sLabel, $aAttributes=array() */ ) {
         
@@ -52,8 +50,6 @@ 
      * 
      * @return      array       The updated attribute array. 
      * @since       3.5.3
-     * @param       string      $sKey       The array element key of the radio button. 
-     * It is assumed that there is an array that holds multiple radio buttons and each of them has an array key.
      */
     public function getAttributesByKey( /* $sKey */ ) {
         

Spacing +10 added lines, -10 removed lines

@@ -36,12 +36,12 @@ 
         
         // Output
         return 
-            "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
-                . "<input " . $this->getAttributes( $_aAttributes ) . " />" 
-            . "</{$this->aOptions['input_container_tag']}>"
-            . "<{$this->aOptions['label_container_tag']} " . $this->getAttributes( $this->aOptions['label_container_attributes'] ) . ">"
+            "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
+                . "<input ".$this->getAttributes( $_aAttributes )." />" 
+            . "</{$this->aOptions[ 'input_container_tag' ]}>"
+            . "<{$this->aOptions[ 'label_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'label_container_attributes' ] ).">"
                 . $_sLabel
-            . "</{$this->aOptions['label_container_tag']}>"
+            . "</{$this->aOptions[ 'label_container_tag' ]}>"
             ;
         
     }    
@@ -58,22 +58,22 @@ 
     public function getAttributesByKey( /* $sKey */ ) {
         
         // Parameters
-        $_aParams       = func_get_args() + array( 0 => '', );
-        $_sKey           = $_aParams[ 0 ];        
+        $_aParams = func_get_args() + array( 0 => '',);
+        $_sKey = $_aParams[ 0 ];        
         
         // Result
         return $this->getElementAsArray( $this->aAttributes, $_sKey, array() )
             + array(
                 'type'          => 'radio',
-                'checked'       => isset( $this->aAttributes['value'] ) && $this->aAttributes['value'] == $_sKey 
+                'checked'       => isset( $this->aAttributes[ 'value' ] ) && $this->aAttributes[ 'value' ] == $_sKey 
                     ? 'checked' 
                     : null,
                 'value'         => $_sKey,
                 // 'id'            => $this->aField['input_id'] . '_' . $_sKey,
-                'id'            => $this->getAttribute( 'id' ) . '_' . $_sKey,
+                'id'            => $this->getAttribute( 'id' ).'_'.$_sKey,
                 // 'data-default'  => $this->aField['default'],        // refered by the repeater script
                 // 'data-id'       => $this->aField['input_id'],       // refered by the JavaScript scripts such as the revealer script.
-                'data-id'       => $this->getAttribute( 'id' ),       // refered by the JavaScript scripts such as the revealer script.
+                'data-id'       => $this->getAttribute( 'id' ), // refered by the JavaScript scripts such as the revealer script.
             ) 
             + $this->aAttributes; 
             

development/factory/_common/form/input/AdminPageFramework_Input_select.php

Doc Comments +3 added lines, -1 removed lines

@@ -122,7 +122,7 @@ 
          * @internal
          * @param       string      $sInputID           The input ID that will be the base of each generated option tag ID.
          * @param       array       $aLabels            The array holding labels.
-         * @param       array       $aAttributes        The attribute arrays. Accepts the following arguments.
+         * @param       array       $aBaseAttributes        The attribute arrays. Accepts the following arguments.
          * - optgroup
          * - option
          */     
@@ -160,6 +160,7 @@ 
             /**
              * Returns an HTML output of optgroup tag.
              * @since       3.5.3
+             * @param string $sInputID
              * @return      string      an HTML output of optgroup tag.
              */
             private function _getOptGroup( array $aBaseAttributes, $sInputID, $sKey, $asLabel ) {
@@ -179,6 +180,7 @@ 
             /**
              * 
              * @since        3.5.3
+             * @param string $sInputID
              */
             private function _getOptionTagAttributes( array $aBaseAttributes, $sInputID, $sKey, $aValues ) {
             

Spacing +14 added lines, -14 removed lines

@@ -69,14 +69,14 @@ 
         );    
 
         return  
-            "<{$this->aOptions[ 'input_container_tag' ]} " . $this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ) . ">"
-                . "<select " . $this->getAttributes( $this->_getSelectAttributes( $_aAttributes ) ) . " >"
+            "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
+                . "<select ".$this->getAttributes( $this->_getSelectAttributes( $_aAttributes ) )." >"
                     . $this->_getDropDownList( 
                         $this->getAttribute( 'id' ),
                         $this->getAsArray(
                             isset( $_aLabels ) 
                                 ? $_aLabels
-                                : $this->aField[ 'label' ],    // backward compatibility
+                                : $this->aField[ 'label' ], // backward compatibility
                             true
                         ),
                         $_aAttributes
@@ -104,9 +104,9 @@ 
                         ? 'multiple' 
                         : null,
                     'name'      => $_bIsMultiple 
-                        ? $this->getAttribute( 'name' ) . '[]'
+                        ? $this->getAttribute( 'name' ).'[]'
                         : $this->getAttribute( 'name' ),
-                    'data-id'   => $this->getAttribute( 'id' ),       // referenced by the JavaScript scripts such as the revealer script.
+                    'data-id'   => $this->getAttribute( 'id' ), // referenced by the JavaScript scripts such as the revealer script.
                 )
             );            
             
@@ -128,12 +128,12 @@ 
          */     
         private function _getDropDownList( $sInputID, array $aLabels, array $aBaseAttributes ) {
             
-            $_aOutput   = array();
-            foreach( $aLabels as $__sKey => $__asLabel ) {
+            $_aOutput = array();
+            foreach ( $aLabels as $__sKey => $__asLabel ) {
                 
                 // For an optgroup tag,
                 if ( is_array( $__asLabel ) ) {
-                    $_aOutput[] = $this->_getOptGroup(
+                    $_aOutput[ ] = $this->_getOptGroup(
                         $aBaseAttributes, 
                         $sInputID, 
                         $__sKey, 
@@ -143,8 +143,8 @@ 
                 }
                 
                 // A normal option tag,
-                $_aOutput[] = $this->_getOptionTag( 
-                    $__asLabel,   // the text label the user sees to be selected
+                $_aOutput[ ] = $this->_getOptionTag( 
+                    $__asLabel, // the text label the user sees to be selected
                     $this->_getOptionTagAttributes( 
                         $aBaseAttributes, 
                         $sInputID, 
@@ -170,7 +170,7 @@ 
                 $_aOptGroupAttributes = array(
                     'label' => $sKey,
                 ) + $_aOptGroupAttributes;
-                return "<optgroup " . $this->getAttributes( $_aOptGroupAttributes ) . ">"
+                return "<optgroup ".$this->getAttributes( $_aOptGroupAttributes ).">"
                         . $this->_getDropDownList( $sInputID, $asLabel, $aBaseAttributes )
                     . "</optgroup>";
              
@@ -188,7 +188,7 @@ 
                     $aValues 
                 );
                 return array(      
-                        'id'        => $sInputID . '_' . $sKey,
+                        'id'        => $sInputID.'_'.$sKey,
                         'value'     => $sKey,
                         'selected'  => in_array( ( string ) $sKey, $aValues ) 
                             ? 'selected' 
@@ -204,8 +204,8 @@ 
              * @sicne       3.4.0
              * @return      string      The generated option tag HTML output.
              */
-            private function _getOptionTag( $sLabel, array $aOptionTagAttributes=array() ) {
-                return "<option " . $this->getAttributes( $aOptionTagAttributes ) . " >"    
+            private function _getOptionTag( $sLabel, array $aOptionTagAttributes = array() ) {
+                return "<option ".$this->getAttributes( $aOptionTagAttributes )." >"    
                         . $sLabel
                     . "</option>";
             }

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_SystemInformation.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
      * Returns an array of constants.
      * 
      * @since       3.4.6
-     * @param       array|string      $asCategory      The category key names of the returning array.
+     * @param       array|string      $asCategories      The category key names of the returning array.
      */
     static public function getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {
         

Spacing +11 added lines, -11 removed lines

@@ -41,7 +41,7 @@ 
         $_sOutput = preg_replace(
             array(
                 '#^.*<body>(.*)</body>.*$#ms', '#<h2>PHP License</h2>.*$#ms',
-                '#<h1>Configuration</h1>#',  "#\r?\n#", "#</(h1|h2|h3|tr)>#", '# +<#',
+                '#<h1>Configuration</h1>#', "#\r?\n#", "#</(h1|h2|h3|tr)>#", '# +<#',
                 "#[ \t]+#", '#&nbsp;#', '#  +#', '# class=".*?"#', '%&#039;%',
                 '#<tr>(?:.*?)" src="(?:.*?)=(.*?)" alt="PHP Logo" /></a>'
                     .'<h1>PHP Version (.*?)</h1>(?:\n+?)</td></tr>#',
@@ -52,11 +52,11 @@ 
                 '#</tr>#'
             ),
             array(
-                '$1', '', '', '', '</$1>' . "\n", '<', ' ', ' ', ' ', '', ' ',
+                '$1', '', '', '', '</$1>'."\n", '<', ' ', ' ', ' ', '', ' ',
                 '<h2>PHP Configuration</h2>'."\n".'<tr><td>PHP Version</td><td>$2</td></tr>'.
                 "\n".'<tr><td>PHP Egg</td><td>$1</td></tr>',
                 '<tr><td>PHP Credits Egg</td><td>$1</td></tr>',
-                '<tr><td>Zend Engine</td><td>$2</td></tr>' . "\n" . '<tr><td>Zend Egg</td><td>$1</td></tr>',
+                '<tr><td>Zend Engine</td><td>$2</td></tr>'."\n".'<tr><td>Zend Egg</td><td>$1</td></tr>',
                 ' ',
                 '%S%',
                 '%E%'
@@ -68,7 +68,7 @@ 
         unset( $_aSections[ 0 ] );
 
         $_aOutput = array();
-        foreach( $_aSections as $_sSection ) {
+        foreach ( $_aSections as $_sSection ) {
             $_iIndex = substr( $_sSection, 0, strpos( $_sSection, '</h2>' ) );
             preg_match_all(
                 '#%S%(?:<td>(.*?)</td>)?(?:<td>(.*?)</td>)?(?:<td>(.*?)</td>)?%E%#',
@@ -76,12 +76,12 @@ 
                 $_aAskApache, 
                 PREG_SET_ORDER
             );
-            foreach( $_aAskApache as $_aMatches ) {
-                if ( ! isset( $_aMatches[ 1 ], $_aMatches[ 2 ] ) ) {
+            foreach ( $_aAskApache as $_aMatches ) {
+                if ( !isset( $_aMatches[ 1 ], $_aMatches[ 2 ] ) ) {
                     array_slice( $_aMatches, 2 );
                     continue;
                 }
-                $_aOutput[ $_iIndex ][ $_aMatches[ 1 ] ] = ! isset( $_aMatches[ 3 ] ) || $_aMatches[ 2 ] == $_aMatches[ 3 ]
+                $_aOutput[ $_iIndex ][ $_aMatches[ 1 ] ] = !isset( $_aMatches[ 3 ] ) || $_aMatches[ 2 ] == $_aMatches[ 3 ]
                     ? $_aMatches[ 2 ] 
                     : array_slice( $_aMatches, 2 );
             }
@@ -97,7 +97,7 @@ 
      * @since       3.4.6
      * @param       array|string      $asCategory      The category key names of the returning array.
      */
-    static public function getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {
+    static public function getDefinedConstants( $asCategories = null, $asRemovingCategories = null ) {
         
         $_aCategories           = is_array( $asCategories ) ? $asCategories : array( $asCategories );
         $_aCategories           = array_filter( $_aCategories );
@@ -124,8 +124,8 @@ 
     static public function getPHPErrorLogPath() {
                 
         $_aPHPInfo = self::getPHPInfo();
-        return isset( $_aPHPInfo['PHP Core']['error_log'] ) 
-            ? $_aPHPInfo['PHP Core']['error_log']
+        return isset( $_aPHPInfo[ 'PHP Core' ][ 'error_log' ] ) 
+            ? $_aPHPInfo[ 'PHP Core' ][ 'error_log' ]
             : '';
         
     }
@@ -134,7 +134,7 @@ 
      * Returns a PHP error log.
      * @since       3.4.6
      */
-    static public function getPHPErrorLog( $iLines=1 ) {
+    static public function getPHPErrorLog( $iLines = 1 ) {
         
         $_sLog = self::getFileTailContents( self::getPHPErrorLogPath(), $iLines );
         

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_URL.php

Doc Comments +2 added lines

@@ -23,6 +23,7 @@ 
      * 
      * @since       2.0.0
      * @since       3.5.7       Moved from `AdminPageFramework_Utility`.
+     * @param string $sQueryKey
      * @return      string|null
      */ 
     static public function getQueryValueInURLByKey( $sURL, $sQueryKey ) {
@@ -72,6 +73,7 @@ 
         /**
          * Returns the port suffix in the currently loading url.
          * @since       3.5.7
+         * @param boolean $_bSSL
          * @return      string
          */
         static private function _getURLPortSuffix( $_bSSL ) {

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
             0 => '',
             1 => 's',
         );
-        $_sProtocol         = substr( $_sServerProtocol, 0, strpos( $_sServerProtocol, '/' ) )
+        $_sProtocol = substr( $_sServerProtocol, 0, strpos( $_sServerProtocol, '/' ) )
             . $_aProrocolSuffix[ ( int ) $_bSSL ];
         
         // Port: e.g. :80
@@ -66,7 +66,7 @@ 
                 : $_SERVER[ 'SERVER_NAME' ] 
             );
             
-        return $_sProtocol . '://' . $_sHost . $_sPort . $_SERVER[ 'REQUEST_URI' ];
+        return $_sProtocol.'://'.$_sHost.$_sPort.$_SERVER[ 'REQUEST_URI' ];
         
     }
         /**
@@ -79,10 +79,10 @@ 
                 ? ( string ) $_SERVER[ 'SERVER_PORT' ]
                 : '';
             $_aPort     = array(
-                0 => ':' . $_sPort,
+                0 => ':'.$_sPort,
                 1 => '',
             );
-            $_bPortSet  = ( ! $_bSSL && '80' === $_sPort ) || ( $_bSSL && '443' === $_sPort );
+            $_bPortSet = ( !$_bSSL && '80' === $_sPort ) || ( $_bSSL && '443' === $_sPort );
             return $_aPort[ ( int ) $_bPortSet ];
         }
     

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Hook.php

Doc Comments +1 added lines, -9 removed lines

@@ -89,11 +89,6 @@ 
      * @access      public
      * @since       2.0.0
      * @remark      Accepts variadic parameters.
-     * @param       object      $oCallerObject      the object that holds the callback method that matches the action hook name.
-     * @param       array       $aActionHooks       a numerically index array consisting of action hook names that serve as the callback method names. 
-     * @param       mixed       $vArgs1             the argument to pass to the hook callback functions.
-     * @param       mixed       $vArgs2             another argument to pass to the hook callback functions.
-     * @param       mixed       $_and_more          add as many arguments as necessary to the next parameters.
      * @return      void
      */ 
     static public function addAndDoActions( /* $oCallerObject, $aActionHooks, $vArgs1=null, $vArgs2=null, $_and_more=null */ ) {
@@ -192,10 +187,6 @@ 
      * `
      * 
      * @since       2.0.0
-     * @param       object      $oCallerObject
-     * @param       string      $sFilter            The filter hook name.
-     * @param       mixed       $vData              The filtering data
-     * @param       mixed       $vArgs              The arguments.
      */
     static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) { 
         
@@ -233,6 +224,7 @@ 
      * 
      * @since       2.0.0
      * @access      public
+     * @param string $sPrefix
      * @return      array       Returns an array consisting of the filters.
      */ 
     static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse=false ) {

Spacing +17 added lines, -17 removed lines

@@ -26,7 +26,7 @@ 
      * @since       3.4.2
      * @todo        Create a similar function for filters.
      */
-    static public function registerAction( $sActionHook, $oCallable, $iPriority=10 ) {
+    static public function registerAction( $sActionHook, $oCallable, $iPriority = 10 ) {
       
         if ( did_action( $sActionHook ) ) {
             return call_user_func_array( $oCallable, array() );
@@ -63,13 +63,13 @@ 
      * @param       mixed       $_and_more      add as many arguments as necessary to the next parameters.
      * @return      void        does not return a value.
      */     
-    static public function doActions( $aActionHooks, $vArgs1=null, $vArgs2=null, $_and_more=null ) {
+    static public function doActions( $aActionHooks, $vArgs1 = null, $vArgs2 = null, $_and_more = null ) {
         
         $aArgs          = func_get_args();     
         $aActionHooks   = $aArgs[ 0 ];
-        foreach( ( array ) $aActionHooks as $sActionHook  ) {
+        foreach ( ( array ) $aActionHooks as $sActionHook ) {
             $aArgs[ 0 ] = $sActionHook;
-            call_user_func_array( 'do_action' , $aArgs );
+            call_user_func_array( 'do_action', $aArgs );
         }
 
     }
@@ -101,12 +101,12 @@ 
         $aArgs          = func_get_args();    
         $oCallerObject  = $aArgs[ 0 ];
         $aActionHooks   = $aArgs[ 1 ];
-        foreach( ( array ) $aActionHooks as $sActionHook ) {
-            if ( ! $sActionHook ) { 
+        foreach ( ( array ) $aActionHooks as $sActionHook ) {
+            if ( !$sActionHook ) { 
                 continue; 
             }
             $aArgs[ 1 ] = $sActionHook;    
-            call_user_func_array( array( get_class(), 'addAndDoAction' ) , $aArgs );     
+            call_user_func_array( array( get_class(), 'addAndDoAction' ), $aArgs );     
         }
         
     }
@@ -125,7 +125,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sActionHook    = $_aArgs[ 1 ];
-        if ( ! $_sActionHook ) { 
+        if ( !$_sActionHook ) { 
             return; 
         }
         
@@ -137,7 +137,7 @@ 
 
         // Remove the first element, the caller object.
         array_shift( $_aArgs ); 
-        call_user_func_array( 'do_action' , $_aArgs );
+        call_user_func_array( 'do_action', $_aArgs );
         
     }
     
@@ -165,12 +165,12 @@ 
         $_aFilters      = $_aArgs[ 1 ];
         $_vInput        = $_aArgs[ 2 ];
 
-        foreach( ( array ) $_aFilters as $_sFilter ) {
-            if ( ! $_sFilter ) { 
+        foreach ( ( array ) $_aFilters as $_sFilter ) {
+            if ( !$_sFilter ) { 
                 continue; 
             }
             $_aArgs[ 1 ] = $_sFilter;
-            $_aArgs[ 2 ] = $_vInput;    // assigns the updated value as it is filtered in previous iterations
+            $_aArgs[ 2 ] = $_vInput; // assigns the updated value as it is filtered in previous iterations
             $_vInput = call_user_func_array( 
                 array( get_class(), 'addAndApplyFilter' ), 
                 $_aArgs 
@@ -204,7 +204,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sFilter        = $_aArgs[ 1 ];
-        if ( ! $_sFilter ) { 
+        if ( !$_sFilter ) { 
             return $_aArgs[ 2 ]; 
         }
         
@@ -235,17 +235,17 @@ 
      * @access      public
      * @return      array       Returns an array consisting of the filters.
      */ 
-    static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse=false ) {
+    static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse = false ) {
                 
         $_aFilters = array();
         if ( $sTabSlug && $sPageSlug ) {
-            $_aFilters[] = "{$sPrefix}{$sPageSlug}_{$sTabSlug}";
+            $_aFilters[ ] = "{$sPrefix}{$sPageSlug}_{$sTabSlug}";
         }
         if ( $sPageSlug ) {
-            $_aFilters[] = "{$sPrefix}{$sPageSlug}";     
+            $_aFilters[ ] = "{$sPrefix}{$sPageSlug}";     
         }
         if ( $sClassName ) {
-            $_aFilters[] = "{$sPrefix}{$sClassName}";
+            $_aFilters[ ] = "{$sPrefix}{$sClassName}";
         }
         return $bReverse 
             ? array_reverse( $_aFilters ) 

Braces +6 added lines, -6 removed lines

@@ -102,7 +102,7 @@ 
         $oCallerObject  = $aArgs[ 0 ];
         $aActionHooks   = $aArgs[ 1 ];
         foreach( ( array ) $aActionHooks as $sActionHook ) {
-            if ( ! $sActionHook ) { 
+            if ( ! $sActionHook ) {
                 continue; 
             }
             $aArgs[ 1 ] = $sActionHook;    
@@ -125,7 +125,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sActionHook    = $_aArgs[ 1 ];
-        if ( ! $_sActionHook ) { 
+        if ( ! $_sActionHook ) {
             return; 
         }
         
@@ -158,7 +158,7 @@ 
      * 
      * @since       2.0.0
      */    
-    static public function addAndApplyFilters( /* $oCallerObject, $aFilters, $vInput, $vArgs... */ ) { 
+    static public function addAndApplyFilters( /* $oCallerObject, $aFilters, $vInput, $vArgs... */ ) {
             
         $_aArgs         = func_get_args();    
         // $oCallerObject  = $_aArgs[ 0 ];
@@ -166,7 +166,7 @@ 
         $_vInput        = $_aArgs[ 2 ];
 
         foreach( ( array ) $_aFilters as $_sFilter ) {
-            if ( ! $_sFilter ) { 
+            if ( ! $_sFilter ) {
                 continue; 
             }
             $_aArgs[ 1 ] = $_sFilter;
@@ -197,14 +197,14 @@ 
      * @param       mixed       $vData              The filtering data
      * @param       mixed       $vArgs              The arguments.
      */
-    static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) { 
+    static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) {
         
         // Parameters
         $_iArgs          = func_num_args();
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sFilter        = $_aArgs[ 1 ];
-        if ( ! $_sFilter ) { 
+        if ( ! $_sFilter ) {
             return $_aArgs[ 2 ]; 
         }
         

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Page.php

Doc Comments +1 added lines

@@ -284,6 +284,7 @@
              * Checkes whether the passed base url name is of the admin index page.
              * @since       3.5.3
              * return       boolean      Whether the passed base url name is of the admin index page.
+             * @param string $sPageNow
              */
             static private function _isInAdminIndex( $sPageNow ) {
                 return in_array(

Braces +7 added lines, -9 removed lines

@@ -29,7 +29,7 @@ 
      */
     static public function getCurrentPostType() {
                          
-        if ( isset( self::$_sCurrentPostType ) ) { 
+        if ( isset( self::$_sCurrentPostType ) ) {
             return self::$_sCurrentPostType; 
         }
         self::$_sCurrentPostType = self::_getCurrentPostType();
@@ -141,7 +141,7 @@ 
     static public function isPostDefinitionPage( $asPostTypes=array() ) {
         
         // If it's not the post definition page, 
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) { 
+        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -160,7 +160,7 @@ 
         $_aPostTypes = self::getAsArray( $asPostTypes );        
         
         // If the parameter is empty, 
-        if ( empty( $_aPostTypes ) ) { 
+        if ( empty( $_aPostTypes ) ) {
             return true;
         }
 
@@ -176,13 +176,13 @@ 
      */
     static public function isPostListingPage( $asPostTypes=array() ) {
                 
-        if ( 'edit.php' != self::getPageNow() ) { 
+        if ( 'edit.php' != self::getPageNow() ) {
             return false;
         }
         
         $_aPostTypes = self::getAsArray( $asPostTypes );    
         
-        if ( ! isset( $_GET[ 'post_type' ] )  ) { 
+        if ( ! isset( $_GET[ 'post_type' ] )  ) {
             return in_array( 'post', $_aPostTypes );
         }
 
@@ -273,11 +273,9 @@ 
                 
                 if ( is_network_admin() ) {
                     $_sNeedle = '#/wp-admin/network/?(.*?)$#i';
-                }
-                else if ( is_user_admin() ) {
+                } else if ( is_user_admin() ) {
                     $_sNeedle = '#/wp-admin/user/?(.*?)$#i';
-                }
-                else {
+                } else {
                     $_sNeedle = '#/wp-admin/?(.*?)$#i';
                 }                
                 preg_match( $_sNeedle, $_SERVER[ 'PHP_SELF' ], $_aMatches );

Spacing +11 added lines, -11 removed lines

@@ -57,7 +57,7 @@ 
                 'getPostTypeByTypeNow',
                 'getPostTypeByScreenObject',
                 'getPostTypeByREQUEST',
-                'getPostTypeByPostObject',  // 3.6.0+ Moved to the last as it is not reliable.
+                'getPostTypeByPostObject', // 3.6.0+ Moved to the last as it is not reliable.
             );
             foreach ( $_aMethodsToTry as $_sMethodName ) {
                 $_sPostType = call_user_func( array( __CLASS__, $_sMethodName ) );
@@ -121,9 +121,9 @@ 
      * @param       array|string        The post type slug(s) to check. If this is empty, the method just checks the current page is a taxonomy page.
      * @return      boolean
      */    
-    static public function isCustomTaxonomyPage( $asPostTypes=array() ) {
+    static public function isCustomTaxonomyPage( $asPostTypes = array() ) {
         
-        if ( ! in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', 'term.php' ) ) ) {
+        if ( !in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', 'term.php' ) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -138,10 +138,10 @@ 
      * Otherwise, it will check if the page belongs to the given post type(s).
      * @return      boolean
      */
-    static public function isPostDefinitionPage( $asPostTypes=array() ) {
+    static public function isPostDefinitionPage( $asPostTypes = array() ) {
         
         // If it's not the post definition page, 
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) { 
+        if ( !in_array( self::getPageNow(), array( 'post.php', 'post-new.php',) ) ) { 
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -174,7 +174,7 @@ 
      * 
      * @since 3.0.0
      */
-    static public function isPostListingPage( $asPostTypes=array() ) {
+    static public function isPostListingPage( $asPostTypes = array() ) {
                 
         if ( 'edit.php' != self::getPageNow() ) { 
             return false;
@@ -182,7 +182,7 @@ 
         
         $_aPostTypes = self::getAsArray( $asPostTypes );    
         
-        if ( ! isset( $_GET[ 'post_type' ] )  ) { 
+        if ( !isset( $_GET[ 'post_type' ] ) ) { 
             return in_array( 'post', $_aPostTypes );
         }
 
@@ -220,7 +220,7 @@ 
             0 => '_getPageNow_FrontEnd',
             1 => '_getPageNow_BackEnd',
         );
-        $_sMethodName  = $_aMethodNames[ ( integer ) is_admin() ];
+        $_sMethodName = $_aMethodNames[ ( integer ) is_admin() ];
         self::$_sPageNow = self::$_sMethodName();            
         return self::$_sPageNow;          
         
@@ -315,7 +315,7 @@ 
         
         if ( isset( $GLBOALS[ 'page_hook' ] ) ) {
             return is_network_admin() 
-                ? $GLBOALS[ 'page_hook' ] . '-network'
+                ? $GLBOALS[ 'page_hook' ].'-network'
                 : $GLBOALS[ 'page_hook' ];
         }
         
@@ -329,11 +329,11 @@ 
      * @since       3.7.0
      * @return      boolean
      */
-    static public function doesMetaBoxExist( $sContext='' ) {
+    static public function doesMetaBoxExist( $sContext = '' ) {
         
         $_aDimensions = array( 'wp_meta_boxes', $GLOBALS[ 'page_hook' ] );
         if ( $sContext ) {
-            $_aDimensions[] = $sContext;
+            $_aDimensions[ ] = $sContext;
         }
         $_aSideMetaBoxes = self::getElementAsArray( 
             $GLOBALS, 

development/factory/post_type/_model/AdminPageFramework_PostType_Model__FlushRewriteRules.php

Doc Comments +1 added lines

@@ -26,6 +26,7 @@
      * Sets up hooks and properties.
      * 
      * @internal
+     * @param AdminPageFramework_PostType_Model $oFactory
      */
     public function __construct( $oFactory ) {
                 

Indentation +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
         );        
     }    
     
-       /**
+        /**
          * Extracts meta box form fields options array from the given options array of an admin page.
          * 
          * @since       3.5.6

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function __construct( $oFactory ) {
                 
-        if ( ! $this->_shouldProceed( $oFactory ) ) {
+        if ( !$this->_shouldProceed( $oFactory ) ) {
             return;
         }        
         
@@ -54,10 +54,10 @@ 
          */
         private function _shouldProceed( $oFactory ) {
 
-            if ( ! $oFactory->oProp->bIsAdmin ) {
+            if ( !$oFactory->oProp->bIsAdmin ) {
                 return false;
             }        
-            if ( ! $oFactory->oProp->sCallerPath ) {
+            if ( !$oFactory->oProp->sCallerPath ) {
                 return false;
             }
             return 'plugin' === $oFactory->oProp->sScriptType;
@@ -87,7 +87,7 @@ 
         }
                 
         // If the execution flow in the plugin activation hook, schedule flushing rewrite rules.
-        if ( did_action( 'activate_' . plugin_basename( $this->oFactory->oProp->sCallerPath ) ) ) {
+        if ( did_action( 'activate_'.plugin_basename( $this->oFactory->oProp->sCallerPath ) ) ) {
             add_action( 'shutdown', array( $this, '_replyToFlushRewriteRules' ) );
         }
 

Braces +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
      * @callback    action      activate_{plugin base name}     
      * @since       3.7.6
      */
-    public function _replyToSetUpPostType() {            
+    public function _replyToSetUpPostType() {
         do_action( "set_up_{$this->oFactory->oProp->sClassName}", $this );
     }        
     

development/factory/post_type/_model/AdminPageFramework_PostType_Model__SubMenuOrder.php

Doc Comments +3 added lines

@@ -26,6 +26,7 @@ 
      * Sets up hooks and properties.
      * 
      * @internal
+     * @param AdminPageFramework_PostType_Model $oFactory
      */
     public function __construct( $oFactory ) {
           
@@ -82,6 +83,7 @@ 
     
         /**
          * @since       3.7.4
+         * @param string $sSubMenuSlug
          */
         private function _setSubMenuSlugForSorting( $sSubMenuSlug ) {
                 
@@ -94,6 +96,7 @@ 
         
         /**
          * @since       3.7.4
+         * @param string $sSubMenuSlug
          */
         private function _setSubMenuItemIndex( $sSubMenuSlug ) {                
                                         

Spacing +12 added lines, -12 removed lines

@@ -31,7 +31,7 @@ 
           
         $this->oFactory = $oFactory;
           
-        if ( ! $oFactory->oProp->bIsAdmin ) {
+        if ( !$oFactory->oProp->bIsAdmin ) {
             return;
         }
         
@@ -47,7 +47,7 @@ 
         
         add_action(
             'admin_menu',
-            array( $this, 'sortAdminSubMenu' ),  // defined in the framework utility class.
+            array( $this, 'sortAdminSubMenu' ), // defined in the framework utility class.
             9999
         );            
             
@@ -62,15 +62,15 @@ 
             
         // Check the post type `show_ui` and other related UI arguments.
         $_bsShowInMeenu = $this->getShowInMenuPostTypeArgument( $this->oFactory->oProp->aPostTypeArgs );
-        if ( ! $_bsShowInMeenu ) {
+        if ( !$_bsShowInMeenu ) {
             return;
         }
         
         // If the user sets a menu slug to the 'show_in_menu' argument, use that. 
         // It is used to set a custom post type sub-menu belong to another menu.
-        $_sSubMenuSlug  = is_string( $_bsShowInMeenu )
+        $_sSubMenuSlug = is_string( $_bsShowInMeenu )
             ? $_bsShowInMeenu
-            : 'edit.php?post_type=' . $this->oFactory->oProp->sPostType;
+            : 'edit.php?post_type='.$this->oFactory->oProp->sPostType;
         
         // Set the index to the framework specific global array for sorting.
         $this->_setSubMenuSlugForSorting( $_sSubMenuSlug );
@@ -131,11 +131,11 @@ 
 
                 // @remark      This is the partial link url set in the third element ( index of 2 ) in the third dimension of submenu global array element.
                 // This is not the submenu slug.
-                $_sLinkSlugManage = 'edit.php?post_type=' . $this->oFactory->oProp->sPostType;
+                $_sLinkSlugManage = 'edit.php?post_type='.$this->oFactory->oProp->sPostType;
                                 
                 $_aLinkSlugs = array(
                     $_sLinkSlugManage => $_nSubMenuOrderManage,
-                    'post-new.php?post_type=' . $this->oFactory->oProp->sPostType => $_nSubMenuOrderAddNew,
+                    'post-new.php?post_type='.$this->oFactory->oProp->sPostType => $_nSubMenuOrderAddNew,
                 );
 
                 // If the user does not set a custom value, unset it
@@ -144,8 +144,8 @@ 
                 }
                 
                 // If the user does not want to show the Add New sub menu, no need to change the order.
-                if ( ! $_bShowAddNew || 10 == $_nSubMenuOrderAddNew ) {
-                    unset( $_aLinkSlugs[ 'post-new.php?post_type=' . $this->oFactory->oProp->sPostType ] );
+                if ( !$_bShowAddNew || 10 == $_nSubMenuOrderAddNew ) {
+                    unset( $_aLinkSlugs[ 'post-new.php?post_type='.$this->oFactory->oProp->sPostType ] );
                 }                   
                 
                 return $_aLinkSlugs;
@@ -165,9 +165,9 @@ 
          */
         private function _setSubMenuIndexByLinksSlugs( $sSubMenuSlug, array $aLinkSlugs ) {
 
-            foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sSubMenuSlug ) ) as $_nIndex => $_aSubMenuItem ) {
+            foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sSubMenuSlug ) ) as $_nIndex => $_aSubMenuItem ) {
                 
-                foreach( $aLinkSlugs as $_sLinkSlug => $_nOrder ) {
+                foreach ( $aLinkSlugs as $_sLinkSlug => $_nOrder ) {
                     
                     $_bIsSet = $this->_setSubMenuIndexByLinksSlug( $sSubMenuSlug, $_nIndex, $_aSubMenuItem, $_sLinkSlug, $_nOrder ); 
                     
@@ -193,7 +193,7 @@ 
             private function _setSubMenuIndexByLinksSlug( $sSubMenuSlug, $nIndex, $aSubMenuItem, $sLinkSlug, $nOrder ) {
                 
                 // The third item is the link slug.
-                if ( ! isset( $aSubMenuItem[ 2 ] ) ) { 
+                if ( !isset( $aSubMenuItem[ 2 ] ) ) { 
                     return false; 
                 }
                 if ( $aSubMenuItem[ 2 ] !== $sLinkSlug ) {

Braces +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
  * @subpackage      PostType
  * @internal
  */
-class AdminPageFramework_PostType_Model__SubMenuOrder extends AdminPageFramework_FrameworkUtility {    
+class AdminPageFramework_PostType_Model__SubMenuOrder extends AdminPageFramework_FrameworkUtility {
     
     /**
      * Stores a post type factory object.
@@ -95,7 +95,7 @@ 
         /**
          * @since       3.7.4
          */
-        private function _setSubMenuItemIndex( $sSubMenuSlug ) {                
+        private function _setSubMenuItemIndex( $sSubMenuSlug ) {
                                         
             // Only if custom values are set, set them.                
             $this->_setSubMenuIndexByLinksSlugs( 
@@ -193,7 +193,7 @@ 
             private function _setSubMenuIndexByLinksSlug( $sSubMenuSlug, $nIndex, $aSubMenuItem, $sLinkSlug, $nOrder ) {
                 
                 // The third item is the link slug.
-                if ( ! isset( $aSubMenuItem[ 2 ] ) ) { 
+                if ( ! isset( $aSubMenuItem[ 2 ] ) ) {
                     return false; 
                 }
                 if ( $aSubMenuItem[ 2 ] !== $sLinkSlug ) {