Issue with field.watch

[JedWatson]

As reported on the google group here: https://groups.google.com/forum/#!topic/keystonejs/snCB2F_KcVU

I am trying to use the watch feature and can not get it to watch another field.

This works:

total: { type: Types.Number, initial:true, default: '0.00' },
totalreceived: { type: Types.Number, initial:true, default: '0.00', label:'Received Total'  },
complete: { type: Boolean, watch: true, value: checkTotal }

...

function checkTotal() {
    return this.total<=this.totalreceived ? true:false;
}

If I try and watch the totalreceived field it does not work

complete: { type: Boolean, watch: 'totalreceived', value: checkTotal }

[DerekDomino]

Hello,

I confirm

platform: { type: Types.Select, options: 'node, ruby, python' },
usesKeystone: { type: String, watch: 'platform', value: testup},

...
var testup = function() {
    return "3";
}

Does the field watching occurs at the UI level when using a string for the watch entry of the Field definition?

[Ventis]

When is a fix for this expected?

[kraigh]

This is working for me, except it is actually watching the field above the "watched" field.

This is what my model looks like:

integration: { type: Types.Text },
source: { type: Types.Text },
lastUpdated: { type: Types.Datetime, default: Date.now, watch: 'integration', value: Date.now },

Updating the "integration" field does nothing, but updating the "source" field will cause the lastUpdated field to be updated.

[JedWatson]

@DerekDomino it happens server-side, thanks to pre('save') middleware function that is added to the schema.

@kraigh that's really weird behaviour, hopefully it'll help me track down the issue.

@Ventis I'll try and tackle this one soon, I'd really like to fix it. If anyone else can dive in and help me figure out what's going wrong that'd be great, the relevant code is here:

https://github.com/JedWatson/keystone/blob/master/lib/field.js#L79

[Globegitter]

Just to make this bug report complete:
headerPostTime: { type: Types.Datetime, default: null, label: 'Header Post', watch: { 'headerPost': true }, value: Date.now } doesn't seem to work either. Will try to look into this issue though.

[Globegitter]

@JedWatson still looking into the issue, but saw that at least for _.isObject(this.options.watch) the issue seems to be that item.get('path') returns undefined (https://github.com/JedWatson/keystone/blob/master/lib/field.js#L105`
Will keep you updated/post a PR if I make progress.

Edit: Ok was an easy fix :P Will see if I can fix the rest and send a PR

[JedWatson]

Thanks @Globegitter! I've merged that, and will push it in a new version to npm soon.