This of course depends on the definition of 'empty' :)

But I have field settings that keep giving differences, and this is what I saw:

  $default = array(
    'name' => 'name',
    'settings' => array(),
  );

  $normal = array(
    'name' => 'name',
    'settings' => array(
      'empty_value' => NULL,
    ),
  );

_features_sanitize removes the 'settings' key from $default.

But it does not remove the 'settings' key from $normal; it leaves 'settings => array()' in there. I'm guessing that this is not as desired.

Patch attached.

CommentFileSizeAuthor
features-sanitize-really-empty.patch1.03 KBroderik

Comments

mpotter’s picture

Status: Needs review » Fixed

Seems like an easy one. Committed to 19b952d.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.