These classes need to be autoloaded with the module:

-files[] = lib/OAuth.php
-files[] = includes/DrupalOAuthServer.inc
-files[] = includes/DrupalOAuthDataStore.inc
-files[] = includes/DrupalOAuthRequest.inc
-files[] = includes/DrupalOAuthToken.inc
-files[] = includes/DrupalOAuthConsumer.inc
-files[] = includes/DrupalOAuthClient.inc
-files[] = includes/OAuthSignatureMethod_HMAC.inc

CommentFileSizeAuthor
#3 2013927-psr0-3.patch80.2 KBpcambra
#1 2013927-psr0-1.patch81.11 KBpcambra
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pcambra’s picture

Status: Active » Needs review
FileSize
81.11 KB
juampynr’s picture

Status: Needs review » Needs work

Weird output when applying. See:

juampy@juampy-bot:~/projects/drupal8/modules/contrib/oauth (8.x-1.x)$ git apply -v --index 2013927-psr0-1.patch
Checking patch includes/DrupalOAuthClient.inc...
Checking patch includes/DrupalOAuthConsumer.inc...
Checking patch includes/DrupalOAuthDataStore.inc...
Checking patch includes/DrupalOAuthRequest.inc...
Checking patch includes/DrupalOAuthServer.inc...
Checking patch includes/DrupalOAuthToken.inc...
Checking patch includes/OAuthSignatureMethod_HMAC.inc...
Checking patch lib/Drupal/oauth/DrupalOAuthClient.php...
Checking patch lib/Drupal/oauth/DrupalOAuthConsumer.php...
Checking patch lib/Drupal/oauth/DrupalOAuthDataStore.php...
Checking patch lib/Drupal/oauth/DrupalOAuthRequest.php...
Checking patch lib/Drupal/oauth/DrupalOAuthServer.php...
Checking patch lib/Drupal/oauth/DrupalOAuthToken.php...
Checking patch lib/Drupal/oauth/OAuthSignatureMethod_HMAC.php...
Checking patch oauth.authorizations.inc...
Checking patch oauth.consumers.inc...
Checking patch oauth.inc...
Checking patch oauth.info...
error: oauth.info: does not exist in index
Checking patch oauth.info.yml...
error: oauth.info.yml: already exists in index
Checking patch oauth.module...
Checking patch oauth.pages.inc...
pcambra’s picture

Status: Needs work » Needs review
FileSize
80.2 KB

Solving conflicts with myself :P

juampynr’s picture

Status: Needs review » Closed (won't fix)

Closing because #2027863: Make Oauth module an AuthenticationProvider uses PECLs OAuth provider insted of the custom library.