8000 Added support for guards when advancing workflow from a command by GDIBass · Pull Request #23906 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

Added support for guards when advancing workflow from a command #23906

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

Merged
merged 12 commits into from
Oct 5, 2017
Merged

Added support for guards when advancing workflow from a command #23906

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b044ffb
Added support for guards when advancing workflow from a command
GDIBass Aug 16, 2017
324c208
Code standards update
GDIBass Aug 16, 2017
8ab59cb
Updated if statement
Aug 17, 2017
22b44e2
Changed automatic token generation to throw an exception instead
Aug 18, 2017
3a8b2ed
Code standard fixes
Aug 18, 2017
2ebc71a
Created new Exception to throw and modified tests
Aug 18, 2017
92308b4
Created new Exception to throw and modified tests.
Aug 18, 2017
2c9d1e2
Removed newline
Aug 18, 2017
49839e3
Ahh, I see. It actually wants a newline!
Aug 18, 2017
8f2fa6b
changed exception message
Aug 25, 2017
fefc202
newline at end of file
Aug 25, 2017
15fd863
8000 Updated Test name and exception name to be more accurate
Sep 15, 2017
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
Next Next commit
Added support for guards when advancing workflow from a command
  • Loading branch information
@GDIBass
GDIBass committed Aug 16, 2017
commit b044ffb4a23d109d62724e7c62855f4d03d72a32
6 changes: 6 additions & 0 deletions src/Symfony/Component/Workflow/EventListener/GuardListener.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
namespace Symfony\Component\Workflow\EventListener;

use Symfony\Component\Security\Core\Authentication\AuthenticationTrustResolverInterface;
use Symfony\Component\Security\Core\Authentication\Token\AnonymousToken;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
use Symfony\Component\Security\Core\Role\RoleHierarchyInterface;
Expand Down Expand Up @@ -55,6 +56,11 @@ private function getVariables(GuardEvent $event)
{
$token = $this->tokenStorage->getToken();

if($token == null) {
$token = new AnonymousToken('secret','anon',[]);
$this->tokenStorage->setToken($token);
}

if (null !== $this->roleHierarchy) {
$roles = $this->roleHierarchy->getReachableRoles($token->getRoles());
} else {
Expand Down
14 changes: 14 additions & 0 deletions src/Symfony/Component/Workflow/Tests/EventListener/GuardListenerTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,20 @@ public function testWithSupportedEventAndAccept()
$this->assertTrue($event->isBlocked());
}

public function testWithNoTokenStorage()
Copy link
Member

Choose a reason for hiding this comment

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

actually, there is a token storage but no token

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated!

{
$event = $this->createEvent();
$this->tokenStorage = null;
Copy link
Member

Choose a reason for hiding this comment

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

this does not work.
You should configure the token storage to return null

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool, thanks for your patience here. First time contributing!

I'll get this done right away


$this->listener->onTransition($event, 'event_name_a');

$this->assertFalse($event->isBlocked());

$this->listener->onTransition($event, 'event_name_b');

$this->assertTrue($event->isBlocked());
}

private function createEvent()
{
$subject = new \stdClass();
Expand Down
0