From 294db65d6c539310199c6ddd5280c2ac11a44410 Mon Sep 17 00:00:00 2001 From: Olivier Berger Date: Thu, 16 Aug 2012 14:49:02 +0000 Subject: [PATCH] Make sure to notify if enable_uploads is unset, and no target images dir will be created by cronjob, which will prevent uploading --- src/plugins/mediawiki/www/plugin_admin.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/plugins/mediawiki/www/plugin_admin.php b/src/plugins/mediawiki/www/plugin_admin.php index ea89263..686545a 100644 --- a/src/plugins/mediawiki/www/plugin_admin.php +++ b/src/plugins/mediawiki/www/plugin_admin.php @@ -83,6 +83,11 @@ $group_logo = $wgUploadDirectory . "/.wgLogo.png"; $group_logo_url = util_make_url("/plugins/mediawiki/wiki/" . $group_unix_name . "/images/.wgLogo.png"); +/* As the cronjob creates images subdirs in project data only if the uploads are enabled, there are chances the upload may fail */ +if (! forge_get_config('enable_uploads', 'mediawiki')) { + $error_msg .= _("Mediawiki plugin's configuration may require to enable uploads ('enable_uploads'). Contact your admin."); +} + if (getStringFromRequest("logo_submit")) { $userfile = getUploadedFile('userfile'); $userfile_name = $userfile['name']; -- 2.1.4