search

tripal / tripal_galaxy

This module is for integration of Tripal and a remote Galaxy Instance.
https://tripal-galaxy.readthedocs.io/en/latest/
GNU General Public License v2.0
2 stars 4 forks source link

Table columns with not NULL #129

Closed spficklin closed 5 years ago

spficklin commented 5 years ago

In the tripal_galaxy.install file many of the table columns that don't allow NULL values have a key of 'not NULL', but Drupal isn't case insensitive and ignores this. It results in our table columns that should have a not null constraint missing that constraint. We should fix that in an update and test to make sure we don't break anything.

spficklin commented 5 years ago

Because folks are already using the module, it would be potentially disruptive to fix this and since the "NOT NULL" wasn't being recognized we may have built in functionality that didn't expect it and it's too much to sort through at this point in time.