Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #603 #604

Closed
wants to merge 9 commits into from
Closed

Fix #603 #604

wants to merge 9 commits into from

Commits on Feb 14, 2013

  1. A test case that proves a strange behavior.

    TestSuite output:
    
        $ phpunit test/testsuite/misc/MoreRelationTest.php
        PHPUnit 3.7.13 by Sebastian Bergmann.
    
        Configuration read from /Users/marc/Propel/phpunit.xml.dist
    
        F
    
        Time: 0 seconds, Memory: 18.75Mb
    
        There was 1 failure:
    
        1) MoreRelationTest::testContentsDeletion
        We assigned a collection of only one item.
        Failed asserting that 4 matches expected 1.
    
        /Users/marc/Propel/test/testsuite/misc/MoreRelationTest.php:97
    
        FAILURES!
        Tests: 1, Assertions: 2, Failures: 1.
    MArcJ committed Feb 14, 2013
    Configuration menu
    Copy the full SHA
    35dfd06 View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2013

  1. Fixed propelorm#603.

    Added a test set that proves the bug.
    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    0fe5e3e View commit details
    Browse the repository at this point in the history
  2. typo

    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    ebf84cd View commit details
    Browse the repository at this point in the history
  3. Added new method isAtLeastOneForeignPrimaryKey and `getForeignPrima…

    …ryKeys` at class ForeignKey
    
    and use it in the PHP5ObjectBuilder to check whether we have to backup the data of the relation entries
    to be able to manipulate it later.
    Added these methods because `isForeignPrimaryKey` returns only true if _all_ foreign columns are pks,
    which is not useful in this case.
    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    06f4873 View commit details
    Browse the repository at this point in the history
  4. Use clone instead of unserialize(serialize. Added another test that…

    … proves that 'onDelete=setnull' works, too.
    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    0f1bcbc View commit details
    Browse the repository at this point in the history
  5. lol :D

    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    604222c View commit details
    Browse the repository at this point in the history
  6. blank lines

    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    456fe80 View commit details
    Browse the repository at this point in the history
  7. Fixed propelorm#603

    MArcJ committed Feb 15, 2013
    Configuration menu
    Copy the full SHA
    bb9ae3d View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    015ac3a View commit details
    Browse the repository at this point in the history