Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.2.1
    • Fix Version/s: None
    • Component/s: Migrations
    • Labels:
      None
    • Environment:
      Symfony 1.4.3, Linux, PostgreSQL, Pgpool II

      Description

      I attached a testcase that reproduces all problem that I noticed (previously reported in "serious symfony doctrine:migrate issues - symfony 1.4" on symfony-dev)

      There is a problem with an empty ON UPDATE when migrate to version 5

      • SQLSTATE[42601]: Syntax error: 7 BŁĄD: błąd składni w lub blisko "ON"
        LINE 1: ... (object_c_id) REFERENCES object_c(id) ON UPDATE ON DELETE ...
        ^. Failing Query: "ALTER TABLE object_b ADD CONSTRAINT object_b_object_c_id_object_c_id FOREIGN KEY (object_c_id) REFERENCES object_c(id) ON UPDATE ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE"

      When migrateing from 5 to 3
      Failing Query: "DROP INDEX object_b_object_c_id_idx" - this index doesn't exist - should be "object_b_object_c_id"

      Also
      $this->changeColumn('object_a', 'name', 'string', '2048', array(
      ));
      $this->changeColumn('object_b', 'name', 'string', '2048', array(
      ));
      doesn't work for me

        Activity

        MichalKJP created issue -
        Hide
        Ludo added a comment - - edited

        My own Quick Fix for DROP INDEX (into Export class)
        Skip Formatter->getIndexName() in Export->dropIndex()
        see : http://www.pastie.org/1145916

        Show
        Ludo added a comment - - edited My own Quick Fix for DROP INDEX (into Export class) Skip Formatter->getIndexName() in Export->dropIndex() see : http://www.pastie.org/1145916
        Hide
        MichalKJP added a comment -

        I already tested this solution and I can confirm that also works for me.

        I am afraid that Johnatan spends much time on the new version and did not have time to correct these bugs for the old version. So we still repair migrations manualy.

        Show
        MichalKJP added a comment - I already tested this solution and I can confirm that also works for me. I am afraid that Johnatan spends much time on the new version and did not have time to correct these bugs for the old version. So we still repair migrations manualy.
        Hide
        James Bell added a comment -

        We've had this issue for a while - while Ludo's fix does work, there is an SQL injection issue in that the '$name' doesn't get escaped.

        You can do this: $name = $this->conn->quoteIdentifier($name); which also steps around the issue for now.

        The problem appears to be that the indexes aren't being created with the _idx string on the end when using the migrations. This is probably down to the index creation process rather than the index removal process.

        Show
        James Bell added a comment - We've had this issue for a while - while Ludo's fix does work, there is an SQL injection issue in that the '$name' doesn't get escaped. You can do this: $name = $this->conn->quoteIdentifier($name); which also steps around the issue for now. The problem appears to be that the indexes aren't being created with the _idx string on the end when using the migrations. This is probably down to the index creation process rather than the index removal process.

        This list may be incomplete, as errors occurred whilst retrieving source from linked applications:

        • Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DC-617, expand=changesets[0:20].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)

          People

          • Assignee:
            Jonathan H. Wage
            Reporter:
            MichalKJP
          • Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: