[php-nextgen] Discriminator class detection uses wrong namespace #22811
+12
−4
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When using discriminator - for example:
then the object serializer is trying to find matching class ChangeNewVO.
ObjectSerializerclass inside generated package was using hard-coded namespace{{invokerPackage}}\Modelinstead of proper{{modelPackage}}.Error is happening only when non-standard
modelPackageconfig option is set.@jebentier @dkarlovi @mandrean @jfastnacht @ybelenko @renepardon
Summary by cubic
Fix discriminator subclass resolution in PHP NextGen to use the configured modelPackage, so polymorphic models resolve correctly when a custom modelPackage is set. Also refactors a TypeScript sample to build request options without executing the request.
Bug Fixes
Refactors
Written for commit 93c7b0b. Summary will update on new commits.