Skip to content

Commit

Permalink
Merge pull request #3738 from NateWr/i8592_notification_subscriptions
Browse files Browse the repository at this point in the history
pkp/pkp-lib#8592 Fix site-wide notification subscriptions form
  • Loading branch information
NateWr authored Feb 2, 2023
2 parents a9bad2d + c160e16 commit 384f241
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion classes/notification/form/NotificationSettingsForm.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class NotificationSettingsForm extends PKPNotificationSettingsForm
/**
* @copydoc PKPNotificationSettingsForm::getNotificationSettingsCategories()
*/
public function getNotificationSettingCategories(Context $context)
public function getNotificationSettingCategories(?Context $context = null)
{
$categories = parent::getNotificationSettingCategories($context);
for ($i = 0; $i < count($categories); $i++) {
Expand Down
1 change: 1 addition & 0 deletions dbscripts/xml/upgrade.xml
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,7 @@
<migration class="PKP\migration\upgrade\v3_4_0\I8508_ConvertCurrentLogFile"/>
<migration class="APP\migration\upgrade\v3_4_0\I8027_DoiVersioning"/>
<migration class="APP\migration\upgrade\v3_4_0\I6306_EnableCategories"/>
<migration class="PKP\migration\upgrade\v3_4_0\I8592_SiteNotificationSubscriptions"/>
<data file="dbscripts/xml/upgrade/3.4.0_preupdate_email_templates.xml" />
<note file="docs/release-notes/README-3.4.0" />
</upgrade>
Expand Down

0 comments on commit 384f241

Please sign in to comment.