Do update implicit hydrogens when adding explicit hydrogens

And removed popup item 'Update Implicit Hydrogens' in JCP solves bug 3134
bioclipse · Aug 14, 2012 · f6e5618 · f6e5618
1 parent 0d1750c
commit f6e5618
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/plugins/net.bioclipse.cdk.jchempaint/plugin.xml b/plugins/net.bioclipse.cdk.jchempaint/plugin.xml
@@ -879,15 +879,6 @@
        <separator
              name="net.bioclipse.cdk.jchempaint.managerActions">
        </separator>
-       <command
-             commandId="net.bioclipse.cdk.jchempaint.manager.action"
-             label="Update Implicit Hydrogens"
-             style="push">
-          <parameter
-                name="jcp.manager.method"
-                value="updateImplicitHydrogenCounts">
-          </parameter>
-       </command>
        <command
              commandId="net.bioclipse.cdk.jchempaint.manager.action"
              label="Add Explicit Hydrogens"

diff --git a/...bioclipse.cdk.jchempaint/src/net/bioclipse/cdk/jchempaint/business/JChemPaintManager.java b/...bioclipse.cdk.jchempaint/src/net/bioclipse/cdk/jchempaint/business/JChemPaintManager.java
@@ -783,6 +783,7 @@ public void addExplicitHydrogens() {
         JChemPaintEditor editor = findActiveEditor();
         if (editor != null) {
             IChemModelRelay relay = editor.getControllerHub();
+            relay.updateImplicitHydrogenCounts();
             IChemModel model = relay.getIChemModel();
             List<IAtomContainer> containers =
                 ChemModelManipulator.getAllAtomContainers(model);