Tharos / LeanQuery

www.leanmapper.com
MIT License
6 stars 5 forks source link

\LeanQuery\DomainQuery::getEntity netreba pridat limit? #5

Open achtan opened 10 years ago

achtan commented 10 years ago

aktualny stav getEntity() je:

    public function getEntity()
    {
        $entities = $this->getEntities();
        return ($entity = reset($entities)) !== false ? $entity : null;
    }

cize vytiahne vsetky entity a potom uz len php vyberie prvu.

Nebolo by efektivnesie pridat limit 1 a az potom vytahovat entity ?

Tharos commented 10 years ago

Samozřejmě. Určitě tuhle metodu ještě upravím, až přibude podpora pro limit a offset (viz #4).

Aktuální implementace je taková naivní v tom, že předpokládá, že $entities bude stejně obsahovat jenom jednu entitu. Tj. že se bude číst podle PK nebo jiných jednoznačně identifikujících restrikcí.