Skip to content

Commit

Permalink
Merge pull request #540 from creative-commoners/pulls/6/protect-hooks
Browse files Browse the repository at this point in the history
API Set extension hook implementation visibility to protected
  • Loading branch information
GuySartorelli authored May 21, 2024
2 parents 920831d + 6aa04af commit 7161f05
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/Extension/AccountReset/MFAResetExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class MFAResetExtension extends Extension
/**
* @param Member&MemberExtension $member
*/
public function handleAccountReset(Member $member)
protected function handleAccountReset(Member $member)
{
foreach ($member->RegisteredMFAMethods() as $method) {
$method->delete();
Expand Down
2 changes: 1 addition & 1 deletion src/Extension/AccountReset/MemberExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ class MemberExtension extends DataExtension
'AccountResetExpired' => 'Datetime',
];

public function updateCMSFields(FieldList $fields)
protected function updateCMSFields(FieldList $fields)
{
$fields->removeByName(['AccountResetHash', 'AccountResetExpired']);

Expand Down
2 changes: 1 addition & 1 deletion src/Extension/MemberExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ public function getRegisteredMethodNames(): string
return implode(', ', $arr);
}

public function updateCMSFields(FieldList $fields): FieldList
protected function updateCMSFields(FieldList $fields): FieldList
{
$fields->removeByName(['DefaultRegisteredMethodID', 'HasSkippedMFARegistration', 'RegisteredMFAMethods']);

Expand Down
2 changes: 1 addition & 1 deletion src/Extension/SiteConfigExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ class SiteConfigExtension extends DataExtension
'MFARequired' => false,
];

public function updateCMSFields(FieldList $fields)
protected function updateCMSFields(FieldList $fields)
{
Requirements::javascript('silverstripe/mfa: client/dist/js/bundle-cms.js');
Requirements::css('silverstripe/mfa: client/dist/styles/bundle-cms.css');
Expand Down

0 comments on commit 7161f05

Please sign in to comment.