Self-registration property ExpireAfterDays used to set enabled date not disabled

externalSubjects.autoaddGroupExpireAfterDays = 7
externalSubjects.autoaddGroups = test:autoProvisionGroup
externalSubjects.registerRequiresInvite = false

edu.internet2.middleware.grouper.externalSubjects.ExternalSubject.assignGroups(String groups, String actions, boolean isInsert, final int expireAfterDays)

...

          Group group = GroupFinder.findByName(grouperSession, groupName, true);

          group.addMember(subject, false);

          if (expireAfterDays > 0) {

            Membership membership = group.getImmediateMembership(Group.getDefaultList(), subject, true, true);

        **  membership.setEnabledTime(new Timestamp(System.currentTimeMillis() + ((long)expireAfterDays*24*60*60*1000)));

            membership.update();

          }

...

This probably needs to be membership.setDisabledTime, right? When auto provisioning, memberships are set to start 7 days in the future, which doesn't make sense.