Skip to content

Commit c3b98d5

Browse files
ADmadothercorey
andauthored
Apply suggestions from code review
Co-authored-by: othercorey <corey.taylor.fl@gmail.com>
1 parent 2df7eda commit c3b98d5

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

en/appendices/6-0-migration-guide.rst

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ Event
7777

7878
- The signature of ``EventManagerInterface::on()`` has changed. The 2nd and 3rd
7979
parameters have been swapped. Calls which pass an options array as the 2nd
80-
argument will need to be updated to pass it as the 3rd argument instead.
80+
argument will need to be updated to pass it as the 3rd argument instead or
81+
use named parameters.
8182

8283
Datasource
8384
----------

en/core-libraries/events.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -412,7 +412,7 @@ for event listeners::
412412
$callback = [$this, 'doSomething'];
413413
$this->getEventManager()->on(
414414
'Order.afterPlace',
415-
$callback
415+
$callback,
416416
['priority' => 2],
417417
);
418418

0 commit comments

Comments
 (0)