How does this.isSet in custom validation behave?

[luixal]

Hi, I'm facing something kind of weird...

I have this definition for a field inside a schema:

pin: {
    type: Number,
    label: function() { return I18N.translate('employee_pin'); },
    optional: true,
    custom: function() {

      if (Meteor.isClient && this.isSet) {
        console.log(this); // --> this gets fired ALWAYS!!!
        if ( (this.value <= 0) || (this.value >= 100000000) ) {
          Employees.simpleSchema().namedContext("modalForm").addInvalidKeys([{name: "pin", type: "outOfRange"}]);
          return 'outOfRange';
        }
        Meteor.call(
          'getEmployeeByPIN', Meteor.user().currentProjectId, this.value,
          function(error, result) {
            if (result) {
              Employees.simpleSchema().namedContext("modalForm").addInvalidKeys([{name: "pin", type: "notUnique"}]);
              return 'notUnique';
            }
          }
        );
      }
    }
  },

that console logs shows this object:

...
docId: "6QQTMm7CrQx7MdeAM"
genericKey: "pin"
isFromTrustedCode: false
isInsert: false
isLocalCollection: false
isSet: true
isUpdate: true
isUpsert: false
key: "pin"
operator: "$unset"
userId: "4dy58r789bBqYKky6"
value: ""

this happens when updating a doc using AutoForm's quickform.

Any idea why isSet is set to true if field is not being filled in? It's madness.

[aldeed]

The Meteor SimpleSchema package is no longer maintained other than critical fixes to keep it running with each latest Meteor release. Closing non-critical issues in this repo. Anyone who cares about this may do one or more of the following:

• Switch to the NPM package. Be sure to adjust for the breaking changes. There can be a lot of work involved in switching, but it is worth the effort.
• If this issue still occurs in the NPM package, submit an issue there. Be sure to include a link to a reproduction repo or a PR that adds a failing test for the issue.
• If you have more time than money, submit a pull request to resolve any issue labeled "help wanted" in the NPM package repo.
• If you have more money than time, donate to help support development if you are able