The module currently hard-codes 'user/login' as the redirect path for when the user needs to authenticate. It would better to at least make this a variable. A simple use case - I want to implement a mobile-friendly login page for a site that has an otherwise mobile-unfriendly theme.

CommentFileSizeAuthor
#6 1710752-6.patch1.98 KBrjbrown99
#1 1710752-1.patch701 bytespwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

Title: Add a varialbe for the login page path » Add a variable for the login page path
Status: Active » Needs review
FileSize
701 bytes

simple patch - no UI, though that can be added if desired.

juampynr’s picture

Status: Needs review » Fixed

Committed using a constant instead of user/login. Also added a form field at the module's settings page.

Many thanks!

http://drupalcode.org/project/oauth.git/commit/be6f5a1
http://drupalcode.org/project/oauth.git/commitdiff/914ad04

pwolanin’s picture

Thanks. Does this need to be backported to 6.x?

juampynr’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Fixed » Active

Yes it should, I would appreciate a backport patch.

rjbrown99’s picture

Status: Active » Patch (to be ported)

Updating to the proper status per the issue queue handbook description.

rjbrown99’s picture

Status: Patch (to be ported) » Needs review
FileSize
1.98 KB

Turns out I needed this, so here's a patch rolled against the 6.x-3.x tree. Should be identical to the 7.x commits.

  • juampynr committed be6f5a1 on 8.x-2.x authored by pwolanin
    Issue #1710752 by pwolanin: Added a variable for the login page path.
    
  • juampynr committed 914ad04 on 8.x-2.x
    Issue #1710752 by pwolanin and juampy: Added a variable for the login...