foreach/sql errors when creating CCK fields

nicholasThompson - November 7, 2008 - 22:56

Project:	Relevant Content
Version:	5.x-1.3
Component:	Code
Category:	bug report
Priority:	critical
Assigned:	nicholasThompson
Status:	patch (to be ported)

#1

bribiz - November 10, 2008 - 22:51

I'm also have the same error in 5.x version. This error occurs on the page that is attempting to display the CCK field. Let me know if I can assist further.

# warning: Invalid argument supplied for foreach() in /chroot/home/mydir/mydomain/html/sites/all/modules/relevant_content/relevant_content_cck/relevant_content_cck.module on line 99.
# warning: Invalid argument supplied for foreach() in /chroot/home/mydir/mydomain/html/sites/all/modules/relevant_content/relevant_content_cck/relevant_content_cck.module on line 211

#2

nicholasThompson - November 10, 2008 - 22:56

Interesting...

Although I think this is slightly different. My errors were in CCK's content.module whereas yours are inside my module.

Cheers for bringing this up though!

#3

geverest - November 13, 2008 - 18:54

I'm having the exact same issue, in that the error is coming from the Relevant Content module cck function.

One of my users is reporting the issue and it seems it might be happening from using the preview without saving and then trying to add content to another cck field. Although I am unable to duplicate it.

On the other hand, to fix it, I simply remove all of the entries my user has placed in the taxonomy field and enter one single term into the taxonomy field (I require at least one entry) and save. At that point the problem goes away and I can do anything I want to the page.

#4

awakenedvoice - December 4, 2008 - 20:32

Version:

6.x-1.x-dev

» 6.x-1.4

I'm seeing foreach errors as well but for line 226 of the relevant content CCK module. Subscribing.

Update: I noticed that I only get the foreach error when the relevant content field is enabled to display one the teaser which I'm displaying in a view I created. When I click through to the node there are no errors. For the moment I have hidden the relevant content field from the teaser view which is at http://learnbythedrop.com/drops.

Update 2: I have narrowed down the instances when the error occurs even further. It appears that the error is being thrown when a node is displayed that has no relevant posts associated with it. I'm seeing the error thrown on the post at http://learnbythedrop.com/drop/70. It just so happens that the teaser for the same post was shown in the view that I reference earlier.

#5

dropchew - December 5, 2008 - 04:20

Hi there, I had the same error as nicholasThompson, in cck instead of relevant content.

warning: Invalid argument supplied for foreach() in D:\xampp\htdocs\demo\sites\all\modules\cck\content.module on line 1270. This error repeats for each return results.

I notice this only happens when I tried to display as teasers. Where'as when displaying as full nodes, I get a whole "page not available/The connection to the server was reset while the page was loading" error. It working fine when displaying as title and title link.

Drupal 6.6
CCK 2.1 and latest dev (2008-Dec-04)
Relevant content 6.14 and latest dev (2008-Nov-07)

#6

nicholasThompson - December 16, 2008 - 06:04

#7

dan_aka_jack - January 16, 2009 - 16:12

Status:

active

» needs review

Hi there,

I was getting the following error:

warning: Invalid argument supplied for foreach() in /home/jack/workspace/peckhampower/trunk/sites/all/modules/relevant_content/relevant_content_cck/relevant_content_cck.module on line 228.

As awakenedvoice mentioned in comment #4, it seems that the error is generated when a node has a unique tag (i.e. a tag which is only applied to the current node). I did a bit of tinkering and it turns out that the variable $element['items']['#item'] is null when the error occurs. So I've added a quick if statement to test if the $element['items']['#item'] variable is set before continuing with the foreach statement.

This is what lines 227 to 230 in relevant_content_cck.module currently looks like in version 6.x-1.4 of the module:

<?php
    foreach ($element['items']['#item'] as $delta => $item) {
      $items_output .= '<div class="field-item '. ($count % 2 ? 'odd' : 'even') .'">'. theme('relevant_content_cck_field_format_'. $format, $item) .'</div>';
      $count++;
    }
?>

I have replaced the code above with the following code:

<?php
  if ($element['items']['#item']) {
    foreach ($element['items']['#item'] as $delta => $item) {
      $items_output .= '<div class="field-item '. ($count % 2 ? 'odd' : 'even') .'">'. theme('relevant_content_cck_field_format_'. $format, $item) .'</div>';
      $count++;
    }
  }
?>

i.e. I've simply wrapped the foreach loop in an if statement.

My modification seems to have fixed the problem for me. I haven't yet noticed any side-effects.

#8

my.wahyu - February 28, 2009 - 03:30

I have been activied All CCK MODUL 28 Feb 2009, here .. is my module

Content 6.x-2.1 Allows administrators to define new content types.
Required by: Content Copy (enabled), Content Permissions (enabled), Fieldgroup (enabled), Node Reference (enabled), Number (enabled), Option Widgets (enabled), Text (enabled), User Reference (enabled)

Content Copy 6.x-2.1 Enables ability to import/export field definitions.
Depends on: Content (enabled)

Content Permissions 6.x-2.1 Set field-level permissions for CCK fields.
Depends on: Content (enabled)

Content Templates 6.x-1.0 Create templates to customize output of teaser and body content.

Fieldgroup 6.x-2.1 Create display groups for CCK fields.
Depends on: Content (enabled)

Node Reference 6.x-2.1 Defines a field type for referencing one node from another.
Depends on: Content (enabled), Text (enabled), Option Widgets (enabled)

Number 6.x-2.1 Defines numeric field types.
Depends on: Content (enabled)

