From 2d1c184c716fa88da3a5b8b7d5a2b99cb1d5cd8e Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Thu, 29 Mar 2018 10:45:30 -0300
Subject: [PATCH] Bug 20428: Make about.pl inform about missing upload_tmp_path
 entry

This patch makes about.pl warn about a missing upload_tmp_path entry in
koha-conf.xml. It also mentions the effectively used tmp dir.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
---
 about.pl                                         | 8 ++++++++
 koha-tmpl/intranet-tmpl/prog/en/modules/about.tt | 8 ++++++++
 2 files changed, 16 insertions(+)

diff --git a/about.pl b/about.pl
index 3997d4992a..587d85f336 100755
--- a/about.pl
+++ b/about.pl
@@ -45,6 +45,7 @@ use Koha::Caches;
 use Koha::Config::SysPrefs;
 use Koha::Illrequest::Config;
 use Koha::SearchEngine::Elasticsearch;
+use Koha::UploadedFiles;
 
 use C4::Members::Statistics;
 
@@ -259,6 +260,13 @@ if ( ! defined C4::Context->config('upload_path') ) {
     }
 }
 
+if ( ! defined C4::Context->config('upload_tmp_path') ) {
+    push @xml_config_warnings, {
+        error                    => 'uploadtmppath_entry_missing',
+        effective_upload_tmp_dir => Koha::UploadedFile->temporary_directory
+    }
+}
+
 # Test QueryParser configuration sanity
 if ( C4::Context->preference( 'UseQueryParser' ) ) {
     # Get the QueryParser configuration file name
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
index c35a01ef87..c0dbb19280 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
@@ -409,6 +409,14 @@
                         Also note that you need to properly set the OPACBaseURL preference for the file upload plugin to work.
                     </td>
                 </tr>
+                [% ELSIF config_entry.error == 'uploadtmppath_entry_missing' %]
+                <tr>
+                    <th scope="row"><b>Warning</b></th>
+                    <td>You are missing the &lt;upload_tmp_path&gt; entry in your koha-conf.xml file. Please
+                        add it, pointing to the configured temporary file upload directory for your Koha instance.
+                        The effective temporary file upload directory is '[% config_entry.effective_upload_tmp_dir %]'
+                    </td>
+                </tr>
                 [% END %]
             [% END %]
             </table>
-- 
2.15.1 (Apple Git-101)