tree 919355183c7b899facd722eb9ae7c06a353e46f7
parent aa8bea6d8d3dcdaa313a15a1f41b6c14a426cc77
author Dave Borowitz <dborowitz@google.com> 1519224383 -0500
committer David Ostrovsky <david@ostrovsky.org> 1519232210 +0100

Schema_159: Don't set closed changes as WIP/private

Although the data model allows it, it makes no sense to have a change
that is both merged and WIP, and Gerrit's current behavior is to reject
submitting a WIP change.

It is cleanest to apply this logic even when migrating draft changes to
private changes. This means that some previously-draft *patch sets* of
merged/abandoned changes may become visible, even though the admin
prefers the security of hiding draft *changes*.

Bug: Issue 8346
Change-Id: I91cfce3a7db2e8759e9b5b9835baefdf0388e5dc