Bug #16642

TCEmain: Wrong syntax on checking NEW items

Added by Oliver Hader over 14 years ago. Updated about 14 years ago.

Should have
Target version:
Start date:
Due date:
% Done:


Estimated time:
TYPO3 Version:
PHP Version:
Is Regression:
Sprint Focus:


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.


(issue imported from #M4384)


tcemain.diff (1.41 KB) tcemain.diff Administrator Admin, 2006-10-13 13:05
tcemain_2.diff (2.02 KB) tcemain_2.diff Administrator Admin, 2006-10-13 13:42

Updated by Oliver Hader over 14 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 over 14 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 over 14 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.

Also available in: Atom PDF