michaeluno / admin-page-framework

development/factory/_abstract/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 +24 added lines, -24 removed lines

@@ -44,8 +44,8 @@ 
         // For backward compatibility.
         
         // If the $aField property is set, extract certain elements from it and set them to the attribute array.
-        if ( isset( $this->aField['is_multiple'] ) ) {
-            $this->aAttributes['select']['multiple'] = $this->aField['is_multiple']
+        if ( isset( $this->aField[ 'is_multiple' ] ) ) {
+            $this->aAttributes[ 'select' ][ 'multiple' ] = $this->aField[ 'is_multiple' ]
                 ? 'multiple'
                 : $this->getElement( $this->aAttributes, array( 'select', 'multiple' ) );
         }
@@ -69,20 +69,20 @@ 
         );    
 
         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
                     )
                 . "</select>"
-            . "</{$this->aOptions['input_container_tag']}>"
+            . "</{$this->aOptions[ 'input_container_tag' ]}>"
             ;
             
     }
@@ -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,13 +143,13 @@ 
                 }
                 
                 // 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, 
                         $__sKey,
-                        $this->getAsArray( $aBaseAttributes['value'], true )
+                        $this->getAsArray( $aBaseAttributes[ 'value' ], true )
                     ) 
                 );
                     
@@ -164,13 +164,13 @@ 
              */
             private function _getOptGroup( array $aBaseAttributes, $sInputID, $sKey, $asLabel ) {
              
-                $_aOptGroupAttributes = isset( $aBaseAttributes['optgroup'][ $sKey ] ) && is_array( $aBaseAttributes['optgroup'][ $sKey ] )
-                    ? $aBaseAttributes['optgroup'][ $sKey ] + $aBaseAttributes['optgroup']
-                    : $aBaseAttributes['optgroup'];
+                $_aOptGroupAttributes = isset( $aBaseAttributes[ 'optgroup' ][ $sKey ] ) && is_array( $aBaseAttributes[ 'optgroup' ][ $sKey ] )
+                    ? $aBaseAttributes[ 'optgroup' ][ $sKey ] + $aBaseAttributes[ 'optgroup' ]
+                    : $aBaseAttributes[ 'optgroup' ];
                 $_aOptGroupAttributes = array(
                     'label' => $sKey,
                 ) + $_aOptGroupAttributes;
-                return "<optgroup " . $this->getAttributes( $_aOptGroupAttributes ) . ">"
+                return "<optgroup ".$this->getAttributes( $_aOptGroupAttributes ).">"
                         . $this->_getDropDownList( $sInputID, $asLabel, $aBaseAttributes )
                     . "</optgroup>";
              
@@ -188,14 +188,14 @@ 
                     $aValues 
                 );
                 return array(      
-                        'id'        => $sInputID . '_' . $sKey,
+                        'id'        => $sInputID.'_'.$sKey,
                         'value'     => $sKey,
                         'selected'  => in_array( ( string ) $sKey, $aValues ) 
                             ? 'selected' 
                             : null,                                        
-                    ) + ( isset( $aBaseAttributes['option'][ $sKey ] ) && is_array( $aBaseAttributes['option'][ $sKey ] )
-                        ? $aBaseAttributes['option'][ $sKey ] + $aBaseAttributes['option']
-                        : $aBaseAttributes['option'] );
+                    ) + ( isset( $aBaseAttributes[ 'option' ][ $sKey ] ) && is_array( $aBaseAttributes[ 'option' ][ $sKey ] )
+                        ? $aBaseAttributes[ 'option' ][ $sKey ] + $aBaseAttributes[ 'option' ]
+                        : $aBaseAttributes[ 'option' ] );
             
             }
         
@@ -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/_abstract/utility/AdminPageFramework_RegisterClasses.php

Doc Comments +1 added lines

@@ -170,6 +170,7 @@
         
         /**
          * The recursive version of the glob() function.
+         * @param string $sPathPatten
          */
         protected function doRecursiveGlob( $sPathPatten, $nFlags=0, array $aExcludeDirs=array(), array $aExcludeDirNames=array() ) {
 

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * Admin Page Framework
- * 
- * Helps to set up auto-load classes.
- * 
- * http://en.michaeluno.jp/admin-page-framework/
- * Copyright (c) 2013-2015 Michael Uno; Licensed MIT
- * 
- */
+         * Admin Page Framework
+         * 
+         * Helps to set up auto-load classes.
+         * 
+         * http://en.michaeluno.jp/admin-page-framework/
+         * Copyright (c) 2013-2015 Michael Uno; Licensed MIT
+         * 
+         */
 
 /**
  * Registers classes in the given directory to be auto-loaded.

Spacing +27 added lines, -27 removed lines

@@ -37,7 +37,7 @@ 
         'is_recursive'          => true,
         'exclude_dir_paths'     => array(),
         'exclude_dir_names'     => array( 'asset', 'assets', 'css', 'js', 'image', 'images', 'license', 'document', 'documents' ),
-        'allowed_extensions'    => array( 'php', ), // 'inc'
+        'allowed_extensions'    => array( 'php',), // 'inc'
         'include_function'      => 'include',
         'exclude_class_names'   => array( /* 'SomeClass', 'SomeOtherClass' */ ),
     );
@@ -65,13 +65,13 @@ 
      * </code>
      * @remark The directory paths set for the 'exclude_dir_paths' option should use the system directory separator.
      */
-    function __construct( $asScanDirPaths, array $aOptions=array(), array $aClasses=array() ) {
+    function __construct( $asScanDirPaths, array $aOptions = array(), array $aClasses = array() ) {
         
         $_aOptions = $aOptions + self::$_aStructure_Options;
         $this->_aClasses   = $aClasses + $this->_constructClassArray( $asScanDirPaths, $_aOptions );
-        $_sIncludeFunciton = in_array( $_aOptions['include_function'], array( 'require', 'require_once', 'include', 'include_once' ) )
-            ? $_aOptions['include_function']
-            : 'include';    // default      
+        $_sIncludeFunciton = in_array( $_aOptions[ 'include_function' ], array( 'require', 'require_once', 'include', 'include_once' ) )
+            ? $_aOptions[ 'include_function' ]
+            : 'include'; // default      
         $this->_registerClasses( $_sIncludeFunciton );        
         
     }
@@ -95,7 +95,7 @@ 
             return array();
         }
         $_aFilePaths = array();
