Issue with latest 7.x.3.x-dev version

Comments

csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
CreditAttribution: csakiistvan commented

@rollingnet how to i reproduce this? Please write to me step by step how did you get so far out of step

csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
CreditAttribution: csakiistvan commented
Status: Active » Postponed (maintainer needs more info)
rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented

This issue appears in node/add form
Display the same page with another theme (eg: Seven), the alert doesn't appear, so it's Bootstrap specific.

I'm using the following field modules:

Address Field (addressfield) 7.x-1.0-beta5
 
Computed Field (computed_field) 7.x-1.0+0-dev
 
Conditional Fields (conditional_fields) 7.x-3.0-alpha1+12-dev
 
Email (email) 7.x-1.2+5-dev
 
Entity Reference Behavior Example (entityreference_behavior_example) 7.x-1.1+3-dev
 
Entity reference prepopulate (entityreference_prepopulate) 7.x-1.5+5-dev
 
Entity Reference (entityreference) 7.x-1.1+3-dev
 
Field Permissions (field_permissions) 7.x-1.0-beta2+0-dev
 
Field validation extras (field_validation_extras) 7.x-2.3+8-dev
 
Field Validation UI (field_validation_ui) 7.x-2.3+8-dev
 
Field Validation (field_validation) 7.x-2.3+8-dev
 
Name Field (name) 7.x-1.9+9-dev
 
Phone (phone) 7.x-1.0-beta1+0-dev
 
Property Validation (property_validation) 7.x-2.3+8-dev
 
Select (or other) (select_or_other) 7.x-3.x-dev
 

Simple Field Formatter (simple_field_formatter) 7.x-2.0-beta2+0-dev

URL (url) 7.x-1.0+1-dev
csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
CreditAttribution: csakiistvan commented

Great, but how can i reproduce this? Pls write to me, or send a feature for this node type.

rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented

A feature of the relevant data of the site can be download here

tempo22’s picture

tempo22 CreditAttribution: tempo22 commented

I had the same problem, i seem to have fixed it by changing the line 91

from 

 if ($element['#type'] !== 'checkbox' || $element['#type'] !== 'radio' || $element['#type'] !== 'checkboxes' || $element['#type'] !== 'radios') {

to

 if (!empty($element['#type']) && ($element['#type'] !== 'checkbox' || $element['#type'] !== 'radio' || $element['#type'] !== 'checkboxes' || $element['#type'] !== 'radios')) {
rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented
Status: Postponed (maintainer needs more info) » Needs review

I can confirm that the above correction works.
Please someone create the related patch (sorry, but I can do it now) and let the community test it.

markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell commented
Status: Needs review » Needs work

Please create a patch.

wundo’s picture

wundo CreditAttribution: wundo commented
Status: Needs work » Fixed

I've committed the suggested change ;)

  • wundo committed ff5b2ff on 7.x-3.x 
    Issue #2236513 by tempo22, wundo: Fixes notice in...

Status: Fixed » Closed (fixed)

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