Repository navigation
Test PR - #49
Test PR#49Alexander Holstrup (aholstrup1) wants to merge 1 commit into
Conversation
| BingMapsSettings: Record "BingMaps Settings"; | ||
| BingMapsSetup: Codeunit "BingMaps Setup"; | ||
| sessionId: Integer; | ||
| sessionId: Integerz; |
There was a problem hiding this comment.
The declaration uses Integerz, which is not a valid AL type and prevents the page from compiling. Restore the variable declaration to sessionId: Integer;.
Suggested fix (apply manually — could not be anchored as a one-click suggestion):
sessionId: Integer;👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v10.0.7
| ERROR(ErrorText); | ||
| BingMapsSettings."BingMaps Key OK" := true; | ||
| BingMapsSettings.Modify(); | ||
| BingMapsSettings.Modify() |
There was a problem hiding this comment.
The missing semicolon after the second BingMapsSettings.Modify() call causes a syntax error before Commit() and prevents the page from compiling. Terminate the Modify statement with a semicolon.
Suggested fix (apply manually — could not be anchored as a one-click suggestion):
BingMapsSettings.Modify();👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v10.0.7
|
This API page is exposed through PageType = API, but the app's BingMaps permission set has no page execute entry for page "BingMaps Settings". Add page "BingMaps Settings" = X to an appropriate permission set, preferably a dedicated non-default set for this sensitive endpoint, or remove the API exposure. Knowledge: Line mapping was unavailable, so this was posted as an issue comment. 👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v10.0.7 |
No description provided.