-        foreach( ( array ) $asScanDirPaths as $_sClassDirPath ) {
+        foreach ( ( array ) $asScanDirPaths as $_sClassDirPath ) {
             if ( realpath( $_sClassDirPath ) ) {
                 $_aFilePaths = array_merge( $this->getFilePaths( $_sClassDirPath, $aSearchOptions ), $_aFilePaths );
             }
@@ -103,11 +103,11 @@ 
         
         // Store classes in an array.
         $_aClasses = array();
-        foreach( $_aFilePaths as $_sFilePath ) {
+        foreach ( $_aFilePaths as $_sFilePath ) {
             
             // Class name without a file extension.
             $_sClassNameWOExt = pathinfo( $_sFilePath, PATHINFO_FILENAME );
-            if ( in_array( $_sClassNameWOExt, $aSearchOptions['exclude_class_names'] ) ) {
+            if ( in_array( $_sClassNameWOExt, $aSearchOptions[ 'exclude_class_names' ] ) ) {
                 continue;
             }
             $_aClasses[ $_sClassNameWOExt ] = $_sFilePath; 
@@ -131,26 +131,26 @@ 
          */
         protected function getFilePaths( $sClassDirPath, array $aSearchOptions ) {
             
-            $sClassDirPath = rtrim( $sClassDirPath, '\\/' ) . DIRECTORY_SEPARATOR; // ensures the trailing (back/)slash exists. 
-            $_aAllowedExtensions = $aSearchOptions['allowed_extensions'];
-            $_aExcludeDirPaths = ( array ) $aSearchOptions['exclude_dir_paths'];
-            $_aExcludeDirNames = ( array ) $aSearchOptions['exclude_dir_names'];
+            $sClassDirPath = rtrim( $sClassDirPath, '\\/' ).DIRECTORY_SEPARATOR; // ensures the trailing (back/)slash exists. 
+            $_aAllowedExtensions = $aSearchOptions[ 'allowed_extensions' ];
+            $_aExcludeDirPaths = ( array ) $aSearchOptions[ 'exclude_dir_paths' ];
+            $_aExcludeDirNames = ( array ) $aSearchOptions[ 'exclude_dir_names' ];
             $_bIsRecursive = $aSearchOptions[ 'is_recursive' ];
             
             if ( defined( 'GLOB_BRACE' ) ) { // in some OSes this flag constant is not available.
                 $_aFilePaths = $_bIsRecursive
-                    ? $this->doRecursiveGlob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE, $_aExcludeDirPaths, $_aExcludeDirNames )
-                    : ( array ) glob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE );
+                    ? $this->doRecursiveGlob( $sClassDirPath.'*.'.$this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE, $_aExcludeDirPaths, $_aExcludeDirNames )
+                    : ( array ) glob( $sClassDirPath.'*.'.$this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE );
                 return array_filter( $_aFilePaths ); // drop non-value elements.    
             } 
                 
             // For the Solaris operation system.
             $_aFilePaths = array();
-            foreach( $_aAllowedExtensions as $__sAllowedExtension ) {
+            foreach ( $_aAllowedExtensions as $__sAllowedExtension ) {
                                 
                 $__aFilePaths = $_bIsRecursive
-                    ? $this->doRecursiveGlob( $sClassDirPath . '*.' . $__sAllowedExtension, 0, $_aExcludeDirPaths, $_aExcludeDirNames )
-                    : ( array ) glob( $sClassDirPath . '*.' . $__sAllowedExtension );
+                    ? $this->doRecursiveGlob( $sClassDirPath.'*.'.$__sAllowedExtension, 0, $_aExcludeDirPaths, $_aExcludeDirNames )
+                    : ( array ) glob( $sClassDirPath.'*.'.$__sAllowedExtension );
 
                 $_aFilePaths = array_merge( $__aFilePaths, $_aFilePaths );
                 
@@ -162,27 +162,27 @@ 
         /**
          * Constructs the file pattern of the file extension part used for the glob() function with the given file extensions.
          */
-        protected function _getGlobPatternExtensionPart( array $aExtensions=array( 'php', 'inc' ) ) {
+        protected function _getGlobPatternExtensionPart( array $aExtensions = array( 'php', 'inc' ) ) {
             return empty( $aExtensions ) 
                 ? '*'
-                : '{' . implode( ',', $aExtensions ) . '}';
+                : '{'.implode( ',', $aExtensions ).'}';
         }
         
         /**
          * The recursive version of the glob() function.
          */
-        protected function doRecursiveGlob( $sPathPatten, $nFlags=0, array $aExcludeDirs=array(), array $aExcludeDirNames=array() ) {
+        protected function doRecursiveGlob( $sPathPatten, $nFlags = 0, array $aExcludeDirs = array(), array $aExcludeDirNames = array() ) {
 
             $_aFiles = glob( $sPathPatten, $nFlags );    
             $_aFiles = is_array( $_aFiles ) ? $_aFiles : array(); // glob() can return false.
-            $_aDirs = glob( dirname( $sPathPatten ) . DIRECTORY_SEPARATOR . '*', GLOB_ONLYDIR|GLOB_NOSORT );
+            $_aDirs = glob( dirname( $sPathPatten ).DIRECTORY_SEPARATOR.'*', GLOB_ONLYDIR | GLOB_NOSORT );
             $_aDirs = is_array( $_aDirs ) ? $_aDirs : array();
             foreach ( $_aDirs as $_sDirPath ) {
 
                 if ( in_array( $_sDirPath, $aExcludeDirs ) ) { continue; }
                 if ( in_array( pathinfo( $_sDirPath, PATHINFO_DIRNAME ), $aExcludeDirNames ) ) { continue; }
                 
-                $_aFiles = array_merge( $_aFiles, $this->doRecursiveGlob( $_sDirPath . DIRECTORY_SEPARATOR . basename( $sPathPatten ), $nFlags, $aExcludeDirs ) );
+                $_aFiles = array_merge( $_aFiles, $this->doRecursiveGlob( $_sDirPath.DIRECTORY_SEPARATOR.basename( $sPathPatten ), $nFlags, $aExcludeDirs ) );
                 
             }
             return $_aFiles;
@@ -196,25 +196,25 @@ 
      * 
      */
     protected function _registerClasses( $sIncludeFunction ) {
-        spl_autoload_register( array( $this, '_replyToAutoLoad_' . $sIncludeFunction ) );
+        spl_autoload_register( array( $this, '_replyToAutoLoad_'.$sIncludeFunction ) );
     }    
         /**
          * Responds to the PHP auto-loader and includes the passed class based on the previously stored path associated with the class name in the constructor.
          */
         public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             include( $this->_aClasses[ $sCalledUnknownClassName ] );
         }
         public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             include_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
         public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             require( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
         public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             require_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         } 
     

Braces +6 added lines, -5 removed lines

@@ -137,7 +137,8 @@ 
             $_aExcludeDirNames = ( array ) $aSearchOptions['exclude_dir_names'];
             $_bIsRecursive = $aSearchOptions[ 'is_recursive' ];
             
-            if ( defined( 'GLOB_BRACE' ) ) { // in some OSes this flag constant is not available.
+            if ( defined( 'GLOB_BRACE' ) ) {
+// in some OSes this flag constant is not available.
                 $_aFilePaths = $_bIsRecursive
                     ? $this->doRecursiveGlob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE, $_aExcludeDirPaths, $_aExcludeDirNames )
                     : ( array ) glob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE );
@@ -201,19 +202,19 @@ 
         /**
          * Responds to the PHP auto-loader and includes the passed class based on the previously stored path associated with the class name in the constructor.
          */
-        public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {            
+        public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {
             if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             include( $this->_aClasses[ $sCalledUnknownClassName ] );
         }
-        public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {            
+        public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {
             if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             include_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
-        public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {            
+        public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {
             if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             require( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
-        public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {            
+        public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {
             if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { return; }
             require_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         } 

development/factory/_abstract/utility/base_utility/AdminPageFramework_Utility_String.php

Doc Comments +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
      * Converts non-alphabetic characters to underscore.
      * 
      * @since       2.0.0
-     * @return      string|null     The sanitized string.
+     * @return      string     The sanitized string.
      * @todo        Change the method name as it does not tell for what it will sanitized.
      * @todo        Examine why null needs to be returned.
      */ 
@@ -90,7 +90,7 @@ 
     /**
      * Returns the given string length.
      * @since       3.3.0
-     * @return      integer|null        Null if an array is given.
+     * @return      integer        Null if an array is given.
      */
     static public function getStringLength( $sString ) {
         return function_exists( 'mb_strlen' )

Spacing +6 added lines, -6 removed lines

@@ -56,9 +56,9 @@ 
      * @since       2.0.0
      * @return      string|integer      A numeric value will be returned. 
      */ 
-    static public function fixNumber( $nToFix, $nDefault, $nMin='', $nMax='' ) {
+    static public function fixNumber( $nToFix, $nDefault, $nMin = '', $nMax = '' ) {
 
-        if ( ! is_numeric( trim( $nToFix ) ) ) { 
+        if ( !is_numeric( trim( $nToFix ) ) ) { 
             return $nDefault; 
         }
         if ( $nMin !== '' && $nToFix < $nMin ) { 
@@ -80,7 +80,7 @@ 
     static public function minifyCSS( $sCSSRules ) {
         
         return str_replace( 
-            array( "\r\n", "\r", "\n", "\t", '  ', '    ', '    '),     // remove line breaks, tab, and white sspaces.
+            array( "\r\n", "\r", "\n", "\t", '  ', '    ', '    ' ), // remove line breaks, tab, and white sspaces.
             '', 
             preg_replace( '!/\*[^*]*\*+([^/][^*]*\*+)*/!', '', $sCSSRules ) // remove comments
         );
@@ -105,8 +105,8 @@ 
      */     
     static public function getNumberOfReadableSize( $nSize ) {
         
-        $_nReturn     = substr( $nSize, 0, -1 );
-        switch( strtoupper( substr( $nSize, -1 ) ) ) {
+        $_nReturn = substr( $nSize, 0, -1 );
+        switch ( strtoupper( substr( $nSize, -1 ) ) ) {
             case 'P':
                 $_nReturn *= 1024;
             case 'T':
@@ -132,7 +132,7 @@ 
         $_nLog      = log( $nBytes, 1024 );
         $_iPower    = ( int ) $_nLog;
         $_iSize     = pow( 1024, $_nLog - $_iPower );
-        return $_iSize . $_aUnits[ $_iPower ];
+        return $_iSize.$_aUnits[ $_iPower ];
     }
     
     /**

Braces +3 added lines, -3 removed lines

@@ -58,13 +58,13 @@
      */ 
     static public function fixNumber( $nToFix, $nDefault, $nMin='', $nMax='' ) {
 
-        if ( ! is_numeric( trim( $nToFix ) ) ) { 
+        if ( ! is_numeric( trim( $nToFix ) ) ) {
             return $nDefault; 
         }
-        if ( $nMin !== '' && $nToFix < $nMin ) { 
+        if ( $nMin !== '' && $nToFix < $nMin ) {
             return $nMin; 
         }
-        if ( $nMax !== '' && $nToFix > $nMax ) { 
+        if ( $nMax !== '' && $nToFix > $nMax ) {
             return $nMax; 
         }
         return $nToFix;

development/factory/_abstract/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/_abstract/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/_abstract/utility/wp_utility/AdminPageFramework_WPUtility_Hook.php

Doc Comments +3 added lines, -4 removed lines

@@ -115,6 +115,8 @@ 
      * @access      public
      * @since       2.0.0
      * @remark      Accepts variadic parameters.
+     * @param AdminPageFramework_Factory_Router $oCallerObject
+     * @param AdminPageFramework_Factory_Router $vArgs1
      * @return      void
      */ 
     static public function addAndDoAction( $oCallerObject, $sActionHook, $vArgs1=null, $vArgs2=null, $_and_more=null ) {
@@ -178,10 +180,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... */ ) { 
         
@@ -212,6 +210,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 +20 added lines, -20 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 );
         }
 
     }
@@ -96,15 +96,15 @@ 
      * @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 ) {
+    static public function addAndDoActions( $oCallerObject, $aActionHooks, $vArgs1 = null, $vArgs2 = null, $_and_more = null ) {
     
         $aArgs          = func_get_args();    
         $oCallerObject  = $aArgs[ 0 ];
         $aActionHooks   = $aArgs[ 1 ];
-        foreach( ( array ) $aActionHooks as $sActionHook ) {
-            if ( ! $sActionHook ) { continue; }
+        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 );     
         }
         
     }
@@ -117,16 +117,16 @@ 
      * @remark      Accepts variadic parameters.
      * @return      void
      */ 
-    static public function addAndDoAction( $oCallerObject, $sActionHook, $vArgs1=null, $vArgs2=null, $_and_more=null ) {
+    static public function addAndDoAction( $oCallerObject, $sActionHook, $vArgs1 = null, $vArgs2 = null, $_and_more = null ) {
         
         $_iArgs          = func_num_args();
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sActionHook    = $_aArgs[ 1 ];
-        if ( ! $_sActionHook ) { return; }
+        if ( !$_sActionHook ) { return; }
         add_action( $_sActionHook, array( $_oCallerObject, $_sActionHook ), 10, $_iArgs - 2 );
         array_shift( $_aArgs ); // remove the first element, the caller object
-        call_user_func_array( 'do_action' , $_aArgs );
+        call_user_func_array( 'do_action', $_aArgs );
         
     }
     /**
@@ -153,10 +153,10 @@ 
         $_aFilters      = $_aArgs[ 1 ];
         $_vInput        = $_aArgs[ 2 ];
 
-        foreach( ( array ) $_aFilters as $_sFilter ) {
-            if ( ! $_sFilter ) { continue; }
+        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 
@@ -190,7 +190,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sFilter        = $_aArgs[ 1 ];
-        if ( ! $_sFilter ) { 
+        if ( !$_sFilter ) { 
             return $_aArgs[ 2 ]; 
         }
         
@@ -198,7 +198,7 @@ 
         add_filter( $_sFilter, array( $_oCallerObject, $_sFilter ), 10, $_iArgs - 2 ); 
         
         // Remove the first element, the caller object 
-        array_shift( $_aArgs );     // removes the caller object     
+        array_shift( $_aArgs ); // removes the caller object     
         
         // Trigger the magic method __call().
         return call_user_func_array( 'apply_filters', $_aArgs ); // $_aArgs: $vData, $vArgs...
@@ -214,17 +214,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 +3 added lines, -3 removed lines

@@ -146,7 +146,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 ];
@@ -183,14 +183,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/_abstract/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(

Spacing +27 added lines, -27 removed lines

@@ -56,7 +56,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 ) );
@@ -75,28 +75,28 @@ 
              * @since       3.5.3
              */            
             static public function getPostTypeByTypeNow() {
-                if ( isset( $GLOBALS['typenow'] ) && $GLOBALS['typenow'] ) {
-                    return $GLOBALS['typenow'];
+                if ( isset( $GLOBALS[ 'typenow' ] ) && $GLOBALS[ 'typenow' ] ) {
+                    return $GLOBALS[ 'typenow' ];
                 }                
             }
             static public function getPostTypeByScreenObject() {
                 if ( 
-                    isset( $GLOBALS['current_screen']->post_type )
-                    && $GLOBALS['current_screen']->post_type 
+                    isset( $GLOBALS[ 'current_screen' ]->post_type )
+                    && $GLOBALS[ 'current_screen' ]->post_type 
                 ) {
-                    return $GLOBALS['current_screen']->post_type;
+                    return $GLOBALS[ 'current_screen' ]->post_type;
                 }    
             }         
             /**
              * Tries to find the post type from the URL query for type.
              */
             static public function getPostTypeByREQUEST() {
-                if ( isset( $_REQUEST['post_type'] ) ) {
-                    return sanitize_key( $_REQUEST['post_type'] );
+                if ( isset( $_REQUEST[ 'post_type' ] ) ) {
+                    return sanitize_key( $_REQUEST[ 'post_type' ] );
                 }
-                if ( isset( $_GET['post'] ) && $_GET['post'] ) {
+                if ( isset( $_GET[ 'post' ] ) && $_GET[ 'post' ] ) {
                     // It will perform a database query.
-                    return get_post_type( $_GET['post'] );
+                    return get_post_type( $_GET[ 'post' ] );
                 }                                
             }
             /**
@@ -104,10 +104,10 @@ 
              */
             static public function getPostTypeByPostObject() {
                 if ( 
-                    isset( $GLOBALS['post'], $GLOBALS['post']->post_type ) 
-                    && $GLOBALS['post']->post_type 
+                    isset( $GLOBALS[ 'post' ], $GLOBALS[ 'post' ]->post_type ) 
+                    && $GLOBALS[ 'post' ]->post_type 
                 ) {
-                    return $GLOBALS['post']->post_type;
+                    return $GLOBALS[ 'post' ]->post_type;
                 }
             }            
             /**#@-*/
@@ -119,9 +119,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', ) ) ) {
+        if ( !in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php',) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -136,10 +136,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 );
@@ -172,7 +172,7 @@ 
      * 
      * @since 3.0.0
      */
-    static public function isPostListingPage( $asPostTypes=array() ) {
+    static public function isPostListingPage( $asPostTypes = array() ) {
                 
         if ( 'edit.php' != self::getPageNow() ) { 
             return false;
@@ -180,11 +180,11 @@ 
         
         $_aPostTypes = self::getAsArray( $asPostTypes );    
         
-        if ( ! isset( $_GET['post_type'] )  ) { 
+        if ( !isset( $_GET[ 'post_type' ] ) ) { 
             return in_array( 'post', $_aPostTypes );
         }
 
-        return in_array( $_GET['post_type'], $_aPostTypes );
+        return in_array( $_GET[ 'post_type' ], $_aPostTypes );
         
     }
     
@@ -209,8 +209,8 @@ 
         }
         
         // If already set, use that.
-        if ( isset( $GLOBALS['pagenow'] ) ) {
-            self::$_sPageNow = $GLOBALS['pagenow'];
+        if ( isset( $GLOBALS[ 'pagenow' ] ) ) {
+            self::$_sPageNow = $GLOBALS[ 'pagenow' ];
             return self::$_sPageNow;
         }
                         
@@ -230,7 +230,7 @@ 
          * @return      string      The current page url base name.
          */
         static private function _getPageNow_FrontEnd() {
-            if ( preg_match( '#([^/]+\.php)([?/].*?)?$#i', $_SERVER['PHP_SELF'], $_aMatches ) ) {
+            if ( preg_match( '#([^/]+\.php)([?/].*?)?$#i', $_SERVER[ 'PHP_SELF' ], $_aMatches ) ) {
                 return strtolower( $_aMatches[ 1 ] );
             }
             return 'index.php';                
@@ -276,7 +276,7 @@ 
                 else {
                     $_sNeedle = '#/wp-admin/?(.*?)$#i';
                 }                
-                preg_match( $_sNeedle, $_SERVER['PHP_SELF'], $_aMatches );
+                preg_match( $_sNeedle, $_SERVER[ 'PHP_SELF' ], $_aMatches );
                 return preg_replace( '#\?.*?$#', '', trim( $_aMatches[ 1 ], '/' ) );
                 
             }
@@ -311,7 +311,7 @@ 
         
         if ( isset( $GLBOALS[ 'page_hook' ] ) ) {
             return is_network_admin() 
-                ? $GLBOALS[ 'page_hook' ] . '-network'
+                ? $GLBOALS[ 'page_hook' ].'-network'
                 : $GLBOALS[ 'page_hook' ];
         }
         
@@ -325,11 +325,11 @@ 
      * @since       DEVVER
      * @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, 

Braces +7 added lines, -9 removed lines

@@ -33,7 +33,7 @@ 
         
         // Since the current page will be the same throughout the execution of the script, 
         // once it's found, there is no need to find it again.
-        if ( $_sCurrentPostType ) { 
+        if ( $_sCurrentPostType ) {
             return $_sCurrentPostType; 
         }
         $_sCurrentPostType = self::_getCurrentPostType();
@@ -139,7 +139,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 );
@@ -158,7 +158,7 @@ 
         $_aPostTypes = self::getAsArray( $asPostTypes );        
         
         // If the parameter is empty, 
-        if ( empty( $_aPostTypes ) ) { 
+        if ( empty( $_aPostTypes ) ) {
             return true;
         }
 
@@ -174,13 +174,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 );
         }
 
@@ -269,11 +269,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 );

development/factory/admin_page/_model/AdminPageFramework_FormEmail.php

Doc Comments +1 added lines

@@ -202,6 +202,7 @@
          * 
          * @since       3.3.0
          * @since       3.4.2       Moved from the validation class.
+         * @param string $sKey
          */
         private function _getEmailArgument( $aInput, array $aEmailOptions, $sKey, $sSectionID ) {
             

Indentation +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
                 return '';
                 
             }
-       /**
+        /**
          * Sets the mail content type to HTML.
          * @since       3.3.0
          * @since       3.4.2       Moved from the validation class.

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
         }
 
         // Send mail.
-        $_bSent         = wp_mail( 
+        $_bSent = wp_mail( 
             $this->_getEmailArgument( $aInput, $aEmailOptions, 'to', $sSubmitSectionID ),
             $this->_getEmailArgument( $aInput, $aEmailOptions, 'subject', $sSubmitSectionID ),
             $_bIsHTML 
@@ -100,7 +100,7 @@ 
         remove_filter( 'wp_mail_from_name', array( $this, '_replyToSetEmailSenderAddress' ) );
         
         // Clean up.
-        foreach( $this->_aPathsToDelete as $_sPath ) {
+        foreach ( $this->_aPathsToDelete as $_sPath ) {
             unlink( $_sPath );
         }
         
@@ -121,7 +121,7 @@ 
             }
             
             $_aAttachments = $this->getAsArray( $asAttachments );          
-            foreach( $_aAttachments as $_iIndex => $_sPathORURL ) {
+            foreach ( $_aAttachments as $_iIndex => $_sPathORURL ) {
                 
                 // If it is a file path, fine.
                 if ( is_file( $_sPathORURL ) ) {
@@ -211,7 +211,7 @@ 
             }
             
             // If the key element is empty, search the corresponding item in the same section. 
-            if ( ! $aEmailOptions[ $sKey ] ) {
+            if ( !$aEmailOptions[ $sKey ] ) {
                 return $this->getArrayValueByArrayKeys( $aInput, array( $sSectionID, $sKey ) );
             }
             

Braces +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
                 
                 // Download the file. It returns a string value on success. WP Error object on failure.
                 $_sPath = $this->download( $sURL, 10 );
-                if ( is_string( $_sPath ) ) {                    
+                if ( is_string( $_sPath ) ) {
                     $this->_aPathsToDelete[ $_sPath ] = $_sPath;
                     return $_sPath;
                 }

development/factory/admin_page/_model/AdminPageFramework_ImportOptions.php

Doc Comments +5 added lines

@@ -89,6 +89,11 @@
         return $vData;
         
     }
+
+    /**
+     * @param string $vData
+     * @param string $sFormatType
+     */
     public function formatImportData( &$vData, $sFormatType=null ) {
         
         $sFormatType = isset( $sFormatType ) ? $sFormatType : $this->getFormatType();

Spacing +3 added lines, -3 removed lines

@@ -57,11 +57,11 @@ 
         
     }
     
-    private function getElementInFilesArray( array $aFilesImport, $sInputID, $sElementKey='error' ) {
+    private function getElementInFilesArray( array $aFilesImport, $sInputID, $sElementKey = 'error' ) {
 
         $sElementKey = strtolower( $sElementKey );
         return $this->getElement( 
-            $aFilesImport,  // subject array
+            $aFilesImport, // subject array
             array( $sElementKey, $sInputID ), // dimensional keys
             null    // default
         );
@@ -89,7 +89,7 @@ 
         return $vData;
         
     }
-    public function formatImportData( &$vData, $sFormatType=null ) {
+    public function formatImportData( &$vData, $sFormatType = null ) {
         
         $sFormatType = isset( $sFormatType ) ? $sFormatType : $this->getFormatType();
         switch ( strtolower( $sFormatType ) ) {