Bug #16642
closed
TCEmain: Wrong syntax on checking NEW items
0%
Description
There are some lines like in class.t3lib_tcemain.php
if (strstr($pid_value,'NEW'))
The syntax for substr is
string substr ( string string, int start [, int length] )
The code shown above will always return the first argument of the substr-call.
I guess a "preg_match" should have been standing there.
olly
(issue imported from #M4384)
Files
Updated by Oliver Hader about 18 years ago
If this patch should be applied to the trunk version, I've got a feature wish to add (I know wrong type of issue, but correct file):
If a new record was passed to TCEmain and that record was created in database, it could happen that in the same session another field of that record is changed. I added some line that notice that a record already was processed and has a proper uid of it's table.
Have a look to tcemain_2.diff (include the substitution and the substr thingy)
Updated by Oliver Hader about 18 years ago
Just realized, that I must have been drunken on writing the bug-report.
No bug at all - everything is fine ;-)
Updated by Michael Stucki about 18 years ago
OK so I'm closing this bug. Can you please open a new issue for the feature request? Just to keep title and description up to date... Thanks.