smalot / drush-cerbere

lib/Cerbere/Model/Hacked/HackedProject.php

Doc Comments +1 added lines

@@ -123,6 +123,7 @@
 
     /**
      * Get the Human readable title of this project.
+     * @return string
      */
     public function getTitle()
     {

Spacing +3 added lines, -3 removed lines

@@ -157,7 +157,7 @@ 
         $this->project_identified = true;
         $this->existing_version = $this->project->getExistingVersion();
         $this->project_type = 'module';
-        $this->project_info['includes'] = array($this->name . '.module');
+        $this->project_info['includes'] = array($this->name.'.module');
     }
 
     /**
@@ -196,7 +196,7 @@ 
 
         // Logging.
         if (!$this->remote_hashed) {
-            throw new \Exception('Could not hash remote project: ' . strip_tags($this->getTitle()));
+            throw new \Exception('Could not hash remote project: '.strip_tags($this->getTitle()));
         }
     }
 
@@ -230,7 +230,7 @@ 
 
         // Logging.
         if (!$this->local_hashed) {
-            throw new \Exception('Could not hash local project: ' . strip_tags($this->getTitle()));
+            throw new \Exception('Could not hash local project: '.strip_tags($this->getTitle()));
         }
     }
 

commands/cerbere.drush.inc

Spacing +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
     // Action.
     $action = new Update();
     if ($use_cache) {
-        $cache = new FilesystemCache(sys_get_temp_dir() . '/cerbere');
+        $cache = new FilesystemCache(sys_get_temp_dir().'/cerbere');
         $action->setCache($cache);
     }
 

lib/Cerbere/Cerbere.php

Spacing +2 added lines, -2 removed lines

@@ -231,10 +231,10 @@
         $files = glob($pattern, $flags);
 
         if ($nested) {
-            foreach (glob(dirname($pattern) . DIRECTORY_SEPARATOR . '*', GLOB_ONLYDIR | GLOB_NOSORT) as $dir) {
+            foreach (glob(dirname($pattern).DIRECTORY_SEPARATOR.'*', GLOB_ONLYDIR | GLOB_NOSORT) as $dir) {
                 $files = array_merge(
                   $files,
-                  $this->getFilesFromPattern($dir . DIRECTORY_SEPARATOR . basename($pattern), $nested, $flags)
+                  $this->getFilesFromPattern($dir.DIRECTORY_SEPARATOR.basename($pattern), $nested, $flags)
                 );
             }
         }

lib/Cerbere/Event/CerbereLoggerListener.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     public function getLogLevelMapping($eventName)
     {
         if (!isset($this->logLevelMappings[$eventName])) {
-            throw new \DomainException('Unknown event: ' . $eventName);
+            throw new \DomainException('Unknown event: '.$eventName);
         }
 
         return $this->logLevelMappings[$eventName];

lib/Cerbere/Model/Hacked/HackedProjectWebFilesDownloader.php

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
         try {
             $this->extractArchive($local_file, $destination);
         } catch (\Exception $e) {
-            echo $e->getMessage() . "\n";
+            echo $e->getMessage()."\n";
 
             return false;
         }
@@ -90,8 +90,8 @@ 
 
         // Todo: use Symfony's cache objects
         // Check the cache and download the file if needed.
-        $cache_directory = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'hacked-cache';
-        $local = $cache_directory . DIRECTORY_SEPARATOR . basename($parsed_url['path']);
+        $cache_directory = sys_get_temp_dir().DIRECTORY_SEPARATOR.'hacked-cache';
+        $local = $cache_directory.DIRECTORY_SEPARATOR.basename($parsed_url['path']);
 
         if (!file_exists($cache_directory)) {
             @mkdir($cache_directory, 0775, true);

lib/Cerbere/Model/Hacked/HackedFileGroup.php

Spacing +10 added lines, -10 removed lines

@@ -54,7 +54,7 @@ 
     public function computeHashes()
     {
         foreach ($this->files as $filename) {
-            $this->files_hashes[$filename] = $this->hasher->hash($this->base_path . DIRECTORY_SEPARATOR . $filename);
+            $this->files_hashes[$filename] = $this->hasher->hash($this->base_path.DIRECTORY_SEPARATOR.$filename);
         }
     }
 
@@ -65,7 +65,7 @@ 
      */
     public function fileExists($file)
     {
-        return file_exists($this->base_path . DIRECTORY_SEPARATOR . $file);
+        return file_exists($this->base_path.DIRECTORY_SEPARATOR.$file);
     }
 
     /**
@@ -93,7 +93,7 @@ 
         $files = self::scanDirectory($this->base_path, '/.*/', $white_list);
 
         foreach ($files as $file) {
-            $filename = str_replace($this->base_path . DIRECTORY_SEPARATOR, '', $file->filename);
+            $filename = str_replace($this->base_path.DIRECTORY_SEPARATOR, '', $file->filename);
             $this->files[] = $filename;
         }
     }
