[geeklog-cvs] geeklog: Pushing bug fix for 0001126: Installer display messages...

[geeklog-cvs at lists.geeklog.net]

changeset 7883:3dff80c7eefe
url:  http://project.geeklog.net/cgi-bin/hgwebdir.cgi/geeklog/rev/3dff80c7eefe
user: stan <stan at spalatnik.com>
date: Fri Apr 16 12:11:20 2010 -0400
description:
Pushing bug fix for 0001126: Installer display messages that system require 777 for rw files(fixed by erisco)

diffstat:

 public_html/admin/install/index.php |  7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diffs (19 lines):

diff -r 922bafd5b86e -r 3dff80c7eefe public_html/admin/install/index.php
--- a/public_html/admin/install/index.php	Fri Apr 16 12:06:49 2010 -0400
+++ b/public_html/admin/install/index.php	Fri Apr 16 12:11:20 2010 -0400
@@ -1125,9 +1125,14 @@
 
             foreach ($file_list as $file) {
                 if (!is_writable($file)) {
+                    if (is_file($file)) {
+                        $perm_should_be = '666';
+                    } else {
+                        $perm_should_be = '777';
+                    }
                     $permission = sprintf("%3o", @fileperms($file) & 0777);
                     $display_permissions    .= '<p><label class="' . $perms_label_dir . '"><code>' . $file . '</code></label>' . LB
-                                            . ' <span class="permissions-list">' . $LANG_INSTALL[12] . ' 777</span> ('
+                                            . ' <span class="permissions-list">' . $LANG_INSTALL[12] . ' '. $perm_should_be .'</span> ('
                                             . $LANG_INSTALL[13] . ' ' . $permission . ')</p>' . LB ;
                     $chmod_string .= $file . ' ' ;
                     $num_wrong++;