Fix 4796: Prepend global:: to references of generated C# types

[peterwurzinger]

This should fix #4796 by prepending global:: to references of generated C# types.

@deinok: Can you confirm if using a kiota build of this branch to generate your clients resolves the issue? I did some tests with samples that I had at hand and it seems to work, but having confirmation from a real-world example would be nice.

cc @baywet

[baywet]

Thanks for the contribution! Can you also add an entry to the changelog please? (unreleased, changed)

[deinok]

Well, yes, it works more or less.

But now looks like a method called "AsList" does not exist.

[baywet]

@deinok this is because of #4823 Just update your abstractions reference

[peterwurzinger]

Hmm, I see.
At first sight it looks like it could be unrelated. Could you try using a build that's based on the current main branch and see if the problem occurs as well?

Also could you share the OpenAPI documents you're using, or are those internal ones?
I can have a look tomorrow morning if the issue was introduced by the PR - for today we probably are both watching Spain-Italy 😄

EDIT: Alright, seems like this is unrelated as per @baywet s comment

[deinok]

Hmm, I see. At first sight it looks like it could be unrelated. Could you try using a build that's based on the current main branch and see if the problem occurs as well?

Also could you share the OpenAPI documents you're using, or are those internal ones? I can have a look tomorrow morning if the issue was introduced by the PR - for today we probably are both watching Spain-Italy 😄

EDIT: Alright, seems like this is unrelated as per @baywet s comment

Yup, unrelated, its the Remove LINQ change.

Regarding my OpenAPI file, its not the issue as it is. Probably its because i have a layout like this:

So they clash a bit internaly, thats the reason i need the global.

Regarding Spain-Italy, try not to drink too much and have fun 😄

EDIT: A test to check this kind of project layout where two folders contain two kiota instalations would be awesome

[peterwurzinger]

@baywet I added a changelog entry and a dedicated test for the global:: prefix to make sure this does not see any regression.

@deinok Well, Spain would have deserved more goals. 😄
I don't really have more time to create a unit test for that scenario. As far as I can tell, it would become quite complex, as the test assertion would boil down to "does it compile?" after the generation run(s).

[baywet]

Thanks for making the change