Model: move clear_rookie from Users to Users::Class

diff --git a/src/newslash_web/lib/Newslash/Model/Moderations.pm b/src/newslash_web/lib/Newslash/Model/Moderations.pm
index 30e6587..256474b 100644 (file)
--- a/src/newslash_web/lib/Newslash/Model/Moderations.pm
+++ b/src/newslash_web/lib/Newslash/Model/Moderations.pm
@@ -234,7 +234,7 @@ sub create {
         }
         elsif ($reason->{val} > 0) {
             $upmods = 1;
-            $users->clear_rookie(uid => $comment->{uid});
+            $users->class->clear_rookie(uid => $comment->{uid});
             $users->class->update(uid => $comment->{uid},
                                   validated_commenter => { function => 'NOW()'}
                                  );

diff --git a/src/newslash_web/lib/Newslash/Model/Users.pm b/src/newslash_web/lib/Newslash/Model/Users.pm
index 40b4e5a..4256bae 100644 (file)
--- a/src/newslash_web/lib/Newslash/Model/Users.pm
+++ b/src/newslash_web/lib/Newslash/Model/Users.pm
@@ -129,41 +129,6 @@ sub nickname_to_matchname {
     return $nick;
 }
 
-=head2 clear_rookie($uid, uid => $uid, time => $time)
-
-clear Rookie flag.
-
-=over 4
-
-=item Return value
-
-1/0
-
-=back
-
-=cut
-
-sub clear_rookie {
-    my $self = shift;
-    return if $self->check_readonly;
-    my $params = {@_};
-    my $uid = $params->{uid};
-    return if !$uid;
-
-    my $time = $params->{time};
-    if ($time) {
-        return $self->class->update(uid => $uid,
-                                    rookie => 0,
-                                    rookie_cleared => $time);
-    }
-    else {
-        return $self->class->update(uid => $uid,
-                                    rookie => 0,
-                                    rookie_cleared => { FUNCTION => "NOW()" });
-    }
-    return;
-}
-
 sub update_karma {
     my $self = shift;
     my $params = {@_};

diff --git a/src/newslash_web/lib/Newslash/Model/Users/Class.pm b/src/newslash_web/lib/Newslash/Model/Users/Class.pm
index d131956..b13136d 100644 (file)
--- a/src/newslash_web/lib/Newslash/Model/Users/Class.pm
+++ b/src/newslash_web/lib/Newslash/Model/Users/Class.pm
@@ -31,4 +31,60 @@ sub update {
     return $self->generic_update(params => $params);
 }
 
+=head2 clear_rookie($uid, uid => $uid, time => $time)
+
+clear Rookie flag.
+
+=over 4
+
+=item Return value
+
+1/0
+
+=back
+
+=cut
+sub clear_rookie {
+    my $self = shift;
+    return if $self->check_readonly;
+    my $params = {@_};
+    my $uid = $params->{uid};
+    return if !$uid;
+
+    my $time = $params->{time};
+    if ($time) {
+        return $self->update(uid => $uid,
+                             rookie => 0,
+                             rookie_cleared => $time);
+    }
+    else {
+        return $self->update(uid => $uid,
+                             rookie => 0,
+                             rookie_cleared => { function => "NOW()" });
+    }
+    return;
+}
+
+sub _validate {
+    my $self = shift;
+    my $parmas = {@_};
+    return if !$params->{uid} || !$params->{target};
+    my $users = $self->new_instance_of("Users");
+    return if !$users->is_anonymous_uid($params->{uid});
+
+    if ($params->{time}) {
+        return $self->$self->update(uid => $params->{uid},
+                                    $params->{target} => $params->{time}
+                                   );
+    }
+    else {
+        return $self->update(uid => $params->{uid},
+                                    $params->{target} => { function => 'NOW()'}
+                                   );
+    }
+}
+
+sub validate_submitter {
+    return $shift->update(validated_submitter, target => @_);
+
 1;