michaeluno / admin-page-framework

development/factory/_common/utility/AdminPageFramework_RegisterClasses.php

Doc Comments +2 added lines, -1 removed lines

@@ -75,7 +75,7 @@ 
      * Sets up properties and performs registering classes.
      *
      * 
-     * @param    array|string       $asScanDirPath       the target directory path to scan
+     * @param    array|string       $asScanDirPaths       the target directory path to scan
      * @param    array              $aOptions            The recursive settings.
      * <ul>
      *      <li><code>is_recursive</code> - (boolean) determines whether the scan should be performed recursively.</li>
@@ -217,6 +217,7 @@ 
         /**
          * The recursive version of the glob() function.
          * @internal
+         * @param string $sPathPatten
          */
         protected function doRecursiveGlob( $sPathPatten, $nFlags=0, array $aExcludeDirs=array(), array $aExcludeDirNames=array() ) {
 

Spacing +25 added lines, -25 removed lines

@@ -66,7 +66,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' */ ),
     );
@@ -105,10 +105,10 @@ 
      * 
      * @remark The directory paths set for the 'exclude_dir_paths' option should use the system directory separator.
      */
-    public function __construct( $asScanDirPaths, array $aOptions=array(), array $aClasses=array() ) {
+    public function __construct( $asScanDirPaths, array $aOptions = array(), array $aClasses = array() ) {
         
         $_aOptions = $aOptions + self::$_aStructure_Options;
-        $this->_aClasses   = $aClasses + $this->_getClassArray( $asScanDirPaths, $_aOptions );
+        $this->_aClasses = $aClasses + $this->_getClassArray( $asScanDirPaths, $_aOptions );
         $this->_registerClasses( $_aOptions[ 'include_function' ] );
         
     }
@@ -134,7 +134,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 );
             }
@@ -142,11 +142,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; 
@@ -176,26 +176,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 );
                 
@@ -208,28 +208,28 @@ 
          * Constructs the file pattern of the file extension part used for the glob() function with the given file extensions.
          * @internal
          */
-        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.
          * @internal
          */
-        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;
@@ -244,7 +244,7 @@ 
      * @return      void
      */
     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.
@@ -252,25 +252,25 @@ 
          * @return      void
          */        
         public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             include( $this->_aClasses[ $sCalledUnknownClassName ] );
         }
         public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             include_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
         public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             require( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
         public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             require_once( $this->_aClasses[ $sCalledUnknownClassName ] );

Braces +10 added lines, -9 removed lines

@@ -181,7 +181,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 );
@@ -250,26 +251,26 @@ 
          * @internal
          * @return      void
          */        
-        public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             include( $this->_aClasses[ $sCalledUnknownClassName ] );
         }
-        public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             include_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
-        public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             require( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
-        public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             require_once( $this->_aClasses[ $sCalledUnknownClassName ] );

development/utility/class_tester/AdminPageFramework_ClassTester.php

Doc Comments +3 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
      * Creates an object instance with dynamic parameters.
      * 
      * @since       3.7.10
-     * @param       string      $sCalssName     The class name for testing.
+     * @param       string      $sClassName     The class name for testing.
      * @param       array       $aParameters    The parameters to pass to the constructor of the class set in the first parameter.
      * @return      object      An object instance of the class specified in the first parameter.
      */
@@ -61,7 +61,7 @@ 
      * 
      * @since       3.7.10
      * @param       object      $oClass         The subject class object.
-     * @param       string      $sMathodName    The subject method name.
+     * @param       string      $sMethodName    The subject method name.
      * @param       array       $aParameters    The parameters to pass to the method set in the second parameter.    
      * @remark      This supports private methods to be executed.
      */
@@ -87,6 +87,7 @@ 
     }
         /**
          * @since       3.7.10
+         * @param string $sClassName
          * @return      object
          * @internal
          */

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
      * @param       array       $aParameters    The parameters to pass to the constructor of the class set in the first parameter.
      * @return      object      An object instance of the class specified in the first parameter.
      */
