[Workflow] Fixed BC break with MarkingStoreInterface::setMarking()

lyrixx · lyrixx · commit 4f0c5a8adb13 · 2019-03-12T11:54:51.000+01:00
diff --git a/UPGRADE-4.3.md b/UPGRADE-4.3.md
@@ -110,3 +110,28 @@ Yaml
 ----
 
  * Using a mapping inside a multi-line string is deprecated and will throw a `ParseException` in 5.0.
+
+Workflow
+--------
+
+ * `MarkingStoreInterface::setMarking()` will have a third argument in Symfony 5.0.
+
+   Before:
+   ```php
+   class MyMarkingStore implements MarkingStoreInterface
+   {
+     public function setMarking($subject, Marking $marking)
+     {
+     }
+   }
+   ```
+
+   After:
+   ```php
+    class MyMarkingStore implements MarkingStoreInterface
+   {
+     public function setMarking($subject, Marking $marking , array $context = [])
+     {
+     }
+   }
+   ```
diff --git a/UPGRADE-5.0.md b/UPGRADE-5.0.md
@@ -83,7 +83,7 @@ Finder
 
 Form
 ----
- 
+
  * Removed support for using the `format` option of `DateType` and `DateTimeType` when the `html5` option is enabled.
  * Using names for buttons that do not start with a letter, a digit, or an underscore leads to an exception.
  * Using names for buttons that do not contain only letters, digits, underscores, hyphens, and colons leads to an
@@ -350,6 +350,7 @@ Workflow
  * `add` method has been removed use `addWorkflow` method in `Workflow\Registry` instead.
  * `SupportStrategyInterface` has been removed, use `WorkflowSupportStrategyInterface` instead.
  * `ClassInstanceSupportStrategy` has been removed, use `InstanceOfSupportStrategy` instead.
+ * `MarkingStoreInterface::setMarking()` has a third argument: `array $context = []`.
 
 Yaml
 ----
diff --git a/src/Symfony/Component/Workflow/MarkingStore/MarkingStoreInterface.php b/src/Symfony/Component/Workflow/MarkingStore/MarkingStoreInterface.php
@@ -37,6 +37,7 @@ public function getMarking($subject);
      * Sets a Marking to a subject.
      *
      * @param object $subject A subject
+     * @param array  $context Some context
      */
-    public function setMarking($subject, Marking $marking, array $context = []);
+    public function setMarking($subject, Marking $marking /*, array $context = []*/);
 }

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ public function getMarking($subject);`
`37`	`37`	`* Sets a Marking to a subject.`
`38`	`38`	`*`
`39`	`39`	`* @param object $subject A subject`
	`40`	`+ * @param array $context Some context`
`40`	`41`	`*/`
`41`		`- public function setMarking($subject, Marking $marking, array $context = []);`
	`42`	`+ public function setMarking($subject, Marking $marking /, array $context = []/);`
`42`	`43`	`}`