Changeset 8242 for branches/crazyhorse/wp-admin/theme-editor.php

Timestamp:

07/02/2008 11:07:56 PM (17 years ago)

Author:

mdawaffe

Message:

crazyhorse: merge with log:trunk@8151:8240

File:

• branches/crazyhorse/wp-admin/theme-editor.php (modified) (1 diff)

branches/crazyhorse/wp-admin/theme-editor.php

@@ -44 +44 @@
     $theme = urlencode($theme);
     if (is_writeable($real_file)) {
+        //is_writable() not always reliable, check return value. see comments @ http://uk.php.net/is_writable
         $f = fopen($real_file, 'w+');
-        fwrite($f, $newcontent);
-        fclose($f);
-        $location = "theme-editor.php?file=$file&theme=$theme&a=te";
+        if ($f !== FALSE) {
+            fwrite($f, $newcontent);
+            fclose($f);
+            $location = "theme-editor.php?file=$file&theme=$theme&a=te";
+        } else {
+            $location = "theme-editor.php?file=$file&theme=$theme";
+        }
     } else {
         $location = "theme-editor.php?file=$file&theme=$theme";