Uploaded image for project: 'Couchbase PHP client library'
  1. Couchbase PHP client library
  2. PCBC-181

The documentation is wrong for get() in couchbase-api.php

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.2
    • Fix Version/s: 1.1.3
    • Component/s: docs
    • Security Level: Public
    • Labels:
      None
    • Environment:
      MacOS X, PHP 5.4

      Description

      I just discovered this while fixing a bug in Basement.

      The $cas variable callback is not working on get() but works on getMulti().

      Simple example:

      Works:
      $client->set("foo", "bar");
      $cas = null;
      var_dump($client->getMulti(array("foo"), $cas));
      var_dump($cas);

      array(1)

      { ["foo"]=> string(3) "bar" }

      array(1)

      { ["foo"]=> string(20) "11699665754086047744" }

      Doesn't work:
      $client->set("foo", "bar");
      $cas = null;
      var_dump($client->get("foo", $cas));
      var_dump($cas);

      string(3) "bar"
      NULL

      The doc gets returned properly, but the cas value is not set. Also, when you set $cas to "" or false (not null), then it complains with a dubios error:
      Warning: Couchbase::get() expects parameter 2 to be a valid callback, no array or string given in /Users/michael/Webserver/basement-test/index.php on line 10
      Note that its a warning and not an exception on the OOP interface!

      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        daschl Michael Nitschinger created issue -
        daschl Michael Nitschinger made changes -
        Field Original Value New Value
        Priority Critical [ 2 ] Major [ 3 ]
        Component/s docs [ 10135 ]
        Component/s library [ 10134 ]
        ingenthr Matt Ingenthron made changes -
        Assignee Matt Ingenthron [ ingenthr ] Trond Norbye [ trond ]
        ingenthr Matt Ingenthron made changes -
        Fix Version/s 1.1.3 [ 10452 ]
        trond Trond Norbye made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        trond Trond Norbye made changes -
        Summary $cas callback not working on get() The documentation is wrong for get() in couchbase-api.php
        trond Trond Norbye made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        brett19 Brett Lawson made changes -
        Workflow jira [ 23673 ] Couchbase SDK Workflow [ 44387 ]

          People

          • Assignee:
            trond Trond Norbye
            Reporter:
            daschl Michael Nitschinger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes