You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2017/07/22 14:13:14 UTC
camel git commit: CAMEL-11577: apt plugin should generate URL as URL
and not U R L in displayName
Repository: camel
Updated Branches:
refs/heads/master a59e86b29 -> 617cfbb04
CAMEL-11577: apt plugin should generate URL as URL and not U R L in displayName
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/617cfbb0
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/617cfbb0
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/617cfbb0
Branch: refs/heads/master
Commit: 617cfbb04f91dfc4350c94fb8f9b7b8b6589bc26
Parents: a59e86b
Author: Claus Ibsen <da...@apache.org>
Authored: Sat Jul 22 16:12:53 2017 +0200
Committer: Claus Ibsen <da...@apache.org>
Committed: Sat Jul 22 16:12:53 2017 +0200
----------------------------------------------------------------------
.../main/java/org/apache/camel/tools/apt/helper/Strings.java | 6 +++++-
.../test/java/org/apache/camel/tools/apt/StringsTest.java | 8 ++++++++
2 files changed, 13 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/617cfbb0/tooling/apt/src/main/java/org/apache/camel/tools/apt/helper/Strings.java
----------------------------------------------------------------------
diff --git a/tooling/apt/src/main/java/org/apache/camel/tools/apt/helper/Strings.java b/tooling/apt/src/main/java/org/apache/camel/tools/apt/helper/Strings.java
index 6c5d870..7ba92a5 100644
--- a/tooling/apt/src/main/java/org/apache/camel/tools/apt/helper/Strings.java
+++ b/tooling/apt/src/main/java/org/apache/camel/tools/apt/helper/Strings.java
@@ -129,7 +129,11 @@ public final class Strings {
if (first) {
sb.append(Character.toUpperCase(c));
} else if (upper) {
- sb.append(' ');
+ char prev = sb.charAt(sb.length() - 1);
+ if (!Character.isUpperCase(prev)) {
+ // append space if previous is not upper
+ sb.append(' ');
+ }
sb.append(c);
} else {
sb.append(Character.toLowerCase(c));
http://git-wip-us.apache.org/repos/asf/camel/blob/617cfbb0/tooling/apt/src/test/java/org/apache/camel/tools/apt/StringsTest.java
----------------------------------------------------------------------
diff --git a/tooling/apt/src/test/java/org/apache/camel/tools/apt/StringsTest.java b/tooling/apt/src/test/java/org/apache/camel/tools/apt/StringsTest.java
index 3903d53..c554eec 100644
--- a/tooling/apt/src/test/java/org/apache/camel/tools/apt/StringsTest.java
+++ b/tooling/apt/src/test/java/org/apache/camel/tools/apt/StringsTest.java
@@ -18,6 +18,7 @@ package org.apache.camel.tools.apt;
import org.junit.Test;
+import static org.apache.camel.tools.apt.helper.Strings.asTitle;
import static org.apache.camel.tools.apt.helper.Strings.between;
import static org.junit.Assert.assertEquals;
@@ -27,4 +28,11 @@ public class StringsTest {
public void testBetween() {
assertEquals("org.apache.camel.model.OnCompletionDefinition", between("java.util.List<org.apache.camel.model.OnCompletionDefinition>", "<", ">"));
}
+
+ @Test
+ public void testAsTitle() {
+ assertEquals("Broker URL", asTitle("brokerURL"));
+ assertEquals("Expose All Queues", asTitle("exposeAllQueues"));
+ assertEquals("Reply To Concurrent Consumers", asTitle("replyToConcurrentConsumers"));
+ }
}