Permalink
Commits on Oct 9, 2012
  1. merged branch szicsu/UniversalClassLoader-FIX (PR #5692)

    This PR was merged into the master branch.
    
    Commits
    -------
    
    f66f110 FIX [2.1][ClassLoader]UniversalClassLoader not working with AnnotationRegistry::registerLoader
    
    Discussion
    ----------
    
    [2.1][ClassLoader]UniversalClassLoader not working with AnnotationRe...
    
    Bug fix: yes
    Feature addition: no
    Backwards compatibility break: no
    Symfony2 tests pass: yes
    Fixes the following tickets: ~
    Todo: ~
    License of the code: MIT
    Documentation PR: ~
    
    The Doctrine\Common\Annotations\AnnotationRegistry::loadAnnotationClass examines the returning value of the loader and the load is successful only if the loader returns with "TRUE" value.
    This is how method Symfony\Component\ClassLoader\ClassLoader::loadClass works, but it is not true for Symfony\Component\ClassLoader\UniversalClassLoader::loadClass.
    
    ---------------------------------------------------------------------------
    
    by sstok at 2012-10-08T09:25:39Z
    
    As this is a bug fix it should be done on 2.0
    
    ---------------------------------------------------------------------------
    
    by stof at 2012-10-08T12:49:42Z
    
    It is not a bugfix. Nothing enforces an autoloader to return a boolean in PHP.
    
    And Symfony works with the annotation registry since 1.5 year (when it was introduced): https://github.com/symfony/symfony-standard/blob/2.0/app/autoload.php#L34-38
    
    Btw, if you are using 2.1, I would recommend you to use the new ClassLoader instead of the UniversalClassLoader to autoload PSR-0 libraries. It has a simpler API (and returns the boolean needed by Doctrine) while supporting the same classes than the UniversalClasssLoader (both of them are supporting PSR-0 and nothing else)
    fabpot committed Oct 9, 2012
Commits on Oct 7, 2012
  1. FIX [2.1][ClassLoader]UniversalClassLoader not working with Annotatio…

    …nRegistry::registerLoader
    szicsu committed Oct 7, 2012
Commits on Oct 6, 2012
  1. Merge branch '2.1'

    * 2.1:
      fixed CS
      added doc comments
      added doc comments
      [Validator] Updated swedish translation
      Update src/Symfony/Component/Validator/Resources/translations/validators.de.xlf
      [2.1] Exclude tests from zips via gitattributes
      [HttpKernel][Translator] Fixed type-hints
      Updated lithuanian validation translation
      [DomCrawler] Allows using multiselect through Form::setValues().
      [Translation] forced the catalogue to be regenerated when a resource is added (closes symfony/translation#1)
      Unit test for patched method OptionsResolver::validateOptionValues().
      validateOptionValues throw a notice if an allowed value is set and the corresponding option isn't.
      [Form] Hardened code of ViolationMapper against errors
      [HttpFoundation] Fixed #5611 - Request::splitHttpAcceptHeader incorrect result order.
      [Form] Fixed negative index access in PropertyPathBuilder
      Update src/Symfony/Component/Validator/Resources/translations/validators.ro.xlf
    
    Conflicts:
    	src/Symfony/Component/DomCrawler/Form.php
    	src/Symfony/Component/Process/Process.php
    fabpot committed Oct 6, 2012
Commits on Oct 4, 2012
Commits on Sep 28, 2012
  1. Optimize autoload prefix in composer.json

    By having more specific autoload prefixes it is possible to reduce the
    number of stat calls made. Also it prevents conflicts with similar
    namespaces.
    Slamdunk committed Sep 28, 2012
Commits on Sep 6, 2012
  1. udpated composer.json to 2.2

    fabpot committed Sep 6, 2012
Commits on Sep 1, 2012
  1. load test

    eventhorizonpl committed Aug 27, 2012
Commits on Aug 27, 2012
  1. dump test

    fix things pointed out by stof
    
    fix things pointed out by pborreli
    
    fix things pointed out by fabpot
    eventhorizonpl committed Aug 23, 2012
Commits on Aug 25, 2012
  1. merged branch willdurand/fix-composer-components (PR #5318)

    Commits
    -------
    
    933e821 Add minimum-stability (dev) in each component
    
    Discussion
    ----------
    
    Add minimum-stability (dev) in each component
    
    This fixes the ability to run the test suite in each component if a `composer install` is needed.
    
    ---------------------------------------------------------------------------
    
    by stof at 2012-08-22T13:57:14Z
    
    If you really want to run the testsuite standalone, some dev requirements are missing (SecurityBundle needs the FrameworkBundle for its functional tests for instance). If you have some time to check the missing dev requirement, it would be great.
    Anyway, 👍 for this
    
    ---------------------------------------------------------------------------
    
    by willdurand at 2012-08-22T13:59:15Z
    
    Yes I already did that once. I'll try to fix more components later.
    
    On Wed, Aug 22, 2012 at 3:57 PM, Christophe Coevoet <
    notifications@github.com> wrote:
    
    > If you really want to run the testsuite standalone, some dev requirements
    > are missing (SecurityBundle needs the FrameworkBundle for its functional
    > tests for instance). If you have some time to check the missing dev
    > requirement, it would be great.
    > Anyway, [image: 👍] for this
    >
    > —
    > Reply to this email directly or view it on GitHub<symfony/symfony#5318 (comment)>.
    >
    >
    
    ---------------------------------------------------------------------------
    
    by stof at 2012-08-22T14:02:23Z
    
    Well, I think most components should be good now (as some work has been done on them). But the bridges and bundles may need some work (bundles were not having any dev requirements until yesterday when @guilhermeblanco added some on FrameworkBundle)
    
    ---------------------------------------------------------------------------
    
    by pborreli at 2012-08-22T14:14:00Z
    
    what about having for each READ-ONLY repo his own .travis.yml and travisci hook activated ?
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2012-08-22T14:30:13Z
    
    please, don't add more travis files. The main already tests everything, and that's all we need.
    
    ---------------------------------------------------------------------------
    
    by stof at 2012-08-22T14:33:46Z
    
    @pborreli tests should not be different for subtree split repos as the code is the same and the tests are the same (except that more tests could be skipped because of missing deps).
    Note that for the bundles, it is likely to be different currently as I think some skip tests are missing (just like dev requirements are). But fixing this does not require enablign travis.
    
    ---------------------------------------------------------------------------
    
    by pborreli at 2012-08-22T14:42:30Z
    
    ok, i was just thinking about a way to be sure each component is usable individually but yeah that would require to relaunch each tests and add a bunch of travis files + hook
    
    ---------------------------------------------------------------------------
    
    by hason at 2012-08-24T13:12:04Z
    
    @stof, @eriksencosta, @fabpot: Tests are different for Locale component, see #5235
    
    ---------------------------------------------------------------------------
    
    by stof at 2012-08-24T13:35:07Z
    
    @hason no. You also need to do it when running the tests of the Locale component as part of the full run.
    fabpot committed Aug 25, 2012
Commits on Aug 24, 2012
  1. getNamespaces test

    getPrefixes test
    
    loadClass test
    
    getPrefixes test
    
    addPrefix test
    
    getFallbackDirs test
    
    fix things pointed out by stof
    eventhorizonpl committed Aug 23, 2012
Commits on Aug 22, 2012
Commits on Jul 15, 2012
  1. Revert "raised the minimum version of PHP to 5.3.4 (closes #3856)"

    This reverts commit 2dcc44897ef1ceec2721bbf616c4744131340657.
    fabpot committed Jul 15, 2012
Commits on Jul 13, 2012
  1. raised the minimum version of PHP to 5.3.4 (closes #3856)

    We've raised the minimum version of PHP because of a PHP
    bug before 5.3.4:
    
    https://bugs.php.net/bug.php?id=52083
    https://bugs.php.net/bug.php?id=50027
    fabpot committed Jul 13, 2012
Commits on Jul 10, 2012
  1. [ClassLoader] fixed order of interfaces in generated class collection…

    … caches (closes #4841)
    fabpot committed Jul 10, 2012
Commits on Jul 9, 2012
  1. merged branch bamarni/master (PR #4792)

    Commits
    -------
    
    6c9c2ec [ClassCollectionLoader] fixed comment striping on classes in global namespace
    
    Discussion
    ----------
    
    [ClassCollectionLoader] fixed comment striping on classes in global namespace
    
    Comments aren't striped when the class is in the global namespace, this adds a fake namespace to let the fixNamespaceDeclaration method handling the formating, so we can remove a method.
    
    I've also put dev as minimum policy in composer, otherwise I couldn't install dependencies.
    fabpot committed Jul 9, 2012
  2. fixed CS

    fabpot committed Jul 9, 2012
Commits on Jul 8, 2012
Commits on Jul 4, 2012
  1. merged branch fabpot/classloader-optim (PR #4729)

    Commits
    -------
    
    3f9e8ff [ClassLoader] made ClassCollectionLoader::load() automatically include class dependencies
    6f4d281 [ClassLoader] added missing support for PHP 5.4 traits
    
    Discussion
    ----------
    
    Classloader optimization
    
    The first commit fixes support for PHP 5.4 trait.
    
    The second one does several things:
    
     * it optimizes the recent merge so that the reflection class instance is only loaded once;
     * we use the fact that we now get all class dependencies to automatically add all class dependencies to the map.
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2012-07-03T17:26:46Z
    
    I've updated to take into accounts traits.
    
    ---------------------------------------------------------------------------
    
    by bamarni at 2012-07-04T11:58:57Z
    
    great job 👍
    
    I can't see it in the diff as this part hasn't changed, but somewhere in the autoReload block there is :
    ```
    if ($meta[1] != $classes) {
        $reload = true;
    }
    ```
    
    It should be array_unique($classes), otherwise the file would be perpetually regenerated in autoReload mode when the input contains duplicate, because they're implicitely removed when dumping the files.
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2012-07-04T13:20:04Z
    
    @bamarni I've added an `array_unique` call at the top (this bug existed before by the way).
    fabpot committed Jul 4, 2012
  2. [ClassLoader] made ClassCollectionLoader::load() automatically includ…

    …e class dependencies
    fabpot committed Jul 3, 2012
Commits on Jul 3, 2012
  1. [ClassLoader] fixed typo

    fabpot committed Jul 3, 2012
  2. fixed unit tests

    fabpot committed Jul 3, 2012
Commits on Jun 19, 2012
Commits on May 21, 2012
  1. fixed CS

    fabpot committed May 21, 2012
Commits on May 15, 2012
  1. added @ to all chmod() calls to avoid PHP warnings (operation not per…

    …mitted) when using CIFS or NTFSa (closes #2125)
    fabpot committed May 15, 2012
Commits on May 7, 2012
  1. updated minimum PHP version to 5.3.3

    5.3.3 has some interesting fixes and this is the version used by
    Redhat 6 and Debian 6
    fabpot committed May 7, 2012
Commits on May 1, 2012
  1. merged branch willdurand/fix-components (PR #4155)

    Commits
    -------
    
    c195957 [Components] Tests/Autoloading fixes
    
    Discussion
    ----------
    
    Fix components
    
    See #4141
    
    ----
    This PR:
    
    * configures each component to use composer to manage "dev" dependencies instead of env variables;
    * adds phpunit configuration file on Filesystem component;
    * fixes READMEs.
    
    It's mergeable without any problems, but I would recommend to wait a fix in Composer in order to use `self.version` in `require`/`require-dev` sections.
    
    Note: I kept `suggest` sections because it makes sense but this PR doesn't aim to provide useful explanations for each entry. It could be another PR, not that one.
    
    ---------------------------------------------------------------------------
    
    by willdurand at 2012-04-30T20:43:13Z
    
    @fabpot I reviewed each component, one by one. Now `phpunit` always works, even if tests are skipped. A simple `composer install --dev` allows to run the complete test suite. Each commit is well separated from the others. I guess, everything is ok now.
    
    ---------------------------------------------------------------------------
    
    by Tobion at 2012-04-30T20:47:00Z
    
    Please squash, as it makes no sense to have the same commit for each component.
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2012-05-01T14:26:11Z
    
    Can you squash your commits before I merge? Thanks.
    
    ---------------------------------------------------------------------------
    
    by willdurand at 2012-05-01T14:29:38Z
    
    done
    
    ---------------------------------------------------------------------------
    
    by fabpot at 2012-05-01T15:48:25Z
    
    It does not seem that the commits are squashed.
    
    ---------------------------------------------------------------------------
    
    by willdurand at 2012-05-01T15:54:08Z
    
    done
    fabpot committed May 1, 2012
  2. [Components] Tests/Autoloading fixes

    * Switched to Composer to manage "dev" dependencies
    * Fixed READMEs
    * Excluded vendor in phpunit.xml.dist files
    * Fixed message in bootstrap.php files
    * Added autoloader for the component itself
    willdurand committed Apr 27, 2012
  3. fixed CS

    fabpot committed May 1, 2012
Commits on Apr 27, 2012
  1. Revert "merged branch Seldaek/master (PR #4133)"

    This reverts commit 00e7a94a8c761a3e10c388e4d06cb3ce4cd04ad7, reversing
    changes made to a01dec00f4e8de357cce5fdf121ac98729f7f8ff.
    fabpot committed Apr 27, 2012
  2. Update branch aliases

    Seldaek committed Apr 27, 2012
Commits on Apr 26, 2012
  1. [ClassLoader] added CHANGELOG

    fabpot committed Apr 26, 2012
Commits on Apr 25, 2012
  1. merged 2.0

    fabpot committed Apr 25, 2012
Commits on Apr 23, 2012
  1. [2.1][Component][ClassLoader] cs

    gajdaw committed Apr 23, 2012