]> git.mxchange.org Git - core.git/commitdiff
Continued:
authorRoland Häder <roland@mxchange.org>
Thu, 15 Nov 2018 00:00:15 +0000 (01:00 +0100)
committerRoland Häder <roland@mxchange.org>
Thu, 15 Nov 2018 00:00:15 +0000 (01:00 +0100)
- improved find-bad-php.sh
- RegistryIterator->rewind() doesn't need to rewind when there is nothing to
  iterate
- imported deprecated class

contrib/find-bad-php.sh
framework/main/classes/file_directories/directory/class_FrameworkDirectoryPointer.php
framework/main/classes/iterator/registry/class_RegistryIterator.php

index 6febfcfc0d009f5c817dafe93bc27379372c6db4..45d6e7afd6e21594da63dd18b17c75d75e4446e5 100755 (executable)
@@ -5,18 +5,17 @@ PHP=$(find -type f -name "*.php" 2>&1 | grep -v "third_party" | grep -v "/vendor
 
 for SCRIPT in ${PHP};
 do
-       HEADER=$(cat ${SCRIPT} | head -n 1 | grep -v "<?")
+       SHORT_OPEN_TAG=$(cat ${SCRIPT} | head -n 1 | grep -v "<?")
+       CLOSING_TAG=$(cat ${SCRIPT} | tail -n 1 | grep "?>")
 
-       FOOTER=$(cat ${SCRIPT} | tail -n 1 | grep "?>")
-
-       if [ -n "${HEADER}" ];
+       if [ -n "${SHORT_OPEN_TAG}" ];
        then
-               echo "$0: Script '${SCRIPT}' has non-typical header."
+               echo "$0: Script '${SCRIPT}' has short opening tag (<?)."
        fi
 
-       if [ -n "${FOOTER}" ];
+       if [ -n "${CLOSING_TAG}" ];
        then
-               echo "$0: Script '${SCRIPT}' has discouraged footer."
+               echo "$0: Script '${SCRIPT}' has discouraged closing tag (?>)."
        fi
 
        LINT=$(php -l "${SCRIPT}" 2>&1 | grep -v "Constant FUSE_EDEADLK already defined in Unknown on line 0" | grep -v "No syntax errors detected in")
index d7f346a1052a2d274354796e93ecf8b8633e542b..4b2e1107bfd40c60a4db648669951fe84d697ed1 100644 (file)
@@ -7,6 +7,9 @@ use Org\Mxchange\CoreFramework\Filesystem\FrameworkDirectory;
 use Org\Mxchange\CoreFramework\Generic\NullPointerException;
 use Org\Mxchange\CoreFramework\Object\BaseFrameworkSystem;
 
+// Deprecated imports
+use Org\Mxchange\CoreFramework\Deprecated\PathIsNoDirectoryException;
+
 // Import SPL stuff
 use \DirectoryIterator;
 
index 93d54fa86401658ecd0831efecc7e078a6657804..c01722adaf10f25a6b5593dc4aca107d7c1f9d4e 100644 (file)
@@ -39,7 +39,7 @@ class RegistryIterator extends BaseIterator implements IteratableRegistry {
        /**
         * All found registry keys
         */
-       private $registryKeys = array();
+       private $registryKeys = [];
 
        /**
         * Current array key
@@ -109,7 +109,7 @@ class RegistryIterator extends BaseIterator implements IteratableRegistry {
                $entries = $this->getRegistryInstance()->getGenericRegistry();
 
                // Init registry keys array
-               $this->registryKeys['generic'] = array();
+               $this->registryKeys['generic'] = [];
 
                // Anything in there?
                if (count($entries) > 0) {
@@ -121,7 +121,7 @@ class RegistryIterator extends BaseIterator implements IteratableRegistry {
                $entries = $this->getRegistryInstance()->getInstanceRegistry();
 
                // Init registry keys array
-               $this->registryKeys['instance'] = array();
+               $this->registryKeys['instance'] = [];
 
                // Anything in there?
                if (count($entries) > 0) {
@@ -178,7 +178,7 @@ class RegistryIterator extends BaseIterator implements IteratableRegistry {
                                //* NOISY-DEBUG: */ self::createDebugInstance(__CLASS__, __LINE__)->debugOutput(sprintf('REGISTRY-ITERATOR[instance]: key=%s,entry[]=%s - Adding ...', $key, gettype($entry)));
 
                                // Add key to array
-                               $this->registryKeys['instance'][$key] = array();
+                               $this->registryKeys['instance'][$key] = [];
                        } // END - foreach
                } // END - if
        }
@@ -233,6 +233,12 @@ class RegistryIterator extends BaseIterator implements IteratableRegistry {
         * @return      void
         */
        public function rewind () {
+               // Is the registry empty?
+               if (count($this->registryKeys) == 0) {
+                       // Then no need to continue
+                       return;
+               } // END - if
+
                // Debugging:
                /* DEBUG-DIE: */ die(sprintf('[%s:%d]: this->key(%d)[%s]=%s,this->valid=%d,this->registryKeys=%s', __METHOD__, __LINE__, strlen($this->key()), gettype($this->key()), $this->key(), intval($this->valid()), print_r($this->registryKeys, TRUE)));
        }