Skip to content

Commit 580864a

Browse files
Master dev kit (#403)
* changes to get symfony 3.4 working without deprecations * testing app directory structure and php version reducing * DevKit updates * make provider public, enable bundle.php * make alias public only * apply style fixes
1 parent ef01605 commit 580864a

34 files changed

+58
-37
lines changed

.gitignore

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
phpunit.xml
22
composer.lock
33
vendor
4-
tests/Resources/app/cache
5-
tests/Resources/app/logs
4+
tests/Fixtures/App/var/

.travis.yml

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -26,31 +26,30 @@ cache:
2626
- $HOME/.composer/cache/files
2727

2828
env:
29-
matrix: SYMFONY_VERSION=3.4
29+
matrix: SYMFONY_VERSION=^3.4@dev
3030
global:
31-
- SYMFONY_DEPRECATIONS_HELPER=48
31+
- SYMFONY_DEPRECATIONS_HELPER=0
3232
- SYMFONY_PHPUNIT_DIR=.phpunit SYMFONY_PHPUNIT_REMOVE="symfony/yaml"
33+
- KERNEL_CLASS=Symfony\Cmf\Bundle\RoutingBundle\Tests\Fixtures\App\Kernel
3334
- SYMFONY_PHPUNIT_VERSION=5.7
3435
- TEST_INSTALLATION=false
3536

3637
matrix:
3738
include:
3839
- php: 7.1
39-
env: SYMFONY_VERSION=3.4.*
40+
env: DEPS=dev SYMFONY_VERSION=^4@dev
41+
- php: 7.1
42+
env: DEPS=dev SYMFONY_VERSION=^3.4@dev
4043
- php: 7.1
41-
env: DEPS=dev SYMFONY_VERSION=3.3.*
42-
- php: 5.6
4344
env: COMPOSER_FLAGS="--prefer-lowest" SYMFONY_VERSION=2.8.* SYMFONY_DEPRECATIONS_HELPER=weak
44-
- php: 7.0
45-
env: SYMFONY_VERSION=3.1.*
46-
- php: 7.0
47-
env: SYMFONY_VERSION=3.2.*
45+
- php: 7.1
46+
env: SYMFONY_VERSION=3.3.*
4847
- env: TEST_INSTALLATION=true
4948

5049
fast_finish: true
5150
allow_failures:
5251
- php: 7.1
53-
env: SYMFONY_VERSION=3.4.*
52+
env: DEPS=dev SYMFONY_VERSION=^4@dev
5453
- env: TEST_INSTALLATION=true
5554

5655

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
Changelog
22
=========
33

4+
* **2017-11-11**: Removed php 5.6 and 7.0 support, removed Symfony 3.0.* and 3.1.* support
5+
46
2.0.1
57
-----
68

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ ifdef BRANCH
2020
VERSION=dev-${BRANCH}
2121
endif
2222
PACKAGE=symfony-cmf/routing-bundle
23-
23+
export KERNEL_CLASS=Symfony\Cmf\Bundle\RoutingBundle\Tests\Fixtures\App\Kernel
2424
list:
2525
@echo 'test: will run all tests'
2626
@echo 'unit_tests: will run unit tests only'

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ controller for redirection routes.
2323

2424
## Requirements
2525

26-
* PHP 5.6 / 7.0 / 7.1
27-
* Symfony 2.8 / 3.1 / 3.2 / 3.3
26+
* PHP 7.1
27+
* Symfony 2.8 / 3.3 / ^3.4@dev
2828
* See also the `require` section of [composer.json](composer.json)
2929

3030
## Documentation

composer.json

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,18 +12,18 @@
1212
}
1313
],
1414
"require": {
15-
"php": "^5.6|^7.0",
15+
"php": "^7.1",
1616
"symfony-cmf/routing": "^2.0",
1717
"symfony/framework-bundle": "^2.8|^3.0",
18-
"symfony/twig-bundle": "^2.8|^3.0",
19-
"symfony-cmf/testing": "^2.1@dev"
18+
"symfony/twig-bundle": "^2.8|^3.0"
2019
},
2120
"require-dev": {
22-
"doctrine/phpcr-odm": "^1.4",
21+
"doctrine/phpcr-odm": "^1.4|^2.0",
2322
"symfony/phpunit-bridge": "^3.2",
2423
"matthiasnoback/symfony-dependency-injection-test": "~0.6",
2524
"matthiasnoback/symfony-config-test": "^1.3.1",
2625
"doctrine/orm": "^2.5",
26+
"symfony-cmf/testing": "^2.1@dev",
2727
"doctrine/data-fixtures": "^1.0.0"
2828
},
2929
"suggest": {
@@ -48,5 +48,7 @@
4848
"branch-alias": {
4949
"dev-master": "2.1-dev"
5050
}
51-
}
51+
},
52+
"minimum-stability":"dev",
53+
"prefer-stable": true
5254
}

phpunit.xml.dist

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,4 @@
2929
</exclude>
3030
</whitelist>
3131
</filter>
32-
33-
<php>
34-
<server name="KERNEL_DIR" value="tests/Resources/app" />
35-
</php>
3632
</phpunit>

src/DependencyInjection/CmfRoutingExtension.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ private function setupDynamicRouter(array $config, ContainerBuilder $container,
124124

125125
if (isset($config['route_provider_service_id'])) {
126126
$container->setAlias('cmf_routing.route_provider', $config['route_provider_service_id']);
127+
$container->getAlias('cmf_routing.route_provider')->setPublic(true);
127128
$hasProvider = true;
128129
}
129130

src/Resources/config/provider-orm.xml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,10 @@
1717
<argument>%cmf_routing.dynamic.limit_candidates%</argument>
1818
</service>
1919

20-
<service id="cmf_routing.route_provider" class="Symfony\Cmf\Bundle\RoutingBundle\Doctrine\Orm\RouteProvider">
20+
<service
21+
id="cmf_routing.route_provider"
22+
class="Symfony\Cmf\Bundle\RoutingBundle\Doctrine\Orm\RouteProvider"
23+
public="true">
2124
<argument type="service" id="doctrine" />
2225
<argument type="service" id="cmf_routing.orm_candidates" />
2326
<argument>%cmf_routing.dynamic.persistence.orm.route_class%</argument>

src/Resources/config/provider-phpcr.xml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,10 @@
1414
<call method="setRouteCollectionLimit"><argument>%cmf_routing.route_collection_limit%</argument></call>
1515
</service>
1616

17-
<service id="cmf_routing.route_provider" alias="cmf_routing.phpcr_route_provider"/>
17+
<service
18+
id="cmf_routing.route_provider"
19+
alias="cmf_routing.phpcr_route_provider"
20+
public="true"/>
1821

1922
<service id="cmf_routing.phpcr_candidates_prefix" class="Symfony\Cmf\Bundle\RoutingBundle\Doctrine\Phpcr\PrefixCandidates">
2023
<argument>%cmf_routing.dynamic.persistence.phpcr.route_basepaths%</argument>

0 commit comments

Comments
 (0)