joyfullservice / msaccess-vcs-addin

Synchronize your Access Forms, Macros, Modules, Queries, Reports, and more with a version control system.
Other
211 stars 41 forks source link

Module clsVCSIndex - procedure LoadFromFile - Runtime error 13 - datatype mismatch in CallByName #452

Closed ollypsilon closed 12 months ago

ollypsilon commented 1 year ago

https://github.com/joyfullservice/msaccess-vcs-addin/blob/a666c9dbe161d00580c517874955222b1f86e142/Version%20Control.accda.src/modules/clsVCSIndex.cls#L86

With latest VCS 4.0.25 from dev branch I get this error, no matter if I try to export all, export just one selected object or click on "Open Add-in". image

With 4.0.9 all was fine, see attached export.log. Export.log

ollypsilon commented 1 year ago

Looks like this is caused by changing language in Office setup from German to English. Date/time value can't be parsed correct I guess. Maybe VCS add-in options should be converted and saved in a fixed NLS.

image

hecon5 commented 1 year ago

I think another way to deal with it is to use ISO8601 export type, and then it's a non issue. See also #354 as I'm pretty sure it's the same issue.

ollypsilon commented 1 year ago

Thanks, going to check on Monday

joyfullservice commented 12 months ago

The simple fix is to delete the index file and run an export to regenerate it. (This will update all the dates to your current local format.)

I have spent some time pondering @hecon5's proposed suggestion of using UTC/ISO 8601 for storing dates, and I will respond to that discussion in #354.

hecon5 commented 12 months ago

This should be fixed on the latest dev now; you'll need do an export and the dates should reset. Also deleting the existing index file may help this. Since there's nothing critical in the index (it's used only for change detection) not having an index will just slow down the first export and act more like a full export.

ollypsilon commented 12 months ago

Thanks! Export with new version succeeded! Well, spotted two other issues, will open separate threads.