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

Comments

pwolanin’s picture

pwolanin commented
Title: Add a varialbe for the login page path » Add a variable for the login page path
Status: Active » Needs review
StatusFileSize
new 1710752-1.patch701 bytes

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

juampynr’s picture

juampynr commented
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

pwolanin commented

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

juampynr’s picture

juampynr commented
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

rjbrown99 commented
Status: Active » Patch (to be ported)

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

rjbrown99’s picture

rjbrown99 commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 1710752-6.patch1.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...