#1460 ImplicitH set to zero casue error loading query molecule from ket

Fix bug. Add UT.
epam · Dec 26, 2023 · 0ebaab1 · 0ebaab1
1 parent fd3fa7d
commit 0ebaab1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/api/tests/integration/tests/formats/ref/ket_with_implicit_h_count.ket b/api/tests/integration/tests/formats/ref/ket_with_implicit_h_count.ket
@@ -23,7 +23,8 @@
                     3.866025447845459,
                     -3.549999952316284,
                     0.0
-                ]
+                ],
+                "implicitHCount": 0
             },
             {
                 "label": "C",

diff --git a/core/indigo-core/molecule/src/molecule_json_loader.cpp b/core/indigo-core/molecule/src/molecule_json_loader.cpp
@@ -443,7 +443,7 @@ void MoleculeJsonLoader::parseAtoms(const rapidjson::Value& atoms, BaseMolecule&
             else
             {
                 int count = a["implicitHCount"].GetInt();
-                if (count < 1)
+                if (count < 0)
                     throw Error("Wrong value for implicitHCount: %d", count);
                 _pqmol->resetAtom(atom_idx,
                                   QueryMolecule::Atom::und(_pqmol->releaseAtom(atom_idx), new QueryMolecule::Atom(QueryMolecule::ATOM_IMPLICIT_H, count)));