search

vjekob / al-objid

Manage object IDs in multi-user environments with mind-boggling simplicity.
MIT License
29 stars 18 forks source link

Bug: license check doesn't check temporary tables #40

Closed waldo1001 closed 1 year ago

waldo1001 commented 2 years ago

Temporary tables does not need to be in the license.. image

vjekob commented 2 years ago

Yeah, this is a bug. I'll look into this.

pri-kise commented 1 year ago

@vjekob I'd really like to get this fixed with a pull request, but I couldn't find the information in your "semi-private" repository of your AL-Types / AL Parser what properties are stored in the ALObject Class.

https://github.com/vjekob/al-objid/blob/08a794c399e253ece9a82f9c769eab874d59e258/vscode-extension/src/lib/BCLicense.ts#L57

If the TableType is included in some generic property then it shouldn't be to difficult so skip the error at this position: https://github.com/vjekob/al-objid/blob/08a794c399e253ece9a82f9c769eab874d59e258/vscode-extension/src/lib/BCLicense.ts#L69

vjekob commented 1 year ago

Fixed in release 2.10.0 (to be released this week)