Permalink
Commits on Mar 2, 2015
  1. feature #13808 [OptionsResolver] removed deprecated functionality (To…

    …bion)
    
    This PR was merged into the 3.0-dev branch.
    
    Discussion
    ----------
    
    [OptionsResolver] removed deprecated functionality
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | no
    | New feature?  | no
    | BC breaks?    | yes
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets |
    | License       | MIT
    | Doc PR        | -
    
    Commits
    -------
    
    295bba2 [OptionsResolver] removed deprecated functionality
    Tobion committed Mar 2, 2015
  2. Merge branch '2.7'

    Conflicts:
    	src/Symfony/Component/OptionsResolver/OptionsResolverInterface.php
    Tobion committed Mar 2, 2015
  3. Merge branch '2.6' into 2.7

    Tobion committed Mar 2, 2015
  4. bug #13816 [OptionsResolver] fix allowed values with null (Tobion)

    This PR was merged into the 2.6 branch.
    
    Discussion
    ----------
    
    [OptionsResolver] fix allowed values with null
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets |
    | License       | MIT
    | Doc PR        | -
    
    1. setNormalizers was missing deprecated annotation
    2. setAllowedValues/addAllowedValues with `null` was broken because `(array) null === array()`
    
    Commits
    -------
    
    cb37fbe [OptionsResolver] fix allowed values with null
    e3e9a4d [OptionsResolver] add missing deprecated phpdoc for setNormalizers
    Tobion committed Mar 2, 2015
Commits on Mar 1, 2015
Commits on Feb 28, 2015
  1. feature #13809 [OptionsResolver] add missing deprecation triggers (To…

    …bion)
    
    This PR was merged into the 2.7 branch.
    
    Discussion
    ----------
    
    [OptionsResolver] add missing deprecation triggers
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | no
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets |
    | License       | MIT
    | Doc PR        | -
    
    Commits
    -------
    
    9607e82 replace deprecated options resolver uses by new ones
    7e9d2e6 [OptionsResolver] add missing deprecation triggers
    Tobion committed Feb 28, 2015
