SiT! Bugs - SiT!
View Issue Details
0000197SiT!triggerspublic2008-10-28 15:422008-11-21 17:26
kieran 
kieran 
highminoralways
closedfixed 
 
3.403.40 
0000197: Replace brackets in trigger rules
We allowed brackets in triggers for logic reasons. A (far-fetched but possible) side-effect of this is someone could post their own form to avoid the lack of input and specify some functions to execute in the check.

For 3.40 we will remove the brackets buttons for input, str_replace() any brackets from checks and write an upgrade function to remove them from any existing checks.
No tags attached.
Issue History
2008-10-28 15:42kieranNew Issue
2008-10-28 15:42kieranStatusnew => assigned
2008-10-28 15:42kieranAssigned To => kieran
2008-10-28 15:43kieranNote Added: 0000143
2008-10-28 15:47kieranNote Added: 0000144
2008-10-29 15:44kieranNote Added: 0000148
2008-10-29 15:44kieranStatusassigned => resolved
2008-10-29 15:44kieranResolutionopen => fixed
2008-10-29 15:44kieranFixed in Version => 3.40
2008-11-21 17:26kieranStatusresolved => closed

Notes
(0000143)
kieran   
2008-10-28 15:43   
18:08 < ericthefish> can do UPDATE `triggers` SET `checks` = REPLACE(`checks`, '(', '');
18:08 < ericthefish> and for the other bracket too I guess
(0000144)
kieran   
2008-10-28 15:47   
This can wait until after beta if needed.
(0000148)
kieran   
2008-10-29 15:44   
fixed in trunk r4098.