Merge pull request 'Development' (#770) from Development into master

Merging development into master after successful testing.

PROMS/ReferencedObjects/Exe/RefObj/ROEditor/RO_FST.cs

@@ -116,12 +116,12 @@ namespace ROEditor
 		public uint thisoff;
 		public string title;
 
-        public FstTmpSTRC(ushort type, uint offset, string tl)
+		public FstTmpSTRC(ushort type, uint offset, string tl)
 		{
 			thistype = type;
 			thisoff = offset;
 			title = tl;
-        }
+		}
 
 		void WriteString(BinaryWriter bw, string str)
 		{
@@ -150,7 +150,7 @@ namespace ROEditor
 			bw.Write(thisoff);
 			bw.Write(thistype);
 			WriteString(bw,title);
-        }
+		}
 	}
 
 // The Sorted Array was not sorting via the ASCII value of each character in a given string.
@@ -565,8 +565,8 @@ namespace ROEditor
 			string RecIdStr = elem.GetAttribute("RecID");
 			curRecID = System.Convert.ToUInt32(RecIdStr,16);
 
-            // Get the current node's parent id
+			// Get the current node's parent id
-            string ParIdStr = elem.GetAttribute("ParentID");
+			string ParIdStr = elem.GetAttribute("ParentID");
 			elemParentID = Convert.ToUInt32(ParIdStr,16);
 
 			string HasKids = elem.GetAttribute("HasChild");
@@ -771,8 +771,8 @@ namespace ROEditor
 		private ushort SaveROToFST(XmlNode RONode,ArrayList InUseList,string RtnValTmplate, string AccPageIDTplate)
 		{
 			ushort RtnVal;
-            uint startFST = (uint)fhFST.BaseStream.Position;
+			uint startFST = (uint)fhFST.BaseStream.Position;
-            uint RORecID;
+			uint RORecID;
 			uint ParID;
 			byte nullbyte=0;
 
@@ -844,12 +844,8 @@ namespace ROEditor
 				AccPageID = " ";
 			WriteString(AccPageID);
 
-            //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+			// Save the ID and offset entry for the current ID
-            string moddt = ROdatabase.RODB_GetModDateTime(elem.GetAttribute("RecID"), elem.GetAttribute("Table"));
+			IdsAndOffsets.Add(RORecID,startFST);
-            if (!string.IsNullOrEmpty(moddt)) WriteString(moddt);
-
-            // Save the ID and offset entry for the current ID
-            IdsAndOffsets.Add(RORecID, startFST);
 
 			// Save the RecID and Accessory Page id
 			IdsAndAccPgIds[AccPageID] = RORecID;

PROMS/ReferencedObjects/LibSource/RODBInterface/RODBInterface.cs

@@ -322,9 +322,9 @@ namespace RODBInterface
 			get { return _PCChildList; } 
 			set { _PCChildList = value; } 
 		}
-        #endregion
+		#endregion
-        #region abstracts   // need these for each method that must be defined for each database type
+		#region abstracts   // need these for each method that must be defined for each database type
-        public abstract string RODB_GetNextGroupTable();
+		public abstract string RODB_GetNextGroupTable();
 		public abstract string RODB_GetNextRecId(string table);
 		public abstract bool RODB_GetRootGroups(VlnXmlElement root);
 		public abstract bool RODB_DeleteGroup(XmlNode group, string tbname, string toprecid);
@@ -355,10 +355,9 @@ namespace RODBInterface
 		public abstract string RODB_GetDBServerForAbout();
 		public abstract string RODB_HasBeenConverted();
 		public abstract bool RODB_WriteSqlConnectToAccess(string newConectStr);
-		public abstract string RODB_GetModDateTime(string Recid, string table);
+		#endregion
-        #endregion
 
-        public RODB()
+		public RODB()
 		{
 
 		}
@@ -2383,9 +2382,9 @@ namespace RODBInterface
 				RecID = DBE.GetString(0);
 				MyRecID = RecID;
 				Info = DBE.GetString(1);
-                // it's defined in the local table if the first character is "<" which starts
+				// it's defined in the local table if the first character is "<" which starts
-                // the schema definition string.
+				// the schema definition string.
-                if ("<" == Info.Substring(0, 1))
+				if ("<" == Info.Substring(0, 1))
 				{
 					name = ParseEleName(Info);
 					if (name != null)
@@ -2426,7 +2425,7 @@ namespace RODBInterface
 			return retlist;
 		}
 
-        public override string RODB_GetSchemaPiece(string Recid, string table)
+		public override string RODB_GetSchemaPiece(string Recid, string table)
 		{
 			string strGetSchemaPiece;
 			string Info;
@@ -3083,28 +3082,7 @@ namespace RODBInterface
 			return GrpCnt;
 		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+	}
-        //Get the Modification Date / Time for the RO
-        public override string RODB_GetModDateTime(string Recid, string table)
-        {
-            using (System.Data.OleDb.OleDbConnection connection = new System.Data.OleDb.OleDbConnection(strDatabaseConnectionCommand))
-            {
-				connection.Open();
-                using (System.Data.OleDb.OleDbCommand command = connection.CreateCommand())
-                {
-					//Unfortunately Access wont let you paramaterize the table name
-                    command.CommandText = $"SELECT ModDateTime FROM {Regex.Replace(table, "[^a-zA-Z0-9]", "")} WHERE RecID = @Value";
-                    command.Parameters.Add(new System.Data.OleDb.OleDbParameter
-                    {
-                        OleDbType = System.Data.OleDb.OleDbType.VarChar,
-                        ParameterName = "@Value",
-                        Value = Recid
-                    });
-                    return command.ExecuteScalar().ToString();
-                }
-            }
-        }
-    }
 }
 
 

PROMS/ReferencedObjects/LibSource/RODBInterface/SqlRODB.cs

@@ -1803,9 +1803,9 @@ namespace RODBInterface
 							RecID = reader.GetString(0);
 							MyRecID = RecID;
 							Info = reader.GetString(1);
-                            // it's defined in the local table if the first character is "<" which starts
+							// it's defined in the local table if the first character is "<" which starts
-                            // the schema definition string.
+							// the schema definition string.
-                            if ("<" == Info.Substring(0, 1))
+							if ("<" == Info.Substring(0, 1))
 							{
 								name = ParseEleName(Info);
 								if (name != null)
@@ -1859,9 +1859,9 @@ namespace RODBInterface
 								RecID = reader.GetString(0);
 								MyRecID = RecID;
 								Info = reader.GetString(1);
-                                // it's defined in the local table if the first character is "<" which starts
+								// it's defined in the local table if the first character is "<" which starts
-                                // the schema definition string.
+								// the schema definition string.
-                                if ("<" == Info.Substring(0, 1))
+								if ("<" == Info.Substring(0, 1))
 								{
 									name = ParseEleName(Info);
 									if (name != null)
@@ -2653,36 +2653,8 @@ namespace RODBInterface
 			}
 			return GrpCnt;
 		}
