Changeset 2601

Timestamp:

04/06/24 11:14:19 (3 months ago)

Author:

nanardon

Message:

Don't return manager if unexported

Location:

trunk/LATMOS-Accounts/lib/LATMOS/Accounts/Bases

Files:

• Ldap.pm (modified) (1 diff)
• Sql/User.pm (modified) (1 diff)

trunk/LATMOS-Accounts/lib/LATMOS/Accounts/Bases/Ldap.pm

@@ -235 +235 @@
     );
     if ($mesg->code) {
-        $self->log(LA_ERR, "Cannot get object %s: %s", $dn, $mesg->error);
+        $self->log(LA_WARN, "Cannot get object %s: %s", $dn, $mesg->error);
         return;
     }

trunk/LATMOS-Accounts/lib/LATMOS/Accounts/Bases/Sql/User.pm

@@ -770 +770 @@
                     my ($self) = @_;
                     if (my $manager = $self->object->_get_c_field('managerContact')) {
-                        return $manager;
+                        return $manager if ( $self->base->search_objects( 'user', "name=$manager" ) ); # Check in case unexported
                     } elsif (my $department = $self->object->_get_c_field('department')) {
                         my $obj = $self->base->get_object('group', $department);
                         return $obj->_get_c_field('managedBy');
-                    } else {
-                        return;
-                    }
+                    }
+
+                    return;
                 },
                 label => l('Responsible'),