SQL table: ->find(['TEST_ID' => 1]) lists all rows but only with the data from first row ? about mapper HOT 4 CLOSED

@nekufa Perfect, i Will test it again.

Regards

from mapper.

I have found where the bug is causing the issue,
In Repository.php file line 213-219 there is a code:

foreach ($data as $tuple) {
    $instance = $this->getInstance($tuple);
    if ($one) {
        return $this->results[$cacheKey] = $instance;
    }
    $result[] = $instance;
}

I replaced it with:

foreach ($data as $tuple) {
    echo 'tuple:';
    var_dump($tuple);
    $instance = $this->getInstance($tuple);
    echo 'instance:';
    var_dump($instance);
    echo '<hr>';
    if ($one) {
        return $this->results[$cacheKey] = $instance;
    }
    $result[] = $instance;
}

and it seems to me that the request to $this->getInstance($tuple); is only returning the first row even though the tuple is shown with correct data (test my code and you will see).

EDIT:
I changed getTupleKey to the following code in Space.php:

public function getTupleKey(array $tuple)
{
    $key = [];
    foreach ($this->getPrimaryIndex()['parts'] as $part) {
        echo 'part';
        var_dump($part);
        if(isset($part[0]))
            $key[] = $tuple[$part[0]];
        if(isset($part['field']))
            $key[] = $tuple[$part['field']];
        
    }
    return count($key) == 1 ? $key[0] : implode(':', $key);
}

and now it shows all rows data correct, but it still shows all rows, so probably the function getInstance in file Repository.php needs to be adapted for SQL tables to only return the correct data.

regards Zilveer

from mapper.

Hello @zilveer, thanks for you feedback!
I found out why sql-based spaces were not working - index format was changed.
This bug was fixed in the latest version, let me know if something else goes wrong.

from mapper.

Perfect it works good now!

from mapper.