E679 [Workflow] Introduce concept of SupportStrategyInterface by andesk · Pull Request #20751 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Workflow] Introduce concept of SupportStrategyInterface #20751

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 13 commits into from
Closed

[Workflow] Introduce concept of SupportStrategyInterface #20751

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
6421c10
[Workflow] Introduce concept of SupprtStrategyInterface to allow othe…
andesk Dec 4, 2016
bfd3611
[Workflow] Refactoring to resolve BC break and adding deprecated warning
andesk Dec 5, 2016
c0cbc85
[Workflow] Moving and tagging legacy unit tests
andesk Dec 6, 2016
fa448af
[Workflow] Some CS fixes
andesk Dec 10, 2016
19fb676
[Workflow] Removing some docblocks
andesk Dec 10, 2016
faddc97
Merge remote-tracking branch 'upstream/3.2' into 3.2
andesk Dec 10, 2016
a115bdd
Merge remote-tracking branch 'upstream/3.2' into 3.2
andesk Dec 14, 2016
d22fe01
[Workflow] fixing service definition of workflow registry in framewor…
andesk Dec 14, 2016
047a2e7
[Workflow] fixing code style mostly, removing exception
andesk Dec 17, 2016
a34e4af
[Workflow] tiny cs fix in docblock...
andesk Dec 17, 2016
1e65b12
[Workflow] removing deprecated test case for removed InvalidArgumentE…
andesk Dec 17, 2016
f4e0d62
Merge remote-tracking branch 'upstream/3.2' into 3.2
andesk Dec 17, 2016
2d11004
[Workflow] Some more adaptions after code review comments, not finish…
andesk Jan 4, 2017 10000
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[Workflow] fixing service definition of workflow registry in framewor… 
…k bundle
  • Loading branch information
@andesk
andesk committed Dec 14, 2016
commit d22fe018bb141476a3435e2de22522cfb6c27fc8
2 changes: 1 addition & 1 deletion src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -464,7 +464,7 @@ private function registerWorkflowConfiguration(array $workflows, ContainerBuilde
Workflow\SupportStrategy\ClassInstanceSupportStrategy::class, array($supportedClassName)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should use a use statement instead of using the FQCN here.

);
$strategyDefinition->setPublic(false);
$registryDefinition->addMethodCall('add', array(new Reference($workflowId), $supportedClassName));
$registryDefinition->addMethodCall('add', array(new Reference($workflowId), $strategyDefinition));
}
}
if (isset($workflow['support_strategy'])) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could probably be an elseif as the two options are exclusive

Expand Down
0