From f68c7a751f353b8c203c329b5dcd0461915cf867 Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Wed, 31 Jul 2019 10:17:14 +0200 Subject: [PATCH] Polish "Escape reserved asciidoc character in generated documentation" See gh-17663 --- .../boot/configurationdocs/SingleConfigurationTableEntry.java | 3 ++- .../configurationdocs/SingleConfigurationTableEntryTests.java | 3 +-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/main/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntry.java b/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/main/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntry.java index 3ba620b297..f1832a6944 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/main/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntry.java +++ b/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/main/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntry.java @@ -76,7 +76,8 @@ class SingleConfigurationTableEntry extends ConfigurationTableEntry { builder.append("|"); } else { - builder.append("|+++", this.description.replace("|", "\\|"), "+++"); + String cleanedDescription = this.description.replace("|", "\\|"); + builder.append("|+++", cleanedDescription, "+++"); } } diff --git a/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/test/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntryTests.java b/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/test/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntryTests.java index ef84a2459e..8da4cc76c7 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/test/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntryTests.java +++ b/spring-boot-project/spring-boot-tools/spring-boot-configuration-docs/src/test/java/org/springframework/boot/configurationdocs/SingleConfigurationTableEntryTests.java @@ -90,13 +90,12 @@ class SingleConfigurationTableEntryTests { void descriptionWithPipe() { ConfigurationMetadataProperty property = new ConfigurationMetadataProperty(); property.setId("spring.test.prop"); - property.setDefaultValue("first\\second"); property.setDescription("This is a description with a | pipe."); property.setType("java.lang.String"); SingleConfigurationTableEntry entry = new SingleConfigurationTableEntry(property); AsciidocBuilder builder = new AsciidocBuilder(); entry.write(builder); - assertThat(builder.toString()).isEqualTo("|`+spring.test.prop+`" + NEWLINE + "|`+first\\\\second+`" + NEWLINE + assertThat(builder.toString()).isEqualTo("|`+spring.test.prop+`" + NEWLINE + "|" + NEWLINE + "|+++This is a description with a \\| pipe.+++" + NEWLINE); }