Right now ownership can be either 1 (public) or 2 (private), but integer is not really the right data type here: a tree is either public or private and that's it. This should probably be changed to a public column that can be either true or false.
Renaming it will also prevent our current problem of having both an owner column and an ownership column that refer to very different things.
durkie
commented
8 years ago
Right now
ownershipcan be either1(public) or2(private), but integer is not really the right data type here: a tree is either public or private and that's it. This should probably be changed to apubliccolumn that can be either true or false.Renaming it will also prevent our current problem of having both an
ownercolumn and anownershipcolumn that refer to very different things.