improvement: use the same code to check upload readiness in admin>photos>add

screen and in web API.

bug fixed: UploadForm partly takes into account the configurable upload
directory (must be propagated everywhere in the UploadForm process)

git-svn-id: http://piwigo.org/svn/trunk@6051 68402e56-0260-453c-a942-63ccdbb3a9ee

1 files changed, 18 insertions, 11 deletions

diff --git a/include/ws_functions.inc.php b/include/ws_functions.inc.php
index 87cb72393..162cd4371 100644
--- a/include/ws_functions.inc.php
+++ b/include/ws_functions.inc.php
@@ -2136,16 +2136,28 @@ function ws_images_checkUpload($params, &$service)
     return new PwgError(401, 'Access denied');
   }
 
-  $relative_dir = preg_replace('#^'.PHPWG_ROOT_PATH.'#', '', $conf['upload_dir']);
-
-  $ret['message'] = null;
+  $ret['message'] = ready_for_upload_message();
   $ret['ready_for_upload'] = true;
   
+  if (!empty($ret['message']))
+  {
+    $ret['ready_for_upload'] = false;
+  }
+  
+  return $ret;
+}
+
+function ready_for_upload_message()
+{
+  global $conf;
+
+  $relative_dir = preg_replace('#^'.PHPWG_ROOT_PATH.'#', '', $conf['upload_dir']);
+
   if (!is_dir($conf['upload_dir']))
   {
     if (!is_writable(dirname($conf['upload_dir'])))
     {
-      $ret['message'] = sprintf(
+      return sprintf(
         l10n('Create the "%s" directory at the root of your Piwigo installation'),
         $relative_dir
         );
@@ -2159,7 +2171,7 @@ function ws_images_checkUpload($params, &$service)
       
       if (!is_writable($conf['upload_dir']))
       {
-        $ret['message'] = sprintf(
+        return sprintf(
           l10n('Give write access (chmod 777) to "%s" directory at the root of your Piwigo installation'),
           $relative_dir
           );
@@ -2167,11 +2179,6 @@ function ws_images_checkUpload($params, &$service)
     }
   }
 
-  if (!empty($ret['message']))
-  {
-    $ret['ready_for_upload'] = false;
-  }
-  
-  return $ret;
+  return null;
 }
 ?>