]> git.cworth.org Git - turbot/commit
Don't put an attribute into the database for 'url' that is left blank
authorCarl Worth <cworth@cworth.org>
Thu, 31 Dec 2020 18:29:54 +0000 (11:29 -0700)
committerCarl Worth <cworth@cworth.org>
Thu, 31 Dec 2020 19:07:24 +0000 (12:07 -0700)
commite80df93e84e6e20933431a48d2c248fe663a3fa3
tree1961935834cb46cdc60397e3f0aa0cb64b10b921
parent99dfffda6befec46a1ecee6cae093224ee7cc6d0
Don't put an attribute into the database for 'url' that is left blank

The URL field, (for either a hunt or a puzzle), is an optional
parameter, (the user creating the hunt or puzzle may or may not
provide a URL). Immediately prior to this commit, if a user left the
URL field blank, the submission handler was receiving a value of None
and inserting that as an attribute in the database.

This was problematic as subsequent code, (such as sheet creation),
would be prepared for an item to have no url key at all, but was not
prepared for the key to be present but for the value to be empty.

We fix this in this commit by explicitly _not_ inserting a None value
for url as an attribute in the database item, (instead, not supplying
a url attribute at all).
turbot/interaction.py