8000 feature #22180 [Workflow] Added 'workflow_marked_places' twig functio… · symfony/symfony@47740ce · GitHub
[go: up one dir, main page]
Skip to content
8000

Commit 47740ce

Browse files
lyrixxlyrixx
committed
feature #22180 [Workflow] Added 'workflow_marked_places' twig function (lyrixx)
This PR was merged into the 3.3-dev branch. Discussion ---------- [Workflow] Added 'workflow_marked_places' twig function | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - --- When someone uses a custom MarkingStore, the value in the $subject::marking can be really different from the value inside Marking::getPlaces(). This occurs, for example, when the value stored in the subject is a bit mask. So it's always safer to get the places names from the marking, and so with this new function. --- usage: ```twig {{ workflow_marked_places(post)|join(',') }} ``` Commits ------- ead6f14 [Workflow] Added 'workflow_marked_places' twig function
2 parents ae7bcae + ead6f14 commit 47740ce

File tree

3 files changed

+43
-2
lines changed

3 files changed

+43
-2
lines changed

‎src/Symfony/Bridge/Twig/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ CHANGELOG
55
-----
66

77
* added a `workflow_has_marked_place` function
8+
* added a `workflow_marked_places` function
89

910
3.2.0
1011
-----

‎src/Symfony/Bridge/Twig/Extension/WorkflowExtension.php

Lines changed: 32 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ public function getFunctions()
3333
new \Twig_SimpleFunction('workflow_can', array($this, 'canTransition')),
3434
new \Twig_SimpleFunction('workflow_transitions', array($this, 'getEnabledTransitions')),
3535
new \Twig_SimpleFunction('workflow_has_marked_place', array($this, 'hasMarkedPlace')),
36+
new \Twig_SimpleFunction('workflow_marked_places', array($this, 'getMarkedPlaces')),
3637
);
3738
}
3839

@@ -63,9 +64,38 @@ public function getEnabledTransitions($subject, $name = null)
6364
return $this->workflowRegistry->get($subject, $name)->getEnabledTransitions($subject);
6465
}
6566

66-
public function hasMarkedPlace($object, $place, $name = null)
67+
/**
68+
* Returns true if the place is marked.
69+
*
70+
* @param object $subject A subject
71+
* @param string $placeName A place name
72+
* @param string $name A workflow name
73+
*
74+
* @return bool true if the transition is enabled
75+
*/
76+
public function hasMarkedPlace($subject, $placeName, $name = null)
77+
{
78+
return $this->workflowRegistry->get($subject, $name)->getMarking($subject)->has($placeName);
79+
}
80+
81+
/**
82+
* Returns marked places.
83+
*
84+
* @param object $subject A subject
85+
* @param string $placesNameOnly If true, returns only places name. If false returns the raw representation
86+
* @param string $name A workflow name
87+
*
88+
* @return string[]|int[]
89+
*/
90+
public function getMarkedPlaces($subject, $placesNameOnly = true, $name = null)
6791
{
68-
return $this->workflowRegistry->get($object, $name)->getMarking($object)->has($place);
92+
$places = $this->workflowRegistry->get($subject, $name)->getMarking($subject)->getPlaces();
93+
94+
if ($placesNameOnly) {
95+
return array_keys($places);
96+
}
97+
98+
return $places;
6999
}
70100

71101
public function getName()

‎src/Symfony/Bridge/Twig/Tests/Extension/WorkflowExtensionTest.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,4 +75,14 @@ public function testHasMarkedPlace()
7575
$this->assertTrue($this->extension->hasMarkedPlace($subject, 'waiting_for_payment'));
7676
$this->assertFalse($this->extension->hasMarkedPlace($subject, 'processed'));
7777
}
78+
79+
public function testGetMarkedPlaces()
80+
{
81+
$subject = new \stdClass();
82+
$subject->marking = array();
83+
$subject->marking = array('ordered' => 1, 'waiting_for_payment' => 1);
84+
85+
$this->assertSame(array('ordered', 'waiting_for_payment'), $this->extension->getMarkedPlaces($subject));
86+
$this->assertSame($subject->marking, $this->extension->getMarkedPlaces($subject, false));
87+
}
7888
}

0 commit comments

Comments
 (0)
0