Skip to content

a redundant comma appears after data class generated without sort by alphabet #225

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

Jump to bottom
Closed
TedZen opened this issue Aug 16, 2019 · 0 comments
Closed

a redundant comma appears after data class generated without sort by alphabet #225

TedZen opened this issue Aug 16, 2019 · 0 comments
Assignees
@Karthik2007
Labels
bug L3 Difficulty level 3 (Maximum is 9)
Milestone
3.5.0

Comments

@TedZen
Copy link
Collaborator

TedZen commented Aug 16, 2019

with json string like [{"a":0,"b":""},{"a":null,"b":null}] and not choosing Order By Alphabet, an additional comma will occur after last property.

data class a(
val a: Int?,
val b: String?,
)
@TedZen TedZen added bug L3 Difficulty level 3 (Maximum is 9) labels Aug 16, 2019
@Karthik2007 Karthik2007 self-assigned this Aug 16, 2019
@wuseal wuseal added this to the 3.5.0 milestone Aug 19, 2019
Karthik2007 pushed a commit to Karthik2007/JsonToKotlinClass that referenced this issue Aug 22, 2019
bug fixed wuseal#225
1355953
@Karthik2007 Karthik2007 mentioned this issue Aug 22, 2019
Merged
Karthik2007 pushed a commit to Karthik2007/JsonToKotlinClass that referenced this issue Aug 23, 2019
Test case added for Issue wuseal#225
892c3cb
Karthik2007 pushed a commit to Karthik2007/JsonToKotlinClass that referenced this issue Aug 23, 2019
Test case content added for Issue wuseal#225
ad63652
@wuseal wuseal added the fixed but not released this issue has been solved but is not released yet label Aug 23, 2019
@wuseal wuseal closed this as completed in 1c488e5 Sep 2, 2019
@wuseal wuseal removed the fixed but not released this issue has been solved but is not released yet label Mar 21, 2020
wuseal pushed a commit that referenced this issue Sep 20, 2021
@Karthik2007 @wuseal
Bug fix #225 (#232)
e4878a9 
* Fix Issue #123 - File Already exists

* bug fixed #225

* Test case added for Issue #225

* Test case content added for Issue #225
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Karthik2007 Karthik2007

Labels
bug L3 Difficulty level 3 (Maximum is 9)
Projects
None yet
Milestone
3.5.0
Development

No branches or pull requests
3 participants
@wuseal @Karthik2007 @TedZen