using md5_file() in contents reader

AngryBytes · Sep 15, 2014 · ff466a0 · ff466a0
1 parent 10833f8
commit ff466a0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 11 deletions.
diff --git a/src/Naneau/ProjectVersioner/Reader/Finder/Contents.php b/src/Naneau/ProjectVersioner/Reader/Finder/Contents.php
@@ -24,20 +24,18 @@ class Contents extends Finder
      **/
     public function read($directory)
     {
-        $contents = '';
+        $hash = '';
         foreach ($this->getFinder() as $file) {
-            var_dump($file);
 
-            $fileContents = file_get_contents(
-                $file->getPath()
-                . DIRECTORY_SEPARATOR . $file->getFilename()
+            // MD5 hash of the file
+            $fileHash = md5_file(
+                $file->getPath() . DIRECTORY_SEPARATOR . $file->getFilename()
             );
 
-            var_dump($contents, $fileContents);
-
-            $contents = md5($contents . $fileContents);
+            // Mix into result hash
+            $hash = md5($hash . $fileHash);
         }
 
-        return substr($contents, 0, 6);
+        return substr($hash, 0, 6);
     }
 }
diff --git a/tests/Reader/FinderTest.php b/tests/Reader/FinderTest.php
@@ -25,9 +25,8 @@ public function testContents()
     {
         $versioner = new Versioner(array(new ContentsReader('*.php')));
 
-        $hash = 'ee53e4';
         $this->assertEquals(
-            $hash,
+            'db9d80',
             $versioner->get(
                 __DIR__ . '/../projects/finder'
             )