-
+		#endregion
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		public static bool TestConnect(string constring)
-        //Get the Modification Date / Time for the RO
-        public override string RODB_GetModDateTime(string Recid, string table)
-        {
-            using (SqlConnection connection = new SqlConnection(strDatabaseConnectionCommand))
-            {
-				connection.Open();
-                using (SqlCommand command = connection.CreateCommand())
-                {
-                    command.CommandText = $"SELECT ModDateTime FROM ROALL WHERE RecID = @Value AND ROTable = @table";
-                    command.Parameters.Add(new SqlParameter
-                    {
-                        SqlDbType = SqlDbType.VarChar,
-                        ParameterName = "@Value",
-                        Value = Recid
-                    });
-                    command.Parameters.Add(new SqlParameter
-                    {
-                        SqlDbType = SqlDbType.VarChar,
-                        ParameterName = "@table",
-                        Value = table
-                    });
-
-                    return command.ExecuteScalar().ToString();
-                }
-            }
-        }
-        #endregion
-        public static bool TestConnect(string constring)
 		{
 			bool success = false;
 			using (SqlConnection connection = new SqlConnection(constring))

PROMS/VEPROMS User Interface/PROMSFixes.Sql

@@ -13541,6 +13541,34 @@ IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = OBJECT_ID(N'[deleteAllDocVers
     DROP PROCEDURE [deleteAllDocVersionPdfs];
 GO
 	
+/*****************************************************************************
+   Volian Enterprises - Proprietary Information - DO NOT COPY OR DISTRIBUTE
+       Copyright 2017 - Volian Enterprises, Inc. All rights reserved.
+*****************************************************************************/
+CREATE PROCEDURE [dbo].[deleteAllDocVersionPdfs]
+
+(
+	@VersionID int
+)
+WITH EXECUTE AS OWNER
+AS
+BEGIN TRY -- Try Block
+	BEGIN TRANSACTION
+	DELETE [Pdfs]
+		WHERE [DocID] IN(select EE.DocID from vefn_GetVersionItems(cast(@VersionID as varchar(20))) VI
+		Join Entries EE ON EE.ContentID= VI.ContentID)
+	IF( @@TRANCOUNT > 0 ) COMMIT
+END TRY
+BEGIN CATCH -- Catch Block
+	IF( @@TRANCOUNT = 1 ) ROLLBACK -- Only rollback if top level
+ 	ELSE IF( @@TRANCOUNT > 1 ) COMMIT -- Otherwise commit.  Top level will rollback
+	EXEC vlnErrorHandler
+END CATCH
+GO
+-- Display the status of Proc creation
+IF (@@Error = 0) PRINT 'Procedure Creation: deleteAllDocVersionPdfs Succeeded'
+ELSE PRINT 'Procedure Creation: deleteAllDocVersionPdfs Error on Creation'
+GO
 /****** Object:  StoredProcedure [addFiguresByROFstIDandImageIDs] ******/
 IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = OBJECT_ID(N'[addFiguresByROFstIDandImageIDs]') AND OBJECTPROPERTY(id,N'IsProcedure') = 1)
     DROP PROCEDURE [addFiguresByROFstIDandImageIDs];
@@ -17141,7 +17169,6 @@ GO
 			[roid] [varchar](50) NOT NULL,
 			[appid] [varchar](max) NULL,
 			[value] [varchar](max) NULL,
-            [moddatetime] [datetime] NULL,
 		 CONSTRAINT [PK_RofstChild] PRIMARY KEY CLUSTERED 
 		(
 			[RofstChildID] ASC
@@ -17276,23 +17303,6 @@ GO
 	End -- Rofst Tables
 	Go
 
-	-- =============================================
-	-- Author:		Matthew Schill
-	-- Create date: 03/30/2026
-	-- Description:	Store RO Modification date/time
-	-- =============================================
-
-	--- Add Column to store RO  Modification date/time if it does not already exist
-	IF NOT EXISTS(SELECT *
-			  FROM   INFORMATION_SCHEMA.COLUMNS
-			  WHERE  TABLE_NAME = 'RofstChild'
-					 AND COLUMN_NAME = 'moddatetime') 
-	ALTER TABLE RofstChild ADD moddatetime datetime NULL;
-	go
-	-- Display the status
-	IF (@@Error = 0) PRINT 'Altered table [RofstChild] Succeeded for moddatetime'
-	ELSE PRINT 'Altered table [RofstChild] Error on Alter for moddatetime'
-	go
 
 	/*
 	----------------------------------------------------------------------------------
@@ -19695,7 +19705,6 @@ GO
 		@roid VarChar(50),
 		@appid VarChar(Max) = null,
 		@value VarChar(Max) = null,
-		@ModDateTime DateTime = null,
 		@missingDefaultValue VarChar(Max) = null
 	)
 	With Execute as Owner
@@ -19713,8 +19722,8 @@ GO
 			Set @missingDefaultValue = '[TBD]';
 
 		-- Create Rofst Child/Group Record --> [Roid = (12) Digits]
-		Insert Into RofstChild (RofstID, ID, ParentID, dbiID, [type], title, roid, appid, moddatetime, [value])
+		Insert Into RofstChild (RofstID, ID, ParentID, dbiID, [type], title, roid, appid, [value])
-		Values (@RofstID, @ID, @ParentID, @dbiID, @type, @title, @roid, @appid, @ModDateTime, REPLACE(REPLACE(@value, '&123;', '{'), '&125;', '}'));
+		Values (@RofstID, @ID, @ParentID, @dbiID, @type, @title, @roid, @appid, REPLACE(REPLACE(@value, '&123;', '{'), '&125;', '}'));
 
 
 		-- Check for appid, if exists, then insert the default value for each return type if multi-value
@@ -24768,146 +24777,6 @@ IF (@@Error = 0) PRINT 'Procedure Creation: [GetMissingDocsByUnit] Succeeded'
 ELSE PRINT 'Procedure Creation: [GetMissingDocsByUnit] Error on Creation'
 GO
 
-
-IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = OBJECT_ID(N'[vefn_ROFST_changes]') AND OBJECTPROPERTY(id,N'IsTableFunction') = 1)
-    DROP FUNCTION [vefn_ROFST_changes];
-GO
-
-SET ANSI_NULLS ON
-GO
-SET QUOTED_IDENTIFIER ON
-GO
-	
-/*
-==========================================================================================================
- Author:		Matthew Schill
- Create Date:	03/31/2026
- Description:	Function for ROs that updated in latest RO FST Load
-==========================================================================================================
-*/
-CREATE FUNCTION [dbo].[vefn_ROFST_changes](@OrigFSTID int, @NewFSTid int, @VersionID int)
-RETURNS @ROIDs TABLE
-(
-[roid] varchar(50)
-)
-WITH EXECUTE AS OWNER
-AS
-BEGIN
-
-    insert into @ROIDs
-	  SELECT DISTINCT ISNULL(RofstChild.roid,previous.roid)
-	    FROM 
-			(SELECT * FROM RofstChild where RofstChild.RofstID = @NewFSTid) RofstChild
-			FULL OUTER JOIN 
-			(SELECT * FROM RofstChild previous where previous.RofstID = @OrigFSTID) previous
-			ON previous.dbiID = RofstChild.dbiID AND previous.ID = RofstChild.ID 
-	    where	
-	    ISNULL(previous.RofstID,'') != ISNULL(RofstChild.RofstID,'')
-		AND
-	    (
-		 (RofstChild.RofstID = @NewFSTid OR RofstChild.RofstID IS NULL)
-		  AND
-		 (previous.RofstID = @OrigFSTID OR previous.RofstID IS NULL)
-		)
-		AND
-		(previous.moddatetime IS NULL 
-        OR RofstChild.moddatetime IS NULL 
-		OR RofstChild.moddatetime != previous.moddatetime
-		OR RofstChild.title != previous.title
-		OR RofstChild.value != previous.value
-		)
-
-	RETURN
-END
-go
-
-IF (@@Error = 0) PRINT 'TableFunction [vefn_ROFST_changes] Succeeded'
-ELSE PRINT 'TableFunction [vefn_ROFST_changes] Error on Creation'
-go
-
-/****** Object:  StoredProcedure [deleteDocVersionPdfsWithNewROs] ******/
-IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = OBJECT_ID(N'[deleteDocVersionPdfsWithNewROs]') AND OBJECTPROPERTY(id,N'IsProcedure') = 1)
-    DROP PROCEDURE [deleteDocVersionPdfsWithNewROs];
-GO
-	
-/*
-==========================================================================================================
- Author:		Matthew Schill
- Create Date:	03/31/2026
- Description:	Delete all PDFs with ROs that will need re-resolved
-==========================================================================================================
-*/
-CREATE PROCEDURE [dbo].[deleteDocVersionPdfsWithNewROs]
-
-(
-	@VersionID int,
-    @OrigFSTid int,
-    @NewFSTid int
-)
-WITH EXECUTE AS OWNER
-AS
- BEGIN
-	DELETE [Pdfs]
-		WHERE [DocID] IN 
-            (
-               select EE.DocID from vefn_GetVersionItems(cast(@VersionID as varchar(20))) VI
-		       Join Entries EE ON EE.ContentID= VI.ContentID
-			   Join DRoUsages ON DRoUsages.DocID = EE.DocID
-               Join dbo.vefn_ROFST_changes(@OrigFSTID, @NewFSTid, @VersionID) ROFST_changes on LEFT(DRoUsages.ROID,12) = ROFST_changes.roid
-            )
-
-	RETURN
- END
-
-GO
-
--- Display the status of Proc creation
-IF (@@Error = 0) PRINT 'Procedure Creation: deleteDocVersionPdfsWithNewROs Succeeded'
-ELSE PRINT 'Procedure Creation: deleteDocVersionPdfsWithNewROs Error on Creation'
-GO
-
-
-/****** Object:  StoredProcedure [getItemsWithNewROs] ******/
-IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = OBJECT_ID(N'[getItemsWithNewROs]') AND OBJECTPROPERTY(id,N'IsProcedure') = 1)
-    DROP PROCEDURE [getItemsWithNewROs];
-GO
-
-/*
-==========================================================================================================
- Author:		Matthew Schill
- Create Date:	03/31/2026
- Description:	Get Items with New ROs that will need resolved
-==========================================================================================================
-*/
-CREATE PROCEDURE [dbo].[getItemsWithNewROs]
-
-(
-	@VersionID int,
-    @OrigFSTid int,
-    @NewFSTid int
-)
-WITH EXECUTE AS OWNER
-AS
- BEGIN
-	select DISTINCT tblItems.ItemID FROM
-	dbo.vefn_ROFST_changes(@OrigFSTID, @NewFSTid, @VersionID) ROFST_changes
-	INNER JOIN RoUsages ON LEFT(RoUsages.ROID,12) = ROFST_changes.roid
-	INNER JOIN tblItems ON RoUsages.ContentID = tblItems.ContentID
-	OUTER APPLY (Select VersionID =  dbo.vefn_GetVersionIDByItemID(tblItems.ItemID)) ver
-	INNER JOIN DocVersions DV ON DV.VersionID = ver.VersionID
-	INNER JOIN Associations ON Associations.VersionID = DV.VersionID
-	where ver.VersionID = @VersionID
-
-	RETURN
- END
-
-GO
-
--- Display the status of Proc creation
-IF (@@Error = 0) PRINT 'Procedure Creation: getItemsWithNewROs Succeeded'
-ELSE PRINT 'Procedure Creation: getItemsWithNewROs Error on Creation'
-GO
-
 /*
   ---------------------------------------------------------------------------
   |                       ADD New Code Before this Block                    |
@@ -24941,8 +24810,8 @@ BEGIN TRY -- Try Block
 	DECLARE @RevDate varchar(255)
 	DECLARE @RevDescription varchar(255)
 
-	set @RevDate = '04/21/2026 7:00 AM'
+	set @RevDate = '03/13/2026 7:00 AM'
-	set @RevDescription = 'Store RO Modification date/time'
+	set @RevDescription = 'Added Get Missing Docs by Unit for Generating Pdf table'
 
 	Select  cast(@RevDate as datetime) RevDate, @RevDescription  RevDescription
 	PRINT 'SQL Code Revision ' + @RevDate + ' - ' + @RevDescription

PROMS/VEPROMS User Interface/dlgApproveProcedure.cs

@@ -1343,31 +1343,23 @@ namespace VEPROMS
                     //// so change bars update
                     //// on any open StepPanel
 
-                    //B2026-019 Attempt to prevent an Access Error by utilizing a different Refresh if a Procedure is Open
+                    newproc = ItemInfo.ResetProcedure(pi.ItemID);
-                    DisplayTabItem dti = MyFrmVEPROMS.GetTabContainingProcedure(pi.ItemID);
+
+					//B2026-019 Attempt to prevent an Access Error by utilizing a different Refresh if a Procedure is Open
+					DisplayTabItem dti = MyFrmVEPROMS.GetTabContainingProcedure(pi.ItemID);
 					if (dti != null)
 					{
 						if (!dti.MyStepTabPanel.MyStepPanel.ContainsFocus)
 							dti.MyStepTabPanel.MyStepPanel.Focus();
 
-						foreach (EditItem eitm in dti.MyStepTabPanel.MyStepPanel.Controls.OfType<EditItem>())
+						dti.MyStepTabPanel.MyStepTabRibbon.RefreshProcedure();
-                        {
+						Application.DoEvents();
-                            eitm.ChangeBar = eitm.MyItemInfo.HasChangeBar;
-                        }
 
-                        dti.MyStepTabPanel.MyStepTabRibbon.RefreshProcedure();
+					}
-                        Application.DoEvents();
-                        newproc = ProcedureInfo.Get(pi.ItemID);
 
-                    }
+					//since in a separate form, need to update the tree view
-					else
+					//so "Showing Change Bars" Content Menu Item is correct
-					{
+					MyFrmVEPROMS.RefreshProcedureNode(newproc);
-                        newproc = ItemInfo.ResetProcedure(pi.ItemID);
-                    }
-
-						//since in a separate form, need to update the tree view
-						//so "Showing Change Bars" Content Menu Item is correct
-						MyFrmVEPROMS.RefreshProcedureNode(newproc);
 				}
 				else
 					UpdateProcedureConfig(pi, ap.RevNumber, ap.RevDate, myDTS, selectedSlave);

PROMS/VEPROMS User Interface/frmBatchRefresh.cs

@@ -1036,11 +1036,8 @@ namespace VEPROMS
 			ROFstInfo roFstInfo = dq.DocVersionAssociations[0].MyROFst;
 			string rofstPath = roFstInfo.MyRODb.FolderPath + @"\ro.fst";
 
-            //must get id before ROFST gets updated so know what to refresh later
+			//if (!pathExists(rofstPath))
-            int origfstid = roFstInfo.ROFstID;
+			if (!File.Exists(rofstPath))
-
-            //if (!pathExists(rofstPath))
-            if (!File.Exists(rofstPath))
 			{
 				ProgressBar.ColorTable = eProgressBarItemColor.Error;
 				FinalProgressBarMessage = "No existing RO.FST";
@@ -1067,8 +1064,8 @@ namespace VEPROMS
 					roFstInfo = dq.DocVersionAssociations[0].MyROFst;
 				}
 				roFstInfo.ROTableUpdate += new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
-                ROFstInfo.RefreshROFstAtItemLevel(DocVersionInfo.Get(dv.VersionID), DoProgressBarRefresh, txtProcess, roFstInfo, origfstid, roFstInfo.ROFstID);
+				ROFst newrofst = ROFstInfo.RefreshROFst(dv, roFstInfo, DoProgressBarRefresh, txtProcess);
-                roFstInfo.ROTableUpdate -= new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
+				roFstInfo.ROTableUpdate -= new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 			}
 
 			Cursor = Cursors.Default;

PROMS/VEPROMS User Interface/frmVEPROMS.cs

@@ -1594,7 +1594,7 @@ namespace VEPROMS
 					displayRO.MyROFST = SelectedROFst;
                     // B2023-021: force Load of Step Prop/RO panel RO tree by passing in
                     //   true to LoadTree
-                    if (!_WeAreExitingPROMS) displayRO.LoadTree(true);
+                    displayRO.LoadTree(true);
 				}
 			}
 
@@ -2395,7 +2395,6 @@ namespace VEPROMS
 			tv.MySessionInfo = MySessionInfo;
 			tv.MyUserInfo = MyUserInfo;
 			StepTabRibbon.MySessionInfo = MySessionInfo;
-            displayRO.MySessionInfo = MySessionInfo;
 
             // Initialize Caption with Server name and Database name.
             SetCaption(tv.TopNode as VETreeNode);
@@ -4554,15 +4553,6 @@ namespace VEPROMS
 			SetCaption(tv.SelectedNode as VETreeNode);
 			displayApplicability.MyDisplayTabItem = tc.SelectedDisplayTabItem;
 
-            //C2026-008 Re-Architect RO.FST to include RO Modification date/time
-            // DisplayTab was changed
-            // need to clear the RTB selected
-            // without resetting the DVI or FST
-            // (those will be set manually / individually)
-            // this will prevent DVI and FST from changing to null then back again
-            // which will trigger reloads            
-            displayRO.ClearRTB();
-
             if (tc.SelectedDisplayTabItem.MyItemInfo.MyDocVersion.DocVersionAssociationCount > 0)
 			{
 				displayRO.MyROFST = tc.SelectedDisplayTabItem.MyItemInfo.MyDocVersion.DocVersionAssociations[0].MyROFst;
@@ -4572,24 +4562,9 @@ namespace VEPROMS
 				displayRO.MyROFST = null;
 			}
 
-            //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+			// B2022-026 RO Memory reduction coding (Jakes Merge)
-			// set the DocVersionInfo so that the FST and docversion are in sync
+			// B2022-123: RO Tab Treeview not showing correct RO values when switching between procedures. (Added True for the forceLoad parameter)
-            SelectedDVI = tc.SelectedDisplayTabItem.MyItemInfo.MyDocVersion;
+			displayRO.LoadTree(true);
-            if (displayRO.MyDvi != SelectedDVI)
-            {
-                displayRO.MyDvi = SelectedDVI;
-            }
-			Application.DoEvents();
-
-            // B2022-026 RO Memory reduction coding (Jakes Merge)
-            // B2022-123: RO Tab Treeview not showing correct RO values when switching between procedures. (Added True for the forceLoad parameter)
-            //C2026-008 Re-Architect RO.FST to include RO Modification date/time
-            // if ROFST got updated,
-			// set the Selected FST so it is in sync
-            if (!_WeAreExitingPROMS && displayRO.LoadTree(true))
-			{
-				SelectedROFst = displayRO.MyROFST;
-            }
 
 			lblUser.Text = tc.SelectedDisplayTabItem.MyUserRole;
 
@@ -4790,13 +4765,7 @@ namespace VEPROMS
 					// B2022-026 RO Memory reduction coding (Jakes Merge)
 					displayRO.ProgressBar = bottomProgBar;
 					displayRO.MyRTB = args.MyEditItem.MyStepRTB;
-                    //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+					displayRO.LoadTree();
-                    // if ROFST got updated,
-                    // set the Selected FST so it is in sync
-                    if (!_WeAreExitingPROMS && displayRO.LoadTree())
-                    {
-                        SelectedROFst = displayRO.MyROFST;
-                    }
 
                     displayBookMarks.MyEditItem = args.MyEditItem;
 					displayHistory.MyEditItem = args.MyEditItem;
@@ -4826,7 +4795,7 @@ namespace VEPROMS
 
 					// B2022-026 RO Memory reduction coding (Jakes Merge)
 					displayRO.SetFindDocROButton(false);
-					if (!_WeAreExitingPROMS) displayRO.LoadTree();
+					displayRO.LoadTree();
 
                     //C2019-036 View Only mode work with Checked Out Procedures
                     //In View Only Mode - Step Properties should be disabled
@@ -4959,14 +4928,8 @@ namespace VEPROMS
 			displayRO.MyRTB = SelectedStepTabPanel.MyStepPanel.SelectedEditItem.MyStepRTB;
 			displayRO.CurROLink = args.MyLinkText.MyRoUsageInfo;
 
-            // B2022-026 RO Memory reduction coding (Jakes Merge)
+			// B2022-026 RO Memory reduction coding (Jakes Merge)
-            //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+			displayRO.LoadTree();
-            // if ROFST got updated,
-            // set the Selected FST so it is in sync
-            if (displayRO.LoadTree())
-            {
-                SelectedROFst = displayRO.MyROFST;
-            }
         }
 
 		#endregion

PROMS/VEPROMS User Interface/frmVersionsProperties.cs

@@ -1527,11 +1527,8 @@ namespace VEPROMS
 
 				Cursor = Cursors.WaitCursor;
 
-                //must get id before ROFST gets updated so know what to refresh later
+				// RO changes placed in file in the Documents\VEPROMS folder
-                int origfstid = SelectedROFst.ROFstID;
+				swROUpdate = new System.IO.StreamWriter(ROFstInfo.ROUpdateResultsPath(_DocVersionConfig.MyDocVersion.MyDocVersionInfo));
-
-                // RO changes placed in file in the Documents\VEPROMS folder
-                swROUpdate = new System.IO.StreamWriter(ROFstInfo.ROUpdateResultsPath(_DocVersionConfig.MyDocVersion.MyDocVersionInfo));
 				DocVersion dv = _DocVersionConfig.MyDocVersion;
 				// B2022-026 RO Memory Reduction code - first load the new ro.fst so that we can assign the ROTableUpdate event to the correct roFstInfo
 				if (dv.ROfstLoadingFigures || dv.NewerRoFst)    // B2017-125 see if loading figures was completed
@@ -1543,9 +1540,9 @@ namespace VEPROMS
 				ContentInfo.StaticContentInfoChange += ContentInfo_StaticContentInfoChange; // write changes to a text file
 				SelectedROFst.ROTableUpdate += new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 
-				ROFstInfo.RefreshROFstAtItemLevel(DocVersionInfo.Get(dv.VersionID), DoProgressBarRefresh, null, SelectedROFst, origfstid, SelectedROFst.ROFstID);
+				ROFst newrofst = ROFstInfo.RefreshROFst(_DocVersionConfig.MyDocVersion, SelectedROFst, DoProgressBarRefresh, null);
 
-                ContentInfo.StaticContentInfoChange -= ContentInfo_StaticContentInfoChange;
+				ContentInfo.StaticContentInfoChange -= ContentInfo_StaticContentInfoChange;
 				SelectedROFst.ROTableUpdate -= new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 
 				swROUpdate.Close();

PROMS/VEPROMS.CSLA.Library/Config/ROFSTLookup.cs

@@ -97,8 +97,7 @@ namespace VEPROMS.CSLA.Library
 			public string appid;
 			public int ID;
 			public int ParentID;
-			public DateTime? ModDateTime;  //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+			public rochild[] children;
-            public rochild[] children;
 		};
 
 		[Serializable]
@@ -111,8 +110,7 @@ namespace VEPROMS.CSLA.Library
 			public string roid;         // roid unique identifier
 			public string appid;        // accessory page id - user specified unique id
 			public string value;        // return value, can be multiple values
-            public DateTime? ModDateTime;  //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+			public rochild[] children;
-            public rochild[] children;
 		};
 
 		public class RoExtension
@@ -688,19 +686,15 @@ namespace VEPROMS.CSLA.Library
 			//return string.Format("{0,10:#####0.00}", TimeSpan.FromTicks(DateTime.Now.Ticks - dtStart.Ticks).TotalSeconds);
 		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		#endregion
-		//return new ROFstID if there is a newer ID that matches the same ROFSTDB
-        public int GetNewerFSTID() => RofstGetLatestID(RofstID);
 
-        #endregion
+		#endregion
 
-        #endregion
+		#region Private Methods
 
-        #region Private Methods
+		#region (Database Calls)
 
-        #region (Database Calls)
+		private bool RofstDataExists(int rofstID)
-
-        private bool RofstDataExists(int rofstID)
 		{
 			int headerStatusID = RofstDataGetHeaderLoadStatus(rofstID);
 
@@ -894,7 +888,7 @@ namespace VEPROMS.CSLA.Library
 			}
 		}
 
-		private void RofstChildInsert(int rofstID, int id, int parentID, int dbiID, int type, string title, string roid, string appid, string value, DateTime? dt = null)
+		private void RofstChildInsert(int rofstID, int id, int parentID, int dbiID, int type, string title, string roid, string appid, string value)
 		{
 			try
 			{
@@ -923,10 +917,7 @@ namespace VEPROMS.CSLA.Library
 						if (!string.IsNullOrEmpty(this.RoMissingDefaultValue))
 							cmd.Parameters.Add(new SqlParameter("@missingDefaultValue", SqlDbType.VarChar)).Value = this.RoMissingDefaultValue;
 
-                        if (dt != null)
+						cmd.ExecuteNonQuery();
-                            cmd.Parameters.Add(new SqlParameter("@ModDateTime", SqlDbType.DateTime)).Value = dt;
-
-                        cmd.ExecuteNonQuery();
 					}
 				}
 			}
@@ -1308,37 +1299,11 @@ namespace VEPROMS.CSLA.Library
 			}
 		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		#endregion
-        //return new ROFstID if there is a newer ID that matches the same ROFSTDB
-        private int RofstGetLatestID(int rofstID)
-        {
-            try
-            {
-                using (SqlConnection cn = Database.VEPROMS_SqlConnection)
-                {
-                    using (SqlCommand cmd = cn.CreateCommand())
-                    {
-                        cmd.CommandTimeout = 0;
-                        cmd.CommandType = CommandType.Text;
-                        cmd.CommandText = "SELECT ISNULL((SELECT TOP 1 ROFsts.ROFstID FROM ROFsts INNER JOIN ROFsts curFST ON curFST.RODbID = ROFsts.RODbID WHERE curFST.ROFstID = @FSTid order by ROFsts.DTS desc),-1)";
 
-                        cmd.Parameters.Add(new SqlParameter("@FSTid", SqlDbType.Int)).Value = rofstID;
+		#region (Core/Base logic for RO Values & UnitInfo RO Values) 
 
-						return (int) cmd.ExecuteScalar();
+		private ROFSTLookup.rochild RofstDataGetChildByRoid(int rofstID, string roid, bool loadChildren = false, bool loadAllChildren = false)
-                    }
-                }
-            }
-            catch (Exception ex)
-            {
-                throw new DbCslaException("RofstData.RofstGetLatestID", ex);
-            }
-        }
-
-        #endregion
-
-        #region (Core/Base logic for RO Values & UnitInfo RO Values) 
-
-        private ROFSTLookup.rochild RofstDataGetChildByRoid(int rofstID, string roid, bool loadChildren = false, bool loadAllChildren = false)
 		{
 			try
 			{
@@ -1550,7 +1515,7 @@ namespace VEPROMS.CSLA.Library
 
 					myOffset += (7 + slen);
 
-                    ROFSTLookup.rogrp tmpg = LoadGroup(ab, childOffset, tableID);
+					ROFSTLookup.rogrp tmpg = LoadGroup(ab, childOffset, tableID);
 
 					tmp.ID = tmpg.ID;
 					tmp.ParentID = tmpg.ParentID;
@@ -1558,12 +1523,8 @@ namespace VEPROMS.CSLA.Library
 					tmp.appid = tmpg.appid;
 					tmp.roid = tableID.ToString("X4") + tmp.ID.ToString("X8");
 					tmp.children = tmpg.children;
-                    if (tmpg.ModDateTime != null)
-                    {
-                        tmp.ModDateTime = tmpg.ModDateTime;
-                    }
 
-                    int j;
+					int j;
 
 					for (j = i - 1; j >= 0 && tmp.ID < myGrp.children[j].ID; j--)
 					{
@@ -1581,13 +1542,7 @@ namespace VEPROMS.CSLA.Library
 				int slen2 = StringLength(ab, offset + 13 + slen);
 				myGrp.appid = Encoding.Default.GetString(ab, offset + 13 + slen, slen2);
 
-                //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+				_dbRoCnt++;
-                if (myGrp.value != "" && DateTime.TryParseExact(Encoding.Default.GetString(ab, offset + 14 + slen + slen2, 14), "yyyyMMddHHmmss", null, System.Globalization.DateTimeStyles.None, out DateTime dt))
-                {
-                    myGrp.ModDateTime = dt;
-                }
-
-                _dbRoCnt++;
 			}
 
 			return myGrp;
@@ -1700,7 +1655,7 @@ namespace VEPROMS.CSLA.Library
 		private void LoadChild(int rofstID, int dbiID, ROFSTLookup.rochild child)
 		{
 			//Insert Rofst Child
-			RofstChildInsert(rofstID, child.ID, child.ParentID, dbiID, child.type, child.title, child.roid, child.appid, child.value, child.ModDateTime);
+			RofstChildInsert(rofstID, child.ID, child.ParentID, dbiID, child.type, child.title, child.roid, child.appid, child.value);
 
 			//Increment RO Count if RoChild has a return value
 			if (!string.IsNullOrEmpty(child.value)) _curRoCnt++;

PROMS/VEPROMS.CSLA.Library/Extension/ItemExt.cs

@@ -1093,17 +1093,8 @@ namespace VEPROMS.CSLA.Library
 				}
 			}
 		}
-
+		// B2022-026 RO Memory Reduction code - pass in ROFstInfo
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		internal static void MyRefreshReferenceObjects(ItemInfo itemInfo, IVEDrillDownReadOnly itemParent, SectionInfo sectionInfo, DocVersionInfo docVersionInfo, ROFstInfo origROFst)
-		//Refresh at item level
-        public static void RefreshReferenceObjects(ItemInfo tmp, ROFstInfo origROFst)
-        {
-            if (tmp.MyDocVersion.DocVersionConfig.SelectedSlave <= 0)
-                MyRefreshReferenceObjects(tmp, null, tmp.GetSectionInfo(), tmp.MyDocVersion, origROFst);
-        }
-
-        // B2022-026 RO Memory Reduction code - pass in ROFstInfo
-        internal static void MyRefreshReferenceObjects(ItemInfo itemInfo, IVEDrillDownReadOnly itemParent, SectionInfo sectionInfo, DocVersionInfo docVersionInfo, ROFstInfo origROFst)
 		{
 			if (itemInfo.MyContent.ContentPartCount > 0)
 			{
@@ -1228,47 +1219,31 @@ namespace VEPROMS.CSLA.Library
 			return true;
 		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		#region Debug Code
-        public static List<ItemInfo> GetItemInfoWithChangedROs(int docversionid, int origfstid, int newfstid) 
-		{
-            List<ItemInfo> lst = new List<ItemInfo>();
 
-            foreach (DataRow r in Data_GetItemsWithChangedROs(docversionid, origfstid, newfstid).Rows)
+		//private static void ShowDifference(string oldText, string newText)
-            {
+		//{
-                using (ItemInfo itm = Get((int)r["ItemID"]))
+		//	string nt = newText.Replace(@"\u8209?", "-").Replace(@"\u160?", " ").Replace("\xA0", " ");
-				{
+		//	string ot = oldText.Replace(@"\u8209?", "-").Replace(@"\u160?", " ").Replace("\xA0", " ");
-                    lst.Add(itm);
+		//	ShowText("OldText", ot);
-				}
+		//	ShowText("NewText", nt);
-			}
+		//}
+		//private static void ShowText(string title, string newText)
+		//{
+		//	StringBuilder sb = new StringBuilder();
+		//	foreach (char c in newText)
+		//	{
+		//		if(c<' ' || c> '\x7F')
+		//			sb.Append(string.Format("\\x{0:X2}",((int) c)));
+		//		else
+		//			sb.Append(c);
+		//	}
+		//	Console.WriteLine("{0}='{1}'",title,sb.ToString());
+		//}
 
-			return lst;
+		#endregion // debug
-        }
 
-        #region Debug Code
+		internal static void SetParentSectionAndDocVersionPageNum(ItemInfo itemInfo, IVEDrillDownReadOnly itemParent, SectionInfo sectionInfo, ProcedureInfo procInfo, DocVersionInfo docVersionInfo, TransitionLookup tranLookup)
-
-        //private static void ShowDifference(string oldText, string newText)
-        //{
-        //	string nt = newText.Replace(@"\u8209?", "-").Replace(@"\u160?", " ").Replace("\xA0", " ");
-        //	string ot = oldText.Replace(@"\u8209?", "-").Replace(@"\u160?", " ").Replace("\xA0", " ");
-        //	ShowText("OldText", ot);
-        //	ShowText("NewText", nt);
-        //}
-        //private static void ShowText(string title, string newText)
-        //{
-        //	StringBuilder sb = new StringBuilder();
-        //	foreach (char c in newText)
-        //	{
-        //		if(c<' ' || c> '\x7F')
-        //			sb.Append(string.Format("\\x{0:X2}",((int) c)));
-        //		else
-        //			sb.Append(c);
-        //	}
-        //	Console.WriteLine("{0}='{1}'",title,sb.ToString());
-        //}
-
-        #endregion // debug
-
-        internal static void SetParentSectionAndDocVersionPageNum(ItemInfo itemInfo, IVEDrillDownReadOnly itemParent, SectionInfo sectionInfo, ProcedureInfo procInfo, DocVersionInfo docVersionInfo, TransitionLookup tranLookup)
 		{
 			if (itemInfo.MyContent.ContentPartCount > 0)
 			{
@@ -1915,44 +1890,10 @@ namespace VEPROMS.CSLA.Library
 					foreach (ItemInfo itemInfo in partInfo.MyItems)
 						itemInfo.SpinThroughChildren();
 		}
-
+		#endregion
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		#region LoadAtOnce
-        private static DataTable Data_GetItemsWithChangedROs(int docversionid, int origfstid, int newfstid)
+		// Method to Get Item and children
-        {
+		public static ItemInfo GetItemAndChildren(int? itemID, int? parentID)
-            try
-            {
-                using (SqlConnection cn = Database.VEPROMS_SqlConnection)
-                {
-                    using (SqlCommand cm = cn.CreateCommand())
-                    {
-                        cm.CommandType = CommandType.StoredProcedure;
-                        cm.CommandText = "getItemsWithNewROs";
-                        cm.Parameters.AddWithValue("@VersionID", docversionid);
-                        cm.Parameters.AddWithValue("@OrigFSTid", origfstid);
-                        cm.Parameters.AddWithValue("@NewFSTid", newfstid);
-                        cm.CommandTimeout = Database.DefaultTimeout;
-
-                        using (SqlDataAdapter da = new SqlDataAdapter(cm))
-                        {
-                            DataTable dt = new DataTable();
-                            da.Fill(dt);
-                            return dt;
-                        }
-                    }
-
-                }
-            }
-            catch (Exception ex)
-            {
-                Database.LogException("ItemInfoList.Data_GetItemsWithChangedROs", ex);
-                throw new DbCslaException("ItemInfoList.Data_GetItemsWithChangedROs", ex);
-            }
-        }
-
-        #endregion
-        #region LoadAtOnce
-        // Method to Get Item and children
-        public static ItemInfo GetItemAndChildren(int? itemID, int? parentID)
 		{
 			try
 			{

PROMS/VEPROMS.CSLA.Library/Extension/PdfExt.cs

@@ -61,47 +61,57 @@ namespace VEPROMS.CSLA.Library
 				throw new DbCslaException("Pdf.DataPortal_Delete", ex);
 			}
 		}
+		// used to remove word section PDFs to force ROs to be updated when printed or saved
+		public static void DeleteAllDocVersion(int versionID)
+		{
+			if (!CanDeleteObject())
+				throw new System.Security.SecurityException("User not authorized to remove a Pdf");
+			try
+			{
+				DataPortal.Delete(new VersionIDCriteria(versionID));
+			}
+			catch (Exception ex)
+			{
+				throw new DbCslaException("Error on Pdf.DeleteAllDocVersion", ex);
+			}
+		}
+		[Serializable()]
+		protected class VersionIDCriteria
+		{
+			private int _VersionID;
+			public int VersionID
+			{ get { return _VersionID; } }
+			public VersionIDCriteria(int versionID)
+			{
+				_VersionID = versionID;
+			}
+		}
+		[Transactional(TransactionalTypes.TransactionScope)]
+		private void DataPortal_Delete(VersionIDCriteria criteria)
+		{
+			if (_MyLog.IsDebugEnabled) _MyLog.DebugFormat("[{0}] Pdf.DataPortal_Delete", GetHashCode());
+			try
+			{
+				using (SqlConnection cn = Database.VEPROMS_SqlConnection)
+				{
+					using (SqlCommand cm = cn.CreateCommand())
+					{
+						cm.CommandType = CommandType.StoredProcedure;
+						cm.CommandTimeout = Database.SQLTimeout;
+						cm.CommandText = "deleteAllDocVersionPdfs";
+						cm.Parameters.AddWithValue("@VersionID", criteria.VersionID);
+						cm.ExecuteNonQuery();
+					}
+				}
+			}
+			catch (Exception ex)
+			{
+				if (_MyLog.IsErrorEnabled) _MyLog.Error("Pdf.DataPortal_Delete", ex);
+				_ErrorMessage = ex.Message;
+				throw new DbCslaException("Pdf.DataPortal_Delete", ex);
+			}
+		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
-        // used to remove word section PDFs to force ROs to be updated when printed or saved
-        public static void DeleteDocVersionPDFsWithNewROs(int versionID, int origfstid, int newfstid)
-        {
-            if (!CanDeleteObject())
-                throw new System.Security.SecurityException("User not authorized to remove a Pdf");
-            try
-            {
-                DeleteWithNewROs(versionID, origfstid, newfstid);
-            }
-            catch (Exception ex)
-            {
-                throw new DbCslaException("Error on Pdf.DeleteAllDocVersion", ex);
-            }
-        }
-
-        static private void DeleteWithNewROs(int docversionID, int origfstid, int newfstid)
-        {
-            try
-            {
-                using (SqlConnection cn = Database.VEPROMS_SqlConnection)
-                {
-                    using (SqlCommand cm = cn.CreateCommand())
-                    {
-                        cm.CommandType = CommandType.StoredProcedure;
-                        cm.CommandTimeout = Database.SQLTimeout;
-                        cm.CommandText = "deleteDocVersionPdfsWithNewROs";
-                        cm.Parameters.AddWithValue("@VersionID", docversionID);
-                        cm.Parameters.AddWithValue("@OrigFSTid", origfstid);
-                        cm.Parameters.AddWithValue("@NewFSTid", newfstid);
-                        cm.ExecuteNonQuery();
-                    }
-                }
-            }
-            catch (Exception ex)
-            {
-                if (_MyLog.IsErrorEnabled) _MyLog.Error("Pdf.DeleteWithNewROs", ex);
-                throw new DbCslaException("Pdf.DeleteWithNewROs", ex);
-            }
-        }
 	}
 	public partial class PdfInfo
 	{

PROMS/VEPROMS.CSLA.Library/Extension/ROFSTExt.cs

@@ -225,54 +225,41 @@ namespace VEPROMS.CSLA.Library
 			}
 		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		// B2022-026 RO Memory Reduction code - pass in the ROFstInfo
-        // Refresh at item level
+		public static int RefreshROFst(DocVersionInfo dvi, ROFstInfoProgressBarRefresh myProgressBarRefresh, System.Windows.Forms.TextBox tbStatus, ROFstInfo origROFst)
-        public static int RefreshROFstAtItemLevel(DocVersionInfo dvi, ROFstInfoProgressBarRefresh myProgressBarRefresh, System.Windows.Forms.TextBox tbStatus, ROFstInfo localROFst, int origfstid, int newfstid)
+		{
-        {
+			int fixedROs = 0;
-            int fixedROs = 0;
 
-            if (dvi.DocVersionConfig.SelectedSlave <= 0)
+			if (dvi.DocVersionConfig.SelectedSlave <= 0)
-            {
+			{
-                // remove word section PDFs to force update of RO values when printed
+				myProgressBarRefresh(1, 100, "Update MS Word ROs");
-                myProgressBarRefresh(1, 100, "Updating MS Word ROs In Progress");
-                Pdf.DeleteDocVersionPDFsWithNewROs(dvi.VersionID, origfstid, newfstid);
 
-                //Loop Through ROs in this docversion that are different than the previous fst
+				Pdf.DeleteAllDocVersion(dvi.VersionID); // remove word section PDFs to force update of RO values when printed
-                myProgressBarRefresh(50, 100, "Updating ROs In PROMS Steps In Progress");
+				int i = 0;
-                int i = 0;
-				List<ItemInfo> list = ItemInfo.GetItemInfoWithChangedROs(dvi.VersionID, origfstid, newfstid);
-                foreach (ItemInfo itm in list)
-                {
-                    DateTime start = DateTime.Now;
 
-                    ItemInfo.ResetROCounters();
+				foreach (ProcedureInfo proc in dvi.Procedures)
-                    myProgressBarRefresh(++i, list.Count, string.Format("{0} ({1}/{2} ROs {3})", itm.MyProcedure.DisplayNumber, i, list.Count, fixedROs));
+				{
-                    ItemInfo.RefreshReferenceObjects(itm, localROFst);
+					DateTime start = DateTime.Now;
-                    fixedROs += ItemInfo.ROFixCount;
 
-                    TimeSpan ts = DateTime.Now - start;
+					ProcedureInfo.ResetROCounters();
+					myProgressBarRefresh(++i, dvi.Procedures.Count, string.Format("{0} ({1}/{2} ROs {3})", proc.DisplayNumber, i, dvi.Procedures.Count, fixedROs));
+					ProcedureInfo.RefreshReferenceObjects(proc, origROFst);
+					fixedROs += ProcedureInfo.ROFixCount;
 
-                    if (tbStatus != null)
+					TimeSpan ts = DateTime.Now - start;
-                        tbStatus.AppendText(string.Format("Procedure:  {1}{0}, Checked {2} Referenced Objects{0}, Fixed {3} Referenced Objects{0}, Elapsed Seconds:{4}{0}{0}", Environment.NewLine, itm.MyProcedure.DisplayNumber, ItemInfo.ROCheckCount, ItemInfo.ROFixCount, ts.TotalSeconds));
-                }
 
-				//Update the DocVersion Associations to link to the new RO FST id and current date/time
+					if (tbStatus != null)
-                using (DocVersion dv = DocVersion.Get(dvi.VersionID))
+						tbStatus.AppendText(string.Format("Procedure:  {1}{0}, Checked {2} Referenced Objects{0}, Fixed {3} Referenced Objects{0}, Elapsed Seconds:{4}{0}{0}", Environment.NewLine, proc.DisplayNumber, ProcedureInfo.ROCheckCount, ProcedureInfo.ROFixCount, ts.TotalSeconds));
-                {
+				}
-                    if (dvi.DocVersionAssociations[0].MyROFst.ROFstID != newfstid)
+			}
-                    {
-                        dv.DocVersionAssociations[0].MyROFst = localROFst.GetJustROFst();
-                        SetAssociationLastCompleted(dv, DateTime.Now.ToString());
-                    }
-                }
-            }
 
-            return fixedROs;
+			return fixedROs;
-        }
+		}
 
-        //C2022-028 for Admin tool to check for bad RO links
+
-        //B2022-144 we now loop through checked procedures list from Admin Tools and call this method for each procedure we want to process
+		//C2022-028 for Admin tool to check for bad RO links
-        public static int CheckROLinksInThisProcedure(ProcedureInfo proc,  System.Windows.Forms.TextBox tbStatus)
+		//B2022-144 we now loop through checked procedures list from Admin Tools and call this method for each procedure we want to process
+		public static int CheckROLinksInThisProcedure(ProcedureInfo proc,  System.Windows.Forms.TextBox tbStatus)
 		{
 			int FoundBadROLinks = 0;
 			DocVersionInfo dvi = DocVersionInfo.Get(proc.MyDocVersion.VersionID);
@@ -304,6 +291,34 @@ namespace VEPROMS.CSLA.Library
 			return Environment.GetFolderPath(Environment.SpecialFolder.MyDocuments) + @"\VEPROMS\ROUpdateReport_" + ValidFileName.FixFileName(dvi.MyFolder.Name.Replace(" ", "_") + "_" + DateTime.Now.ToString("MM-dd-yyyy_HH-mm-ss") + ".txt");
 		}
 
+		// B2022-026 RO Memory Reduction code - moved the call to UpdateROFst() to before we call RefreshROFst
+		// so that we used the correct ROFstInfo which as the needed event methods set when updated RO Table types
+		public static ROFst RefreshROFst(DocVersion docver, ROFstInfo origROFst, ROFstInfoProgressBarRefresh myProgressBarRefresh, System.Windows.Forms.TextBox tbStatus)
+		{
+			ROFst rofst = null;
+
+			rofst = docver.DocVersionAssociations[0].MyROFst;
+
+			DocVersionInfo dvi = DocVersionInfo.Get(docver.VersionID);
+
+			SetAssociationLastCompleted(docver, string.Empty);
+			int fixedROs = RefreshROFst(dvi, myProgressBarRefresh, tbStatus, origROFst);
+			SetAssociationLastCompleted(docver, DateTime.Now.ToString());  // RO Update completed successfully and un-interrupted, save the date/time to the Doc Version Association config
+
+			myProgressBarRefresh(100, 100, "RO Update Complete"); // update the progress bar
+			System.Windows.Forms.Application.DoEvents();
+
+			// pop up a message window telling the user the RO Update has completed and how many ROs were updated
+			// If we are updating RO from the Admin Tools (from the V button) and we are updating more than on procedure set, then just append the "RO Update Complete" text
+			// To the MessageList.  Once completed will all procedure sets, Admin Tools will display one message box with all the results in it.
+			if (MessageList == null)
+				FlexibleMessageBox.Show(fixedROs == 0 ? "No ROs Required Updating" : string.Format("{0} ROs Updated for {1}", fixedROs, dvi.MyFolder.Name), "RO Update Complete");
+			else
+				MessageList.AppendLine((fixedROs == 0 ? "No ROs Required Updating for " : string.Format("{0} ROs Updated for ", fixedROs)) + dvi.MyFolder.Name);
+
+			return rofst;
+		}
+
 		/// <summary>
 		/// Updates an ro.fst into a sql database.  
 		/// </summary>
@@ -655,7 +670,188 @@ namespace VEPROMS.CSLA.Library
 			return sb.ToString();
 		}
 
-        private static DateTime ShowDuration(DateTime dtLast, string message)
+		private static void UpdateROValuesText(ROFstInfo origROFstInfo, ROFst newROFst, DocVersionInfo dvi, ROFstInfoProgressBarRefresh myProgressBarRefresh, List<string> MyChangedFigureROIDs)
+		{
+			if (myProgressBarRefresh != null) myProgressBarRefresh(0, 100, "Update Ro Values");
+
+			string versionList = dvi.VersionID.ToString();
+
+			ROFSTLookup origLookup = new ROFSTLookup(origROFstInfo.ROFstID, dvi);
+			ROFSTLookup newLookup = new ROFSTLookup(newROFst.ROFstID, dvi);
+
+			List<string> delList = new List<string>();
+			List<string> chgList = newLookup.GetValueDifferences(origROFstInfo.ROFstID, ref delList);
+
+			// Any figures which have been changed will be included in the list of values that have changed.
+			if (MyChangedFigureROIDs != null)
+			{
+				foreach (string roid in MyChangedFigureROIDs)
+				{
+					if (!chgList.Contains(roid)) chgList.Add(roid);
+				}
+			}
+
+			string RoidList = GetRoidList(newROFst.RODbID, chgList);
+
+			if (myProgressBarRefresh != null) myProgressBarRefresh(0, chgList.Count, "Getting List of ROs Used");
+			List<string> activeRoids = BuildActiveROIDsForRoUsages12(RoidList, versionList);
+
+			if (myProgressBarRefresh != null) myProgressBarRefresh(0, chgList.Count, "Updating RO Values");
+			int iCount = 0;
+
+			if (activeRoids.Count > 0)
+			{
+				foreach (string chg in chgList)
+				{
+					if (myProgressBarRefresh != null) myProgressBarRefresh(++iCount, chgList.Count, "Updating RO Values");
+					if (activeRoids.Contains(chg.Substring(0, 12)))
+					{
+						ROFSTLookup.rochild roch = newLookup.GetRoChild(chg);
+						string desc = string.Format("Change in RO Values: Old value = {0}, New value = {1}", origLookup.GetRoChild(chg).value, roch.value);
+
+						// roid's are stored in database as 16 characters long in the RoUsages table.  They may be stored as 12 characters in the ro.fst.
+						// string padroid = chg.Length <= 12 ? chg + "0000" : chg;
+						// B2022-088: Find Doc Ro button not working in Word Sections
+						string padroid = ROFSTLookup.FormatRoidKey(chg, true);
+
+						using (RoUsageInfoList affected = RoUsageInfoList.GetAffected(origROFstInfo.MyRODb.RODbID, padroid, desc, "Changed", versionList))
+						{
+							foreach (RoUsageInfo roUsg in affected)
+							{
+								using (Content content = Content.Get(roUsg.MyContent.ContentID))
+								{
+									foreach (ItemInfo ii in roUsg.MyContent.ContentItems)
+									{
+										string val = newLookup.GetTranslatedRoValue(padroid, ii.ActiveFormat.PlantFormat.FormatData.SectData.ConvertCaretToDelta, ii.ActiveFormat.PlantFormat.FormatData.SectData.UseTildaPoundCharsForSuperSubScriptInROValues, false, ii);
+										content.FixContentText(roUsg, val, roch.type, origROFstInfo, true);
+
+										if (content.IsDirty)
+										{
+											// Update UserID and DTS when RO Value is updated.
+											content.UserID = Volian.Base.Library.VlnSettings.UserID;
+											content.DTS = DateTime.Now;
+											content.Save();
+
+											if (content.MyGrid != null)
+											{
+												GridInfo.Refresh(content.MyGrid);
+											}
+										}
+									}
+								}
+							}
+						}
+					}
+				}
+			}
+
+			if (myProgressBarRefresh != null) myProgressBarRefresh(0, chgList.Count, "Getting List of ROs Used");
+			List<string> activeDRoids = BuildActiveROIDsForDRoUsages12(RoidList, versionList);
+			iCount = 0;
+
+			if (activeDRoids.Count > 0)
+			{
+				foreach (string chg in chgList)
+				{
+					// B2022-088: Find Doc Ro button not working in Word Sections
+					// string padroid = chg.Length <= 12 ? chg + "0000" : chg;
+					string padroid = ROFSTLookup.FormatRoidKey(chg, true);
+
+					if (myProgressBarRefresh != null) myProgressBarRefresh(++iCount, chgList.Count, "Updating RO Values");
+
+					if (activeDRoids.Contains(chg.Substring(0, 12)))
+					{
+						ROFSTLookup.rochild roch = newLookup.GetRoChild(chg);
+
+						string desc = string.Format("Change in RO Values: Old value = {0}, New value = {1}", origLookup.GetRoChild(chg).value, roch.value);
+
+						// roid's are stored in database as 16 characters long in the rousages table.  They may be stored as 12 characters in the ro.fst.
+						using (DROUsageInfoList affected = DROUsageInfoList.GetAffected(origROFstInfo.MyRODb.RODbID, padroid, desc, "Changed", versionList))
+						{
+							foreach (DROUsageInfo droUsg in affected)
+							{
+								Pdf.DeleteAll(droUsg.DocID);
+							}
+						}
+					}
+				}
+			}
+
+			iCount = 0;
+			string RoidDelList = GetRoidList(newROFst.RODbID, delList);
+			if (myProgressBarRefresh != null) myProgressBarRefresh(0, chgList.Count, "Getting List of ROs Used");
+			activeRoids = BuildActiveROIDsForRoUsages12(RoidDelList, versionList);
+
+			if (activeRoids.Count > 0)
+			{
+				foreach (string del in delList)
+				{
+					// B2022-088: Find Doc Ro button not working in Word Sections
+					//string padroiddel = del.Length <= 12 ? del + "0000" : del;
+					string padroiddel = ROFSTLookup.FormatRoidKey(del, true);
+
+					if (myProgressBarRefresh != null) myProgressBarRefresh(++iCount, chgList.Count, "Removing Old RO Values");
+					string desc = string.Format("Deleted RO: Value = {0}", origLookup.GetRoChild(del).value);
+
+					if (activeRoids.Contains(del.Substring(0, 12).ToUpper()))
+					{
+						using (RoUsageInfoList affected = RoUsageInfoList.GetAffected(origROFstInfo.MyRODb.RODbID, padroiddel, desc, "Deleted", versionList))
+						{
+							foreach (RoUsageInfo roUsg in affected)
+							{
+								using (Content content = Content.Get(roUsg.MyContent.ContentID))
+								{
+									content.FixContentText(roUsg, "?", 0, origROFstInfo);
+									if (content.IsDirty)
+									{
+										// Update UserID and DTS when RO Value is updated.
+										content.UserID = Volian.Base.Library.VlnSettings.UserID;
+										content.DTS = DateTime.Now;
+										content.Save();
+									}
+								}
+							}
+						}
+					}
+				}
+			}
+
+			if (myProgressBarRefresh != null) myProgressBarRefresh(0, chgList.Count, "Getting List of ROs Used");
+			activeDRoids = BuildActiveROIDsForDRoUsages12(RoidDelList, versionList);
+			iCount = 0;
+
+			if (activeDRoids.Count > 0)
+			{
+				foreach (string del in delList)
+				{
+					
+					// B2022-088: Find Doc Ro button not working in Word Sections
+					string padroiddel = ROFSTLookup.FormatRoidKey(del, true);
+					if (myProgressBarRefresh != null) myProgressBarRefresh(++iCount, chgList.Count, "Removing Old RO Values");
+
+					string desc = string.Format("Deleted RO: Value = {0}", origLookup.GetRoChild(del).value);
+
+					// If there's an issue then maybe try getting the RoChild with the Padded roid instead
+					//string desc = string.Format("Deleted RO: Value = {0}", origLookup.GetRoChild(padroiddel).value);
+
+					if (activeDRoids.Contains(del.Substring(0, 12).ToUpper()))
+					{
+						using (DROUsageInfoList Daffected = DROUsageInfoList.GetAffected(origROFstInfo.MyRODb.RODbID, padroiddel, desc, "Deleted", versionList))
+						{
+							foreach (DROUsageInfo droUsg in Daffected)
+							{
+								if (myProgressBarRefresh != null) myProgressBarRefresh(++iCount, chgList.Count, "Removing Old RO Values");
+								Pdf.DeleteAll(droUsg.DocID);
+							}
+						}
+					}
+				}
+			}
+
+			if (myProgressBarRefresh != null) myProgressBarRefresh(100, 100, "RO Values Updated");
+		}
+
+		private static DateTime ShowDuration(DateTime dtLast, string message)
 		{
 			DateTime dtNext = DateTime.Now;
 			Console.WriteLine("{0,10:#####0.00},'{1}'", TimeSpan.FromTicks(dtNext.Ticks - dtLast.Ticks).TotalSeconds, message);

PROMS/Volian.Controls.Library/AnnotationDetails.cs

@@ -1,16 +1,17 @@
+using JR.Utils.GUI.Forms;
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
-using System.Drawing;
 using System.Data;
+using System.Diagnostics;
+using System.Drawing;
 using System.Text;
+using System.Text.RegularExpressions;
 using System.Windows.Forms;
+using System.Xml;
 using VEPROMS.CSLA.Library;
 using Volian.Base.Library;
 using Volian.Pipe.Library;
-using System.Xml;
-using System.Diagnostics;
-using JR.Utils.GUI.Forms;
 
 namespace Volian.Controls.Library
 {
@@ -220,7 +221,10 @@ namespace Volian.Controls.Library
 			{
 				if (!DesignMode) // B2019-043 need to check if we are just saving changes to the user interface
 				{
-					rtxbComment.Text = value;
+					if (value != null)
+					{
+						rtxbComment.Text = Regex.Replace(value, @"\\u([0-9]{1,4})\?", m => Convert.ToChar(int.Parse(m.Groups[1].Value)).ToString());
+					}
 					if (rtxbComment.Text != string.Empty)
 						rtxbComment.SelectionStart = rtxbComment.TextLength; // position cursor to end of text
 				}

PROMS/Volian.Controls.Library/DisplayRO.cs

@@ -59,11 +59,11 @@ namespace Volian.Controls.Library
 		private DisplayTags displayTags;
 
 
-        #endregion
+		#endregion
 
-        #region Properties
+		#region Properties
 
-        public ProgressBarItem ProgressBar
+		public ProgressBarItem ProgressBar
 		{
 			get { return _progressBar; }
 			set { _progressBar = value; }
@@ -89,10 +89,7 @@ namespace Volian.Controls.Library
 					(_myRTB != null && (_myRTB.IsRoTable != lastRTBwasROTable || _myRTB.IsRoFigure != lastRTBwasROFigure)); }
 		}
 
-		//This flag will be set when the docversion gets updated to let it know that the full tree needs a refresh
+		public ROFstInfo MyROFST
-		private bool changedDocVersion = false;
-
-        public ROFstInfo MyROFST
 		{
 			get { return _myROFST; }
 			set
@@ -131,22 +128,16 @@ namespace Volian.Controls.Library
 				// B2022-135 Submitted for Admin Tools (Check RO Links tool)
 				if (_docVersionInfo == null || _docVersionInfo != value || _docVersionInfo.VersionID != value.VersionID)
 				{
-					//Set flag for modified the docversion - should check for new ROs
+					_docVersionInfo = value;
-                    if (_docVersionInfo?.VersionID != value?.VersionID)
-                    {
-                        changedDocVersion = true;
-                    }
-
-                    _docVersionInfo = value;
 
 					if (_myRTB != null && (_docVersionInfo == null || _docVersionInfo.VersionID != _myRTB.MyDVI.VersionID))
 					{
 						_docVersionInfo = _myRTB.MyDVI;
 					}
-                }
+				}
 
-                // B2022-123: RO Tab Treeview not showing correct RO values when switching between procedures.
+				// B2022-123: RO Tab Treeview not showing correct RO values when switching between procedures.
-                _currDocVersionID = (_docVersionInfo != null) ? (int?)_docVersionInfo.VersionID : null;
+				_currDocVersionID = (_docVersionInfo != null) ? (int?)_docVersionInfo.VersionID : null;
 			}
 		}
 
@@ -210,11 +201,7 @@ namespace Volian.Controls.Library
 					// B2022-098: [JPR] ROs not being resolved in Word Sections
 					CurROLink = null;
 					_savCurROLink = null;
-
+				}
-                    //Clear flag for modified the docversion  (if should check for new ROs)
-					//clearing this flag here prevents it from checking twice
-                    changedDocVersion = false;
-                }
 				else if(_myRTB != value)
 				{
 					if (_myRTB != null)
@@ -234,9 +221,9 @@ namespace Volian.Controls.Library
 						CurROLink = null;
 						_savCurROLink = null;
 					}
-                    // B2023-004 assign the doc version info associated with the current RTB (rich text box)
+					// B2023-004 assign the doc version info associated with the current RTB (rich text box)
-                    //           this fixes an issue where unit designators could not be linked in the step edit (BNPP data)
+					//           this fixes an issue where unit designators could not be linked in the step edit (BNPP data)
-                    MyDvi = _myRTB.MyItemInfo.MyDocVersion;
+					MyDvi = _myRTB.MyItemInfo.MyDocVersion;
 
 					MyROFST = (_myRTB.MyItemInfo.MyDocVersion.DocVersionAssociationCount > 0) ? _myRTB.MyItemInfo.MyDocVersion.DocVersionAssociations[0].MyROFst : null;
 				}
@@ -264,13 +251,11 @@ namespace Volian.Controls.Library
 			set { _myUserInfo = value; }
 		}
 
-        public SessionInfo MySessionInfo { get; set; }
+		#endregion
 
-        #endregion
+		#region Constructors
 
-        #region Constructors
+		public DisplayRO()
-
-        public DisplayRO()
 		{
 			InitializeComponent();
 
@@ -292,62 +277,48 @@ namespace Volian.Controls.Library
 			// Initialize the DisplayTags object
 			displayTags = new DisplayTags();
 
+			_progressBar = null;
 		}
 
-        #endregion
+		#endregion
-        #region Event Handlers
-        #region (Progress Bar)
 
-        private string InitialProgressBarMessage
+		#region Event Handlers
-        {
-            set
-            {
-                if (ProgressBar == null) return;
-                ProgressBar.Value = 0;
-                ProgressBar.Maximum = 100;
-                ProgressBar.Text = value;
-                Application.DoEvents();
-            }
-        }
-        private string FinalProgressBarMessage
-        {
-            set
-            {
-                if (ProgressBar == null) return;
-                ProgressBar.Value = 100;
-                ProgressBar.Maximum = 100;
-                ProgressBar.Text = value;
-                Application.DoEvents();
-            }
-        }
-        private void DoProgressBarRefresh(int value, int max, string text)
-        {
-            if (ProgressBar == null) return;
-            ProgressBar.Maximum = max;
-            ProgressBar.Value = value;
-            ProgressBar.Text = text;
-            Application.DoEvents();
-        }
 
-        #endregion
+		#region (Progress Bar)
 
-        #region (RTB)
+		private void ProgressBar_Initialize(int max, string desc)
+		{
+			if (_progressBar != null)
+			{
+				_progressBar.Maximum = max;
+				_progressBar.Text = desc;
+				_progressBar.TextVisible = true;
+			}
+		}
 
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+		private void ProgressBar_SetValue(int curval)
-        // DisplayTab was changed
+		{
-        // need to clear the RTB selected
+			if (_progressBar != null)
-        // without resetting the DVI or FST
+			{
-        // (those will be set manually / individually)
+				_progressBar.Value = curval;
-        // this will prevent DVI and FST from changing to null then back again
+			}
-        // which will trigger reloads
+		}
-        public void ClearRTB()
-        {
-            _myRTB = null;
-            CurROLink = null;
-            _savCurROLink = null;
-        }
 
-        public void MyRTB_LinkChanged(object sender, StepPanelLinkEventArgs args)
+		private void ProgressBar_Clear()
+		{
+			if (_progressBar != null)
+			{
+				_progressBar.Text = string.Empty;
+				_progressBar.Maximum = 0;
+				_progressBar.Value = 0;
+			}
+		}
+
+		#endregion
+
+		#region (RTB)
+
+		public void MyRTB_LinkChanged(object sender, StepPanelLinkEventArgs args)
 		{
 			CurROLink = null;
 			if (MyRTB.MyLinkText != null) CurROLink = args.MyLinkText.MyRoUsageInfo;
@@ -594,102 +565,55 @@ namespace Volian.Controls.Library
 			}
 		}
 
-        private void lbFound_SelectedValueChanged(object sender, EventArgs e)
+		private void lbFound_SelectedValueChanged(object sender, EventArgs e)
-        {
-            if (lbFound.Visible && lbFound.SelectedIndex >= 0 && lbFound.SelectedValue != null)
-            {
-                ExpandNode(Convert.ToString(lbFound.SelectedValue));
-            }
-        }
-
-        #endregion
-
-        #region (ROUpdate)
-        public List<string> roFstInfo_ROTableUpdate(object sender, ROFstInfoROTableUpdateEventArgs args)
-        {
-            return VlnFlexGrid.ROTableUpdate(sender, args);
-        }
-
-        #endregion
-
-        #endregion
-
-        #region Public Methods
-
-        //C2026-008 Re-Architect RO.FST to include RO Modification date/time
-        // changed to return true if the RO FST got updated
-        public bool LoadTree(bool forceReload = false)
 		{
-			bool updatedROs = false;
+			if (lbFound.Visible && lbFound.SelectedIndex >= 0 && lbFound.SelectedValue != null)
+			{
+				ExpandNode(Convert.ToString(lbFound.SelectedValue));
+			}
+		}
 
+		#endregion
+
+		#endregion
+
+		#region Public Methods
+
+		public void LoadTree(bool forceReload = false)
+		{
 			if (MyROFST == null)
 			{
 				tvROFST.Nodes.Clear();
 				lbFound.Visible = false;
-				return updatedROs;
+				return;
 			}
 
-			if (changedDocVersion || forceReload || RoTreeNeedsReloaded || tvROFST.Nodes == null || tvROFST.Nodes.Count <= 0)
+			if (forceReload || RoTreeNeedsReloaded || tvROFST.Nodes == null || tvROFST.Nodes.Count <= 0)
 			{
-                //C2026-008 Re-Architect RO.FST to include RO Modification date/time
+				//B2025-008
-				//check if newer data - if there is, check if user wants to update data
+				//in cases where a RO table is clicked on
-                int fstid = MyROFSTLookup.GetNewerFSTID();
+				//it will refresh the list
-				int origfstid = MyROFSTLookup.RofstID;
+				//temp store what the ROID was before the list refreshes
+				//so can go to it after the refresh
+				string tmpROID = CurROLink?.ROID;
 
-                if (_docVersionInfo != null && fstid != -1 && fstid != MyROFSTLookup.RofstID)
+				ROFSTLookup.rodbi[] dbs = MyROFSTLookup.GetRODatabaseList(true);
-				{
-                    string message = string.Empty;
-                    if (!MySessionInfo.CanCheckOutItem(_docVersionInfo.VersionID, CheckOutType.DocVersion, ref message))
-                    {
-                        FlexibleMessageBox.Show(this, message, "Working Draft Has Items Already Checked Out", MessageBoxButtons.OK, MessageBoxIcon.Warning);
-                        FinalProgressBarMessage = "Cannot check-out Working Draft";
-                    }
-                    else if (changedDocVersion && MessageBox.Show($"There exists a newer ROFST for this RO database that was loaded for other sets.\r\n\r\nDo you want to update this set's ROs to be consistent/use the latest loaded ROFST?", "Load ROs", MessageBoxButtons.YesNo) == DialogResult.Yes)
-					{
-
-                         InitialProgressBarMessage = "Updating ROs";
-
-                         ROFstInfo roFstInfo = ROFstInfo.Get(fstid);
-                         roFstInfo.ROTableUpdate += new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
-                         ROFstInfo.RefreshROFstAtItemLevel(_docVersionInfo, DoProgressBarRefresh, null, roFstInfo, origfstid, fstid);
-                         roFstInfo.ROTableUpdate -= new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
-
-						 Application.DoEvents();
-
-                         FinalProgressBarMessage = "ROs values updated";
-                         MyROFST = roFstInfo;
-
-                         updatedROs = true;
-                    }
-                }
-
-                //B2025-008
-                //in cases where a RO table is clicked on
-                //it will refresh the list
-                //temp store what the ROID was before the list refreshes
-                //so can go to it after the refresh
-                string tmpROID = CurROLink?.ROID;
-
-				ROFSTLookup.rodbi[] dbs = MyROFSTLookup?.GetRODatabaseList(true);
 
 				// B2022-123: RO Tab Treeview not showing correct RO values when switching between procedures.
 				// Added optional parameter "forceReload" and cleared out any existing nodes before reloading the tree
 				// the clear nodes code below has to be after the GetRODatabaseList database call because of races conditions in the code
 				tvROFST.Nodes.Clear();
 				ResetSearch(); //B2023-050 need to reset the SaveRO, and any RO info that was selected last time the tree was loaded
-				if (dbs != null)
+				for (int i = 0; i < dbs.Length; i++)
 				{
-                    for (int i = 0; i < dbs.Length; i++)
+					ROFSTLookup.rodbi db = dbs[i];
-                    {
-                        ROFSTLookup.rodbi db = dbs[i];
 
-                        TreeNode tn = new TreeNode(db.dbiTitle);
+					TreeNode tn = new TreeNode(db.dbiTitle);
-                        tn.Tag = db;
+					tn.Tag = db;
-                        tvROFST.Nodes.Add(tn);
+					tvROFST.Nodes.Add(tn);
 
-                        AddDummyGroup(db, tn);
+					AddDummyGroup(db, tn);
-                    }
+				}
-                }
 
 				_currRofstID = (IsRofstValid) ? (int?)_myROFST.ROFstID : null;
 				_currDocVersionID = null;
@@ -697,13 +621,7 @@ namespace Volian.Controls.Library
 				if(_docVersionInfo != null) _currDocVersionID = (int?)_docVersionInfo.VersionID;
 
 				if (tmpROID != null) ExpandNode(ROFSTLookup.FormatRoidKey(tmpROID, true));
-
+			}
-				//doc version would have updated (if needed) so reset flag
-				if (_progressBar?.Text != "Cannot check-out Working Draft")
-                {
-                    changedDocVersion = false;
-                }
-            }
 
 			var unitInfoNode = tvROFST.Nodes.Cast<TreeNode>().Where(x => x.Text == "Unit Information").FirstOrDefault();
 
@@ -754,9 +672,7 @@ namespace Volian.Controls.Library
 			}
 
 			_curROTypeFilter = _roTypeFilter;
-
+		}
-            return updatedROs;
-        }
 
 		public void SetFindDocROButton(bool enabled)
 		{

PROMS/Volian.Controls.Library/StepTabRibbon.cs

@@ -3556,10 +3556,7 @@ namespace Volian.Controls.Library
 			MyEditItem.SaveContents();
 			using (DocVersion dv = DocVersion.Get(Mydvi.VersionID))
 			{
-				//must get id before ROFST gets updated so know what to refresh later
+				swROUpdate = new System.IO.StreamWriter(ROFstInfo.ROUpdateResultsPath(Mydvi));
-				int origfstid = roFstInfo.ROFstID;
-
-                swROUpdate = new System.IO.StreamWriter(ROFstInfo.ROUpdateResultsPath(Mydvi));
 				// B2022-026 RO Memory Reduction code - first load the new ro.fst so that we can assign the ROTableUpdate event to the correct roFstInfo
 				if (dv.ROfstLoadingFigures || dv.NewerRoFst)    // B2017-125 see if loading figures was completed
 				{
@@ -3569,8 +3566,8 @@ namespace Volian.Controls.Library
 				}
 				roFstInfo.ROTableUpdate += new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 				ContentInfo.StaticContentInfoChange += ContentInfo_StaticContentInfoChange; // write changes to a text file
-				ROFstInfo.RefreshROFstAtItemLevel(DocVersionInfo.Get(dv.VersionID), DoProgressBarRefresh, null, roFstInfo, origfstid, roFstInfo.ROFstID);         
+				ROFst newrofst = ROFstInfo.RefreshROFst(dv, roFstInfo, DoProgressBarRefresh, null);
-                swROUpdate.Close();
+				swROUpdate.Close();
 				ContentInfo.StaticContentInfoChange -= ContentInfo_StaticContentInfoChange;
 				roFstInfo.ROTableUpdate -= new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 				StepPanelTabDisplayEventArgs args = new StepPanelTabDisplayEventArgs("DisplayROUpdateROFST");

PROMS/Volian.Controls.Library/vlnTreeView.cs

@@ -2465,11 +2465,7 @@ namespace Volian.Controls.Library
 			}
 			Cursor = Cursors.WaitCursor;    // C2023-002: move wait cursor after check out error
 			int ownerid = MySessionInfo.CheckOutItem(MyDVI.VersionID, CheckOutType.DocVersion);
-
+			using (DocVersion dv = DocVersion.Get(MyDVI.VersionID))
-            //must get id before ROFST gets updated so know what to refresh later
-            int origfstid = roFstInfo.ROFstID;
-
-            using (DocVersion dv = DocVersion.Get(MyDVI.VersionID))
 			{
 				swROUpdate = new System.IO.StreamWriter(ROFstInfo.ROUpdateResultsPath(MyDVI)); // RO changes placed in file in the Documents\VEPROMS folder
 																							   // B2022-026 RO Memory Reduction code - first load the new ro.fst so that we can assign the ROTableUpdate event to the correct roFstInfo
@@ -2481,8 +2477,8 @@ namespace Volian.Controls.Library
 				}
 				roFstInfo.ROTableUpdate += new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 				ContentInfo.StaticContentInfoChange += ContentInfo_StaticContentInfoChange; // write changes to a text file
-                ROFstInfo.RefreshROFstAtItemLevel(DocVersionInfo.Get(dv.VersionID), DoProgressBarRefresh, null, roFstInfo, origfstid, roFstInfo.ROFstID);
+				ROFst newrofst = ROFstInfo.RefreshROFst(dv, roFstInfo, DoProgressBarRefresh, null);
-                swROUpdate.Close();
+				swROUpdate.Close();
 				ContentInfo.StaticContentInfoChange -= ContentInfo_StaticContentInfoChange;
 				roFstInfo.ROTableUpdate -= new ROFstInfoROTableUpdateEvent(roFstInfo_ROTableUpdate);
 				OnTabDisplay(this, new StepPanelTabDisplayEventArgs("DisplayROUpdateROFST"));