Resolving Issue #2912

Resolve navigation caching array to string conversion issues

CHANGELOG

@@ -39,6 +39,7 @@ Cacti CHANGELOG
 -issue#2907: SSL column for multiple pollers can be incorrectly set causing SQL errors
 -issue#2908: When URL_PATH is blank, it should assume that it is '/'
 -issue#2909: Correct usage of affect vs effect in strings
+-issue#2912: Resolve navigation caching array to string conversion issues
 -issue#2913: Selection "Graph type" not saved and restored after going to page two
 
 1.2.5

lib/functions.php

@@ -2385,6 +2385,11 @@ function draw_navigation_text($type = 'url') {
 	$navigation      = api_plugin_hook_function('draw_navigation_text', $navigation);
 	$current_page    = get_current_page();
 
+	// Do an error check here for bad plugins manipulating the cache
+	if (!is_array($nav_level_cache)) {
+		$nav_level_cache = array();
+	}
+
 	if (!isempty_request_var('action')) {
 		get_filter_request_var('action', FILTER_VALIDATE_REGEXP, array('options' => array('regexp' => '/^([-a-zA-Z0-9_\s]+)$/')));
 	}