Couchbase PHP client library

Undefined Index from existing keys in getMulti result

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0.1
  • Fix Version/s: 1.0.4
  • Component/s: None
  • Security Level: Public
  • Labels:
    None

Description

Bug reporter is noticing some strange behavior from the getMulti function: keys which exist and are visible when print_r'ing the response give an undefined index error when reporter tries to access them.

Full details are in http://www.couchbase.com/forums/thread/php-undefined-index-existing-keys-getmulti-result

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
greg schoen added a comment -
I just noticed this issue as well when trying to switch over from Memcache to Couchbase. I came up with the following hack until it is corrected in the library.

$get = $couchbase->getMulti($keys);

foreach($get as $k => $v)
{
$hack_array[$k] = $v;
}
$get = $hack_array;

It's interesting to note that you can iterate through the array with a foreach and setting it into a new array lets you access the values.
Show
greg schoen added a comment - I just noticed this issue as well when trying to switch over from Memcache to Couchbase. I came up with the following hack until it is corrected in the library. $get = $couchbase->getMulti($keys); foreach($get as $k => $v) { $hack_array[$k] = $v; } $get = $hack_array; It's interesting to note that you can iterate through the array with a foreach and setting it into a new array lets you access the values.
Hide
Permalink
Matt Ingenthron added a comment -
I think this is something at the extension level. Asking Laruence to have a look.
Show
Matt Ingenthron added a comment - I think this is something at the extension level. Asking Laruence to have a look.
Hide
Permalink
Matt Ingenthron added a comment -
Discussion with Laruence indicates that it was a libcouchbase bug which has been fixed. We'll add the referenced test to the suite and verify that it works with the latest libcouchbase.
Show
Matt Ingenthron added a comment - Discussion with Laruence indicates that it was a libcouchbase bug which has been fixed. We'll add the referenced test to the suite and verify that it works with the latest libcouchbase.
Hide
Permalink
Matt Ingenthron added a comment -
Update: it was actually a bug fixed in the extension now verified. This was actually fixed in 1.0.4 and this issue is a dupe of PCBC-66 (now, anyway).
Show
Matt Ingenthron added a comment - Update: it was actually a bug fixed in the extension now verified. This was actually fixed in 1.0.4 and this issue is a dupe of PCBC-66 (now, anyway).

People

  • Assignee:
    Matt Ingenthron
    Reporter:
    Matt Ingenthron
Vote (2)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: