Download & Extend

"Invalid argument supplied for foreach() in .../skinr/skinr.module on line 348." caused by poor documentation and bad example

Project:Fusion
Version:6.x-1.x-dev
Component:Documentation
Category:bug report
Priority:major
Assigned:Unassigned
Status:active

Issue Summary

So these are the lines in the fusion_starter.info file:

; Disable 12 column grid block widths inherited from fusion_core
skinr[grid12-width] =

Which causes the error Invalid argument supplied for foreach() in .../skinr/skinr.module on line 348. to show for every file that has an instance of this or similar settings.

I went and used devel to print out the array that is return by skinr_process_info_files(); and found that whenever you set the above setting then when the info file is parsed that parser continues down until the next available character. So in most instances this would be the equivalent of putting the following in your info file:

skinr[grid12-width] = ';'

This is since the code looks like this by default:
; Disable 12 column grid block widths inherited from fusion_core
skinr[grid12-width] =

; Sample Skinr style (uncomment to use, see Skinr section in CSS)

I ended up changing my info file to look like:
skinr[grid12-width] = ''

and the errors went away. However, I had to correct that line in every info fusion-based info file on my site. This includes fusion_starter.info as an offender.

Please update documentation and change the fusion_starter.info to reflect this.