Merged PR 392: V3.1.13 Release 3

* Set Transport Layer Security TLS 1.2 protocol to prevent https errors (missed earlier) in PubChem Search
* Fix incorrect position of implicit Hydrogen for a lone atom (as seen in Microsoft Word)

Related work items: #760, #761

Author: MikeWilliams-UK

src/Chem4Word.V3/Data/Chem4Word-Versions.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Version>
-  <Number>3.1.12 Release 2</Number>
+  <Number>3.1.13 Release 3</Number>
   <IsBeta>false</IsBeta>
-  <Released>06-Sep-2020</Released>
+  <Released>17-Sep-2020</Released>
 </Version>

src/Chem4Word.V3/Data/This-Version.xml

@@ -15,9 +15,13 @@ <h1>Chemistry for Word Add-In 2020</h1>
             <td>Setup Bootstrapper</td>
             <td><a href="/files3-1/Chem4Word-Setup.exe">Chem4Word-Setup</a></td>
         </tr>
+        <tr>
+            <td>Version 3.1.13 - Release 3</td>
+            <td><a href="/files3-1/Chem4Word-Setup.3.1.13.Release.3.msi">Chem4Word-Setup 3.1.13 Release 3</a></td>
+        </tr>
         <tr>
             <td>Version 3.1.12 - Release 2</td>
-            <td><a href="/files3-1/Chem4Word-Setup.3.1.12.Release.2.msi">Chem4Word-Setup 3.1.12 Release 2</a></td>
+            <td>Chem4Word-Setup 3.1.12 Release 2</td>
         </tr>
         <tr>
             <td>Version 3.1.11 - Release 1</td>

src/Chem4Word.V3/Data/index.html

@@ -35,5 +35,5 @@
 //      Build Number
 //      Revision
 //
-[assembly: AssemblyVersion("3.1.12.7554")]
-[assembly: AssemblyFileVersion("3.1.12.7554")]
+[assembly: AssemblyVersion("3.1.13.7565")]
+[assembly: AssemblyFileVersion("3.1.13.7565")]

src/Chem4Word.V3/Properties/AssemblyInfo.cs

@@ -721,6 +721,15 @@ public static void PerformEdit()
                                     }
                                 }
 
+                                if (isNewDrawing)
+                                {
+                                    Globals.Chem4WordV3.Telemetry.Write(module, "Information", $"Starting create new structure {fullTag}");
+                                }
+                                else
+                                {
+                                    Globals.Chem4WordV3.Telemetry.Write(module, "Information", $"Starting edit existing structure {fullTag}");
+                                }
+
                                 var used1D = ChemistryHelper.GetUsed1D(doc, guidString);
 
                                 editor.Cml = beforeCml;
@@ -765,11 +774,11 @@ public static void PerformEdit()
 
                                     if (isNewDrawing)
                                     {
-                                        Globals.Chem4WordV3.Telemetry.Write(module, "Information", $"Creating new structure {fullTag}");
+                                        Globals.Chem4WordV3.Telemetry.Write(module, "Information", $"Finished creating new structure {fullTag}");
                                     }
                                     else
                                     {
-                                        Globals.Chem4WordV3.Telemetry.Write(module, "Information", $"Editing existing structure {fullTag}");
+                                        Globals.Chem4WordV3.Telemetry.Write(module, "Information", $"Finished editing existing structure {fullTag}");
                                     }
 
                                     // Copy back CustomXmlPartGuid which will get lost if edited via ChemDoodle Web
@@ -871,9 +880,9 @@ public static void PerformEdit()
                                             // Delete the temporary file now we are finished with it
                                             try
                                             {
-                                                // Only delete file in release mode
 #if !DEBUG
-                                        File.Delete(tempfileName);
+                                                // Only delete file in release mode
+                                                File.Delete(tempfileName);
 #endif
                                             }
                                             catch

src/Chem4Word.V3/Ribbon/Chem4WordRibbon.cs

@@ -35,5 +35,5 @@
 //      Build Number
 //      Revision
 //
-[assembly: AssemblyVersion("3.1.12.7554")]
-[assembly: AssemblyFileVersion("3.1.12.7554")]
+[assembly: AssemblyVersion("3.1.13.7565")]
+[assembly: AssemblyFileVersion("3.1.13.7565")]

src/Chem4WordTests/Properties/AssemblyInfo.cs

@@ -305,18 +305,18 @@ public override void Render()
             Atom endAtom = ParentBond.EndAtom;
 
             // Only continue if we have both a start and end atom
-            if (startAtom != null && endAtom != null)
-            {
+            //if (startAtom != null && endAtom != null)
+            //{
                 //set up the shared variables first
                 Point startPoint = startAtom.Position;
                 Point endPoint = endAtom.Position;
 
                 double bondLength = ParentBond.Model.XamlBondLength;
-                bool firstVisualExists = ChemicalVisuals.ContainsKey(startAtom);
-                bool secondVisualExists = ChemicalVisuals.ContainsKey(endAtom);
+                //bool firstVisualExists = ChemicalVisuals.ContainsKey(startAtom);
+                //bool secondVisualExists = ChemicalVisuals.ContainsKey(endAtom);
 
                 // Only continue if bond length is not zero and we have both a startVisual and endVisual
-                if (startPoint != endPoint && firstVisualExists && secondVisualExists)
+                if (startPoint != endPoint ) // && firstVisualExists && secondVisualExists)
                 {
                     //now get the geometry of start and end atoms
                     AtomVisual startVisual = (AtomVisual)ChemicalVisuals[startAtom];
@@ -489,7 +489,7 @@ public override void Render()
                             break;
                     }
                 }
-            }
+            //}
 
             //local function
             void DrawHitTestOverlay(DrawingContext dc)