10
6

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 5 years have passed since last update.

TypeORMのリレーション定義に気をつけよう。死ぬぞ。

Last updated at Posted at 2018-07-30

いつものようにあら便利カレンダーらしく、死んだ話です。

TypeORMというTypeScriptで使えるORMがあるんですが、
OneToManyなリレーションを作ったら死んでしまいました。

tl;dr

@ManyToOne(type => User, user => user.interests)
user: User;

このデコレータの中の引数かかないとts的なエラーを吐かずにMySQLがSyntaxError吐いて死にます。

原因

class Interest {
    @ManyToOne(type => User, user => user.interests)
    user: User;
}

これどうなっているかもぐっていくと SelectQueryBuilder.ts にたどりつきます。
createJoinExpression()というJOINされる部分のクエリを組み立てる実装がどうなっているか見てみる。

const condition = relation.inverseRelation!.joinColumns.map(joinColumn => {
    return destinationTableAlias + "." + relation.inverseRelation!.propertyPath + 
    "." + joinColumn.referencedColumn!.propertyPath + "=" +
    parentAlias + "." + joinColumn.referencedColumn!.propertyPath;
}).join(" AND ");

return " " + joinAttr.direction + " JOIN " + this.getTableName(destinationTableName) + " " + 
    this.escape(destinationTableAlias) + " ON " + this.replacePropertyNames(condition + appendedCondition);

このONのあとのJOIN条件を組み立てる部分が動きません。

SELECT ... 
FROM ... 
LEFT JOIN ... 
ON

でクエリが終わってしまってそりゃSyntaxError出るよなっていう話でした。
jsレイヤーでエラー吐けや!!!

余談

なんとこのクエリ、たまたま1480文字で死んでいたので最初はクエリがちゃんと送信できてない!みたいにしてネットワーク周りの見直しをしてました。
開発環境がdockerなのでMTUが問題になってるんじゃ!?みたいな勘違いぶちかまして情けないことに2人日くらいまるまるふっとびました。
マジなさけねえ...

ちなみに、もう一回ちゃんと調べてみようってことでTypeORMをcloneしてちゃんとデバッグした結果わかったことなので社内では誇らしい顔しながら土下座しました。

10
6
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
10
6

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?