Doctrine 1

AddPendingJoinConditions doesn't work if alias equals model name

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Critical Critical
  • Resolution: Unresolved
  • Affects Version/s: 1.2.1
  • Fix Version/s: None
  • Component/s: Query
  • Labels:
    None

Description

If the alias in a from-query is equal to the model name, I get an exception when executing the query after adding a "pendingJoinCondition".

See attached testCase.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
David Abdemoulaie added a comment -

I fail to see why you would use an alias that is equal to the model name.

Show
David Abdemoulaie added a comment - I fail to see why you would use an alias that is equal to the model name.
Hide
Permalink
Christian Jaentsch added a comment -

To clarify again: Using an alias equal to the model name is the same as not using an alias at all. The built sql is the same afterwards (and the addPendingJoinCondition behaviour is the same as well).

I would not use an alias equal to the model name myself, but sometimes this is used in Doctrine core, e.g. in Doctrine_Relation_ForeignKey around line 60. So to solve the problem either all the core queries, where no alias or an alias equal to the model name is used, have to be rewritten or the behaviour of a respective query has to be changed somehow so that applying an addPendingJoinCondition is possible even in the given case.

Show
Christian Jaentsch added a comment - To clarify again: Using an alias equal to the model name is the same as not using an alias at all. The built sql is the same afterwards (and the addPendingJoinCondition behaviour is the same as well). I would not use an alias equal to the model name myself, but sometimes this is used in Doctrine core, e.g. in Doctrine_Relation_ForeignKey around line 60. So to solve the problem either all the core queries, where no alias or an alias equal to the model name is used, have to be rewritten or the behaviour of a respective query has to be changed somehow so that applying an addPendingJoinCondition is possible even in the given case.

People

  • Assignee:
    Guilherme Blanco
    Reporter:
    Christian Jaentsch
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: