Homec4science

Save empty fields as no row, not an empty row

Authored by epriestley <git@epriestley.com> on Aug 26 2011, 20:40.

Description

Save empty fields as no row, not an empty row

Summary: When a user stores the empty string in an auxiliary field, simply don't
store it, and delete it if it already exists.

Test Plan: Edited a revision with an empty "Quack" field, got an empty row in
the DB. Applied patch, edited empty again, row went away. Edited empty again,
still no row. Edited and put something in the field, got a row.

Reviewers: jungejason, tuomaspelkonen, aran

Reviewed By: jungejason

CC: aran, jungejason

Differential Revision: 865

Details

Committed
epriestley <git@epriestley.com>Aug 27 2011, 01:26
Pushed
aubortJan 31 2017, 17:16
Parents
rPH99b9ceb6d774: Fix arcanist documentation to include "set -e" in example SVN hooks
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH0334a9262175: Save empty fields as no row, not an empty row (authored by epriestley <git@epriestley.com>).Aug 27 2011, 01:26