HomePhorge

Save empty fields as no row, not an empty row

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

Provenance
epriestleyAuthored on Aug 26 2011, 11:40 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP99b9ceb6d774: Fix arcanist documentation to include "set -e" in example SVN hooks
Branches
Unknown
Tags
Unknown

Event Timeline