Override select orientation for attachment points.

johnmay · johnmay · commit f8a595dd2d9f · 2020-01-25T18:34:32.000Z
diff --git a/tool/sdg/src/main/java/org/openscience/cdk/layout/StructureDiagramGenerator.java b/tool/sdg/src/main/java/org/openscience/cdk/layout/StructureDiagramGenerator.java
@@ -893,18 +893,18 @@ private void refinePlacement(IAtomContainer molecule) {
         LayoutRefiner refiner = new LayoutRefiner(molecule, afix, bfix);
         refiner.refine();
 
-        // choose the orientation in which to display the structure
-        if (selectOrientation) {
-
-            // check for attachment points, these override the direction which we rorate structures
-            IAtom begAttach = null;
-            for (IAtom atom : molecule.atoms()) {
-                if (atom instanceof IPseudoAtom && ((IPseudoAtom) atom).getAttachPointNum() == 1) {
-                    begAttach = atom;
-                    break;
-                }
+        // check for attachment points, these override the direction which we rorate structures
+        IAtom begAttach = null;
+        for (IAtom atom : molecule.atoms()) {
+            if (atom instanceof IPseudoAtom && ((IPseudoAtom) atom).getAttachPointNum() == 1) {
+                begAttach = atom;
+                selectOrientation = true;
+                break;
             }
+        }
 
+        // choose the orientation in which to display the structure
+        if (selectOrientation) {
             // no attachment point, rotate to maximise horizontal spread etc.
             if (begAttach == null) {
                 selectOrientation(molecule, DEFAULT_BOND_LENGTH, 1);
