Yii Framework

lancedevnull писал(а):по pk дефолтное поведение ::find($pk) findOne($pk)[,findAll($pk)]

lancedevnull писал(а):https://github.com/yiisoft/yii2/blob/ma ... n-database

public static function findOne($condition)
    {
        $query = static::find();
        if (ArrayHelper::isAssociative($condition)) {
            // hash condition
            return $query->andWhere($condition)->one();
        } else {
            // query by primary key
            $primaryKey = static::primaryKey();
            if (isset($primaryKey[0])) {
                return $query->andWhere([$primaryKey[0] => $condition])->one();
            } else {
                throw new InvalidConfigException(get_called_class() . ' must have a primary key.');
            }
        }
    }

maleks писал(а):А что вы в scopes() хотите для запроса по первичному ключу предустановить?
Например
->select(поля)
->asArray()
?
Явно ж там не ограничение множества строк, т.к. идет то по PK все равно.

<?php
$post = Post::find()
    ->onlyActive()
    ->onlySomeAnotherCondition()
    ->findByPk($id);

Sam Dark писал(а):Кидайте на github, ещё не поздно.

lancedevnull писал(а):Model::find($pk)->active()->all()

/**
   * @inheritdoc
   */
  public static function find()
  {
      return new ActiveQuery(get_called_class());
  }

vladqa писал(а): Конечно там ограничение количества строк.
Например мне нужно выбрать модель по id, то только такую, которая имеет статус "Active" и удовлетворяет другим доп. условиям:

Код: Выделить всё

<?php
$post = Post::find()
    ->onlyActive()
    ->onlySomeAnotherCondition()
    ->findByPk($id);

Вообще я считаю, что любые фильтры и все варианты доп. условий для выборок надо инкапсулировать scopes (они ж для этого и придуманы), ибо иначе мы получаем неподдерживаемый код, который с трудом поддается рефакторингу.

maleks писал(а):

vladqa писал(а): Конечно там ограничение количества строк.
Например мне нужно выбрать модель по id, то только такую, которая имеет статус "Active" и удовлетворяет другим доп. условиям:

Код: Выделить всё

<?php
$post = Post::find()
    ->onlyActive()
    ->onlySomeAnotherCondition()
    ->findByPk($id);

Вообще я считаю, что любые фильтры и все варианты доп. условий для выборок надо инкапсулировать scopes (они ж для этого и придуманы), ибо иначе мы получаем неподдерживаемый код, который с трудом поддается рефакторингу.

Так Primary key уникален же для всех строк таблицы, и только по нему точно одна строка и найдется и никакая другая.

<?php
$posts = Post::find()
    ->onlyActive()
    ->onlySomeAnotherCondition()
    ->byPK([10, 20, 30])
    ->all();