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

Error 94 "Incorrect use of Null" in Function "IDbComponent_DbObject" #408

Closed monheimx9 closed 1 year ago

monheimx9 commented 1 year ago

Dear joyfullservice

While exporting all sources

In the class "clsDbSharedImage" In the Function "IDbComponent_DbObject" Error 94 on the line:
m_Extension = m_Rst!Extension "Incorrect use of Null"

Tested on version v3.4.23 and tried on v4.0.13-beta as well (I took some long programming vacations, the add-in was working perfectly before) Access Version 2306 Build 16.0.16529.20100 (64Bits)

Here you can find a screenshot of the Immediate Debug Window Capture d’écran 2023-07-02 203716 "<Opération non autorisée pour ce type d'objet>" means "Operation not authorized for this object type" "Vrai" means "True" "Faux" means "False" "Opération non valide" means "Operation not valid" Translated from French

If I can provide you additional informations feel free to ask

Otherwise I want to thank you for this amazing addin, I used it a lot last year and provided me great help in my workflow Cheers

joyfullservice commented 1 year ago

Thanks for reporting this! I have wrapped the field reference in the Nz() function to ensure that we are assigning a string value to m_Extension.

monheimx9 commented 1 year ago

Thank you for your answer, I've made the change on my side as well But might check my comment here https://github.com/joyfullservice/msaccess-vcs-addin/commit/f8d1f229ff03900ab359d6d78388b1ddd909f9c0#r120797607 I think there is a typo

Wish you a nice day :)

monheimx9 commented 1 year ago

Sorry, did not see https://github.com/joyfullservice/msaccess-vcs-addin/commit/60522997954f519da7f8cf5435c30fabccc850f0 Thus my previous comment is irrelevant

joyfullservice commented 1 year ago

Yes, you are correct. I had some other refactoring I was doing in other commits around the same time. 😄 I am hoping to build an updated beta version in the next few days, which should include this and other fixes.

joyfullservice commented 1 year ago

I believe this has been resolved now. Closing out as completed.