-    static public function getInstance( $sClassName, array $aParameters=array() ) {
+    static public function getInstance( $sClassName, array $aParameters = array() ) {
         
         $_oReflection = new ReflectionClass( $sClassName );
         return $_oReflection->newInstanceArgs( $aParameters );               
@@ -69,7 +69,7 @@ 
         // For PHP 5.2.x or below
         if ( version_compare( phpversion(), '<', '5.3.0' ) ) {
             trigger_error(
-                'Program Name' . ': ' 
+                'Program Name'.': ' 
                     . sprintf( 
                         'The method cannot run with your PHP version: %1$s',
                         phpversion()

development/document/factory/AdminPageFramework_Documentation.php

Indentation -1 removed lines

@@ -196,7 +196,6 @@
  * For common hooks throughout the other factory components, see [Base Factory](./package-AdminPageFramework.Common.Factory.html).
  * 
  * <h3>Factory Specific Hooks</h3>
-
  * <h4> Action Hooks</h4>
  * <ul>
  *     <li>**load_{page slug}** – [2.1.0+] triggered when the framework's page is loaded before the header gets sent. This will not be triggered in the admin pages that are not registered by the framework. The first parameter: class object [3.1.2+].</li>

development/factory/user_meta/_model/AdminPageFramework_UserMeta_Model___UserMeta.php

Spacing +1 added lines, -1 removed lines

@@ -21,6 +21,6 @@
     /**
      * The callback function name or the callable object to retrieve meta data.
      */
-    protected $osCallable   = 'get_user_meta'; 
+    protected $osCallable = 'get_user_meta'; 
         
 }

development/factory/_common/_abstract/AdminPageFramework_Factory_View.php

Spacing +12 added lines, -12 removed lines

@@ -41,7 +41,7 @@ 
          * @return      string      the input id attribute
          */    
         public function _replyToGetSectionName( /* $sSectionName, $aSectionset */ ) {
-            $_aParams = func_get_args() + array( null, null, );
+            $_aParams = func_get_args() + array( null, null,);
             return $_aParams[ 0 ];
         }
         
@@ -69,8 +69,8 @@ 
          * @since       3.6.0
          * @return      string
          */
-        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ )  {            
-            $_aParams = func_get_args() + array( null, null, );
+        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ ) {            
+            $_aParams = func_get_args() + array( null, null,);
             return $_aParams[ 0 ];                        
         }
         /**
@@ -80,7 +80,7 @@ 
          * @return      string
          */        
         public function _replyToGetFlatFieldName( /* $sFieldName, $aFieldset */ ) {            
-            $_aParams = func_get_args() + array( null, null, );
+            $_aParams = func_get_args() + array( null, null,);
             return $_aParams[ 0 ];            
         }
         
@@ -102,7 +102,7 @@ 
          * @return      string      the flat input name attribute
          */    
         public function _replyToGetFlatInputName( /* $sFlatNameAttribute, $aField, $sKey */ ) {
-            $_aParams   = func_get_args() + array( null, null, null );
+            $_aParams = func_get_args() + array( null, null, null );
             return $_aParams[ 0 ];
         }
 
@@ -144,14 +144,14 @@ 
                 'if'            => true,
                 'capability'    => '',
             );
-            if ( ! $aElementDefinition[ 'if' ] ) {
+            if ( !$aElementDefinition[ 'if' ] ) {
                 return false;
             }
             // For front-end forms that allow guests, the capability level can be empty. In that case, return true.
-            if ( ! $aElementDefinition[ 'capability' ] ) {
+            if ( !$aElementDefinition[ 'capability' ] ) {
                 return true;
             }
-            if ( ! current_user_can( $aElementDefinition[ 'capability' ] ) ) {
+            if ( !current_user_can( $aElementDefinition[ 'capability' ] ) ) {
                 return false;
             }            
             return $bDefault;
@@ -187,7 +187,7 @@ 
             $this,
             array( 
                 // section_{instantiated class name}_{section id}
-                'section_head_' . $this->oProp->sClassName . '_' . $aSectionset[ 'section_id' ] 
+                'section_head_'.$this->oProp->sClassName.'_'.$aSectionset[ 'section_id' ] 
             ), 
             $sSectionDescription
         );
@@ -204,15 +204,15 @@ 
      */
     public function _replyToGetFieldOutput( $sFieldOutput, $aFieldset ) {
 
-        $_sSectionPart  = $this->oUtil->getAOrB(
+        $_sSectionPart = $this->oUtil->getAOrB(
             isset( $aFieldset[ 'section_id' ] ) && '_default' !== $aFieldset[ 'section_id' ],
-            '_' . $aFieldset[ 'section_id' ],
+            '_'.$aFieldset[ 'section_id' ],
             ''
         );
         return $this->oUtil->addAndApplyFilters(
             $this,
             array( 
-                'field_' . $this->oProp->sClassName . $_sSectionPart . '_' . $aFieldset[ 'field_id' ]
+                'field_'.$this->oProp->sClassName.$_sSectionPart.'_'.$aFieldset[ 'field_id' ]
             ),
             $sFieldOutput,
             $aFieldset // the field array

Braces +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
          * @since       3.6.0
          * @return      string
          */
-        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ )  {            
+        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ ) {
             $_aParams = func_get_args() + array( null, null, );
             return $_aParams[ 0 ];                        
         }
@@ -79,7 +79,7 @@ 
          * @since       3.6.0
          * @return      string
          */        
-        public function _replyToGetFlatFieldName( /* $sFieldName, $aFieldset */ ) {            
+        public function _replyToGetFlatFieldName( /* $sFieldName, $aFieldset */ ) {
             $_aParams = func_get_args() + array( null, null, );
             return $_aParams[ 0 ];            
         }

development/factory/_common/_abstract/_controller/AdminPageFramework_Link_Base.php

Spacing +19 added lines, -19 removed lines

@@ -26,9 +26,9 @@ 
     /**
      * Sets up hooks and properties.
      */ 
-    public function __construct( $oProp, $oMsg=null ) {
+    public function __construct( $oProp, $oMsg = null ) {
         
-        if ( ! $this->_isLoadable( $oProp ) ) {
+        if ( !$this->_isLoadable( $oProp ) ) {
             return;
         }
                 
@@ -40,7 +40,7 @@ 
         // Add an action link in the plugin listing page
         if ( $this->_shouldSetPluginActionLinks() ) {            
             add_filter( 
-                'plugin_action_links_' . plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ),
+                'plugin_action_links_'.plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ),
                 array( $this, '_replyToAddSettingsLinkInPluginListingPage' ), 
                 20     // set a lower priority so that the link will be embedded at the beginning ( the most left hand side ).
             );     
@@ -55,13 +55,13 @@ 
          * @return      boolean
          */
         private function _isLoadable( $oProp ) {
-            if ( ! $oProp->bIsAdmin ) { 
+            if ( !$oProp->bIsAdmin ) { 
                 return false;
             }
             if ( $oProp->bIsAdminAjax ) {
                 return false;
             }
-            return ! $this->hasBeenCalled( 'links_' . $oProp->sClassName );
+            return !$this->hasBeenCalled( 'links_'.$oProp->sClassName );
         }
         
     /**
@@ -72,11 +72,11 @@ 
     protected function _shouldSetPluginActionLinks() {
         
         // It is possible that the sub-objects are not set when the class is considered not loadable.
-        if ( ! isset( $this->oProp ) ) {
+        if ( !isset( $this->oProp ) ) {
             return false;
         }
         
-        if ( ! in_array( $this->oProp->sPageNow, array( 'plugins.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'plugins.php' ) ) ) {
             return false;
         }
         return 'plugin' === $this->oProp->aScriptInfo[ 'sType' ];
@@ -143,7 +143,7 @@ 
                         array(
                             'href'      => $aScriptInfo[ 'sURI' ],
                             'target'    => '_blank',
-                            'title'     => $aScriptInfo[ 'sName' ] . $_sVersion . $_sDescription 
+                            'title'     => $aScriptInfo[ 'sName' ].$_sVersion.$_sDescription 
                         ), 
                         $aScriptInfo[ 'sName' ] 
                     )    
@@ -165,13 +165,13 @@ 
                 $_sAuthorInfo = $this->getAOrB(
                     empty( $aScriptInfo[ 'sAuthor' ] ),
                     $_sAuthorInfo,
-                    ' by ' . $_sAuthorInfo
+                    ' by '.$_sAuthorInfo
                 );
                 
                 // Enclosing the output in a span tag as the outer element is a '<p>' tag. So this cannot be div.
                 // 3.5.7+ Added the class attribute for acceptance testing
                 return "<span class='apf-script-info'>"  
-                        . $_sPluginInfo . $_sAuthorInfo
+                        . $_sPluginInfo.$_sAuthorInfo
                     . "</span>";
           
             }
@@ -202,7 +202,7 @@ 
                         array(
                             'href'      => $aScriptInfo[ 'sURI' ],
                             'target'    => '_blank',
-                            'title'     => $aScriptInfo[ 'sName' ] . $_sVersion . $_sDescription,
+                            'title'     => $aScriptInfo[ 'sName' ].$_sVersion.$_sDescription,
                         ), 
                         $aScriptInfo[ 'sName' ]
                     )                   
@@ -212,16 +212,16 @@ 
                 // 3.5.7+ added the 'apf-credit' class attribute for acceptance testing
                 // 3.7.0+  added the footer-thankyou id attribute.
                 return "<span class='apf-credit' id='footer-thankyou'>" 
-                    . $this->oMsg->get( 'powered_by' ) . '&nbsp;' 
+                    . $this->oMsg->get( 'powered_by' ).'&nbsp;' 
                     . $_sLibraryInfo
                     . ",&nbsp;"
-                    . $this->oMsg->get( 'and' ) . '&nbsp;' 
+                    . $this->oMsg->get( 'and' ).'&nbsp;' 
                     . $this->getHTMLTag( 
                         'a', 
                         array(
                             'href'      => 'https://wordpress.org',
                             'target'    => '_blank',
-                            'title'     => 'WordPress ' . $GLOBALS[ 'wp_version' ]
+                            'title'     => 'WordPress '.$GLOBALS[ 'wp_version' ]
                         ), 
                         'WordPress'
                     )
@@ -237,7 +237,7 @@ 
          */
         protected function _setFooterHooks() {
             
-            add_filter( 'admin_footer_text' , array( $this, '_replyToAddInfoInFooterLeft' ) );
+            add_filter( 'admin_footer_text', array( $this, '_replyToAddInfoInFooterLeft' ) );
             add_filter( 'update_footer', array( $this, '_replyToAddInfoInFooterRight' ), 11 );
             
         }             
@@ -249,7 +249,7 @@ 
              * @callback    filter      admin_footer_text
              * @internal
              */ 
-            public function _replyToAddInfoInFooterLeft( $sLinkHTML='' ) {
+            public function _replyToAddInfoInFooterLeft( $sLinkHTML = '' ) {
 
                 $sLinkHTML = empty( $this->oProp->aScriptInfo[ 'sName' ] )
                     ? $sLinkHTML
@@ -257,7 +257,7 @@ 
              
                 return $this->addAndApplyFilters( 
                     $this->oProp->oCaller, 
-                    'footer_left_' . $this->oProp->sClassName, 
+                    'footer_left_'.$this->oProp->sClassName, 
                     $sLinkHTML
                 );
              
@@ -270,10 +270,10 @@ 
              * @callback    filter      admin_footer_text
              * @internal
              */     
-            public function _replyToAddInfoInFooterRight( $sLinkHTML='' ) {
+            public function _replyToAddInfoInFooterRight( $sLinkHTML = '' ) {
                 return $this->addAndApplyFilters( 
                     $this->oProp->oCaller, 
-                    'footer_right_' . $this->oProp->sClassName, 
+                    'footer_right_'.$this->oProp->sClassName, 
                     $this->oProp->aFooterInfo[ 'sRight' ]
                 );                
             }       

Braces +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
         add_action( 'in_admin_footer', array( $this, '_replyToSetFooterInfo' ) );
         
         // Add an action link in the plugin listing page
-        if ( $this->_shouldSetPluginActionLinks() ) {            
+        if ( $this->_shouldSetPluginActionLinks() ) {
             add_filter( 
                 'plugin_action_links_' . plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ),
                 array( $this, '_replyToAddSettingsLinkInPluginListingPage' ), 
@@ -55,7 +55,7 @@ 
          * @return      boolean
          */
         private function _isLoadable( $oProp ) {
-            if ( ! $oProp->bIsAdmin ) { 
+            if ( ! $oProp->bIsAdmin ) {
                 return false;
             }
             if ( $oProp->bIsAdminAjax ) {

development/factory/_common/_abstract/_model/AdminPageFramework_Factory_Model___Meta_Base.php

Spacing +9 added lines, -9 removed lines

@@ -23,17 +23,17 @@ 
     /**
      * The callback function name or the callable object to retrieve meta data.
      */
-    protected $osCallable   = 'get_post_meta';
+    protected $osCallable = 'get_post_meta';
 
     /**
      * The object ID such as post ID, user ID, and term ID.
      */
-    public $iObjectID       = 0;
+    public $iObjectID = 0;
     
     /**
      * A form fieldsets array.
      */
-    public $aFieldsets      = array();
+    public $aFieldsets = array();
     
     /**
      * Sets up properties.
@@ -57,7 +57,7 @@ 
      */
     public function get() {
 
-        if ( ! $this->iObjectID ) {
+        if ( !$this->iObjectID ) {
             return array();
         }
         return $this->_getSavedDataFromFieldsets(
@@ -76,11 +76,11 @@ 
             
             $_aMetaKeys = $this->_getMetaKeys( $iObjectID );
             $_aMetaData = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
                 
-                if ( '_default' == $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aFieldset ) {
-                        if ( ! in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
+                if ( '_default' == $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aFieldset ) {
+                        if ( !in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
                             continue;
                         }
                         $_aMetaData[ $_aFieldset[ 'field_id' ] ] = call_user_func_array(
@@ -93,7 +93,7 @@ 
                         );
                     }
                 }
-                if ( ! in_array( $_sSectionID, $_aMetaKeys ) ) {
+                if ( !in_array( $_sSectionID, $_aMetaKeys ) ) {
                     continue;
                 }                
                 $_aMetaData[ $_sSectionID ] = call_user_func_array(

Doc Comments +1 added lines

@@ -69,6 +69,7 @@
          * 
          * @since       3.7.0
          * @uses        get_post_meta()
+         * @param integer $iObjectID
          * @return      array
          */
         private function _getSavedDataFromFieldsets( $iObjectID, $aFieldsets ) {

development/factory/_common/form/_view/fieldset/AdminPageFramework_Form_View___Fieldset_Base.php

Braces +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
                 self::$_bIsLoadedJSScripts_Widget = true;
             }
             
-            if ( self::$_bIsLoadedJSScripts ) { 
+            if ( self::$_bIsLoadedJSScripts ) {
                 return; 
             }
             self::$_bIsLoadedJSScripts = true;
@@ -255,7 +255,7 @@ 
      * 
      * @since 3.0.0
      */    
-    protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
+    protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {
     
         $_sScript = <<<JAVASCRIPTS
     jQuery( document ).ready( function() {

Doc Comments +4 added lines, -1 removed lines

@@ -174,6 +174,7 @@ 
      * Returns the repeatable fields script.
      * 
      * @since 2.1.3
+     * @param string $sFieldsContainerID
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
         
@@ -244,6 +245,7 @@ 
             /**
              * @since       3.8.13
              * @param       array       $aArguments
+             * @param string $sSmallButtonSelector
              * @return      string
              */
             private function ___getAddButtonAttribtes( $aArguments, $sFieldsContainerID, $sSmallButtonSelector ) {
@@ -262,7 +264,7 @@ 
             }
             /**
              * @since       3.8.13
-             * @param       array       $aArguments
+             * @param string $sSmallButtonSelector
              * @return      string
              */
             private function ___getRemvoeButtonAttribtes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
@@ -306,6 +308,7 @@ 
      * Returns the sortable fields script.
      * 
      * @since 3.0.0
+     * @param string $sFieldsContainerID
      */    
     protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
     

Indentation +3 added lines, -3 removed lines

@@ -255,8 +255,8 @@ 
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
                         : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                            . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
+                            . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -268,7 +268,7 @@ 
             private function ___getRemvoeButtonAttribtes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
                 $_aMinusButtonAttributes = array(
                     'class'     => 'repeatable-field-remove-button button-secondary repeatable-field-button button'
-                         . $sSmallButtonSelector,
+                            . $sSmallButtonSelector,
                     'title'     => $this->oMsg->get( 'remove' ),
                     'style'     => $iFieldCount <= 1
                         ? 'visibility: hidden'

Spacing +21 added lines, -21 removed lines

@@ -68,7 +68,7 @@ 
      * @param       object      $oMsg                   An object storing the system messages.
      * @param       array       $aCallbacks             An array storing the form-field specific callbacks.     
      */
-    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks=array() ) {
+    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks = array() ) {
 
         // Set up the properties that will be accessed later in the methods.
         $this->aFieldset                = $this->_getFormatted( $aFieldset, $aFieldTypeDefinitions );
@@ -77,13 +77,13 @@ 
         $this->aErrors                  = $this->getAsArray( $aErrors );
         $this->oMsg                     = $oMsg;
         $this->aCallbacks               = $aCallbacks + array(
-            'hfID'              => null,    // the input id attribute
-            'hfTagID'           => null,    // the fieldset/field row container id attribute
-            'hfName'            => null,    // the input name attribute
-            'hfNameFlat'        => null,    // the flat input name attribute                
+            'hfID'              => null, // the input id attribute
+            'hfTagID'           => null, // the fieldset/field row container id attribute
+            'hfName'            => null, // the input name attribute
+            'hfNameFlat'        => null, // the flat input name attribute                
             'hfInputName'       => null,
             'hfInputNameFlat'   => null,
-            'hfClass'           => null,    // the class attribute
+            'hfClass'           => null, // the class attribute
         );        
         
         // 2. Load necessary JavaScript scripts.
@@ -150,9 +150,9 @@ 
          * @since   3.2.0   Added the `$sFieldsType` parameter.
          * @internal
          */
-        private function _loadScripts( $sStructureType='' ) {
+        private function _loadScripts( $sStructureType = '' ) {
 
-            if ( 'widget' === $sStructureType && ! self::$_bIsLoadedJSScripts_Widget ) {
+            if ( 'widget' === $sStructureType && !self::$_bIsLoadedJSScripts_Widget ) {
                 new AdminPageFramework_Form_View___Script_Widget;
                 self::$_bIsLoadedJSScripts_Widget = true;
             }
@@ -177,8 +177,8 @@ 
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
         
-        $_sSmallButtons         = '"' . $this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, true ) . '"';
-        $_sNestedFieldsButtons  = '"' . $this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, false ) . '"';
+        $_sSmallButtons         = '"'.$this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, true ).'"';
+        $_sNestedFieldsButtons  = '"'.$this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, false ).'"';
         $_aJSArray              = json_encode( $aSettings );
         $_sScript               = <<<JAVASCRIPTS
 jQuery( document ).ready( function() {
@@ -224,18 +224,18 @@ 
          * @since       3.8.0
          * @return      string
          */
-        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall=true ) {
+        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall = true ) {
 
             // @todo Move this formatting routine to the field-set formatter class.
             $_oFormatter             = new AdminPageFramework_Form_Model___Format_RepeatableField( $aArguments, $this->oMsg );
             $_aArguments             = $_oFormatter->get();
             $_sSmallButtonSelector   = $bSmall ? ' button-small' : '';
-            return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                    . "<a " . $this->___getRemvoeButtonAttribtes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">-</a>"
-                    . "<a " . $this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">+</a>"
+            return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                    . "<a ".$this->___getRemvoeButtonAttribtes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">-</a>"
+                    . "<a ".$this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">+</a>"
                 . "</div>"
                 . $this->getModalForDisabledRepeatableElement(
-                    'repeatable_field_disabled_' . $sFieldsContainerID,
+                    'repeatable_field_disabled_'.$sFieldsContainerID,
                     $_aArguments[ 'disabled' ]
                 );
 
@@ -254,9 +254,9 @@ 
                     'data-id'   => $sFieldsContainerID,
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
-                        : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                        : '#TB_inline?width='.$aArguments[ 'disabled' ][ 'box_width' ]
+                          . '&height='.$aArguments[ 'disabled' ][ 'box_height' ]
+                          . '&inlineId='.'repeatable_field_disabled_'.$sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -283,10 +283,10 @@ 
              * @return      string
              */
             private function ___getContainerAttributes( $aArguments ) {
-                $_aContainerAttributes   = array(
+                $_aContainerAttributes = array(
                     'class' => $this->getClassAttribute(
                         'admin-page-framework-repeatable-field-buttons',
-                        ! empty( $aArguments[ 'disabled' ] )
+                        !empty( $aArguments[ 'disabled' ] )
                             ? 'disabled'
                             : ''
                     ),
@@ -299,7 +299,7 @@ 
                     unset( $aArguments[ 'disabled' ] );
                 }
                 return $this->getAttributes( $_aContainerAttributes )
-                    . ' ' . $this->getDataAttributes( $aArguments );
+                    . ' '.$this->getDataAttributes( $aArguments );
             }
 
     /**

development/factory/_common/form/_view/generator/field/AdminPageFramework_Form_View___Generate_FieldName.php

Spacing +5 added lines, -5 removed lines

@@ -26,8 +26,8 @@ 
         return $_sResult;
     }
         
-    public function getModel()     {
-        return $this->get() . '[' . $this->sIndexMark . ']';
+    public function getModel() {
+        return $this->get().'['.$this->sIndexMark.']';
     }
         
         /**
@@ -35,14 +35,14 @@ 
          */
         protected function _getFieldName() {
 
-            $_aFieldPath   = $this->aArguments[ '_field_path_array' ];
-            if ( ! $this->_isSectionSet() ) {
+            $_aFieldPath = $this->aArguments[ '_field_path_array' ];
+            if ( !$this->_isSectionSet() ) {
                 return $this->_getInputNameConstructed( $_aFieldPath );
             }
             
             $_aSectionPath = $this->aArguments[ '_section_path_array' ];
             if ( $this->_isSectionSet() && isset( $this->aArguments[ '_section_index' ] ) ) {
-                $_aSectionPath[] = $this->aArguments[ '_section_index' ];
+                $_aSectionPath[ ] = $this->aArguments[ '_section_index' ];
             }
             $_sFieldName = $this->_getInputNameConstructed( 
                 array_merge( $_aSectionPath, $_aFieldPath )

Braces +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         return $_sResult;
     }
         
-    public function getModel()     {
+    public function getModel() {
         return $this->get() . '[' . $this->sIndexMark . ']';
     }