Don't Run Foreach Loop If No Unique Fields Defined

Created on 16 March 2017, almost 8 years ago

Updated 2 May 2024, 8 months ago

I'm seeing watchdog messages like following after having module 'unique_field' installed:

Warning: Invalid argument supplied for foreach() in unique_field_form_validate() (line 306 of /var/www/modules/contrib/unique_field/unique_field.module)

In those 3 hook_...form_validate methods, the foreach loops shouldn't be executed if no unique fields defined for some node types, taxonomies and users. A patch file is included to address the issue.

🐛 Bug report

Status

Fixed

Version

2.0

Component

Code

Created by

🇨🇦Canada deminy Canada

Live updates comments and jobs are added and updated live.

Incomplete comments

Merge Requests

!6Don't Run Foreach Loop If No Unique Fields Defined
Merged
🇺🇦Ukraine AstonVictor
updated 8 months ago

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

First commit to issue fork.
Merge request !6Issue #2861317 - Fix loop → (Merged) created by AstonVictor
Comment 8 months ago →
System Message

AstonVictor → committed 095a8bdb on 8.x-2.x
Issue #2861317 - Fix loop
Status changed to Fixed 8 months ago8:31am 18 April 2024
Comment 8 months ago →
🇺🇦Ukraine AstonVictor
Created and merged a new MR.

Added the validation.
Comment 8 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024

Don't Run Foreach Loop If No Unique Fields Defined

Merge Requests

!6Don't Run Foreach Loop If No Unique Fields DefinedMerged

Comments & Activities

!6Don't Run Foreach Loop If No Unique Fields Defined
Merged