From 3de73b7b070c7166a67d1a3ff5c87374a38ce5a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Cl=C3=A9ment=20Oudot?= <clement@oodo.net>
Date: Thu, 21 Jul 2011 09:39:53 +0000
Subject: [PATCH] Do not check notification twice is user is already connected
 (#352)

---
 .../lib/Lemonldap/NG/Portal/Simple.pm                | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/modules/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Simple.pm b/modules/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Simple.pm
index e7f1605f8..8e6c0a88c 100644
--- a/modules/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Simple.pm
+++ b/modules/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Simple.pm
@@ -1506,15 +1506,23 @@ sub controlUrlOrigin {
 sub checkNotifBack {
     my $self = shift;
     if ( $self->{notification} and grep( /^reference/, $self->param() ) ) {
+        $self->lmLog( "User was on a notification step", 'debug' );
         unless ( $self->{notifObject}->checkNotification($self) ) {
+            $self->lmLog(
+                "All notifications have not been accepted, display them again",
+                'debug'
+            );
             $self->{_notification} =
               $self->{notifObject}->getNotification($self);
             return PE_NOTIFICATION;
         }
         else {
-            $self->{error} = $self->_subProcess(
-                qw(checkNotification issuerDBInit authInit issuerForAuthUser autoRedirect)
+            $self->lmLog(
+"All notifications have been accepted, follow the authentication process",
+                'debug'
             );
+            $self->{error} = $self->_subProcess(
+                qw(issuerDBInit authInit issuerForAuthUser autoRedirect) );
             return $self->{error} || PE_DONE;
         }
     }