Closed (fixed)
Project:
Term Fields
Version:
6.x-1.0
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 Feb 2009 at 04:01 UTC
Updated:
31 May 2010 at 14:10 UTC
I have this message on all pages just after installing the module. It goes away after setting the first field.
warning: Invalid argument supplied for foreach() in ...\sites\all\modules\term_fields\views\term_fields.views.inc on line 16.
You should check if $fields is not an empty array before using foreach.
After playing with it, it is not exactly what i was looking for. I need to add fields to the listing of nodes so i can only use views of type node. But it is still a good module, maybe i will find a use for it later.
Comments
Comment #1
archard commentedThanks... yeah I forgot about that. I also fixed one when deleting vocabularies. Just released 1.1 which fixes the issue.
BTW I'm not sure when you get this on every page after installation... views.inc should only be included when on the views admin interface.
Comment #2
archard commentedComment #4
manumad40 commentedThere are some minor syntax errors inside the code of term_fields.views.inc which caused a blank screen at /admin/build/modules after installing 6.13 release.
The absence of commas after several of the closing parentheses when defining arrays is causing that.
Moreover, Apache's error_log file showed:
PHP Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING, expecting ')' in /.../sites/all/modules/term_fields/views/term_fields.views.inc on line 111When I put the commas and semicolons in their corresponding places, this error disappeared.