warning: Invalid argument supplied for foreach() in sites/all/modules/storm/storm/stormtask/stormtask.module on line 1092.

i get this message every time i want to create a task. after this the task is stored but every time i want to edit, the same problmem!

CommentFileSizeAuthor
#1 Issue_1083726_fixing_warning_stormtask.patch1.24 KBcarsten müller

Comments

carsten müller’s picture

carsten müller commented
StatusFileSize
new Issue_1083726_fixing_warning_stormtask.patch1.24 KB

Hi FuXXz,

you assigned a stormteam to the task without any members in the team, right?

Here is a patch so fix this warning. When it passed testing it can be commited.

carsten müller’s picture

carsten müller commented
Assigned: FuXXz » carsten müller
Status: Active » Needs review
FuXXz’s picture

FuXXz commented

Thank you, the patch works fine!
But the warning also appears if i create a task without assign a member to it! but with the patch, its ok!

juliangb’s picture

juliangb commented
Title: warning: invalid argument / storm.module » PHP warning when assigning an empty team 
+++ b/stormtask/stormtask.module
@@ -1086,12 +1086,14 @@
+        elseif ('stormteam' == $assignees_node->type) {

Very minor, might be worth swapping the order to match the if() above for clarity.

Otherwise, seems like a useful fix.

@FuXXz - just to clarify your comment #3, were you saying that there is still a warning with the patch applied, or that it was all solved?

Powered by Dreditor.

Status: Needs review » Needs work

The last submitted patch, Issue_1083726_fixing_warning_stormtask.patch, failed testing.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs work » Needs review

#1: Issue_1083726_fixing_warning_stormtask.patch queued for re-testing.

juliangb’s picture

juliangb commented

#1: Issue_1083726_fixing_warning_stormtask.patch queued for re-testing.

juliangb’s picture

juliangb commented
Status: Needs review » Postponed (maintainer needs more info)

Just tested this, but I can't reproduce it.

Would it be possible to check with the latest 6.x-2.x-dev and if the error still exists, to post more instructions?

juliangb’s picture

juliangb commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

I couldn't reproduce the bug. Please reopen if it is still an issue.