Permalink
Commits on Oct 4, 2011
  1. merged branch SongoQ/polish_translations (PR #2315)

    Commits
    -------
    
    31840b9 Updated Polish validator translations (trans-unit id="47" and id="48")
    
    Discussion
    ----------
    
    Updated Polish validator translations (trans-unit id="47" and id="48")
    fabpot committed Oct 4, 2011
  2. merged branch alifity/id-trans-unit-47-48 (PR #2316)

    Commits
    -------
    
    8404b35 Updated indonesian translations for trans-unit 47 and 48
    
    Discussion
    ----------
    
    Updated indonesian translations for trans-unit 47 and 48
    fabpot committed Oct 4, 2011
  3. merged 2.0

    fabpot committed Oct 4, 2011
Commits on Oct 3, 2011
  1. merged branch excelwebzone/hebrew_translations (PR #2314)

    Commits
    -------
    
    eac2a77  Oops.. translated the variable by mistake
    51b3b59  Updated Hebrew validator translations (trans-unit id="48")
    
    Discussion
    ----------
    
    Updated Hebrew validator translations (trans-unit id="48")
    fabpot committed Oct 3, 2011
  2. [Security] changed a RuntimeException to LogicException for consisten…

    …cies between the different Token classes (closes #2310)
    fabpot committed Oct 3, 2011
  3. merged branch dpb587/patch-appkern (PR #2312)

    Commits
    -------
    
    5419638 [HttpKernel] Show the actual directory needing to be created.
    
    Discussion
    ----------
    
    [HttpKernel] Show the actual directory needing to be created.
    
    Bug fix: yes
    Feature addition: no
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: -
    
    before: Unable to create the logs directory (/home/symfony/root/app)
    after: Unable to create the logs directory (/home/symfony/root/app/logs)
    fabpot committed Oct 3, 2011
  4. merged branch aboks/translations_nl (PR #2307)

    Commits
    -------
    
    0299d38 [FrameworkBundle] Updated Dutch validator translations
    
    Discussion
    ----------
    
    [FrameworkBundle] Updated Dutch validator translations
    
    Bug fix: yes
    Feature addition: no
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: -
    fabpot committed Oct 3, 2011
Commits on Oct 1, 2011
Commits on Sep 30, 2011
  1. merged branch jalliot/missing-guessers (PR #2306)

    Commits
    -------
    
    ee0fe7a Added guessers for Size and SizeLength constraints
    
    Discussion
    ----------
    
    Added guessers for Size and SizeLength constraints
    
    Bug fix: no
    Feature addition: yes
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: -
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/30 13:40:37 -0700
    
    BTW, I've noticed that some constraints currently don't have guessers in 2.0:
    
    * ``False`` (which could be guessed as a checkbox)
    * ``True`` (which could be guessed as a required checkbox)
    * ``Choice`` (which could be guessed as a choice type with medium confidence and with the choice list being guessed as the list provided for the constraint)
    
    Are there any reasons why this is not implemented in 2.0 or should I try to make a PR for it?
    
    There is also the ``Collection`` case but I guess it would be too difficult for this one...
    fabpot committed Sep 30, 2011
  2. merged branch jalliot/trans-fr (PR #2305)

    Commits
    -------
    
    f9b2be9 Added french translation for SizeLength and UserPassword constraints
    
    Discussion
    ----------
    
    Added french translation for SizeLength and UserPassword constraints
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2011/09/30 13:45:46 -0700
    
    47 is already taken for "This value should be the user current password". Should be 48.
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/30 13:59:01 -0700
    
    @fabpot Fixed and added translation for 47 as well.
    fabpot committed Sep 30, 2011
  3. merged branch Gregwar/proxy_file_issue (PR #1968)

    Commits
    -------
    
    0e00e3f [DoctrineBundle] CS
    0c4b793 [DoctrineBundle] Fixed performances issues on "On-demand" proxy file generation
    e866a67 [DoctrineBundle] Tries to auto-generate the missing proxy files on the autoloader
    
    Discussion
    ----------
    
    [DoctrineBundle] Tries to auto-generate the missing proxy files on the autoloaded
    
    See:
    symfony#1965
    symfony#1535
    
    This fix is not really clean and there's maybe a factorizing work to do on it, but this work and avoid me spending my day deleting session cookies each time I clear cache.
    
    ---------------------------------------------------------------------------
    
    by stloyd at 2011/08/23 10:37:28 -0700
    
    You should follow Symfony2 CS (http://symfony.com/doc/current/contributing/code/standards.html).
    
    ---------------------------------------------------------------------------
    
    by ruudk at 2011/09/26 02:50:13 -0700
    
    +1
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2011/09/27 07:01:51 -0700
    
    It looks like a bug fix, so this PR should be closed and a new one based on the 2.0 branch should be open. @beberlei: are you fine with this patch?
    
    ---------------------------------------------------------------------------
    
    by beberlei at 2011/09/29 04:24:22 -0700
    
    What is this for? I dont understand the bug and the solution here screams cache slam.
    
    ---------------------------------------------------------------------------
    
    by beberlei at 2011/09/29 04:34:02 -0700
    
    Ok i get the problem but the solution is still a monsterous hack. Can we find a real solution to the problem? There has to be one.
    
    ---------------------------------------------------------------------------
    
    by Gregwar at 2011/09/29 04:34:25 -0700
    
    @beberlei, when an user is authenticated for instance, there can be proxies serialized in session.
    
    Si if you clear the cache in dev environment you'll get an error because the matching proxy classes won't exist and you'll be forced to clear your cookies and reauth, which can be annoying
    
    ---------------------------------------------------------------------------
    
    by Gregwar at 2011/09/29 04:38:45 -0700
    
    @beberlei, yes, I agree that we should do something more elegant, but the problem is that when PHP "meet" the proxy class we can't really know what was the "original" class it is supposed to extend
    
    And as @schmittjoh said, this will only be executed very rarely
    
    ---------------------------------------------------------------------------
    
    by beberlei at 2011/09/29 05:18:35 -0700
    
    You agree you want something more suitable and still want this to be merged?
    
    To ease the immediate pain wr could allow this however only in debug mode. A real solution here is maybe to move the proxy files out of the env folders. Rhey dont depend ont the env after all.
    
    ---------------------------------------------------------------------------
    
    by stloyd at 2011/09/29 05:21:33 -0700
    
    Proxy is not depending on env, but generation of proxy is... So this solution will be hard IMO, or even unacceptable...
    
    ---------------------------------------------------------------------------
    
    by Gregwar at 2011/09/29 05:25:39 -0700
    
    @beberlei what I meant is that I agree that's dirty but I don't think of anything better to solve this...
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2011/09/29 06:23:03 -0700
    
    Even if the current patch is not the best solution, we should probably apply it to fix the problem and think about a better solution afterwards. Does it sound good for everybody?
    fabpot committed Sep 30, 2011
  4. merged branch danielholmes/patch-2 (PR #2302)

    Commits
    -------
    
    84808f4 [DependencyInjection] Removed unused property
    
    Discussion
    ----------
    
    [DependencyInjection] Removed unused property
    
    Bug fix: no
    Feature addition: no
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: -
    fabpot committed Sep 30, 2011
  5. [DependencyInjection] Removed unused property

    Bug fix: no
    Feature addition: no
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: -
    danielholmes committed Sep 30, 2011
  6. merged branch dpb587/patch-monoweb (PR #2296)

    Commits
    -------
    
    e70c884 [Bridge/Monolog] Fix WebProcessor to accept a Request object.
    
    Discussion
    ----------
    
    [Bridge/Monolog] Fix WebProcessor to accept a Request object.
    
    WebProcessor.php is type hinting a bad interface. Adds a simple test case.
    
    Bug fix: yes
    Feature addition: no
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: -
    
    Before:
    
        1) Symfony\Tests\Bridge\Monolog\Processor\WebProcessorTest::testUsesRequestServerData
        Argument 1 passed to Symfony\Bridge\Monolog\Processor\WebProcessor::__construct() must be an instance of Symfony\Bridge\Monolog\Processor\RequestInterface, instance of Symfony\Component\HttpFoundation\Request given, called in ./symfony/vendor/symfony/tests/Symfony/Tests/Bridge/Monolog/Processor/WebProcessorTest.php on line 31 and defined
    
        ./symfony/vendor/symfony/src/Symfony/Bridge/Monolog/Processor/WebProcessor.php:24
        ./symfony/vendor/symfony/tests/Symfony/Tests/Bridge/Monolog/Processor/WebProcessorTest.php:31
    
        FAILURES!
        Tests: 1, Assertions: 0, Errors: 1.
    
    After:
    
        Starting test 'Symfony\Tests\Bridge\Monolog\Processor\WebProcessorTest::testUsesRequestServerData'.
        OK (1 test, 3 assertions)
    
    Global:
    
        OK, but incomplete or skipped tests!
        Tests: 4337, Assertions: 10590, Incomplete: 36, Skipped: 23.
    fabpot committed Sep 30, 2011
  7. updated CHANGELOG for 2.1

    fabpot committed Sep 30, 2011
Commits on Sep 29, 2011
  1. merged branch jalliot/event-subscriber (PR #2021)

    Commits
    -------
    
    3223c5a Removed now useless test
    21cf0ac Backported new behaviour from PR #2148 and removed check for interface at run-time
    8b240d4 Implementation of kernel.event_subscriber tag for services.
    
    Discussion
    ----------
    
    Added missing kernel.event_subscriber tag (closes #2012)
    
    This PR adds a ``kernel.event_subscriber`` tag which allows to register services as event subscribers in the same way ``kernel.event_listener`` allows to register them as event listeners.
    The service is still lazy loaded and the DIC does not need to be recompiled for every modification in the service's code.
    
    There is one important thing to remember:
    If the service is created by a factory, the class parameter **MUST** reflect the real class of the service, although it is not needed at the moment for the DIC. For that issue, we could either forbid services created by factories or add a note to the documentation.
    
    This PR closes #2012.
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/08/24 06:42:18 -0700
    
    I'm not sure the test is good enough so feel free to add some more.
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/08/25 03:46:20 -0700
    
    I re-implemented the check for EventSubscriberInterface in ContainerAwareEventDispatcher because I think the overhead is minimum and it allows to use this method even without the tag (at run-time).
    I also added some tests for RegisterKernelListenersPass.
    
    ---------------------------------------------------------------------------
    
    by stof at 2011/09/04 02:42:00 -0700
    
    @jalliot Your branch conflicts with the current master. could you rebase it ?
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/04 02:57:03 -0700
    
    Rebased
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/13 02:19:46 -0700
    
    @fabpot What do you think about this PR? At the moment, the subscribers are not really usable in Symfony2 because of the lack of this tag.
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2011/09/13 04:17:46 -0700
    
    I don't like subscribers. There are other PRs on adding more support for them, but the reality is that they are complex for no added benefit. I'm wondering if it wouldn't be better to just remove them altogether.
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/13 04:38:20 -0700
    
    @fabpot Well I prefer listeners too but I think that if Symfony2 does support subscribers (which it does at the moment), it should do it properly and completely, thus allowing to register subscriber services like here or to register several methods for one same event like in #2148.
    But I guess that if you merged those 2 PRs (well actually this one would have to be modified first if #2148 is merged but I'll do it then), many use cases would be covered and people should stop asking for more support :) (except maybe for removing the static modifier but this would be wrong IMO and prevent entirely this PR).
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2011/09/28 11:47:10 -0700
    
    @jalliot: #2148 has been merged. Can you update this PR accordingly? thanks.
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/28 12:00:44 -0700
    
    Sure thing. Will do it as well as removing the check for the interface tonight or tomorrow :)
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/29 08:53:17 -0700
    
    @fabpot Check for interface removed and #2148 merged. Also rebased on latest master.
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2011/09/29 09:09:11 -0700
    
    Tests do not pass.
    
    ---------------------------------------------------------------------------
    
    by jalliot at 2011/09/29 09:18:48 -0700
    
    @fabpot Fixed
    fabpot committed Sep 29, 2011
  2. Removed now useless test

    jalliot committed Sep 29, 2011