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>

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Tomas Cohen Arazi 2018-03-29 10:45:30 -03:00 committed by Jonathan Druart
parent 2b0c27beaf
commit da2886132e
2 changed files with 16 additions and 0 deletions

View File

@ -46,6 +46,7 @@ use Koha::Caches;
use Koha::Config::SysPrefs;
use Koha::Illrequest::Config;
use Koha::SearchEngine::Elasticsearch;
use Koha::UploadedFiles;
use C4::Members::Statistics;
@ -260,6 +261,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

View File

@ -414,6 +414,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>