Fix issue pyrus#12 - Lack of file_exists handling

helgi · May 27, 2011 · 5ee1a38 · 5ee1a38
1 parent c511d1a
commit 5ee1a38
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/PEAR2/Pyrus/Developer/PackageFile/Commands.php b/src/PEAR2/Pyrus/Developer/PackageFile/Commands.php
@@ -792,14 +792,24 @@ function processFile($ext, $source, $dest, $replace)
 
     function saveConfigM4($ext)
     {
-        $m4 = file_get_contents(dirname($this->skeleton) . '/config.m4');
+        $filename = dirname($this->skeleton) . '/config.m4';
+        if (!file_exists($filename)) {
+            return false; // FIXME proper handling
+        }
+
+        $m4 = file_get_contents($filename);
         file_put_contents($ext . '/config.m4', str_replace(array('@EXTNAME@', '@extname@'),
                                                            array(strtoupper($ext), $ext), $m4));
     }
 
     function saveConfigW32($ext)
     {
-        $w32 = file_get_contents(dirname($this->skeleton) . '/config.w32');
+        $filename = dirname($this->skeleton) . '/config.w32';
+        if (!file_exists($filename)) {
+            return false; // FIXME proper handling
+        }
+
+        $w32 = file_get_contents($filename);
         file_put_contents($ext . '/config.w32', str_replace(array('@EXTNAME@', '@extname@'),
                                                             array(strtoupper($ext), $ext), $w32));
     }