Add a constant OR use TRUE/FALSE for active storage vs. inactive storage and other properties

Or, @yched suggested making it explicitly a boolean, TRUE or FALSE, instead of 0 or 1.

Un-postponing.

Also each Configurable has public $status property and default Enable/Disable actions that used by views only, related issue is #1965910: Remove enable/disable entity operations
Change notice http://drupal.org/node/1926376

I still consider #1965910: Remove enable/disable entity operations a wontfix, and the status flag is also relevant to filters, blocks, etc.

Looks fine :-)

Suppose better to include more places

Even better :-)

And 2 more places

This looks like good clean-up.

Committed and pushed to 8.x. Thanks!

I *think* this needs a change notice?

I don't think a change notice is needed, this is only improving consistency on existing code by filling actual booleans instead of "various things that are == to TRUE or FALSE" on values that are treated as booleans by the API.

But there is no API change underneath, the underlying code makes no additional checks on those values being strict booleans, and there is no intention of doing so (none of core does).

Closing for now, but feel free to correct me.