Changeset 1950 for trunk/LATMOS-Accounts/lib/LATMOS/Accounts/Bases/Sql/Employment.pm

Timestamp:

02/14/17 21:39:12 (7 years ago)

Author:

nanardon

Message:

Cleanup useless code

File:

• trunk/LATMOS-Accounts/lib/LATMOS/Accounts/Bases/Sql/Employment.pm (modified) (2 diffs)

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

@@ -343 +343 @@
     my $res = $self->SUPER::set_fields(%data);
 
-    $self->applyToUser or return;
-    return $res;
-}
-
-=head2 applyToUser()
-
-Set user's attributes if need from this employement.
-
-=cut
-
-sub applyToUser {
-    my ($self) = @_;
-
     my $user = $self->base->get_object('user', $self->get_attributes('user')) or do {
         $self->base->log(LA_ERR, "Cannot fetch user %s to apply employment", $self->get_attributes('user'));
@@ -361 +348 @@
     };
 
-    $user->base->log(LA_DEBUG, "Applying Employement %s to user %s", $self->id, $user->id);
-
-    my $currentemployment = $user->get_attributes('currentEmployment') || '';
-
-    if (!$currentemployment) {
-        return $user->_resetEmployment
-    } elsif ($currentemployment eq $self->id) {
-        # No sync to do if this employment is not currently applied
-        return $user->applyCurrentEmployment;
-    } else {
-        return $user->computeEmploymentDate;
-    }
+    $user->applyCurrentEmployment;
+
+    return $res;
 }