Hi,

I found an error, When I configure o365. Error because of the missing abstract methods in o365 plugin. Created patch for that. Please review.

Error:
( ! ) Fatal error: Class Drupal\o365\Plugin\Oauth2Client\O365OAuth2Client contains 3 abstract methods and must therefore be declared abstract or implement the remaining methods (Drupal\oauth2_client\Plugin\Oauth2Client\Oauth2ClientPluginInterface::storeAccessToken, Drupal\oauth2_client\Plugin\Oauth2Client\Oauth2ClientPluginInterface::retrieveAccessToken, Drupal\oauth2_client\Plugin\Oauth2Client\Oauth2ClientPluginInterface::clearAccessToken) in C:\wamp64\www\drupal-9.3.2\modules\contrib\o365\src\Plugin\Oauth2Client\O365OAuth2Client.php on line 22

CommentFileSizeAuthor
#2 o365-3265482-2.patch875 bytessaranya ashokkumar

Issue fork o365-3265482

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

saranya ashokkumar created an issue. See original summary.

saranya ashokkumar’s picture

saranya ashokkumar commented
Status: Active » Needs review
StatusFileSize
new o365-3265482-2.patch875 bytes

codebymikey made their first commit to this issue’s fork.

codebymikey opened merge request !14

codebymikey’s picture

codebymikey commented

This issue also affects version 2.0.x when you visit: /admin/config/system/oauth2-client/o365.

fabianderijk made their first commit to this issue’s fork.

fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented
Status: Needs review » Fixed

This MR has been merged into the 2.0.x and 3.0.x branch. Releases will be created in a later moment.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

codebymikey opened merge request !26

codebymikey closed merge request !26