LaravelのEloquent ORMは複合主キーに対応していなかった

注意：Laravel4の話です

2013-09-19追記
こちらの記事ですが、以下の記事の内容を行うと、insert等ができなくなるようになってしまいました・・・。
どうやら最後のIDが取得出来ないからこうなってしまうようです。

互換性などの点からもIDは付けるようにしましょう。複合ユニークキーとインデックスをつければそれなりにちゃんと速くなるはず

こんにちは。ファガイです。

先日から個人的にサイトを制作していたのですが、衝撃の事実がわかりまして・・・

Eloquent ORMは複合主キーに対応していない！

うん、複合主キーに対応していない。

皆見たらわかると思うんだ、Illuminate\Database\Eloquent\Builderのfindメソッドを見てみよう。

public function find($id, $columns = array('*'))
{
　　　　$this->query->where($this->model->getKeyName(), '=', $id);

　　　　return $this->first($columns);
}

あとは、Illuminate\Database\Eloquent\Modelとかも。

public static function find($id, $columns = array('*'))
{
    $instance = new static;

    if (is_array($id))
    {
        return $instance->newQuery()->whereIn($instance->getKeyName(), $id)->get($columns);
    }

    return $instance->newQuery()->find($id, $columns);
}

そもそも、primaryKeyは１つであると確定しているのも問題じゃないのかと思ってる。
リレーションを組もうとしたが、この仕様だと、primaryKeyは１つでなければいけない。複合主キーで組もうと思ったらリレーションが組めない。
結構大問題だった。
大問題でも無かった。Query Builderで取ってきてくっつければ普通に取得できた。
Eloquent Modelのfindメソッドの動き的にid複数を取得して来れる的な仕様と見れる。だがQuery Builderには実装されていない。まだ粗はありそう。

対処方法

上記で出したメソッドにidがarrayで来たとかprimaryKeyがarrayだったとか判定が必要。
あと、コアとなっているEloquent\Relations\BelongsTo等のaddConstrationsをいじる必要がある。
これを修正しないと、リレーションを組むことが出来ませんでした。

自分の場合はこのような形に変更しています。例えば、BelongsToの場合。

public function addConstraints()
{
    if (static::$constraints)
    {
        $key = $this->related->getKeyName();

        $table = $this->related->getTable();

        $count = count($key);
        if($count > 1) {
            for($i = 0; $i < $count; ++$i) {
                $this->query->where($table.'.'.$key[$i], '=', $this->parent->{$this->foreignKey[$i]});
            }
        }else{
            $this->query->where($table.'.'.$key, '=', $this->parent->{$this->foreignKey});
        }
    }
}

コアとなっている部分をいじってしまっているので、composer updateをやってしまうともう一度直さなきゃいけなさそうですね。
Eloquent Modelの方は一つBaseEloquentとかモデルを用意して、それを継承させる形にすればよいでしょう。

ではではー。

2013-09-08追記
複合主キーは拡張性が無くなったり、変更に強くないため主キーが１つなのかもしれませんね・・。しかしながらスキーマビルダーは複合主キーが定義出来る・・・なんだろうこの矛盾