From 86b7cdb5d357ac6346fb348d760b2f1acd991ae7 Mon Sep 17 00:00:00 2001
From: ondrej <ondrej@lazarus-ide.org>
Date: Mon, 25 Jan 2016 22:15:26 +0000
Subject: [PATCH] ideintf: menuintf: update current section even if the
 sections in front need update.

Solves issue #29311 and probably also a lot of different "TIDEMenuSection.UpdateNeedTopSeparator inconsistency" issues.

git-svn-id: trunk@51418 -
---
 components/ideintf/menuintf.pas | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/components/ideintf/menuintf.pas b/components/ideintf/menuintf.pas
index e4da3c5646..b04a0eea62 100644
--- a/components/ideintf/menuintf.pas
+++ b/components/ideintf/menuintf.pas
@@ -1091,8 +1091,9 @@ begin
     debugln('TIDEMenuSection.UpdateMenuStructure Front Section="',Section.Name,'" Name="',Name,'" Section.Invalid=',dbgs(Section.FInvalidChildStartIndex),'..',dbgs(Section.FInvalidChildEndIndex),' Count=',dbgs(Count),' SectionIndex=',dbgs(SectionIndex));
     {$ENDIF}
     Section.UpdateMenuStructure;
-  end else if FInvalidChildStartIndex<Count then begin
-    // the sections in front are uptodate
+  end;
+  // the sections in front are now uptodate, lets continue updating the current section
+  if FInvalidChildStartIndex<Count then begin
     ContainerMenuItem:=GetContainerMenuItem;
     if ChildrenAsSubMenu then
       ContainerMenuIndex:=0