Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

Commit 02db271

Browse files
authored
Merge pull request #192 from damienflament/patch-1
Follow naming changes made in component.
2 parents b5b0e28 + f43a8eb commit 02db271

File tree

3 files changed

+13
-13
lines changed

3 files changed

+13
-13
lines changed

src/Adapter/PhpcrOdmAdapter.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ public function removeAutoRoute(AutoRouteInterface $autoRoute)
109109
/**
110110
* {@inheritdoc}
111111
*/
112-
public function createAutoRoute(UriContext $uriContext, $contentDocument, $autoRouteTag)
112+
public function createAutoRoute(UriContext $uriContext, $contentDocument, $locale)
113113
{
114114
$basePath = $this->baseRoutePath;
115115
$document = $parentDocument = $this->dm->find(null, $basePath);
@@ -142,7 +142,7 @@ public function createAutoRoute(UriContext $uriContext, $contentDocument, $autoR
142142
return $this->migrateGenericToAutoRoute(
143143
$existingDocument,
144144
$contentDocument,
145-
$autoRouteTag,
145+
$locale,
146146
AutoRouteInterface::TYPE_PRIMARY
147147
);
148148
}
@@ -161,7 +161,7 @@ public function createAutoRoute(UriContext $uriContext, $contentDocument, $autoR
161161
$headRoute->setContent($contentDocument);
162162
$headRoute->setName($headName);
163163
$headRoute->setParentDocument($document);
164-
$headRoute->setAutoRouteTag($autoRouteTag);
164+
$headRoute->setLocale($locale);
165165
$headRoute->setType(AutoRouteInterface::TYPE_PRIMARY);
166166

167167
foreach ($uriContext->getDefaults() as $key => $value) {
@@ -229,12 +229,12 @@ private function getPathFromUri($uri)
229229
*
230230
* @param Generic $document
231231
* @param object $contentDocument
232-
* @param string $autoRouteTag
232+
* @param string $locale
233233
* @param string $routeType
234234
*
235235
* @return AutoRouteInterface
236236
*/
237-
private function migrateGenericToAutoRoute(Generic $document, $contentDocument, $autoRouteTag, $routeType)
237+
private function migrateGenericToAutoRoute(Generic $document, $contentDocument, $locale, $routeType)
238238
{
239239
$autoRouteClassName = $this->autoRouteFqcn;
240240
$mapper = $this->dm->getConfiguration()->getDocumentClassMapper();
@@ -256,7 +256,7 @@ private function migrateGenericToAutoRoute(Generic $document, $contentDocument,
256256
}
257257

258258
$autoRoute->setContent($contentDocument);
259-
$autoRoute->setAutoRouteTag($autoRouteTag);
259+
$autoRoute->setLocale($locale);
260260
$autoRoute->setType($routeType);
261261

262262
return $autoRoute;

src/Model/AutoRoute.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
*/
2323
class AutoRoute extends Route implements AutoRouteInterface
2424
{
25-
const DEFAULT_KEY_AUTO_ROUTE_TAG = '_auto_route_tag';
25+
const DEFAULT_KEY_AUTO_ROUTE_LOCALE = '_auto_route_tag';
2626

2727
/**
2828
* @var AutoRouteInterface
@@ -32,17 +32,17 @@ class AutoRoute extends Route implements AutoRouteInterface
3232
/**
3333
* {@inheritdoc}
3434
*/
35-
public function setAutoRouteTag($autoRouteTag)
35+
public function setLocale($locale)
3636
{
37-
$this->setDefault(self::DEFAULT_KEY_AUTO_ROUTE_TAG, $autoRouteTag);
37+
$this->setDefault(self::DEFAULT_KEY_AUTO_ROUTE_LOCALE, $locale);
3838
}
3939

4040
/**
4141
* {@inheritdoc}
4242
*/
43-
public function getAutoRouteTag()
43+
public function getLocale()
4444
{
45-
return $this->getDefault(self::DEFAULT_KEY_AUTO_ROUTE_TAG);
45+
return $this->getDefault(self::DEFAULT_KEY_AUTO_ROUTE_LOCALE);
4646
}
4747

4848
/**

tests/Functional/EventListener/AutoRouteListenerTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ public function testPersistBlog()
6565
$this->assertCount(1, $routes);
6666
$this->assertInstanceOf('Symfony\Cmf\Bundle\RoutingAutoBundle\Model\AutoRoute', $routes[0]);
6767
$this->assertEquals('unit-testing-blog', $routes[0]->getName());
68-
$this->assertEquals(PhpcrOdmAdapter::TAG_NO_MULTILANG, $routes[0]->getAutoRouteTag());
68+
$this->assertEquals(PhpcrOdmAdapter::TAG_NO_MULTILANG, $routes[0]->getLocale());
6969
$this->assertEquals(array(
7070
'_auto_route_tag' => 'no-multilang',
7171
'type' => 'cmf_routing_auto.primary',
@@ -257,7 +257,7 @@ public function testMultilangArticle($data, $expectedPaths)
257257

258258
$this->assertNotNull($route, 'Route: '.$expectedPath);
259259
$this->assertInstanceOf('Symfony\Cmf\Bundle\RoutingAutoBundle\Model\AutoRoute', $route);
260-
$this->assertEquals($expectedLocale, $route->getAutoRouteTag());
260+
$this->assertEquals($expectedLocale, $route->getLocale());
261261

262262
$content = $route->getContent();
263263

0 commit comments

Comments
 (0)