Skip to content

Commit bd3b5b0

Browse files
Merge branch '4.1' into 4.2
* 4.1: [Workflow] Fixed BC break for Workflow metadata [Routing] ignore trailing slash for non-GET requests
2 parents 20a0a93 + 69ca34d commit bd3b5b0

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

DependencyInjection/FrameworkExtension.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -547,7 +547,7 @@ private function registerWorkflowConfiguration(array $config, ContainerBuilder $
547547
}
548548
if ($transition['metadata']) {
549549
$transitionsMetadataDefinition->addMethodCall('attach', array(
550-
$transitionDefinition,
550+
new Reference($transitionId),
551551
$transition['metadata'],
552552
));
553553
}
@@ -569,7 +569,7 @@ private function registerWorkflowConfiguration(array $config, ContainerBuilder $
569569
}
570570
if ($transition['metadata']) {
571571
$transitionsMetadataDefinition->addMethodCall('attach', array(
572-
$transitionDefinition,
572+
new Reference($transitionId),
573573
$transition['metadata'],
574574
));
575575
}

Tests/DependencyInjection/FrameworkExtensionTest.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -258,10 +258,8 @@ public function testWorkflows()
258258
$this->assertSame('attach', $transitionsMetadataCall[0]);
259259
$params = $transitionsMetadataCall[1];
260260
$this->assertCount(2, $params);
261-
$this->assertInstanceOf(Definition::class, $params[0]);
262-
$this->assertSame(Workflow\Transition::class, $params[0]->getClass());
263-
$this->assertSame(array('submit', 'start', 'travis'), $params[0]->getArguments());
264-
$this->assertSame(array('title' => 'transition submit title'), $params[1]);
261+
$this->assertInstanceOf(Reference::class, $params[0]);
262+
$this->assertSame('state_machine.pull_request.transition.0', (string) $params[0]);
265263

266264
$serviceMarkingStoreWorkflowDefinition = $container->getDefinition('workflow.service_marking_store_workflow');
267265
/** @var Reference $markingStoreRef */

0 commit comments

Comments
 (0)