From 106f889f37efd3e5f303049d4da54be05561b838 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Tue, 9 Feb 2021 20:44:46 +0100
Subject: [PATCH] Fix automation context-menu state when reordering plugins

Previously displayed lanes (those with save "visibility" GUI
attributes) were always marked as visible in the context-menu.
---
 gtk2_ardour/route_time_axis.cc | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gtk2_ardour/route_time_axis.cc b/gtk2_ardour/route_time_axis.cc
index 9e1748f6fa..3c0bf77dab 100644
--- a/gtk2_ardour/route_time_axis.cc
+++ b/gtk2_ardour/route_time_axis.cc
@@ -1900,7 +1900,7 @@ RouteTimeAxisView::add_processor_to_subplugin_menu (boost::weak_ptr<Processor> p
 		boost::shared_ptr<AutomationTimeAxisView> atav = pan->view;
 		bool visible;
 		if (atav && atav->get_gui_property ("visible", visible)) {
-			mitem->set_active(true);
+			mitem->set_active(visible);
 		} else {
 			mitem->set_active(false);
 		}
@@ -1934,7 +1934,8 @@ RouteTimeAxisView::processor_menu_item_toggled (RouteTimeAxisView::ProcessorAuto
 		redraw = true;
 	}
 
-	if (pan->view && pan->view->set_marked_for_display (showit)) {
+	boost::shared_ptr<AutomationTimeAxisView> atav = pan->view;
+	if (atav && atav->set_marked_for_display (showit)) {
 		redraw = true;
 	}