Navigation - Make required fields NOT NULL

[colemanw]

Overview

Cleans up civicrm_navigation to make required fields NOT NULL.

Before

Fields weight and is_active can ambiguously be 0 or NULL, which effectively means the same thing.

After

Now they will always have a value.

Comments

IMO we should do this for every weight and is_active field in the DB, but I'm starting with this one entity to see how it goes.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[colemanw]

retest this please

[eileenmcnaughton]

I agree with this change & the general principle that boolean fields should all be required & have a default. I think it makes sense for weight too