Commits on Feb 27, 2015
Commits on Feb 26, 2015
  1. minor #13800 [FrameworkBundle] Update assets configuration tests (jak…

    …zal)
    
    This PR was merged into the 3.0-dev branch.
    
    Discussion
    ----------
    
    [FrameworkBundle] Update assets configuration tests
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | no
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | -
    | License       | MIT
    | Doc PR        | -
    
    Previously, assets where enabled if `templating` didn't contain asset configuration. This has been removed in #13666.
    
    Instead of enabling it back I propose to keep it disabled and update the tests instead, since we already discussed disabling all FrameworkBundle features by default for 3.0 (#13703).
    
    Commits
    -------
    
    5007b41 [Form] Replace use of bind() with submit() in a test.
    494e300 [SecurityBundle] Enable assets in functional tests.
    4c26875 [FrameworkBundle] Fix a default config test case and add a new one for enabling assets.
    b0f6a19 [FrameworkBundle] Remove a legacy test.
    fabpot committed Feb 26, 2015
  2. [FrameworkBundle] Fix a default config test case and add a new one fo…

    …r enabling assets.
    jakzal committed Feb 26, 2015
Commits on Feb 25, 2015
  1. Merge branch '2.7'

    * 2.7:
      [Form] NativeRequestHandler file handling fix
      [VarDumper] Workaround stringy numeric keys
      [HttpKernel] Throw double-bounce exceptions
      [FrameworkBundle] silence E_USER_DEPRECATED in insulated clients
      [minor] composer.json fix
      [minor] composer.json fix
      Fixed absolute_url for absolute paths
      minor #13377 [Console] Change greater by greater or equal for isFresh in FileResource
      [2.3] [HttpFoundation] fixed param order for Nginx's x-accel-redirect
    nicolas-grekas committed Feb 25, 2015
  2. Merge branch '2.6' into 2.7

    * 2.6:
      [Form] NativeRequestHandler file handling fix
      [VarDumper] Workaround stringy numeric keys
      [HttpKernel] Throw double-bounce exceptions
      minor #13377 [Console] Change greater by greater or equal for isFresh in FileResource
      [2.3] [HttpFoundation] fixed param order for Nginx's x-accel-redirect
    nicolas-grekas committed Feb 25, 2015
  3. Merge branch '2.3' into 2.6

    * 2.3:
      [Form] NativeRequestHandler file handling fix
      [HttpKernel] Throw double-bounce exceptions
      minor #13377 [Console] Change greater by greater or equal for isFresh in FileResource
      [2.3] [HttpFoundation] fixed param order for Nginx's x-accel-redirect
    nicolas-grekas committed Feb 25, 2015
  4. bug #13744 minor #13377 [Console] Change greater by greater or equal …

    …for isFresh in FileResource (bijibox)
    
    This PR was merged into the 2.3 branch.
    
    Discussion
    ----------
    
    minor #13377 [Console] Change greater by greater or equal for isFresh in FileResource
    
    | Q             | A
    | ------------- | ---
    | Fixed tickets | #13377
    | License       | MIT
    
    FileResource and tests update
    
    Commits
    -------
    
    87800ae minor #13377 [Console] Change greater by greater or equal for isFresh in FileResource
    fabpot committed Feb 25, 2015
  5. bug #13708 [2.3] [HttpFoundation] fixed param order for Nginx's x-acc…

    …el-mapping (phansys)
    
    This PR was merged into the 2.3 branch.
    
    Discussion
    ----------
    
    [2.3] [HttpFoundation] fixed param order for Nginx's x-accel-mapping
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | kinda
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | #13502
    | License       | MIT
    | Doc PR        | n/a
    
    Inverted path and location directives for x-accel-mapping header (fixes #13502).
    
    Before:
    ```proxy_set_header X-Accel-Mapping /internal/=/var/www/example.com/```
    
    After:
    ```proxy_set_header X-Accel-Mapping /var/www/example.com/=/internal/```
    
    It could be a BC break since the response will fail if someone sends this header
    honoring the previous signature, thus I need some feedback in order to choose the right branch for this change.
    
    Commits
    -------
    
    9f9f230 [2.3] [HttpFoundation] fixed param order for Nginx's x-accel-redirect
    fabpot committed Feb 25, 2015
  6. bug #13767 [HttpKernel] Throw double-bounce exceptions (nicolas-grekas)

    This PR was merged into the 2.3 branch.
    
    Discussion
    ----------
    
    [HttpKernel] Throw double-bounce exceptions
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | -
    | License       | MIT
    | Doc PR        | -
    
    Commits
    -------
    
    0ebcf63 [HttpKernel] Throw double-bounce exceptions
    fabpot committed Feb 25, 2015
  7. bug #13785 [VarDumper] Workaround stringy numeric keys (nicolas-grekas)

    This PR was merged into the 2.6 branch.
    
    Discussion
    ----------
    
    [VarDumper] Workaround stringy numeric keys
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | #13784
    | License       | MIT
    | Doc PR        | -
    
    Commits
    -------
    
    46bea3a [VarDumper] Workaround stringy numeric keys
    fabpot committed Feb 25, 2015
  8. bug #13769 [Form] NativeRequestHandler file handling fix (mpajunen)

    This PR was squashed before being merged into the 2.3 branch (closes #13769).
    
    Discussion
    ----------
    
    [Form] NativeRequestHandler file handling fix
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | #13746
    | License       | MIT
    | Doc PR        | n/a
    
    `NativeRequestHandler` reused the local variable for the form name `$name` as a loop variable for processing the `$_FILES` array. A separate variable is now used.
    
    Two new test cases are included:
    - Handling a request with multiple files
    - Handling a request with file upload and a nameless form
    
    Both tests fail without the fix. The test cases could probably be better though.
    
    Commits
    -------
    
    9b3421f [Form] NativeRequestHandler file handling fix
    fabpot committed Feb 25, 2015
Commits on Feb 24, 2015
  1. Merge branch '2.6' into 2.7

    * 2.6:
      [FrameworkBundle] silence E_USER_DEPRECATED in insulated clients
    nicolas-grekas committed Feb 24, 2015
  2. Merge branch '2.3' into 2.6

    * 2.3:
      [FrameworkBundle] silence E_USER_DEPRECATED in insulated clients
    nicolas-grekas committed Feb 24, 2015
  3. bug #13779 [FrameworkBundle] silence E_USER_DEPRECATED in insulated c…

    …lients (nicolas-grekas)
    
    This PR was merged into the 2.3 branch.
    
    Discussion
    ----------
    
    [FrameworkBundle] silence E_USER_DEPRECATED in insulated clients
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | -
    | License       | MIT
    | Doc PR        | -
    
    Commits
    -------
    
    7558c23 [FrameworkBundle] silence E_USER_DEPRECATED in insulated clients
    fabpot committed Feb 24, 2015
  4. bug #13745 Fixed absolute_url for absolute paths (florianv)

    This PR was merged into the 2.7 branch.
    
    Discussion
    ----------
    
    Fixed absolute_url for absolute paths
    
    | Q             | A
    | ------------- | ---
    | Bug fix?      | yes
    | New feature?  | no
    | BC breaks?    | no
    | Deprecations? | no
    | Tests pass?   | yes
    | Fixed tickets | symfony#13724
    | License       | MIT
    
    The issue is that `asset()` and `absolute_url()` cannot be composed because `asset()` already returns an absolute path (using the `PathPackage`) and `absolute_url()` still prepends the base path. So the resulting url contains the base path twice.
    
    So this PR removes appending the base path to absolute paths (starting with `/`) passed to `absolute_url()`.
    
    Commits
    -------
    
    08aa7bc Fixed absolute_url for absolute paths
    fabpot committed Feb 24, 2015
  5. Merge branch '2.6' into 2.7

    * 2.6:
      [minor] composer.json fix
      [minor] composer.json fix
    
    Conflicts:
    	src/Symfony/Component/HttpFoundation/composer.json
    	src/Symfony/Component/Templating/composer.json
    nicolas-grekas committed Feb 24, 2015
  6. [minor] composer.json fix

    nicolas-grekas committed Feb 24, 2015
  7. Merge branch '2.3' into 2.6

    * 2.3:
      [minor] composer.json fix
    
    Conflicts:
    	src/Symfony/Component/HttpFoundation/composer.json
    	src/Symfony/Component/Templating/composer.json
    nicolas-grekas committed Feb 24, 2015