dynamicdan / sn-filesync

2-way sync ServiceNow field values to local files
MIT License
66 stars 37 forks source link

Similar quitting issues #16

Closed stegel closed 8 years ago

stegel commented 8 years ago

Dan,

I'll start brushing up on my node skills and troubleshooting these myself, but until then, I'll share what I find with you.

17:02:02 - debug: -------------------------------------------------------
17:02:02 - info: Received: 
{ table: 'sys_ui_page',
  field: 'html',
  query: 'name=don_cod' }
17:02:02 - debug: Saving meta/hash data for file: /Users/aj.siegel/sites/fs-cod/ui_pages/don_cod.xhtml
17:02:02 - info: Local has no changes or remote in sync; no need for push/send.

/Users/aj.siegel/.Trash/filesync-master/app/src/file-record.js:128
            callback(true);
            ^
TypeError: undefined is not a function
    at method._saveMeta (/Users/aj.siegel/.Trash/filesync-master/app/src/file-record.js:128:13)
    at /Users/aj.siegel/.Trash/filesync-master/app/node_modules/fs-extra/node_modules/graceful-fs/graceful-fs.js:102:5
    at Object.oncomplete (evalmachine.<anonymous>:297:15)
dynamicdan commented 8 years ago

I just patched this yesterday. Can you please download the latest version and try again?

It was simply that the callback function was not set because it was originally not passed through from the start.

dynamicdan commented 8 years ago

Please note, from the error it looks like you're using the version of FileSync in the "Trash". So you are most likely using an old version. Closing issue.

stegel commented 8 years ago

Thanks for the catch, not sure why that happened. I had redownloaded your latest patch, but I missed something

Thanks On Sun, Dec 20, 2015 at 4:41 AM dynamicdan notifications@github.com wrote:

Please note, from the error it looks like you're using the version of FileSync in the "Trash". So you are most likely using an old version. Closing issue.

— Reply to this email directly or view it on GitHub https://github.com/dynamicdan/filesync/issues/16#issuecomment-166099617.