Option to disable login form and access denied on redirect skip pages (inlcuding patches) [#2323975]

Heyho,

I wanted to disable the redirect for some pages. I added them in the backend and found out, that the site then shows an user login form and also the access denied message.
I wanted it to be just a simple access denied page - no messages, no form.

So I added two additional options to make this possible and changed the code a little to make it work.

The patch is attached and pretty simple. Would really appreciate a merge :)

Thanks :)

func0der

Comment	File	Size	Author
#6	interdiff.txt	1 KB	NWOM
#6	r4032login-option-to-disable-2323975-4.patch	4.53 KB	NWOM
#6
#5	interdiff.txt	1017 bytes	NWOM
#5	r4032login-option-to-disable-2323975-4.patch	4.52 KB	NWOM
#5
#4	interdiff.txt	1010 bytes	NWOM
#4	r4032login-option-to-disable-2323975-4.patch	4.52 KB	NWOM
#4
#1	r4032login-no_form_and_message_on_noredirect-2323975-1.patch	4.1 KB	drupalexio
#1
	r4032login-no_form_and_message_on_noredirect.patch	4.1 KB	func0der

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

drupalexio CreditAttribution: drupalexio commented 9 October 2015 at 15:44

File	Size
r4032login-no_form_and_message_on_noredirect-2323975-1.patch	4.1 KB

An empty string should be returned instead of NULL.

Comment #2

func0der CreditAttribution: func0der commented 29 October 2015 at 15:50

Status:

Needs review

» Reviewed & tested by the community

Work's great :)
Thanks, drupalexio.

Comment #3

NWOM CreditAttribution: NWOM commented 7 June 2016 at 14:57

Status:

Reviewed & tested by the community

» Needs work

After applying the patch, a few notices are shown:

Notice: Undefined variable: message in r4032login_redirect() (line 194 of /var/aegir/platforms/panopoly-7.x-1.35/sites/{SITE}/modules/r4032login/r4032login.module).
Notice: Undefined variable: message_type in r4032login_redirect() (line 194 of /var/aegir/platforms/panopoly-7.x-1.35/sites/{SITE}/modules/r4032login/r4032login.module).

Uploading a new patch here in a second.