Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-2069

Spaces are not ignored in the "column" list of a uniqueConstraints

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3.1
    • Component/s: Mapping Drivers
    • Security Level: All
    • Labels:

      Description

      In a YAML configuration file

      Behavior correct for this:

        uniqueConstraints:
          myConstraint:
            columns: column1,column2
      

      However, if I add a space in the "columns" list:

        uniqueConstraints:
          myConstraint:
            columns: column1, column2
      

      I end up with the following exception:

      exception 'Doctrine\DBAL\Schema\SchemaException' with message 'There is no column with name ' column2' on table 'myTable'.
      

      This can be very confusing, especially if (by any chance) you have a line return just on the space in the command line: you never see the space in the exception message.

      It seems that YAML allows spaces in lists (http://en.wikipedia.org/wiki/YAML#Lists), but this line doesn't seem to be a YAML list. If it is parsed by Doctrine (split on ","), spaces should be ignored (or trimmed).

        Activity

        Hide
        Matthieu Napoli added a comment -

        Note that using the correct YAML format works:

        uniqueConstraints:
            myConstraint:
              columns: [column1, column2]
        

        So it's confusing to have 2 possibilities to write the same thing, and that they don't behave the same.

        Show
        Matthieu Napoli added a comment - Note that using the correct YAML format works: uniqueConstraints: myConstraint: columns: [column1, column2] So it's confusing to have 2 possibilities to write the same thing, and that they don't behave the same.
        Show
        Fabio B. Silva added a comment - Fixed by : https://github.com/doctrine/doctrine2/commit/814f2f9e039caf712b75af7f61d28c361189bcab

          People

          • Assignee:
            Fabio B. Silva
            Reporter:
            Matthieu Napoli
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: