From 827c72e1f535046bdeefb57701fc42779c516af0 Mon Sep 17 00:00:00 2001
From: Jean-Marc Lasgouttes <lasgouttes@lyx.org>
Date: Mon, 30 Mar 2009 12:45:33 +0000
Subject: [PATCH] Simplify and fix the TOGGLE_INSET code for branch inset. If
 it turns out that the extra complexity was intended, I would be interested to
 know what the idea was.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@28979 a592a061-630c-0410-9148-cb99ea01b6c8
---
 src/insets/InsetBranch.cpp | 30 ++++++------------------------
 1 file changed, 6 insertions(+), 24 deletions(-)

diff --git a/src/insets/InsetBranch.cpp b/src/insets/InsetBranch.cpp
index 001ea68621..0072dac43f 100644
--- a/src/insets/InsetBranch.cpp
+++ b/src/insets/InsetBranch.cpp
@@ -144,20 +144,8 @@ void InsetBranch::doDispatch(Cursor & cur, FuncRequest & cmd)
 		break;
 
 	case LFUN_INSET_TOGGLE:
-		if (cmd.argument() == "assign") {
-			// The branch inset uses "assign".
-			if (isBranchSelected()) {
-				if (status(cur.bv()) != Open)
-					setStatus(cur, Open);
-				else
-					cur.undispatched();
-			} else {
-				if (status(cur.bv()) != Collapsed)
-					setStatus(cur, Collapsed);
-				else
-					cur.undispatched();
-			}
-		}
+		if (cmd.argument() == "assign")
+			setStatus(cur, isBranchSelected() ? Open : Collapsed);
 		else
 			InsetCollapsable::doDispatch(cur, cmd);
 		break;
@@ -179,17 +167,11 @@ bool InsetBranch::getStatus(Cursor & cur, FuncRequest const & cmd,
 		break;
 
 	case LFUN_INSET_TOGGLE:
-		if (cmd.argument() == "open" || cmd.argument() == "close" ||
-		    cmd.argument() == "toggle")
-			flag.setEnabled(true);
-		else if (cmd.argument() == "assign" || cmd.argument().empty()) {
-			if (isBranchSelected())
-				flag.setEnabled(status(cur.bv()) != Open);
-			else
-				flag.setEnabled(status(cur.bv()) != Collapsed);
-		} else
+		if (cmd.argument() == "assign") {
 			flag.setEnabled(true);
-		break;
+			break;
+		}
+		//fall through to generic InsetCollapsable implmementation
 
 	default:
 		return InsetCollapsable::getStatus(cur, cmd, flag);
-- 
2.39.2