Changeset 1071 for Scheduling/trunk/cs/bsdx0200GUISourceCode/CGDocumentManager.cs

Timestamp:

Jan 19, 2011, 9:02:59 AM (13 years ago)

Author:

Sam Habiel

Message:

CGDocument:

• Better constructor to set default values.
• Removed OnNewDocument and used constructor plus extra arguments instead.
• Added IsRefreshNeeded method to see if we need to get data from server.

CGDocumentManager:

• Refactored OnNewDocument out (was going to use Application.Run with the CGView form as argument, that didn't work for re-logging in)
• Refactored all ChangeServer and ChangeLogin handlers b/c they got broken with previous work.

CGView:

• this.Activate now in Load to show the form if you use Application.Run(view)
• Added shortcuts for chaning Server, Login, Division
• Position the Grid when doing OpenSelectedSchedule to start at the right day.
• Used IsRefreshNeeded to selectively load the splash screen and contact the database for refresh then close splash screen in RequestRefreshGrid.

File:

• Scheduling/trunk/cs/bsdx0200GUISourceCode/CGDocumentManager.cs (modified) (11 diffs)

Scheduling/trunk/cs/bsdx0200GUISourceCode/CGDocumentManager.cs

@@ -181 +181 @@
             _current = new CGDocumentManager();
 
-            //Get command line options; store in private variables
+            //Get command line options; store in private class wide variables
             var opset = new OptionSet() {
                 { "s=", s => _current.m_Server = s },
@@ -191 +191 @@
 
             opset.Parse(args);
-
             
+            //Init app
             bool isEverythingOkay = _current.InitializeApp();
 
+            //if an error occurred, break out.
             if (!isEverythingOkay) return;
 
             //Create the first empty document
+            //A document holds the resources, appointments, and availabilites
             //SAM: Good place for break point
             CGDocument doc = new CGDocument();
             doc.DocManager = _current;
-            doc.OnNewDocument();
+
+            //Create new View
+            //A view is a specific arrangement of appointments and availabilites that constitute a document
+            CGView view = new CGView();
+            view.InitializeDocView(doc, _current, doc.StartDate, doc.Appointments, _current.WindowText);
+
+            //Handle BMX Event
             Application.DoEvents();
+
+            //Application wide error handler for unhandled errors
+            Application.ThreadException += new ThreadExceptionEventHandler(App_ThreadException);
+
 #if TRACE
             DateTime EndLoadTime = DateTime.Now;
@@ -208 +220 @@
             Debug.Write("Load Time for GUI is " + LoadTime.Seconds + " s & " + LoadTime.Milliseconds + " ms\n");
 #endif
-            //Application wide error handler for unhandled errors
-            Application.ThreadException += new ThreadExceptionEventHandler(App_ThreadException);
-
-            //Run the application
-            //Sam's Note: This is an unusual way to call this. Typically, it's run with
-            //the main form as an argument.
+            
+            view.Show();
+            view.Activate();
+            
             Application.Run();
         }
@@ -368 +378 @@
                     }
                     // My code -- buts looks so ugly!
+                    // Checks the passed parameters stored in the class variables
                     else
                     {
@@ -981 +992 @@
                 }
 
