Skip to content

Feature/sap label support for odata_service_info_for_Z_V_MM03 with Include detailed metadata information#27

Open
viktorashi wants to merge 4 commits intooisee:mainfrom
viktorashi:feature/sap-label-support
Open

Feature/sap label support for odata_service_info_for_Z_V_MM03 with Include detailed metadata information#27
viktorashi wants to merge 4 commits intooisee:mainfrom
viktorashi:feature/sap-label-support

Conversation

@viktorashi
Copy link
Copy Markdown

I've added a little feature (which helped me in what I was doing) which also takes into account the sap:label= from a certain $metadata endpoint, if there exists one.

So for example, if you have an extry like this in the XML $metadata:

<Property Name="A0MATERIAL" Type="Edm.String" MaxLength="40" sap:aggregation-role="dimension" sap:text="A0MATERIAL_T" sap:label="Material" sap:creatable="false" sap:updatable="false"/>

Calling the odata_service_info_for_Z_V_MM03 with Include detailed metadata information turned on, the result will look like:

entity_types_detail/<entity_name>/properties/ entry will also look like:

6:
{
name:"A0MATERIAL",
type:"Edm.String",
nullable:true,
is_key:false,
sap_label:"Material"
}

Also added a test case for it.

Now: Just for a headsup, and I'm sorry in advance: I have no idea how to write Go, I just needed it for something and this is the best GPT5.2-Codex running in Codex CLI could whip out.

In reality, maybe this should be under some flag? And the structure of the patch probably doesn't even match with the way PR's are usually filed in here, but take it with a grain of salt in case anyone needs this.

Again, i know Open source is already inundated with AI slop and I don't wanna contribute to it that much, but yeah, if you need it, feel free to clone / build from my branch

@viktorashi
Copy link
Copy Markdown
Author

Also added some support for sap aggregation roles while i was at it, I don't think there's much point in anyone trying to solve those conflicts since it's just AI, single use-case slop =))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants