From 670d927b0cdc6cea596e35676676aef8f36080bc Mon Sep 17 00:00:00 2001
From: Christian Weiske <cweiske@cweiske.de>
Date: Sun, 15 Apr 2012 22:12:44 +0200
Subject: do not show text field for binary files

---
 src/phorkie/File.php            | 16 ++++++++++++++++
 src/phorkie/Repository/Post.php |  2 +-
 2 files changed, 17 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/phorkie/File.php b/src/phorkie/File.php
index 519413d..53925ee 100644
--- a/src/phorkie/File.php
+++ b/src/phorkie/File.php
@@ -115,6 +115,22 @@ class File
         $tm = new Tool_Manager();
         return $tm->getSuitable($this);
     }
+
+    /**
+     * Tells if the file contains textual content and is editable.
+     *
+     * @return boolean
+     */
+    public function isText()
+    {
+        $ext = $this->getExt();
+        if (!isset($GLOBALS['phorkie']['languages'][$ext]['mime'])) {
+            return false;
+        }
+
+        $type = $GLOBALS['phorkie']['languages'][$ext]['mime'];
+        return substr($type, 0, 5) === 'text/';
+    }
 }
 
 ?>
\ No newline at end of file
diff --git a/src/phorkie/Repository/Post.php b/src/phorkie/Repository/Post.php
index 471cbd8..045bba1 100644
--- a/src/phorkie/Repository/Post.php
+++ b/src/phorkie/Repository/Post.php
@@ -94,7 +94,7 @@ class Repository_Post
                     ->addArgument($file->getFilename())
                     ->execute();
                 $bChanged = true;
-            } else if ($bNew || $file->getContent() != $arFile['content']) {
+            } else if ($bNew || (isset($arFile['content']) && $file->getContent() != $arFile['content'])) {
                 file_put_contents($file->getPath(), $arFile['content']);
                 $command = $vc->getCommand('add')
                     ->addArgument($file->getFilename())
-- 
cgit v1.2.3