Fixing #5462 - Preview Remove Orphans

Package preview apparently making changes to Cacti Templates or there is a Caching issue

CHANGELOG

@@ -162,6 +162,7 @@ Cacti CHANGELOG
 -issue#5458: Unable to delete a Data Query when there are no Graphs using it
 -issue#5459: There is no option to not duplicate Graph Template Data Query association when duplicating a Graph Template
 -issue#5460: When duplicating a Data Template errors can appear in the Cacti log
+-issue#5462: Package preview aparantely making changes to Cacti Templates or there is a Caching issue
 -feature#5375: Add template for Fortinet firewall and Aruba Instant cluster
 -feature#5393: Add template for SNMP printer
 -feature#5418: Display device class before package import

lib/import.php

@@ -918,8 +918,8 @@ function xml_to_graph_template($hash, &$xml_array, &$hash_cache, $hash_version,
 	if (isset($orphaned_items) && cacti_sizeof($orphaned_items)) {
 		$orphan_text = array();
 
-		foreach ($orphaned_items as $item) {
-			if ($remove_orphans) {
+		foreach($orphaned_items as $item) {
+			if (!$preview_only && $remove_orphans) {
 				$orphan_text[] = 'Removed Orphaned Graph Items, Type: ' . $graph_item_types[$item['graph_type_id']] . ', Text Format: ' . $item['text_format'] . ', Value: ' . $item['value'];
 
 				db_execute_prepared('DELETE FROM graph_templates_item
@@ -936,7 +936,7 @@ function xml_to_graph_template($hash, &$xml_array, &$hash_cache, $hash_version,
 
 		$import_debug_info['orphans'] = $orphan_text;
 
-		if ($remove_orphans) {
+		if (!$preview_only && $remove_orphans) {
 			retemplate_graphs($graph_template_id);
 		}
 	}