Volian/SourceCode
8
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

C024-039_Button_on_Applicability_Tab_to_change_all_at_same_level_for_setting_the_item_mode-2 #714

Merged
jjenko merged 5 commits from C024-039_Button_on_Applicability_Tab_to_change_all_at_same_level_for_setting_the_item_mode-2 into Development 2026-02-20 09:06:02 -05:00
Conversation 9 Commits 5 Files Changed 2 -86
1 changed files with 0 additions and 86 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e999bea228 - Show all commits

86
PROMS/Volian.Controls.Library/DisplayApplicability.cs
View File

@@ -46,7 +46,6 @@ namespace Volian.Controls.Library
int apple = -1; int apple = -1;
if(_MyDisplayTabItem.MyStepTabPanel != null) if(_MyDisplayTabItem.MyStepTabPanel != null)
apple = _MyDisplayTabItem.MyStepTabPanel.MyStepPanel.ApplDisplayMode; apple = _MyDisplayTabItem.MyStepTabPanel.MyStepPanel.ApplDisplayMode;
//_MyDisplayTabItem.MyStepTabPanel.MyStepTabRibbon.RefreshProcedure
AddViewMode("Master", "-1", apple == -1); AddViewMode("Master", "-1", apple == -1);
int i = 0; int i = 0;
foreach (string name in names) foreach (string name in names)
@@ -284,11 +283,9 @@ namespace Volian.Controls.Library
} }
private void BtnApplicabilitychg_Click(object sender, System.EventArgs e) private void BtnApplicabilitychg_Click(object sender, System.EventArgs e)
{ {
//ItemInfo
EditItem topEditItem = MyEditItem; EditItem topEditItem = MyEditItem;
Volian.Base.Library.BigNum MasterSlave_ApplicabilityTmp; Volian.Base.Library.BigNum MasterSlave_ApplicabilityTmp;
mschill marked this conversation as resolved
mschill commented 2026-02-19 11:17:54 -05:00
Review
Copy Link

Remove commented out code

Remove commented out code
StepConfig sc = MyItemInfo.MyConfig as StepConfig; StepConfig sc = MyItemInfo.MyConfig as StepConfig;
//sc.MasterSlave_Applicability = Volian.Base.Library.BigNum.MakeBigNum(MyApplicability);
MasterSlave_ApplicabilityTmp = sc.MasterSlave_Applicability; MasterSlave_ApplicabilityTmp = sc.MasterSlave_Applicability;
mschill marked this conversation as resolved
mschill commented 2026-02-19 15:42:26 -05:00
Review
Copy Link

Is this still needed?

Is this still needed?
plarsen commented 2026-02-19 16:13:51 -05:00
Review
Copy Link

It was needed in one solution, but no longer needed.

It was needed in one solution, but no longer needed.
MyEditItem = MyDisplayTabItem.MyStepTabPanel.MyStepPanel.SelectedEditItem; MyEditItem = MyDisplayTabItem.MyStepTabPanel.MyStepPanel.SelectedEditItem;
mschill marked this conversation as resolved
mschill commented 2026-02-19 11:17:51 -05:00
Review
Copy Link

Remove commented out code

Remove commented out code
@@ -302,57 +299,16 @@ namespace Volian.Controls.Library
using (Content cnt = Content.Get(startitm.MyContent.ContentID)) using (Content cnt = Content.Get(startitm.MyContent.ContentID))
{ {
sc2.MasterSlave_Applicability = (MasterSlave_ApplicabilityTmp); sc2.MasterSlave_Applicability = (MasterSlave_ApplicabilityTmp);
//sc.MasterSlave_Applicability = Volian.Base.Library.BigNum.MakeBigNum(MyApplicability);
cnt.Config = sc2.ToString(); cnt.Config = sc2.ToString();
cnt.Save(); cnt.Save();
} }
startitm = startitm.GetNext(); startitm = startitm.GetNext();
} }
//RefreshProcedure();
_MyDisplayTabItem.MyStepTabPanel.MyStepTabRibbon.RefreshProcedure(); _MyDisplayTabItem.MyStepTabPanel.MyStepTabRibbon.RefreshProcedure();
//StepTabRibbon.RefreshProcedureApp(MyEditItem);
} }
//public void RefreshProcedure()
//{
// StepTabPanel stab_Panel = Parent as StepTabPanel;
// if (_MyDisplayTabItem.MyStepTabPanel != null)
// {
// MyItemInfo = _MyDisplayTabItem.MyStepTabPanel.MyStepPanel.SelectedItemInfo;
// MyEditItem = MyDisplayTabItem.MyStepTabPanel.MyStepPanel.SelectedEditItem;
// }
// if (MyEditItem != null)
// {
// if (!MyEditItem.MyStepPanel.ContainsFocus)
// this.Invoke((Action)(() => { MyEditItem.MyStepPanel.Focus(); }));
// E_ViewMode mode = MyEditItem.MyStepPanel.VwMode;
// Application.DoEvents();
// this.Invoke((Action)(() => { MyEditItem.MyStepPanel.ResetAll(); }));
// Application.DoEvents();
// this.Invoke((Action)(() => { MyEditItem.MyStepPanel.Refresh(); }));
// Application.DoEvents();
// MyEditItem.MyStepPanel.VwMode = mode;
// Application.DoEvents();
// }
// else if (stab_Panel != null)
// {
// if (!stab_Panel.MyStepPanel.ContainsFocus)
// this.Invoke((Action)(() => { stab_Panel.MyStepPanel.Focus(); }));
// E_ViewMode mode = stab_Panel.MyStepPanel.VwMode;
// Application.DoEvents();
// this.Invoke((Action)(() => { stab_Panel.MyStepPanel.ResetAll(); }));
// Application.DoEvents();
// this.Invoke((Action)(() => { stab_Panel.MyStepPanel.Refresh(); }));
// Application.DoEvents();
// stab_Panel.MyStepPanel.VwMode = mode;
// Application.DoEvents();
// }
//}
// B2021-149: for procedure, only consider external transitions as invalid // B2021-149: for procedure, only consider external transitions as invalid
private bool IsProcWithNoExternalTrans(ItemInfo ii, List<InvalidTransition> invalidTrans) private bool IsProcWithNoExternalTrans(ItemInfo ii, List<InvalidTransition> invalidTrans)
{ {
@@ -441,48 +397,6 @@ namespace Volian.Controls.Library
set set
{ {
_MyEditItem = value; _MyEditItem = value;
//if (DesignMode) return; // B2019-043 need to check if we are just saving changes to the user interface
//if (!Visible)
//{
// NotVisibleEditItem = value;
// CurItemInfo = null;
// return;
//}
//NotVisibleEditItem = null;
//if (value == null)
//{
// ClearControls();
// return;
//}
//if (CurItemInfo != null && CurItemInfo.ItemID == value.MyItemInfo.ItemID) return;
//if (!value.MyItemInfo.IsStep)
//{
// ClearControls();
// return;
//}
//_MyEditItem = value;
//CurItemInfo = value.MyItemInfo;
//_MyEditItem.MyStepPropertiesPanel = this; // this allows us to access the Step Property pannel and toggle check boxes with shortcut keystrokes
//// originally the change all (for step type) checkbox was initialized based
//// on whether all steps at level were same type, i.e. if same type, default
//// it to checked. This may confuse the user, so we decided to always default
//// to not checked. The code was left in here in case that decision is changed.
//// MRC, JSJ & KBR were in on discussion on 2/27/12:
//// only change all at level if all substeps are of same type
////bool allSameType = true;
////ItemInfo itmp = CurItemInfo.FirstSibling;
////while (itmp != null)
////{
//// if (CurItemInfo.MyContent.Type != itmp.MyContent.Type)
//// {
//// allSameType = false;
//// break;
//// }
//// itmp = itmp.NextItem;
////}
//cbChgAll.Checked = false;
//TagsFillIn();
} }
} }
private void WireCheckboxes() private void WireCheckboxes()