fixed the rewriting models
PR checklist
- [x] Read the contribution guidelines.
- [x] Ran the shell script under
./bin/to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.shand./bin/security/{LANG}-petstore.shif updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\. - [x] Filed the PR against the correct branch:
3.0.0branch for changes related to OpenAPI spec 3.0. Default:master. - [x] Copied the technical committee to review the pull request if your PR is targeting a particular programming language.
Description of the PR
Fixed erasing existing models in "allProcessedModels" if any model has the same names that on in map. Before, in this case, these models had the same position and cause one of them can be erasing. Now we correct position also being guided by model file path.
Comrades, why the first test failed?
Restarted the CI tests and all green now.
@serezha93 do you mind sharing a spec to reproduce the issue?
@wing328 I can attach a swagger file but where is the best place for this?
@serezha93 please send it to my email ([email protected]) or put it on https://gist.github.com.
@serezha93 I probably missed your email. Please send it (the spec) again when you've time.