Closed (fixed)
Project:
Workflow Fields
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
23 Mar 2007 at 09:11 UTC
Updated:
1 Aug 2007 at 22:08 UTC
Jump to comment: Most recent file
* warning: Invalid argument supplied for foreach() in C:\Program Files\xampp\xampp\htdocs\Drupcrm\sites\default\modules\workflow_fields\workflow_fields.module on line 59.
* warning: Invalid argument supplied for foreach() in C:\Program Files\xampp\xampp\htdocs\Drupcrm\sites\default\modules\workflow_fields\workflow_fields.module on line 85.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | patch_workflow_fields.txt | 865 bytes | hbfkf |
Comments
Comment #1
colesidhu commentedI get the same errors.
I first created a workflow.
If I try to add new states without first selecting the node type involved, I get the error messages listed above.
However, if I first select the node type involved and click "Save Workflow Mapping", THEN try to add a new state, I dont get any error messages.
I should note as well that it looks like only 1 node type can be referenced in this way; if I try to add more than 1 node type for a workflow, only the fields for the first one appear.
Actually, even if I do all this and with all different combinations, I still cant get the workflow fields to work.
Comment #2
hbfkf commentedI get the same error.
Comment #3
hbfkf commentedAs noted by colesidhu, the bug does not manifest if you first save a workflow mapping in which at least one workflow is assigned to some content type.
I do not have the complete overview of the module's code, but it looks as if the case of no assigned workflow is not handled correctly. Here's a fix (at least for the original poster's problem, maybe not for the problem colesidhu mentions) that works for me.
I attach this diff also as a text-file.
Could others please check whether this solves the issue?
Comment #4
hbfkf commentedColesidhu, could you maybe post your second issue as a separate problem report (if none already exists)? This would allow this bug to be closed as soon as we have a good fix! Thanks!
Comment #5
syoumans commentedThis fix patch worked for me; I applied it to version 5.x-1.x-dev.
Comment #6
infojunkieLine 50: I changed
to
Please verify and close if everything OK.
Comment #7
(not verified) commented