Doctrine 1
  1. Doctrine 1
  2. DC-197

[patch] default model orderBy option breaks data-load task

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.0-BETA1
    • Fix Version/s: 1.2.0-BETA2
    • Component/s: Query
    • Labels:
      None

      Description

      currently the default orderBy clause one can specify for models gets added to all queries, including DELETE and UPDATE.
      This breaks data-load because it includes in the orderBy the table alias but currently only SELECT queries support aliases.

      For example if I have the following schema:

      Mytable:
      options:

      { orderBy: name ASC }

      columns:
      name:

      { type: string(100), notnull: true }

      The data-load task does a Doctrine::getTable('Mytable')>delete()>execute(); to clear the table before loading the fixtures which results in the following SQL:
      DELETE FROM mytable ORDER BY m.name ASC

      which errors out as the m alias was not defined.

      Attached is the patch that makes only SELECT queries get the default orderBy clause from the option.

        Activity

        There are no comments yet on this issue.

          People

          • Assignee:
            Jonathan H. Wage
            Reporter:
            Ariel Arjona
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: