Skip to content

[kotlin] add option for non public api #4089

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.

Sign up for GitHub

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

Merged
wing328 merged 9 commits into from
Oct 10, 2019
Merged

[kotlin] add option for non public api #4089

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
2f8ef97
[kotlin] add option for non public api
4brunu Oct 7, 2019
2a38b36
Merge branch 'master' into kotlin-visibility
4brunu Oct 9, 2019
836f550
[kotlin] update petstores
4brunu Oct 9, 2019
4f530c9
Merge branch 'master' into kotlin-visibility
4brunu Oct 9, 2019
4d9ae1d
[kotlin] add missing script to kotlin-client-all.sh
4brunu Oct 9, 2019
fecc416
[kotlin] update pet projects
4brunu Oct 9, 2019
708c765
[kotlin] remove public modifier
4brunu Oct 9, 2019
d3a33b6
[kotlin-client] inline visibility modifier
4brunu Oct 10, 2019
b5a1335
Merge branch 'master' into kotlin-visibility
4brunu Oct 10, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into kotlin-visibility 
# Conflicts:
#	modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractKotlinCodegen.java
  • Loading branch information
@4brunu
4brunu committed Oct 9, 2019
commit 2a38b36c9475983ccd4ad7906c4518dbaa8bc456
9 changes: 9 additions & 0 deletions ...api-generator/src/main/java/org/openapitools/codegen/languages/AbstractKotlinCodegen.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -438,6 +438,7 @@ public void processOpts() {
additionalProperties.put(CodegenConstants.NON_PUBLIC_API, nonPublicApi);
}

additionalProperties.put(CodegenConstants.NEEDS_DATACLASS_BODY, this.hasEnums || serializableModel);
additionalProperties.put(CodegenConstants.API_PACKAGE, apiPackage());
additionalProperties.put(CodegenConstants.MODEL_PACKAGE, modelPackage());

Expand Down Expand Up @@ -497,6 +498,14 @@ public void setNonPublicApi(boolean nonPublicApi) {
this.nonPublicApi = nonPublicApi;
}

public boolean isNeedsDataClassBody() {
return needsDataClassBody;
}

public void setNeedsDataClassBody(boolean needsDataClassBody) {
this.needsDataClassBody = needsDataClassBody;
}

/**
* Return the sanitized variable name for enum
*
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.