michaeluno / admin-page-framework

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

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/script/AdminPageFramework_Form_View___Script_RepeatableSection.php

Spacing +12 added lines, -12 removed lines

@@ -441,7 +441,7 @@ 
         if ( empty( $asArguments ) ) {
             return ''; 
         }
-        if ( self::hasBeenCalled( 'repeatable_section_' . $sContainerTagID ) ) {
+        if ( self::hasBeenCalled( 'repeatable_section_'.$sContainerTagID ) ) {
             return '';
         }
 
@@ -449,16 +449,16 @@ 
         $_oFormatter    = new AdminPageFramework_Form_Model___Format_RepeatableSection( $asArguments, $oMsg );
         $_aArguments    = $_oFormatter->get();
         $_sButtons      = "<div class='admin-page-framework-repeatable-section-buttons-outer-container'>"
-                . "<div " . self::___getContainerAttributes( $_aArguments, $oMsg ) . ' >'
-                    . "<a " . self::___getRemoveButtonAttributes( $sContainerTagID, $oMsg, $iSectionCount ) . ">-</a>"
-                    . "<a " . self::___getAddButtonAttributes( $sContainerTagID, $oMsg, $_aArguments ) . ">+</a>"
+                . "<div ".self::___getContainerAttributes( $_aArguments, $oMsg ).' >'
+                    . "<a ".self::___getRemoveButtonAttributes( $sContainerTagID, $oMsg, $iSectionCount ).">-</a>"
+                    . "<a ".self::___getAddButtonAttributes( $sContainerTagID, $oMsg, $_aArguments ).">+</a>"
                 . "</div>"
             . "</div>"
             . AdminPageFramework_Form_Utility::getModalForDisabledRepeatableElement(
-                'repeatable_section_disabled_' . $sContainerTagID,
+                'repeatable_section_disabled_'.$sContainerTagID,
                 $_aArguments[ 'disabled' ]
             );
-        $_sButtonsHTML  = '"' . $_sButtons . '"';
+        $_sButtonsHTML  = '"'.$_sButtons.'"';
         $_aJSArray      = json_encode( $_aArguments );
         $_sScript       = <<<JAVASCRIPTS
 jQuery( document ).ready( function() {
@@ -505,12 +505,12 @@ 
                     empty( $aArguments[ 'disabled' ] ) ? '' : 'disabled'
                 ),
             );
-            unset( $aArguments[ 'disabled' ][ 'message' ] );    // this element can contain HTML tags.
+            unset( $aArguments[ 'disabled' ][ 'message' ] ); // this element can contain HTML tags.
             // Needs to remove it if it is empty as its data attribute will be checked in the JavaScript script.
             if ( empty( $aArguments[ 'disabled' ] ) ) {
                 unset( $aArguments[ 'disabled' ] );
             }
-            return self::getAttributes( $_aAttriubtes ) . ' ' . self::getDataAttributes( $aArguments );
+            return self::getAttributes( $_aAttriubtes ).' '.self::getDataAttributes( $aArguments );
         }
         /**
          * @return  array
@@ -541,10 +541,10 @@ 
                         . 'repeatable-section-button button button-large',
                     'title'     => $oMsg->get( 'add_section' ),
                     'data-id'   => $sContainerTagID,
-                    'href'      => ! empty( $aArguments[ 'disabled' ] )
-                        ? '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                            . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                            . '&inlineId=' . 'repeatable_section_disabled_' . $sContainerTagID
+                    'href'      => !empty( $aArguments[ 'disabled' ] )
+                        ? '#TB_inline?width='.$aArguments[ 'disabled' ][ 'box_width' ]
+                            . '&height='.$aArguments[ 'disabled' ][ 'box_height' ]
+                            . '&inlineId='.'repeatable_section_disabled_'.$sContainerTagID
                         : null,
                 )
             );

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___Format_RepeatableSection.php

Spacing +5 added lines, -5 removed lines

@@ -36,8 +36,8 @@ 
      * @since   3.8.13
      */
     static protected $_aStructure_Disabled = array(
-        'message'    => 'The ability of repeating sections is disabled.',   // will be reassigned
-        'caption'    => 'Warning',  // will be reassigned
+        'message'    => 'The ability of repeating sections is disabled.', // will be reassigned
+        'caption'    => 'Warning', // will be reassigned
         'box_width'  => 300,
         'box_height' => 72,
     );