Option Widgets 6.x-2.1 Defines selection, check box and radio button widgets for text and numeric fields.
Depends on: Content (enabled)
Required by: Node Reference (enabled), User Reference (enabled)

Text 6.x-2.1 Defines simple text field types.
Depends on: Content (enabled)
Required by: Node Reference (enabled), User Reference (enabled)

User Reference 6.x-2.1 Defines a field type for referencing a user from a node.
Depends on: Content (enabled), Text (enabled), Option Widgets (enabled)

My blog appears ..error notification
warning: Invalid argument supplied for foreach() in /home/xxxxxxxx/public_html/sites/all/modules/cck/content.module on line 1270.

So ... my file in the server .. I simply change ... like this IT DANGEROUS with Your System?? ..!! Just waiting UPDATE MODUL NOTIFICATION ... mybe can fix this error AUTOMATICALLY

Parse error: syntax error, unexpected ')' in /home/xxxxxx/public_html/sites/all/modules/cck/content.module on line 1270

So I try to upload CCK modul again!!
and then I choose ... only useful module..!!
Hem ..its not easy .. to understanding code .. Learning by doing because
basically I'm a simple auditor from gastia.com

#9

rbgrn - March 15, 2009 - 01:12

Same error, different line number, not sure if it makes a difference.

This is with D6.10 and latest CCK

warning: Invalid argument supplied for foreach() in /sites/all/modules/relevant_content/relevant_content_cck/relevant_content_cck.module on line 226.

is the full path of where drupal is installed, I just edited it for my own security.

If I get the error on a node, then I get no relevant content. If I get the error in the admin screens somewhere, then most things still work, I just see this error in red. This is definitely a bug.

#10

rbgrn - March 15, 2009 - 03:43

I tried wrapping the foreach in the if statement provided above and that got rid of the error. I recommend that patch.

#11

asb - April 24, 2009 - 08:51

Hi,

I'm getting those errors with version 5.x-1.3:

Invalid argument supplied for foreach() in /var/www/drupal/sites/all/modules/relevant_content/relevant_content_cck/relevant_content_cck.module in Zeile 211.

Has someone already tried applying the patch to the D5 version?

Thanks & greetings, -asb

#12

skizzo - April 25, 2009 - 17:16

I see the error on line 226 (as reported in #4) using CCK 6.x-2.x-dev built on 2009-04-24

warning: Invalid argument supplied for foreach() in /var/www/drupal/sites/all/modules/relevant_content/relevant_content_cck/relevant_content_cck.module on line 226.

#13

spython - May 4, 2009 - 12:01

The bug is still there, even after the patch as described in #7.
If I select the field output as title or full node, it works, if I select to display it as teaser, it's broken.

#14

asb - May 19, 2009 - 17:30

Similar/duplicate issues: 354459 (D5) & 451726 (D6).

Greetings, -asb

#15

icenogle - May 20, 2009 - 21:00

It didn't work for me either, but I got it working with this:

if ($element['items']['#item'] != "") {
.
.

instead of:

if ($element['items']['#item']) {
.
.

This hasn't been extensively tested, but PHP booleans don't always work out the way you expect them to.

#16

gr4phic3r - June 16, 2009 - 08:01

Priority:

normal

» critical

i got these errors when activating both

warning: Invalid argument supplied for foreach() in path_to_drupal_root_directory\includes\form.inc on line 1200.
warning: Invalid argument supplied for foreach() in path_to_drupal_root_directory\includes\form.inc on line 1200.

i added a field to a cck type and all runs fine, but when i tryed to add the existing relevant content field to another cck field i get this one

user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') SELECT vid, nid, FROM content_type_investmentproperty' at line 1 query: INSERT INTO content_field_relcontent (vid, nid, ) SELECT vid, nid, FROM content_type_investmentproperty in path_to_drupal_root_directory\sites\all\modules\cck\includes\content.admin.inc on line 1527.

i´m using the newest D6 version

#17

gr4phic3r - June 16, 2009 - 08:05

and another one when adding a existing "relevant content" to a cck

user warning: Unknown column 'delta' in 'where clause' query: DELETE FROM content_field_relcontent WHERE delta >= 1 in path_to_drupal_root_directory\sites\all\modules\cck\includes\content.admin.inc on line 1492.

#18

dakku - June 26, 2009 - 10:29

yup same here:

Invalid argument supplied for foreach() in /var/www/html/{foo}/relevant_content/relevant_content_cck/relevant_content_cck.module on line 226.

gonna wrap in an IF statement as suggested here:
http://drupal.org/node/331625#comment-1204192

#19

calshei1 - July 19, 2009 - 15:51

I also used the patch, seems to be working just fine now.

#20

tseven - July 28, 2009 - 03:51

Version:	6.x-1.4	» 5.x-1.3
Status:	needs review	» patch (to be ported)

Here is a simple patch for 5.x-1.3. It may work for other versions. I wrapped each foreach in an if (!empty($items)) statement.

Works for me.

Hope this makes it into the code.

Attachment	Size
relevant_content_cck.patch	2.51 KB

#21

asb - July 28, 2009 - 15:53

Hi,

thank you the patch; with "-p0" option the patch applies cleanly on D5.

On D6 it fails ("Hunk #1 FAILED at 96. ... Hunk #2 FAILED at 208")

Greetings, -asb

#22

asb - October 5, 2009 - 08:14

Hi again,

any progress on this issue?

Thanks & greetings, -asb

foreach/sql errors when creating CCK fields

Jump to:

#1

#2

#3

#4

#5

#6

#7

#8

#9

#10

#11

#12

#13

#14

#15

#16

#17

#18

#19

#20

#21

#22

User login

Contributor links