7.x Issue #44 missing braces around chado table names

tripal / brapi

Breeding API client-server implementation (v1 & v2) for Tripal

https://www.drupal.org/project/brapi

0 stars 3 forks source link

7.x Issue #44 missing braces around chado table names #45

Closed dsenalik closed 2 years ago

dsenalik commented 2 years ago

Simple fix for issue #44 to add braces around table names in sql destined for chado_query() calls, or add chado. prefix in the text presented by the installer.

guignonv commented 2 years ago

Good point! Regarding the "DELETE" query specified as a message, specifying "chado." assumes that users have chado in a "chado"-named schema. It may not be the case but we can reasonably assume that people not using "chado" as schema name are smart enough to adapt the query to their schema. ;-) So it's fine for me.