Don't update class field in moira if the year field in the registrar's

data was empty.

diff --git a/regtape/students.pc b/regtape/students.pc
index 8275d5b07be6153b891d0d07d950961583cb6bec..f108c2eb1b78ea7519e27ab3f4b79836ba73229c 100644 (file)
--- a/regtape/students.pc
+++ b/regtape/students.pc
@@ -300,11 +300,13 @@ struct entry *e;
     }
 
     /* See if class changed: if it's different, and the value in the database
-     * is not STAFF or SIPB, then update the database.  Since they were on the
+     * is not STAFF or SIPBMEM, and the value from the tape is actually
+     * meaningful, then update the database.  Since they were on the
      * students tape, make the account usable.
      */
     if (strcmp(e->class, strtrim(class)) &&
-       strcmp(class, "STAFF") && strcmp(class, "SIPB")) {
+       strcmp(class, "STAFF") && strcmp(class, "SIPBMEM") &&
+       e->year[0]) {
        com_err(whoami, 0, "updating class for user %s %s from %s to %s",
                first, last, class, e->class);
        if (status == US_NOT_ALLOWED) status = US_NO_LOGIN_YET;