Don't fail on undefined cache lookup.

michaelherger · May 22, 2024 · 5694038 · 5694038
1 parent 3f8b73d
commit 5694038
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/InfoMenu.pm b/InfoMenu.pm
@@ -219,10 +219,11 @@ sub menuAction {
 	# if we are re-drilling, no need to search, just get our anchor/root
 	if ( defined $itemId ) {
 		my ($key) = $itemId =~ /([^\.]+)/;
-		my $cached = ${$rootFeeds{$key}};
-		main::INFOLOG && $log->is_info && $log->info("re-drilling using cache key: $key");
-		Slim::Control::XMLBrowser::cliQuery('tidal_action', $cached, $request);
-		return;
+		if (my $cached = ${$rootFeeds{$key} || \''}) {
+			main::INFOLOG && $log->is_info && $log->info("re-drilling using cache key: $key");
+			Slim::Control::XMLBrowser::cliQuery('tidal_action', $cached, $request);
+			return;
+		}
 	}
 
 	my $entity  = $request->getRequest(1);
@@ -289,10 +290,11 @@ sub menuBrowse {
 	# if we are descending, no need to search, just get our root
 	if ( defined $itemId ) {
 		my ($key) = $itemId =~ /([^\.]+)/;
-		my $cached = ${$rootFeeds{$key}};
-		main::INFOLOG && $log->is_info && $log->info("re-drilling using cache key: $key");
-		Slim::Control::XMLBrowser::cliQuery('tidal_browse', $cached, $request);
-		return;
+		if (my $cached = ${$rootFeeds{$key} || \''}) {
+			main::INFOLOG && $log->is_info && $log->info("re-drilling using cache key: $key");
+			Slim::Control::XMLBrowser::cliQuery('tidal_browse', $cached, $request);
+			return;
+		}
 	}
 
 	# this key will prefix each action's hierarchy that JSON will sent us which

diff --git a/install.xml b/install.xml
@@ -4,7 +4,7 @@
   <description>PLUGIN_TIDAL_DESC</description>
   <creator>philippe_44, Michael Herger</creator>
   <id>16da8158-263f-4347-8125-184372ea5610</id>
-  <version>1.4.6</version>
+  <version>1.4.7</version>
   <module>Plugins::TIDAL::Plugin</module>
   <importmodule>Plugins::TIDAL::Importer</importmodule>
   <optionsURL>plugins/TIDAL/settings.html</optionsURL>