Handle new error when a deleted step is attempted to be copied.
Added logic to handle an invalid transition (a transition to a disconnected step.
This commit is contained in:
parent
059041c546
commit
b5ded05838
@ -431,6 +431,7 @@ namespace VEPROMS.CSLA.Library
|
|||||||
}
|
}
|
||||||
catch (Exception ex)
|
catch (Exception ex)
|
||||||
{
|
{
|
||||||
|
if (!HandleSqlExceptionOnCopy(ex))
|
||||||
System.Windows.Forms.MessageBox.Show("Details were written to the Error Log.","Paste Failed", System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Error);
|
System.Windows.Forms.MessageBox.Show("Details were written to the Error Log.","Paste Failed", System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Error);
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
@ -457,10 +458,21 @@ namespace VEPROMS.CSLA.Library
|
|||||||
}
|
}
|
||||||
catch (Exception ex)
|
catch (Exception ex)
|
||||||
{
|
{
|
||||||
|
if(!HandleSqlExceptionOnCopy(ex))
|
||||||
System.Windows.Forms.MessageBox.Show("Details were written to the Error Log.", "Paste Failed", System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Error);
|
System.Windows.Forms.MessageBox.Show("Details were written to the Error Log.", "Paste Failed", System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Error);
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
private bool HandleSqlExceptionOnCopy(Exception ex)
|
||||||
|
{
|
||||||
|
if (ex.Message.Contains("This step has been deleted"))
|
||||||
|
{
|
||||||
|
System.Windows.Forms.MessageBox.Show("The step being pasted has been deleted!", "Cannot Paste Deleted Step"
|
||||||
|
, System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Hand);
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
return false;
|
||||||
|
}
|
||||||
private ItemInfo CopyPasteItemInfoFetch(int copyStartID, int itemID, int? type, int? fromType, EAddpingPart addType)
|
private ItemInfo CopyPasteItemInfoFetch(int copyStartID, int itemID, int? type, int? fromType, EAddpingPart addType)
|
||||||
{
|
{
|
||||||
ItemInfo tmp=null;
|
ItemInfo tmp=null;
|
||||||
@ -1225,7 +1237,10 @@ namespace VEPROMS.CSLA.Library
|
|||||||
}
|
}
|
||||||
catch (Exception ex)
|
catch (Exception ex)
|
||||||
{
|
{
|
||||||
if (ex.Message.Contains("has External Transitions and has no next step") || ex.Message.Contains("has External Transitions to it's children"))
|
if (ex.Message.Contains("has External Transitions and has no next step")
|
||||||
|
|| ex.Message.Contains("has External Transitions to it's children")
|
||||||
|
|| ex.Message.Contains("This step has been deleted")
|
||||||
|
)
|
||||||
throw ex;
|
throw ex;
|
||||||
System.Windows.Forms.MessageBox.Show("Details were written to the Error Log.", "Paste Replace Failed", System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Error);
|
System.Windows.Forms.MessageBox.Show("Details were written to the Error Log.", "Paste Replace Failed", System.Windows.Forms.MessageBoxButtons.OK, System.Windows.Forms.MessageBoxIcon.Error);
|
||||||
return itemInfo;
|
return itemInfo;
|
||||||
|
@ -1246,6 +1246,7 @@ namespace Volian.Controls.Library
|
|||||||
//catch (System.Data.SqlClient.SqlException ex)
|
//catch (System.Data.SqlClient.SqlException ex)
|
||||||
catch (Exception ex)
|
catch (Exception ex)
|
||||||
{
|
{
|
||||||
|
if(!HandleSqlExceptionOnCopy(ex))
|
||||||
HandleSqlExceptionOnDelete(ex);
|
HandleSqlExceptionOnDelete(ex);
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
@ -1299,6 +1300,15 @@ namespace Volian.Controls.Library
|
|||||||
MyStepPanel.MyStepTabPanel.MyDisplayTabControl.OnItemPaste(this, new vlnTreeItemInfoPasteEventArgs(newItemInfo, copyStartID, ItemInfo.EAddpingPart.Replace, newItemInfo.MyContent.Type));
|
MyStepPanel.MyStepTabPanel.MyDisplayTabControl.OnItemPaste(this, new vlnTreeItemInfoPasteEventArgs(newItemInfo, copyStartID, ItemInfo.EAddpingPart.Replace, newItemInfo.MyContent.Type));
|
||||||
return newEditItem;
|
return newEditItem;
|
||||||
}
|
}
|
||||||
|
private bool HandleSqlExceptionOnCopy(Exception ex)
|
||||||
|
{
|
||||||
|
if (ex.Message.Contains("This step has been deleted"))
|
||||||
|
{
|
||||||
|
MessageBox.Show("The step being pasted has been deleted", "Cannot Paste Step", MessageBoxButtons.OK, MessageBoxIcon.Hand);
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
return false;
|
||||||
|
}
|
||||||
public void IdentifyChildren(bool highlight)
|
public void IdentifyChildren(bool highlight)
|
||||||
{
|
{
|
||||||
// Highlight children of EditItem:
|
// Highlight children of EditItem:
|
||||||
|
@ -280,7 +280,7 @@ namespace Volian.Controls.Library
|
|||||||
if (args.MyLinkText.LinkInfoText.IndexOf("Transition") > -1)
|
if (args.MyLinkText.LinkInfoText.IndexOf("Transition") > -1)
|
||||||
{
|
{
|
||||||
ItemInfo item = args.MyLinkText.MyTranToItemInfo;
|
ItemInfo item = args.MyLinkText.MyTranToItemInfo;
|
||||||
if (item == null || (item.PreviousID == null && item.ItemPartCount == 0 && item.ItemDocVersionCount == 0))
|
if (item == null || item.MyDocVersion == null || (item.PreviousID == null && item.ItemPartCount == 0 && item.ItemDocVersionCount == 0))
|
||||||
{
|
{
|
||||||
MessageBox.Show("This transition is invalid", "Invalid Transition", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
|
MessageBox.Show("This transition is invalid", "Invalid Transition", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
|
||||||
return; // Not a valid transition
|
return; // Not a valid transition
|
||||||
|
Loading…
x
Reference in New Issue
Block a user