Detect binary file by NULL byte

Author: Patrick-Beuks

src/Gitonomy/Git/Blob.php

@@ -19,6 +19,8 @@
  */
 class Blob
 {
+    private const FIRST_FEW_BYTES = 8000;
+
     /**
      * @var Repository
      */
@@ -39,6 +41,11 @@ class Blob
      */
     protected $mimetype;
 
+    /**
+     * @var bool
+     */
+    protected $text;
+
     /**
      * @param Repository $repository Repository where the blob is located
      * @param string     $hash       Hash of the blob
@@ -103,6 +110,10 @@ public function isBinary()
      */
     public function isText()
     {
-        return (bool) preg_match('#^text/|^application/xml#', $this->getMimetype());
+        if (null === $this->text) {
+            $this->text = !str_contains(substr($this->getContent(), 0, self::FIRST_FEW_BYTES), chr(0));
+        }
+
+        return $this->text;
     }
 }

tests/Gitonomy/Git/Tests/BlobTest.php

@@ -23,6 +23,11 @@ public function getReadmeBlob($repository)
         return $repository->getCommit(self::LONGFILE_COMMIT)->getTree()->resolvePath('README.md');
     }
 
+    public function getImageBlob($repository)
+    {
+        return $repository->getCommit(self::LONGFILE_COMMIT)->getTree()->resolvePath('image.jpg');
+    }
+
     /**
      * @dataProvider provideFoobar
      */
@@ -67,16 +72,20 @@ public function testGetMimetype($repository)
      */
     public function testIsText($repository)
     {
-        $blob = $this->getReadmeBlob($repository);
-        $this->assertTrue($blob->isText());
+        $readmeBlob = $this->getReadmeBlob($repository);
+        $this->assertTrue($readmeBlob->isText());
+        $imageBlob = $this->getImageBlob($repository);
+        $this->assertFalse($imageBlob->isText());
     }
 
     /**
      * @dataProvider provideFoobar
      */
     public function testIsBinary($repository)
     {
-        $blob = $this->getReadmeBlob($repository);
-        $this->assertFalse($blob->isBinary());
+        $readmeBlob = $this->getReadmeBlob($repository);
+        $this->assertFalse($readmeBlob->isBinary());
+        $imageBlob = $this->getImageBlob($repository);
+        $this->assertTrue($imageBlob->isBinary());
     }
 }