Site admin may want to show no message but still let the redirection rule to act. It might be better to avoid strict check 

  if(($login_redirect) && !empty($login_message)){
    // Set Login redirect message.
    drupal_set_message(t($login_message));
    user_default_page_redirect($login_redirect);
  }

And let it work with empty login_message.

CommentFileSizeAuthor
#4 2850704.patch531 bytessaradaprasad17
#2 user_default_page_login.patch596 bytesAshok Negi

Comments

arpitr created an issue. See original summary.

Ashok Negi’s picture

Ashok Negi commented
StatusFileSize
new user_default_page_login.patch596 bytes

Attached Patch for the issue.

saradaprasad17’s picture

saradaprasad17 commented

Fixed. Without Message allowing redirecting to landing page .

saradaprasad17’s picture

saradaprasad17 commented
StatusFileSize
new 2850704.patch531 bytes
saradaprasad17’s picture

saradaprasad17 commented
Status: Active » Needs review
saradaprasad17’s picture

saradaprasad17 commented
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Assigned: arpitr » Unassigned
Status: Needs review » Reviewed & tested by the community

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

rolodmonkey’s picture

rolodmonkey commented

Hiding patch number 4, since it is not the one that was applied.