Bug #6012
Update functions for PHP 5.3
| Status: | Closed | Start date: | 2010-01-13 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assignee: | Reinhard Führicht | % Done: | 100% |
|
| Category: | Code and Architecture | |||
| Target version: | - | Estimated time: | 1.00 hour | |
| Votes: | 0 |
Description
With PHP 5.3 there will be some deprecated functions. See http://us2.php.net/manual/en/migration53.deprecated.php for whole changelog.
Ive tested latest non-svn version (with TYPO3 4.3) with beneath mentioned changes without problems. No need to change parameters, just change functions.
split = explode ereg_replace = preg_replace ereg = preg_match eregi = preg_match
Is there a reason why you used both (ereg and eregi)? RegEx should be enough to control upper and lower case. As far as i can see there is no need to change anything, but im not a regex pro.
History
Updated by Reinhard Führicht over 3 years ago
- Assignee set to Reinhard Führicht
As far as I remember I already replaced "split" in the trunk version, so this should be fixed already.
"ereg" and "eregi" are used in the according error checks and probably somewhere else too. These stem from the migration of th_mailformplus functionality to Formhandler.
In my opinion, these error checks can be deleted, because as you saif "preg_match" will do the job either way.
I will replace alle calls to "ereg" and "ereg_replace" and remove the error checks "ereg" and "eregi".
Thanks for the hint.
Updated by Reinhard Führicht over 3 years ago
- Status changed from New to Closed
- % Done changed from 0 to 100
Committed to trunk.
So far, no errors occurred, so I hope everything still works fine.