Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Save role includes access with uuid null. #57

Merged

Conversation

EdwinBetanc0urt
Copy link
Contributor

@EdwinBetanc0urt EdwinBetanc0urt commented Oct 1, 2024

If there is a form that does not have a UUID, but a role has access to that role, it is not exported to OpenSearch.

"form_access": [
    null,
    "719e204d-4885-4497-a3c2-1cead8ca7525",
    "0253e1db-a4b6-4d9f-9a53-db05b0fa6184",
    null,
    "76a242ef-e7bb-4cc1-9a88-42a0f5e23444",
    "916ee6d6-b56b-4f82-aaa5-ba8c95df9ca7",
    "463e90f4-9af3-4737-bdc2-519ec0a5fbae"
]
2024-10-01T01:23:22.152Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 7958
2024-10-01T01:23:22.153Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8192
2024-10-01T01:23:22.153Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:22.620Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8114
2024-10-01T01:23:22.620Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:22.621Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:22.621Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8114
2024-10-01T01:23:22.621Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:22.621Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8114
2024-10-01T01:23:22.621Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8125
2024-10-01T01:23:22.622Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8007
2024-10-01T01:23:22.771Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:22.771Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8114
2024-10-01T01:23:22.772Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:22.772Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8158
2024-10-01T01:23:22.772Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8158
2024-10-01T01:23:22.772Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8114
2024-10-01T01:23:22.773Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:23.073Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8002
2024-10-01T01:23:23.073Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8114
2024-10-01T01:23:23.074Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8002
2024-10-01T01:23:23.074Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8119
2024-10-01T01:23:23.074Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8121
2024-10-01T01:23:23.074Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8192
2024-10-01T01:23:23.227Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 8192
2024-10-01T01:23:23.227Z WARN  [server] Topic: "role", invalid type: null, expected a string at line 1 column 6905

Now it is exported anyway but it is not considered as part of the role access.

Additional context

fixes solop-develop/frontend-core#2716

src/models/menu_item.rs Outdated Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
src/bin/server.rs Outdated Show resolved Hide resolved
@yamelsenih yamelsenih merged commit 2fceede into adempiere:main Oct 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants