Skip to content

Commit df26385

Browse files
authored
Merge pull request #422 from SentryMan/tojson-invalid
Fix Invalid `toJson` parameter name
2 parents 90b7099 + 00e5702 commit df26385

File tree

2 files changed

+29
-1
lines changed

2 files changed

+29
-1
lines changed

jsonb-generator/src/main/java/io/avaje/jsonb/generator/ClassReader.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,7 @@ private void writeViewBuild(Append writer) {
360360
@Override
361361
public void writeToJson(Append writer) {
362362
try {
363-
final String varName = Util.initLower(shortName);
363+
final String varName = "_" + Util.initLower(shortName);
364364
writer.eol();
365365
writer.append(" @Override").eol();
366366
writer.append(" public void toJson(JsonWriter writer, %s %s) {", shortName, varName).eol();
Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
package io.avaje.jsonb.generator.models.valid.naming;
2+
3+
import io.avaje.jsonb.Json;
4+
5+
@Json
6+
public class PackageNaming {
7+
private final Package queryPackage;
8+
9+
public PackageNaming(Package queryPackage) {
10+
this.queryPackage = queryPackage;
11+
}
12+
13+
public Package getQueryPackage() {
14+
return queryPackage;
15+
}
16+
17+
public static class Package {
18+
private final String ecosystem;
19+
20+
public Package(String ecosystem) {
21+
this.ecosystem = ecosystem;
22+
}
23+
24+
public String getEcosystem() {
25+
return ecosystem;
26+
}
27+
}
28+
}

0 commit comments

Comments
 (0)