Remove the ds-boolean-transform-allow-null feature flag

FEATURES.md

@@ -11,10 +11,6 @@ entry in `config/features.json`.
 
 ## Feature Flags
 
-- `ds-boolean-transform-allow-null` [#4022](https://github.com/emberjs/data/pull/4022)
-
-  Allow `null`/`undefined` values for `boolean` attributes via `DS.attr('boolean', { allowNull: true })`
-
 - `ds-improved-ajax` [#3099](https://github.com/emberjs/data/pull/3099)
 
   This feature allows to customize how a request is formed by overwriting

addon/-private/transforms/boolean.js

@@ -1,6 +1,5 @@
 import Ember from 'ember';
 import Transform from "ember-data/transform";
-import isEnabled from 'ember-data/-private/features';
 
 const { isNone } = Ember;
 
@@ -44,10 +43,8 @@ export default Transform.extend({
   deserialize(serialized, options) {
     var type = typeof serialized;
 
-    if (isEnabled('ds-boolean-transform-allow-null')) {
-      if (isNone(serialized) && options.allowNull === true) {
-        return null;
-      }
+    if (isNone(serialized) && options.allowNull === true) {
+      return null;
     }
 
     if (type === "boolean") {
@@ -62,10 +59,8 @@ export default Transform.extend({
   },
 
   serialize(deserialized, options) {
-    if (isEnabled('ds-boolean-transform-allow-null')) {
-      if (isNone(deserialized) && options.allowNull === true) {
-        return null;
-      }
+    if (isNone(deserialized) && options.allowNull === true) {
+      return null;
     }
 
     return Boolean(deserialized);

config/features.json

@@ -1,5 +1,4 @@
 {
-  "ds-boolean-transform-allow-null": true,
   "ds-improved-ajax": true,
   "ds-pushpayload-return": null,
   "ds-extended-errors": null,

tests/unit/model-test.js

@@ -1131,17 +1131,11 @@ test("a DS.Model can describe Boolean attributes", function(assert) {
   assert.converts('boolean', 1, true);
   assert.converts('boolean', 0, false);
 
-  if (isEnabled('ds-boolean-transform-allow-null')) {
-    assert.converts('boolean', null, null, { allowNull: true });
+  assert.converts('boolean', null, null, { allowNull: true });
 
-    assert.converts('boolean', null, false, { allowNull: false });
+  assert.converts('boolean', null, false, { allowNull: false });
 
-    // duplicating the tests from the else branch here, so once the feature is
-    // enabled and the else branch is deleted, those assertions are kept
-    assert.converts('boolean', null, false);
-  } else {
-    assert.converts('boolean', null, false);
-  }
+  assert.converts('boolean', null, false);
 
   assert.converts('boolean', true, true);
   assert.converts('boolean', false, false);

tests/unit/transform/boolean-test.js

@@ -1,5 +1,4 @@
 import DS from 'ember-data';
-import isEnabled from 'ember-data/-private/features';
 
 import {module, test} from 'qunit';
 
@@ -8,19 +7,14 @@ module("unit/transform - DS.BooleanTransform");
 test("#serialize", function(assert) {
   var transform = new DS.BooleanTransform();
 
-  if (isEnabled('ds-boolean-transform-allow-null')) {
-    assert.equal(transform.serialize(null, { allowNull: true }), null);
-    assert.equal(transform.serialize(undefined, { allowNull: true }), null);
+  assert.equal(transform.serialize(null, { allowNull: true }), null);
+  assert.equal(transform.serialize(undefined, { allowNull: true }), null);
 
-    assert.equal(transform.serialize(null, { allowNull: false }), false);
-    assert.equal(transform.serialize(undefined, { allowNull: false }), false);
+  assert.equal(transform.serialize(null, { allowNull: false }), false);
+  assert.equal(transform.serialize(undefined, { allowNull: false }), false);
 
-    assert.equal(transform.serialize(null, {}), false);
-    assert.equal(transform.serialize(undefined, {}), false);
-  } else {
-    assert.equal(transform.serialize(null), false);
-    assert.equal(transform.serialize(undefined), false);
-  }
+  assert.equal(transform.serialize(null, {}), false);
+  assert.equal(transform.serialize(undefined, {}), false);
 
   assert.equal(transform.serialize(true), true);
   assert.equal(transform.serialize(false), false);
@@ -29,19 +23,14 @@ test("#serialize", function(assert) {
 test("#deserialize", function(assert) {
   var transform = new DS.BooleanTransform();
 
-  if (isEnabled('ds-boolean-transform-allow-null')) {
-    assert.equal(transform.deserialize(null, { allowNull: true }), null);
-    assert.equal(transform.deserialize(undefined, { allowNull: true }), null);
+  assert.equal(transform.deserialize(null, { allowNull: true }), null);
+  assert.equal(transform.deserialize(undefined, { allowNull: true }), null);
 
-    assert.equal(transform.deserialize(null, { allowNull: false }), false);
-    assert.equal(transform.deserialize(undefined, { allowNull: false }), false);
+  assert.equal(transform.deserialize(null, { allowNull: false }), false);
+  assert.equal(transform.deserialize(undefined, { allowNull: false }), false);
 
-    assert.equal(transform.deserialize(null, {}), false);
-    assert.equal(transform.deserialize(undefined, {}), false);
-  } else {
-    assert.equal(transform.deserialize(null), false);
-    assert.equal(transform.deserialize(undefined), false);
-  }
+  assert.equal(transform.deserialize(null, {}), false);
+  assert.equal(transform.deserialize(undefined, {}), false);
 
   assert.equal(transform.deserialize(true), true);
   assert.equal(transform.deserialize(false), false);