@@ -126,11 +126,11 @@ 
         if (is_dir($dir) && $handle = opendir($dir)) {
             while (false !== ($file = readdir($handle))) {
                 if (!in_array($file, $nomask)) {
-                    if (is_dir($dir . DIRECTORY_SEPARATOR . $file) && $recurse) {
+                    if (is_dir($dir.DIRECTORY_SEPARATOR.$file) && $recurse) {
                         // Give priority to files in this folder by merging them in after any subdirectory files.
                         $files = array_merge(
                           self::scanDirectory(
-                            $dir . DIRECTORY_SEPARATOR . $file,
+                            $dir.DIRECTORY_SEPARATOR.$file,
                             $mask,
                             $nomask,
                             $callback,
@@ -143,7 +143,7 @@ 
                         );
                     } elseif ($depth >= $min_depth && preg_match($mask, $file)) {
                         // Always use this match over anything already set in $files with the same $$key.
-                        $filename = $dir . DIRECTORY_SEPARATOR . $file;
+                        $filename = $dir.DIRECTORY_SEPARATOR.$file;
                         $basename = basename($file);
                         $name = substr($basename, 0, strrpos($basename, '.'));
                         $files[$$key] = new \stdClass();
@@ -200,7 +200,7 @@ 
      */
     public function getFileLocation($file)
     {
-        return $this->base_path . DIRECTORY_SEPARATOR . $file;
+        return $this->base_path.DIRECTORY_SEPARATOR.$file;
     }
 
     /**
@@ -218,8 +218,8 @@ 
      */
     public function isNotBinary($file)
     {
-        return is_readable($this->base_path . DIRECTORY_SEPARATOR . $file)
-        && !self::isBinary($this->base_path . DIRECTORY_SEPARATOR . $file);
+        return is_readable($this->base_path.DIRECTORY_SEPARATOR.$file)
+        && !self::isBinary($this->base_path.DIRECTORY_SEPARATOR.$file);
     }
 
     /**
@@ -262,6 +262,6 @@ 
      */
     public function isReadable($file)
     {
-        return is_readable($this->base_path . DIRECTORY_SEPARATOR . $file);
+        return is_readable($this->base_path.DIRECTORY_SEPARATOR.$file);
     }
 }

lib/Cerbere/Model/Hacked/HackedProjectWebDownloader.php

Spacing +5 added lines, -5 removed lines

@@ -36,10 +36,10 @@ 
             $namespace = get_class($this);
         }
 
-        $dir = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'hacked-cache';
+        $dir = sys_get_temp_dir().DIRECTORY_SEPARATOR.'hacked-cache';
 
         if (!empty($namespace)) {
-            $dir .= DIRECTORY_SEPARATOR . preg_replace('/[^0-9A-Z\-_]/i', '', $namespace);
+            $dir .= DIRECTORY_SEPARATOR.preg_replace('/[^0-9A-Z\-_]/i', '', $namespace);
         }
 
         // Build the destination folder tree if it doesn't already exists.
@@ -60,14 +60,14 @@ 
         $name = $this->project->getProject();
         $version = $this->project->getVersion();
 
-        $dir = $this->getTempDirectory() . DIRECTORY_SEPARATOR . $type . DIRECTORY_SEPARATOR . $name;
+        $dir = $this->getTempDirectory().DIRECTORY_SEPARATOR.$type.DIRECTORY_SEPARATOR.$name;
 
         // Build the destination folder tree if it doesn't already exists.
         if (!is_dir($dir)) {
             mkdir($dir, 0775, true);
         }
 
-        return $dir . DIRECTORY_SEPARATOR . $version;
+        return $dir.DIRECTORY_SEPARATOR.$version;
     }
 
     /**
@@ -110,7 +110,7 @@ 
                 if ($entry == '.' || $entry == '..') {
                     continue;
                 }
-                $entry_path = $path . DIRECTORY_SEPARATOR . $entry;
+                $entry_path = $path.DIRECTORY_SEPARATOR.$entry;
                 $this->removeDir($entry_path);
             }
 

lib/Cerbere/Model/Release.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
     public function getDate()
     {
         if ($timestamp = $this->getDatestamp()) {
-            return new \DateTime('@' . $timestamp);
+            return new \DateTime('@'.$timestamp);
         }
 
         return null;

lib/Cerbere/Model/Project.php

Spacing +1 added lines, -1 removed lines

@@ -258,7 +258,7 @@
                 return substr($path, 0, $position);
             }
         } elseif ($this->getProject() == 'drupal' && $this->getCore() == '7.x') {
-            return dirname($this->getFilename()) . '/../..';
+            return dirname($this->getFilename()).'/../..';
         }
 
         return dirname($this->getFilename());