aboutsummaryrefslogtreecommitdiff
path: root/containers
diff options
context:
space:
mode:
authorFrederick Muriuki Muriithi2023-08-02 06:35:37 +0300
committerFrederick Muriuki Muriithi2023-08-02 06:44:33 +0300
commit47c74f8ad6e76c4227ba1ff980d3a49f9ef79a81 (patch)
tree47136d51484446218960f7a5301ecb2f17d0e593 /containers
parent9f62caa68b541d683b2a744af19c0ad4c6d0d206 (diff)
downloadgenenetwork2-47c74f8ad6e76c4227ba1ff980d3a49f9ef79a81.tar.gz
Fix Bug: Unchanged Values Were Being Deleted
The original code (using the homebrew ORM system) would simply ignore values that were `None` when doing updates - the new code using direct queries was not. My (fredmanglis) initial fix to provide a default for values in DB that did not accept NULL was incorrect; instead, I needed to remove any key-value pairs from the incoming data that were set to `None` to fix the bug. The fix still feels incorrect: maybe we should do direct comparisons for all old-new value pairs, and set the data to be updated based on the differences, rather than relying on `None`s.
Diffstat (limited to 'containers')
0 files changed, 0 insertions, 0 deletions