search

NAL-i5K / tripal_eutils

ncbi loader via the eutils interface
GNU General Public License v3.0
4 stars 3 forks source link

Assembly with a missing "Assembly method" field triggers a "Not null violation" error #230

Closed dsenalik closed 1 year ago

dsenalik commented 3 years ago

Not every assembly in NCBI has the Assembly method field. Example: https://www.ncbi.nlm.nih.gov/assembly/GCA_902728035.1 This is used to determine a value for the program and programversion columns of the chado.analysis table, which have a NOT NULL constraint. Current behavior is to generate an error such as:

ERROR (TRIPAL_EUTILS): SQLSTATE[23502]: Not null violation: 7 ERROR:  null value in column "program" violates not-null constraint
DETAIL:  Failing row contains (75, Apium_graveolens, , null, null, , SAMEA6463266, , , 2020-04-02 00:00:00).

Pull request #229 prevents this error by adding a default value for cases where the Assembly method tag is missing.

dsenalik commented 1 year ago

Closed with merged pull request #229