Skip to content

Commit ee85775

Browse files
authored
chore: Misc cleanup related to deprecations (#22172)
* Replace deprecated StringUtils.startsWith methods * Add missing @deprecated annotations
1 parent 94e8997 commit ee85775

File tree

7 files changed

+12
-6
lines changed

7 files changed

+12
-6
lines changed

modules/openapi-generator-cli/src/main/java/org/openapitools/codegen/cmd/GenerateBatch.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
import io.airlift.airline.Command;
3030
import io.airlift.airline.Option;
3131
import org.apache.commons.io.FileUtils;
32+
import org.apache.commons.lang3.Strings;
3233
import org.apache.commons.lang3.StringUtils;
3334
import org.openapitools.codegen.ClientOptInput;
3435
import org.openapitools.codegen.CodegenConfig;
@@ -248,7 +249,7 @@ private void cleanPreviousFiles(final String name, Path outDir) throws IOExcepti
248249
Path filesMeta = Paths.get(outDir.toAbsolutePath().toString(), ".openapi-generator", "FILES");
249250
if (filesMeta.toFile().exists()) {
250251
FileUtils.readLines(filesMeta.toFile(), StandardCharsets.UTF_8).forEach(relativePath -> {
251-
if (!StringUtils.startsWith(relativePath, ".")) {
252+
if (!Strings.CS.startsWith(relativePath, ".")) {
252253
Path file = outDir.resolve(relativePath).toAbsolutePath();
253254
// hack: disallow directory traversal outside of output directory. we don't want to delete wrong files.
254255
if (file.toString().startsWith(outDir.toAbsolutePath().toString())) {

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractPythonConnexionServerCodegen.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import io.swagger.v3.oas.models.parameters.Parameter;
3535
import io.swagger.v3.oas.models.parameters.RequestBody;
3636
import io.swagger.v3.oas.models.security.SecurityScheme;
37+
import org.apache.commons.lang3.Strings;
3738
import org.apache.commons.lang3.StringUtils;
3839
import org.openapitools.codegen.*;
3940
import org.openapitools.codegen.meta.features.DocumentationFeature;
@@ -601,7 +602,7 @@ public String packagePath() {
601602
@Override
602603
public String toModelImport(String name) {
603604
String modelImport;
604-
if (StringUtils.startsWithAny(name, "import", "from")) {
605+
if (Strings.CS.startsWithAny(name, "import", "from")) {
605606
modelImport = name;
606607
} else {
607608
modelImport = "from ";

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/PythonClientCodegen.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
import io.swagger.v3.oas.models.media.Schema;
2121
import io.swagger.v3.oas.models.security.SecurityScheme;
2222
import lombok.Setter;
23-
import org.apache.commons.lang3.StringUtils;
23+
import org.apache.commons.lang3.Strings;
2424
import org.openapitools.codegen.*;
2525
import org.openapitools.codegen.meta.GeneratorMetadata;
2626
import org.openapitools.codegen.meta.Stability;
@@ -355,7 +355,7 @@ public boolean getUseOneOfDiscriminatorLookup() {
355355
@Override
356356
public String toModelImport(String name) {
357357
String modelImport;
358-
if (StringUtils.startsWithAny(name, "import", "from")) {
358+
if (Strings.CS.startsWithAny(name, "import", "from")) {
359359
modelImport = name;
360360
} else {
361361
modelImport = "from ";

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/PythonFastAPIServerCodegen.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import com.fasterxml.jackson.databind.SerializerProvider;
2424
import com.fasterxml.jackson.databind.module.SimpleModule;
2525
import io.swagger.v3.oas.models.media.Schema;
26+
import org.apache.commons.lang3.Strings;
2627
import org.apache.commons.lang3.StringUtils;
2728
import org.openapitools.codegen.*;
2829
import org.openapitools.codegen.meta.GeneratorMetadata;
@@ -203,7 +204,7 @@ public String getName() {
203204
@Override
204205
public String toModelImport(String name) {
205206
String modelImport;
206-
if (StringUtils.startsWithAny(name, "import", "from")) {
207+
if (Strings.CS.startsWithAny(name, "import", "from")) {
207208
modelImport = name;
208209
} else {
209210
modelImport = "from ";

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/PythonPydanticV1ClientCodegen.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import io.swagger.v3.oas.models.media.Schema;
2121
import io.swagger.v3.oas.models.security.SecurityScheme;
2222
import lombok.Setter;
23+
import org.apache.commons.lang3.Strings;
2324
import org.apache.commons.lang3.StringUtils;
2425
import org.openapitools.codegen.*;
2526
import org.openapitools.codegen.meta.GeneratorMetadata;
@@ -353,7 +354,7 @@ public boolean getUseOneOfDiscriminatorLookup() {
353354
@Override
354355
public String toModelImport(String name) {
355356
String modelImport;
356-
if (StringUtils.startsWithAny(name, "import", "from")) {
357+
if (Strings.CS.startsWithAny(name, "import", "from")) {
357358
modelImport = name;
358359
} else {
359360
modelImport = "from ";

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/RustServerCodegen.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1408,6 +1408,7 @@ public String toAnyOfName(List<String> names, Schema composedSchema) {
14081408
*
14091409
* @deprecated Avoid using this - use a different mechanism instead.
14101410
*/
1411+
@Deprecated
14111412
private static String stripNullable(String type) {
14121413
if (type.startsWith("swagger::Nullable<") && type.endsWith(">")) {
14131414
return type.substring("swagger::Nullable<".length(), type.length() - 1);

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/RustServerCodegenDeprecated.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1408,6 +1408,7 @@ public String toAnyOfName(List<String> names, Schema composedSchema) {
14081408
*
14091409
* @deprecated Avoid using this - use a different mechanism instead.
14101410
*/
1411+
@Deprecated
14111412
private static String stripNullable(String type) {
14121413
if (type.startsWith("swagger::Nullable<") && type.endsWith(">")) {
14131414
return type.substring("swagger::Nullable<".length(), type.length() - 1);

0 commit comments

Comments
 (0)