Propel 在使用别名连接表时添加 CROSS JOIN 查询

Question

尝试在 Propel 2 中做一个相当简单的查询。我有一个人 table 和一个财产 table - 人可以拥有许多财产，但每种财产类型只能有一个。所以一个人可以拥有 1 本书、1 辆车等。我正在尝试在 Propel 中编写一个查询，如果他们有车，将 return 所有人连同他们的车名。这是代码和生成的查询：

$x = PersonQuery::create()
  ->groupById()
  ->leftJoinPossession()
  ->addJoinCondition('Possession','Possession.possession_type = ?', 'car')
  ->withColumn('Possession.possession_name', 'CarName')
  ->where('Possession.possession_name IS NOT NULL')
  ->find()
  ->toString();

//resulting sql
SELECT person.id, possession.possession_name as CarName
FROM person
LEFT JOIN possession  ON (person.id=possession.person_id AND possession.possession_type = 'car')
WHERE possession.possession_name IS NOT NULL;

这按预期工作。但是，我需要对财产 table 进行多次连接（例如，为每个人获取书籍），因此我需要使用别名。下面是当我修改前面的查询只是为了使用别名作为财产 table 时发生的情况（并且还为每个人获取这本书）：

$x = PersonQuery::create()
  ->groupById()
  ->leftJoinPossession('p')
  ->addJoinCondition('p','p.possession_type = ?', 'car')
  ->leftJoinPossession('p2') 
  ->addJoinCondition('p2','p2.possession_type = ?', 'book')
  ->withColumn('p.possession_name', 'CarName')
  ->withColumn('p2.possession_name', 'BookName')
  ->where('p.possession_name IS NOT NULL')
  ->find()
  ->toString();

//resulting sql
SELECT person.id, p.possession_name as CarName, p2.possession_name as BookName
FROM person
CROSS JOIN possession
LEFT JOIN possession p ON (person.id=p.person_id AND p.possession_type = 'car')
LEFT JOIN possession p2 ON (person.id=p2.person_id AND p2.possession_type = 'book')
WHERE p.possession_name IS NOT NULL;

如您所见，出于某种原因，Propel 将 "CROSS JOIN possession" 添加到查询中。这不会改变查询的结果，但会使其非常慢。关于如何告诉 Propel 不要使用 CROSS JOIN 同时仍然为我加入的 table 使用别名有什么想法吗？（如果我取出 'where' 子句，CROSS JOIN 也会消失）

Answer 1

你为什么要离开？如果我对您的 post 的理解正确，您只需要实际拥有汽车的人的结果。所以以下应该有效：

PersonQuery::create()
  ->withColumn('Possession.possession_name', 'CarName')
  ->usePossessionQuery()
  ->filterByPossessionType('car')
  ->endUse()
  ->find()
  ->toString();

useXxxQuery() 也可以给一个别名，所以你应该能够完成同样的事情，但更容易:)

编辑：根据您的第二个示例，具有多个连接的示例：

PersonQuery::create()
  ->withColumn('possession_car.possession_name', 'CarName')
  ->withColumn('possession_book.possession_name', 'BookName')
  ->usePossessionQuery('possession_car')
  ->filterByPossessionType('car')
  ->endUse()
  ->usePossessionQuery('possession_book')
  ->filterByPossessionType('book')
  ->endUse()
  ->find()
  ->toString();

Answer 2

找到解决此问题的方法。问题是，当我尝试对同一个 table 进行多个左连接（通过使用别名）时，Propel 也会包含对 table 的交叉连接。但是，我发现如果第一次连接没有使用别名，Propel 不会添加交叉连接。

所以，总而言之，这是行不通的：

$x = PersonQuery::create()
  ->groupById()
  ->leftJoinPossession('p')
  ->addJoinCondition('p','p.possession_type = ?', 'car')
  ->leftJoinPossession('p2') 
  ->addJoinCondition('p2','p2.possession_type = ?', 'book')
  ->withColumn('p.possession_name', 'CarName')
  ->withColumn('p2.possession_name', 'BookName')
  ->where('p.possession_name IS NOT NULL')
  ->find()
  ->toString();

//resulting sql
SELECT person.id, p.possession_name as CarName, p2.possession_name as BookName
FROM person
CROSS JOIN possession
LEFT JOIN possession p ON (person.id=p.person_id AND p.possession_type = 'car')
LEFT JOIN possession p2 ON (person.id=p2.person_id AND p2.possession_type = 'book')
WHERE p.possession_name IS NOT NULL;

但这按预期工作：

$x = PersonQuery::create()
  ->groupById()
  ->leftJoinPossession() //changed
  ->addJoinCondition('Possession','Possession.possession_type = ?', 'car') //changed
  ->leftJoinPossession('p2') 
  ->addJoinCondition('p2','p2.possession_type = ?', 'book')
  ->withColumn('p.possession_name', 'CarName')
  ->withColumn('p2.possession_name', 'BookName')
  ->where('p.possession_name IS NOT NULL')
  ->find()
  ->toString();

//resulting sql
SELECT person.id, p.possession_name as CarName, p2.possession_name as BookName
FROM person
LEFT JOIN possession ON (person.id=posession.person_id AND posession.possession_type = 'car')
LEFT JOIN possession p2 ON (person.id=p2.person_id AND p2.possession_type = 'book')
WHERE p.possession_name IS NOT NULL;

Answer 3

这看起来确实像一个错误。你举报了吗？

一般来说，Propel 会检查 relation/aliases 你的 where 条件指向什么，然后确保所有这些 table 都包含在查询中。如果 relation/alias 没有现成的连接子句，它会将其包含在 CROSS JOIN 中。

->where() 方法没有通过方法参数提供的关系别名（就像 filterByX 有），并且 while propel 看到 where() 条件的 'p.' 部分映射到占有 table，它没有意识到它是通过使用关系别名得出的结论（这是错误），所以 propel 认为 where() 想要检查 possession table（没有别名），因此包含交叉连接。

由于这个错误，您的解决方法只能这样工作，写起来会更清楚：

 ->where('possession.possession_name IS NOT NULL')

... 澄清那里 where() 指的是无别名的财产 table 而不是特定的关系别名。

如果我是正确的，如果您引用了第二个别名，您的解决方法应该不会起作用：

 ->where('p2.possession_name IS NOT NULL')

（它仍然会引用第一个关系）

PS 恕我直言，Propel 应该抛出异常而不是查询的这种自动交叉连接 "fixing"，因为大多数情况下，交叉连接是拼写错误或不正确的不良结果查询方法的使用。

同时，这里有一个帮助代码的要点，让您检查这些条件并在必要时抛出异常：https://gist.github.com/motin/2b00295ca71bb876f9873712544dd077

Propel 在使用别名连接表时添加 CROSS JOIN 查询

Propel adds CROSS JOIN to query when using an alias to JOIN tables

php

mysql

sql

propel

propel2