@@ -54,7 +54,7 @@ 
      */
     public function __construct( /* array $asArguments, $oMsg */ ) {
 
-        $_aParameters       = func_get_args() + array(
+        $_aParameters = func_get_args() + array(
             $this->_aArguments,
             null
         );
@@ -73,8 +73,8 @@ 
     public function get() {
 
         $_aArguments = $this->_aArguments + self::$_aStructure;
-        unset( $_aArguments[ 0 ] );   // remove the 0 index element converted from `'repeatable   => 'true',`.
-        if ( ! empty( $_aArguments[ 'disabled' ] ) ) {
+        unset( $_aArguments[ 0 ] ); // remove the 0 index element converted from `'repeatable   => 'true',`.
+        if ( !empty( $_aArguments[ 'disabled' ] ) ) {
             $_aArguments[ 'disabled' ] = $_aArguments[ 'disabled' ] + array(
                 'message'    => $this->_getDefaultMessage(),
                 'caption'    => $this->_oMsg->get( 'warning_caption' ),

development/factory/_common/form/AdminPageFramework_Form_Utility.php

Spacing +14 added lines, -14 removed lines

@@ -59,7 +59,7 @@ 
      */
     static public function hasNestedFields( array $aFieldset ) {
         
-        if ( ! self::hasFieldDefinitionsInContent( $aFieldset ) ) {
+        if ( !self::hasFieldDefinitionsInContent( $aFieldset ) ) {
             return false;
         }
         // At this point, the `content` argument contains either the definition of nested fields or inline-mixed fields.
@@ -103,7 +103,7 @@ 
      */
     static public function hasFieldDefinitionsInContent( array $aFieldset ) {
         
-        if ( ! isset( $aFieldset[ 'content' ] ) ) {
+        if ( !isset( $aFieldset[ 'content' ] ) ) {
             return false;
         }
         if ( empty( $aFieldset[ 'content' ] ) ) {
@@ -138,10 +138,10 @@ 
      * @since       3.8.13
      */
     static public function isDynamicField( $aField ) {
-        if ( ! empty( $aField[ 'repeatable' ] ) ) {
+        if ( !empty( $aField[ 'repeatable' ] ) ) {
             return true;
         }
-        if ( ! empty( $aField[ 'sortable' ] ) ) {
+        if ( !empty( $aField[ 'sortable' ] ) ) {
             return true;
         }
         return false;
@@ -170,7 +170,7 @@ 
         if ( empty( $sString ) ) {
             return $sString;
         }
-        return $sString . '|';
+        return $sString.'|';
     }
 
     /**
@@ -186,17 +186,17 @@ 
      */
     static public function getFieldsetReformattedBySubFieldIndex( array $aFieldset, $iSubFieldIndex, $bHasSubFields, array $aParentFieldset ) {
 
-        $_oCallerForm   = $aFieldset[ '_caller_object' ];
+        $_oCallerForm = $aFieldset[ '_caller_object' ];
 
         // Add sub-field index to the parent field path for repeated nested items.
-        $aFieldset[ '_parent_field_path' ]   = self::getAOrB(
+        $aFieldset[ '_parent_field_path' ] = self::getAOrB(
             $bHasSubFields,
-            $aFieldset[ '_parent_field_path' ] . '|' . $iSubFieldIndex,
+            $aFieldset[ '_parent_field_path' ].'|'.$iSubFieldIndex,
             $aFieldset[ '_parent_field_path' ]
         );
-        $aFieldset[ '_parent_tag_id' ]       = self::getAOrB(
+        $aFieldset[ '_parent_tag_id' ] = self::getAOrB(
             $bHasSubFields,
-            $aParentFieldset[ 'tag_id' ] . '__' . $iSubFieldIndex,
+            $aParentFieldset[ 'tag_id' ].'__'.$iSubFieldIndex,
             $aParentFieldset[ 'tag_id' ]
         );
 
@@ -205,7 +205,7 @@ 
             $aFieldset,
             $aFieldset[ '_structure_type' ],
             $aFieldset[ 'capability' ],
-            ( integer ) $iSubFieldIndex + 1,   // 1-based count (not index)
+            ( integer ) $iSubFieldIndex + 1, // 1-based count (not index)
             $aFieldset[ '_subsection_index' ],
             $aFieldset[ '_is_section_repeatable' ],
             $aFieldset[ '_caller_object' ]
@@ -214,7 +214,7 @@ 
 
         $_oFieldsetOutputFormatter = new AdminPageFramework_Form_Model___Format_FieldsetOutput(
             $aFieldset,
-            $aFieldset[ '_section_index' ],    // `_section_index` is defined in the ...FieldsetOutput class. Since this is a nested item, it should be already set.
+            $aFieldset[ '_section_index' ], // `_section_index` is defined in the ...FieldsetOutput class. Since this is a nested item, it should be already set.
             $_oCallerForm->aFieldTypeDefinitions
         );
         return $_oFieldsetOutputFormatter->get();
@@ -251,12 +251,12 @@ 
         if ( empty( $aArguments ) ) {
             return '';
         }
-        if ( self::hasBeenCalled( 'disabled_repeatable_elements_modal_' . $sBoxElementID ) ) {
+        if ( self::hasBeenCalled( 'disabled_repeatable_elements_modal_'.$sBoxElementID ) ) {
             return '';
         }
         add_thickbox(); // to display a message to the user.
         return "<div id='{$sBoxElementID}' style='display:none'>"
-                . "<p>" . $aArguments[ 'message' ] . "</p>"
+                . "<p>".$aArguments[ 'message' ]."</p>"
             . "</div>";
 
     }

admin-page-framework-loader.php

Spacing +22 added lines, -22 removed lines

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