Views PHP causing warning: Invalid argument supplied for foreach() [#1134780]

1. On enabling the module, the modules page displays the error message:

warning: Invalid argument supplied for foreach() in /home/752/knowhow/www.knowledge.co.uk/public_html/sites/all/modules/views/views.module on line 80.

After refreshing the modules page, the message disappears.

2. On visiting an existing view, I get another warning message:

warning: Invalid argument supplied for foreach() in /home/752/knowhow/www.knowledge.co.uk/public_html/sites/all/modules/views/includes/admin.inc on line 1060.

Refreshing the Views page, and clearing the Views cache, does not remove the message.

Comment	File	Size	Author
#2	views_php_remove_info_from_plugins-1134780.patch	432 bytes	Chemtox
	myviewexported.txt	15.01 KB	iantresman

Comments

Comment #1

CobraMP commented 22 April 2011 at 08:22

I am seeing this as well and
can't choose php as an access option

Comment #2

Chemtox commented 22 April 2011 at 17:56

Status:

Active

» Needs review

Status	File	Size
new	views_php_remove_info_from_plugins-1134780.patch	432 bytes

Just installed the module to try it out, and decided to have a quick go at the warnings. It was easy to spot after a bit of debugging in views.module: there's an alien 'info' item in views_php_views_plugins() (the dangers of copy-paste! ;). Removing it fixed the warnings.

To trigger the first warning, clear the Theme registry cache; for the second, reload an Edit view page.

Ok, now to finally try the module!

Comment #3

giorgosk

Chios, Greece 🇬🇷

commented 22 April 2011 at 19:36

Status:

Needs review

» Reviewed & tested by the community

warnings are gone with patch

Comment #4

giorgosk

Chios, Greece 🇬🇷

commented 22 April 2011 at 20:29

Status:

Reviewed & tested by the community

» Needs work

even though it clears the warning
it seems that you are throwing away perfectly good code
marked as needs work

Comment #5

Chemtox commented 23 April 2011 at 02:05

It is perfectly good code... in hook_views_handlers(), from where it was probably copy-pasted by mistake. If you care to look at the docs (/help/views/api-plugins), you'll see that there's no "info" plugin type, and that even if there where one, a "path" identifier plus the module directory it's still not a proper plugin definition. But I'll let someone else confirm that.