array(0 => 'some string') == array(0 => 0)

string 'some string' == int 0

Patch changes =! to ==!

CommentFileSizeAuthor
features_type_strict_comparison.patch481 byteshefox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hefox’s picture

hefox CreditAttribution: hefox commented

(Note: bug results in items appearing to be overridden when they are not; may or may not be the cause of some people's 'feature still appears overriden' issues at times)

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Reviewed & tested by the community

Seems like a pretty simple and logically accurate patch.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

+1. This, and some others, seem to have resolved the problems I was experiencing with menu links.

HongPong’s picture

HongPong CreditAttribution: HongPong commented

Is this RTBC??

febbraro’s picture

febbraro CreditAttribution: febbraro commented

I actually committed a fix to this for D7 here.

http://drupalcode.org/project/features.git/commitdiff/2a1a1de?hp=240fc89...

Will back port this.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Version: 6.x-1.0 » 6.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

http://drupalcode.org/project/features.git/commit/b4fc2da

Status: Fixed » Closed (fixed)

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