+        /// <summary>
+        /// Accomplishes Changing the Server to which you connect
+        /// </summary>
+        /// <remarks>
+        /// Parameter relog-in for InitializeApp forces initialize app to use
+        /// 1. The server the user just picked and then BMX saved off to User Preferences
+        /// 2. A new access and verify code pair
+        /// </remarks>
+        /// <param name="sender">unused</param>
+        /// <param name="e">unused</param>
                 private void mnuRPMSServer_Click(object sender, EventArgs e)
                 {
@@ -990 +1011 @@
                         try
                         {
+                // Close All, but tell the Close All method not to call Applicaiton.Exit since we still plan to continue.
+                // Close All does not call Application.Exit, but CGView_Close handler does
                                 m_bExitOK = false;
-                                bool bRetry = true;
-                                BMXNetConnectInfo tmpInfo;
+                CloseAll();
+                m_bExitOK = true;
+                
+                //Used in Do loop
+                bool bRetry = true;
+                                
+                // Do Loop to deal with changing the server and the vagaries of user choices.
                                 do
                                 {
-                                        tmpInfo = m_ConnectInfo;
                                         try
                                         {
-                                                tmpInfo.ChangeServerInfo();
+                        //ChangeServerInfo does not re-login the user
+                        //It only changes the saved server information in the %APPDATA% folder
+                        //so it can be re-used when BMX tries to log in again.
+                        //Access and Verify code are prompted for in InitializeApp
+                                                m_ConnectInfo.ChangeServerInfo();
                                                 bRetry = false;
                                         }
@@ -1006 +1037 @@
                                                 {
                                                         bRetry = false;
+                            Application.Exit();
                                                         return;
                                                 }
@@ -1015 +1047 @@
                                                 {
                                                         bRetry = false;
+                            Application.Exit();
                                                         return;
                                                 }
@@ -1020 +1053 @@
                                 } while (bRetry == true);
 
-                                CloseAll();
-                                m_bExitOK = true;
-                                m_ConnectInfo = tmpInfo;
-
-                                this.InitializeApp();
-
-                                //Create a new document
-                                CGDocument doc = new CGDocument();
-                                doc.DocManager = _current;
-                                doc.OnNewDocument();
-
+                //Parameter for initialize app tells it that this is a re-login and forces a new access and verify code.
+                bool isEverythingOkay = this.InitializeApp(true);
+
+                //if an error occurred, break out. This time we need to call Application.Exit since it's already running.
+                if (!isEverythingOkay)
+                {
+                    Application.Exit();
+                    return;
+                }
+
+                //Otherwise, everything is okay. So open document and view, then show and activate view.
+                CGDocument doc = new CGDocument();
+                doc.DocManager = _current;
+
+                CGView view = new CGView();
+                view.InitializeDocView(doc, _current, doc.StartDate, doc.Appointments, _current.WindowText);
+
+                view.Show();
+                view.Activate();
+
+                //Application.Run need not be called b/c it is already running.
                         }
                         catch (Exception ex)
@@ -1039 +1082 @@
                 }
 
+        /// <summary>
+        /// Accomplishes Re-login into RPMS/VISTA. Now all logic is in this event handler.
+        /// </summary>
+        /// <param name="sender">not used</param>
+        /// <param name="e">not used</param>
                 private void mnuRPMSLogin_Click(object sender, EventArgs e)
                 {
@@ -1048 +1096 @@
                         try
                         {
+                // Close All, but tell the Close All method not to call Applicaiton.Exit since we still plan to continue.
+                // Close All does not call Application.Exit, but CGView_Close handler does
                                 m_bExitOK = false;
                                 CloseAll();
                                 m_bExitOK = true;
-                                //_current.m_ConnectInfo = new BMXNet.BMXNetConnectInfo();//smh redundant
-                                this.InitializeApp(true);
-                                //Create a new document
-                                CGDocument doc = new CGDocument();
-                                doc.DocManager = _current;
-                                doc.OnNewDocument();
+
+                //Parameter for initialize app tells it that this is a re-login and forces a new access and verify code.
+                bool isEverythingOkay = this.InitializeApp(true);
+
+                //if an error occurred, break out. This time we need to call Application.Exit since it's already running.
+                if (!isEverythingOkay)
+                {
+                    Application.Exit();
+                    return;
+                }
+
+                //Otherwise, everything is okay. So open document and view, then show and activate view.
+                CGDocument doc = new CGDocument();
+                doc.DocManager = _current;
+
+                CGView view = new CGView();
+                view.InitializeDocView(doc, _current, doc.StartDate, doc.Appointments, _current.WindowText);
+
+                view.Show();
+                view.Activate();
+
+                //Application.Run need not be called b/c it is already running.
                         }
                         catch (Exception ex)