Doctrine 1
  1. Doctrine 1
  2. DC-744

PHP Deprecated: Function spliti() is deprecated

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2.3
    • Fix Version/s: None
    • 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

        Hide
        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
        Enrico Stahn added a comment - - edited
        Show
        Enrico Stahn added a comment - - edited Should be fixed with DC-828 http://github.com/estahn/doctrine1/compare/master...DC-828
        Hide
        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
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: