操作メニューの「タブ振り分けルール作成」から新規タブを作成するとエラーが発生する不具合を修正

FilterDialogのタブ一覧はコンストラクタ内で生成しているため、
タブ選択画面（新規タブ作成の操作も行える）の後にFilterDialogインスタンスを生成するように順序を変更した

https://osdn.net/projects/opentween/ticket/39821

diff --git a/OpenTween/Resources/ChangeLog.txt b/OpenTween/Resources/ChangeLog.txt
index c49069c..cc7af89 100644 (file)
--- a/OpenTween/Resources/ChangeLog.txt
+++ b/OpenTween/Resources/ChangeLog.txt
@@ -2,6 +2,7 @@
 
 ==== Ver 2.4.3-dev(2019/xx/xx)
  * FIX: 関連発言表示のタブを読み込み中に閉じるとエラーが発生する不具合を修正
+ * FIX: 操作メニューの「タブ振り分けルール作成」から新規タブを作成するとエラーが発生する不具合を修正
 
 ==== Ver 2.4.2(2019/10/23)
  * FIX: タブ更新時にエラーが発生するとプロセスが異常終了する場合がある不具合を修正

diff --git a/OpenTween/Tween.cs b/OpenTween/Tween.cs
index 8536445..60092d5 100644 (file)
--- a/OpenTween/Tween.cs
+++ b/OpenTween/Tween.cs
@@ -7959,16 +7959,16 @@ namespace OpenTween
 
         private void TabMenuItem_Click(object sender, EventArgs e)
         {
-            using (var fltDialog = new FilterDialog())
+            // 選択発言を元にフィルタ追加
+            foreach (var post in this.CurrentTab.SelectedPosts)
             {
-                fltDialog.Owner = this;
+                // タブ選択（or追加）
+                if (!SelectTab(out var tab))
+                    return;
 
-                //選択発言を元にフィルタ追加
-                foreach (var post in this.CurrentTab.SelectedPosts)
+                using (var fltDialog = new FilterDialog())
                 {
-                    //タブ選択（or追加）
-                    if (!SelectTab(out var tab)) return;
-
+                    fltDialog.Owner = this;
                     fltDialog.SetCurrent(tab.TabName);
 
                     if (post.RetweetedBy == null)
@@ -7980,8 +7980,9 @@ namespace OpenTween
                         fltDialog.AddNewFilter(post.RetweetedBy, post.TextFromApi);
                     }
                     fltDialog.ShowDialog(this);
-                    this.TopMost = SettingManager.Common.AlwaysTop;
                 }
+
+                this.TopMost = SettingManager.Common.AlwaysTop;
             }
 
             this.ApplyPostFilters();