[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#707550: marked as done (opu: package php-mdb2/2.5.0b2-1)



Your message dated Sat, 19 Jul 2014 12:46:09 +0100
with message-id <1405770369.9607.17.camel@jacala.jungle.funky-badger.org>
and subject line Re: Bug#707550: opu: package php-mdb2/2.5.0b2-1
has caused the Debian Bug report #707550,
regarding opu: package php-mdb2/2.5.0b2-1
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
707550: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=707550
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: opu

Hi,

This is a follow up on one email thread from <debian-release> list:
  http://lists.debian.org/debian-release/2012/05/msg00182.html

Please apply the attached patch to php-mdb2 package version 2.5.0b2-1
from Debian 6.0 (squeeze) in a future point release (6.0.8 or later).

Cheers

-- System Information:
Debian Release: 7.0
  APT prefers stable
  APT policy: (500, 'stable'), (1, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 3.2.0-4-686-pae (SMP w/2 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
--- /usr/share/php/MDB2.php_2.5.0b2-1	2009-04-25 18:23:22.000000000 +0300
+++ /usr/share/php/MDB2.php	2012-05-03 18:14:11.451133868 +0300
@@ -390,7 +390,7 @@
             return $err;
         }
 
-        $db =& new $class_name();
+        $db = new $class_name();
         $db->setDSN($dsninfo);
         $err = MDB2::setOptions($db, $options);
         if (PEAR::isError($err)) {
@@ -2596,7 +2596,7 @@
     }
 
     // }}}
-    // {{{ function &_wrapResult($result, $types = array(), $result_class = true, $result_wrap_class = false, $limit = null, $offset = null)
+    // {{{ function &_wrapResult($result_resource, $types = array(), $result_class = true, $result_wrap_class = false, $limit = null, $offset = null)
 
     /**
      * wrap a result set into the correct class
@@ -2613,13 +2613,13 @@
      *
      * @access  protected
      */
-    function &_wrapResult($result, $types = array(), $result_class = true,
+    function &_wrapResult($result_resource, $types = array(), $result_class = true,
         $result_wrap_class = false, $limit = null, $offset = null)
     {
         if ($types === true) {
             if ($this->supports('result_introspection')) {
                 $this->loadModule('Reverse', null, true);
-                $tableInfo = $this->reverse->tableInfo($result);
+                $tableInfo = $this->reverse->tableInfo($result_resource);
                 if (PEAR::isError($tableInfo)) {
                     return $tableInfo;
                 }
@@ -2644,7 +2644,7 @@
                     'result class does not exist '.$class_name, __FUNCTION__);
                 return $err;
             }
-            $result =& new $class_name($this, $result, $limit, $offset);
+            $result = new $class_name($this, $result_resource, $limit, $offset);
             if (!MDB2::isResultCommon($result)) {
                 $err =& $this->raiseError(MDB2_ERROR_NOT_FOUND, null, null,
                     'result class is not extended from MDB2_Result_Common', __FUNCTION__);
@@ -2667,7 +2667,7 @@
                     'result wrap class does not exist '.$result_wrap_class, __FUNCTION__);
                 return $err;
             }
-            $result = new $result_wrap_class($result, $this->fetchmode);
+            $result = new $result_wrap_class($result_resource, $this->fetchmode);
         }
         return $result;
     }

--- End Message ---
--- Begin Message ---
Control: tags -1 + wontfix

On Wed, 2014-07-09 at 15:41 +0100, Adam D. Barratt wrote:
> On 2014-07-09 15:08, Thomas Goirand wrote:
> > On 07/09/2014 03:22 AM, Adam D. Barratt wrote:
> >> On Wed, 2014-01-22 at 14:35 +0200, Teodor MICU wrote:
> >>> I don't have the necessary experience to make this source debdiff.
> >>> Thomas, are you still interested in uploading this (simple) patch for
> >>> squeeze?
> >> 
> >> We don't appear to have seen a reply to that, and we're now only a few
> >> days away from the final point release for squeeze.
> [...]
> > Is it ok if I upload now?
> 
> Please could we have a debdiff, just to confirm?

Unfortunately we never got one and as today was the final point release
for squeeze I'm going to close this bug now.

Regards,

Adam

--- End Message ---

Reply to: