Doctrine 1

Versionable behavior not use the right connection

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0.12, 1.2.0, 1.2.1
  • Fix Version/s: 1.2.2
  • Component/s: Behaviors
  • Labels:
    None
  • Environment:
    symfony 1.2.9

Description

I have 2 connections and a table with versionable behavior. The problem is that doctrine use the last connection defined in the database.yml for the article_version table not the connection from the article table.

I think it is also good idea to add a connection option to the behavior.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jonathan H. Wage added a comment -

When I test this it works as expected and I took a look at the code and the generated models from behaviors inherit the connection from its parent.

Show
Jonathan H. Wage added a comment - When I test this it works as expected and I took a look at the code and the generated models from behaviors inherit the connection from its parent.
Hide
Permalink
Ian P. Christian added a comment -

I can reproduce this.... but, not reliably at the moment. There does seem to be an issue though

Show
Ian P. Christian added a comment - I can reproduce this.... but, not reliably at the moment. There does seem to be an issue though
Hide
Permalink
Roman S. Borschel added a comment -

reopening on behalf of pookey/Ian.

Show
Roman S. Borschel added a comment - reopening on behalf of pookey/Ian.
Hide
Permalink
Ian P. Christian added a comment -

whilst I'm still trying to track down exactly how to reproduce this in a nice simple test... I have found this proof that the issue exists.

Modifying AuditLog::getMaxVersion() to include the following:

        $connName1 = Doctrine::getTable('RadiusUserNote')->getConnection()->getName();
        $connName2 = Doctrine::getTable($className)->getConnection()->getName();
        echo " - name of connecion used for 'RadiusUserNote' is {$connName1}, however.... \n";
        echo " the name of connecion used for '{$className}' is {$connName2}\n";
        die();

results in:

- name of connecion used for 'RadiusUserNote' is radius_christina, however.... the name of connecion used for 'RadiusUserNoteVersion' is maildb
Show
Ian P. Christian added a comment - whilst I'm still trying to track down exactly how to reproduce this in a nice simple test... I have found this proof that the issue exists. Modifying AuditLog::getMaxVersion() to include the following: $connName1 = Doctrine::getTable('RadiusUserNote')->getConnection()->getName(); $connName2 = Doctrine::getTable($className)->getConnection()->getName(); echo " - name of connecion used for 'RadiusUserNote' is {$connName1}, however.... \n" ; echo " the name of connecion used for '{$className}' is {$connName2}\n" ; die(); results in: - name of connecion used for 'RadiusUserNote' is radius_christina, however.... the name of connecion used for 'RadiusUserNoteVersion' is maildb
Hide
Permalink
Micah Gersten added a comment - - edited

This patch checks the Manager for a bound component instead of relying on the internal connection of the class at the time.

This is against Doctrine 1.2.1

Show
Micah Gersten added a comment - - edited This patch checks the Manager for a bound component instead of relying on the internal connection of the class at the time. This is against Doctrine 1.2.1
Hide
Permalink
Micah Gersten added a comment -

Affects 1.2.1 as well.

Show
Micah Gersten added a comment - Affects 1.2.1 as well.

People

  • Assignee:
    Jonathan H. Wage
    Reporter:
    Gordon Franke
Vote (1)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: