diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2926da4..5a37ead 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,13 +1,18 @@
CHANGELOG
=========
+7.3
+---
+
+ * Deprecate `Event::getWorkflow()` method
+
7.1
---
* Add method `getEnabledTransition()` to `WorkflowInterface`
* Automatically register places from transitions
* Add support for workflows that need to store many tokens in the marking
- * Add method `getName()` in event classes to build event names in subscribers
+ * Add method `getName()` in event classes to build event names in subscribers
7.0
---
diff --git a/DataCollector/WorkflowDataCollector.php b/DataCollector/WorkflowDataCollector.php
index febc975..0cb7e20 100644
--- a/DataCollector/WorkflowDataCollector.php
+++ b/DataCollector/WorkflowDataCollector.php
@@ -88,21 +88,39 @@ public function getCallsCount(): int
return $i;
}
+ public function hash(string $string): string
+ {
+ return hash('xxh128', $string);
+ }
+
+ public function buildMermaidLiveLink(string $name): string
+ {
+ $payload = [
+ 'code' => $this->data['workflows'][$name]['dump'],
+ 'mermaid' => '{"theme": "default"}',
+ 'autoSync' => false,
+ ];
+
+ $compressed = zlib_encode(json_encode($payload), ZLIB_ENCODING_DEFLATE);
+
+ $suffix = rtrim(strtr(base64_encode($compressed), '+/', '-_'), '=');
+
+ return "https://mermaid.live/edit#pako:{$suffix}";
+ }
+
protected function getCasters(): array
{
return [
...parent::getCasters(),
- TransitionBlocker::class => function ($v, array $a, Stub $s, $isNested) {
- unset(
- $a[\sprintf(Caster::PATTERN_PRIVATE, $v::class, 'code')],
- $a[\sprintf(Caster::PATTERN_PRIVATE, $v::class, 'parameters')],
- );
+ TransitionBlocker::class => static function ($v, array $a, Stub $s) {
+ unset($a[\sprintf(Caster::PATTERN_PRIVATE, $v::class, 'code')]);
+ unset($a[\sprintf(Caster::PATTERN_PRIVATE, $v::class, 'parameters')]);
$s->cut += 2;
return $a;
},
- Marking::class => function ($v, array $a, Stub $s, $isNested) {
+ Marking::class => static function ($v, array $a) {
$a[Caster::PREFIX_VIRTUAL.'.places'] = array_keys($v->getPlaces());
return $a;
@@ -110,11 +128,6 @@ protected function getCasters(): array
];
}
- public function hash(string $string): string
- {
- return hash('xxh128', $string);
- }
-
private function getEventListeners(WorkflowInterface $workflow): array
{
$listeners = [];
diff --git a/Debug/TraceableWorkflow.php b/Debug/TraceableWorkflow.php
index 6d0afd8..c783e63 100644
--- a/Debug/TraceableWorkflow.php
+++ b/Debug/TraceableWorkflow.php
@@ -30,6 +30,7 @@ class TraceableWorkflow implements WorkflowInterface
public function __construct(
private readonly WorkflowInterface $workflow,
private readonly Stopwatch $stopwatch,
+ protected readonly ?\Closure $disabled = null,
) {
}
@@ -90,6 +91,9 @@ public function getCalls(): array
private function callInner(string $method, array $args): mixed
{
+ if ($this->disabled?->__invoke()) {
+ return $this->workflow->{$method}(...$args);
+ }
$sMethod = $this->workflow::class.'::'.$method;
$this->stopwatch->start($sMethod, 'workflow');
diff --git a/DependencyInjection/WorkflowDebugPass.php b/DependencyInjection/WorkflowDebugPass.php
index 634605d..042aaba 100644
--- a/DependencyInjection/WorkflowDebugPass.php
+++ b/DependencyInjection/WorkflowDebugPass.php
@@ -31,6 +31,7 @@ public function process(ContainerBuilder $container): void
->setArguments([
new Reference("debug.{$id}.inner"),
new Reference('debug.stopwatch'),
+ new Reference('profiler.is_disabled_state_checker', ContainerBuilder::IGNORE_ON_INVALID_REFERENCE),
]);
}
}
diff --git a/Event/Event.php b/Event/Event.php
index c3e6a6f..c13818b 100644
--- a/Event/Event.php
+++ b/Event/Event.php
@@ -46,8 +46,13 @@ public function getTransition(): ?Transition
return $this->transition;
}
+ /**
+ * @deprecated since Symfony 7.3, inject the workflow in the constructor where you need it
+ */
public function getWorkflow(): WorkflowInterface
{
+ trigger_deprecation('symfony/workflow', '7.3', 'The "%s()" method is deprecated, inject the workflow in the constructor where you need it.', __METHOD__);
+
return $this->workflow;
}
diff --git a/Tests/Debug/TraceableWorkflowTest.php b/Tests/Debug/TraceableWorkflowTest.php
index 3d8e699..257ad66 100644
--- a/Tests/Debug/TraceableWorkflowTest.php
+++ b/Tests/Debug/TraceableWorkflowTest.php
@@ -21,7 +21,7 @@
class TraceableWorkflowTest extends TestCase
{
- private MockObject|Workflow $innerWorkflow;
+ private MockObject&Workflow $innerWorkflow;
private Stopwatch $stopwatch;
diff --git a/Tests/WorkflowTest.php b/Tests/WorkflowTest.php
index a2236e0..48e2209 100644
--- a/Tests/WorkflowTest.php
+++ b/Tests/WorkflowTest.php
@@ -773,7 +773,7 @@ public function testGetEnabledTransitions()
});
$workflow = new Workflow($definition, new MethodMarkingStore(), $eventDispatcher, 'workflow_name');
- $this->assertEmpty($workflow->getEnabledTransitions($subject));
+ $this->assertSame([], $workflow->getEnabledTransitions($subject));
$subject->setMarking(['d' => 1]);
$transitions = $workflow->getEnabledTransitions($subject);
diff --git a/composer.json b/composer.json
index 44a3000..ef6779c 100644
--- a/composer.json
+++ b/composer.json
@@ -20,15 +20,16 @@
}
],
"require": {
- "php": ">=8.2"
+ "php": ">=8.2",
+ "symfony/deprecation-contracts": "2.5|^3"
},
"require-dev": {
"psr/log": "^1|^2|^3",
"symfony/dependency-injection": "^6.4|^7.0",
- "symfony/event-dispatcher": "^6.4|^7.0",
"symfony/error-handler": "^6.4|^7.0",
- "symfony/http-kernel": "^6.4|^7.0",
+ "symfony/event-dispatcher": "^6.4|^7.0",
"symfony/expression-language": "^6.4|^7.0",
+ "symfony/http-kernel": "^6.4|^7.0",
"symfony/security-core": "^6.4|^7.0",
"symfony/stopwatch": "^6.4|^7.0",
"symfony/validator": "^6.4|^7.0"
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy