Closed (fixed)
Project:
Flag
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Jun 2008 at 23:34 UTC
Updated:
23 Jul 2008 at 10:56 UTC
Jump to comment: Most recent file
Regardless of the options chosen, actions for every flag are executed.
For example, I have 3 flags and 3 send email
actions. Each of the actions is assigned to one individual flag only. I flag content using *one* of the flags but get 3 e-mails (every e-mail associated with a different action).
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | flag.actions.inc-flag-selection.patch | 812 bytes | archetwist |
Comments
Comment #1
archetwist commentedI think I have a fix.
Comment #2
mooffie commentedArchetwist, thanks. Your patch works, but I didn't use it eventually.
Problem was because in PHP the following code returns TRUE.
Fortunately, in_array() gets an optional third parameter, strict, that fixes this. I also sanitized form checkboxes with array_filter.
http://drupal.org/cvs?commit=125234
http://drupal.org/cvs?commit=125236
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.