Added memory config verification

Author: nigelhorne

lib/VWF/Display.pm

@@ -27,7 +27,7 @@ use Template::Plugin::EnvHash;
 use Template::Plugin::Math;
 use Template::Plugin::JSON;
 use HTML::SocialMedia;
-use VWF::Utils qw(create_memory_cache create_disc_cache);
+use VWF::Utils qw(create_memory_cache);
 use Error;
 use Fatal qw(:void open);
 use File::pfopen;

lib/VWF/Utils.pm

@@ -204,6 +204,12 @@ sub _validate_cache_config($config, $cache_type) {
 		croak('Port must be between 1 and 65535')
 			unless $port >= 1 && $port <= 65535;
 	}
+
+	# Validate memory parameters
+	if(exists($cache_config->{'datastore'}) && exists($cache_config->{'global'}) && (my $driver = $cache_config->{'driver'})) {
+		# CHI catches this as well, but putting it here helps to track the error
+		croak('Memory cache cannot have both global and datastore');
+	}
 }
 
 sub _get_default_driver($cache_type, $logger) {