Doctrine 1

PHP Deprecated: Function spliti() is deprecated

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.2.3
  • Fix Version/s: 1.2.4
  • Component/s: Connection
  • Labels:
    None
  • Environment:
    From Symfony 1.4 on Windows with PHP 5.3.2 using MSSQL (thru ODBC)

Description

spliti is deprecated with php 5.3.0+ and this function is used in Connection/Mssql.php lines 171 and 190
Original line:
$aux2 = spliti(' as ', $field_array);

Probably replacing with:
preg_split('/ as /i', $field_array);

Original PHP message :
PHP Deprecated: Function spliti() is deprecated in ....\lib\vendor\symfony\lib\plugins\sfDoctrinePlugin\lib\vendor\doctrine\Doctrine\Connection\Mssql.php on line 190

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Javier Nogueira added a comment -

This bug is also present in doctrine 1.2.2, the version used by the symfony framework 1.4.X

Changing:
$aux2 = spliti(' as ', $field_array);
By:
$aux2 = preg_split('/ as /i', $field_array);

As reported by Jean-Philippe Blais solves the deprecation message problems.

Show
Javier Nogueira added a comment - This bug is also present in doctrine 1.2.2, the version used by the symfony framework 1.4.X Changing: $aux2 = spliti(' as ', $field_array); By: $aux2 = preg_split('/ as /i', $field_array); As reported by Jean-Philippe Blais solves the deprecation message problems.
Hide
Permalink
Enrico Stahn added a comment - - edited

Should be fixed with DC-828

http://github.com/estahn/doctrine1/compare/master...DC-828

Show
Enrico Stahn added a comment - - edited Should be fixed with DC-828 http://github.com/estahn/doctrine1/compare/master...DC-828
Hide
Permalink
Daniel O'Connor added a comment -

Causing http://pear.php.net/bugs/bug.php?id=17835 at the moment.

Who can we bribe to get the preg_split() change in? We're stomping out all of these bugs in PEAR (http://ollehost.dk/media/pear-bugs/) and this would just make life a little less painful

Show
Daniel O'Connor added a comment - Causing http://pear.php.net/bugs/bug.php?id=17835 at the moment. Who can we bribe to get the preg_split() change in? We're stomping out all of these bugs in PEAR ( http://ollehost.dk/media/pear-bugs/ ) and this would just make life a little less painful
Hide
Permalink
Jonathan H. Wage added a comment -

Sorry for the delay, it is fixed here http://trac.doctrine-project.org/changeset/7690

Show
Jonathan H. Wage added a comment - Sorry for the delay, it is fixed here http://trac.doctrine-project.org/changeset/7690

People

  • Assignee:
    Jonathan H. Wage
    Reporter:
    Jean-Philippe Blais
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: