aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Fick <mfick@codeaurora.org>2011-07-22 14:08:45 -0700
committerAndroid Code Review <code-review@android.com>2011-07-22 14:08:45 -0700
commit3deaf9af54df3306e897f7148ea8dc4ce98076d4 (patch)
treea783832e2036b7a2742f3222b170cf20975a5217
parentd46ee8bade08ca3634cf0adf0b95599ba7e5ce6c (diff)
parent74ca597a8a4e35188f16986e13e21b96560bf8d5 (diff)
downloadgerrit-3deaf9af54df3306e897f7148ea8dc4ce98076d4.tar.gz
Merge "Fixed a typo in the name of the ParamertizedString class."
-rw-r--r--gerrit-common/src/main/java/com/google/gerrit/common/data/GitwebLink.java6
-rw-r--r--gerrit-common/src/main/java/com/google/gerrit/common/data/ParameterizedString.java (renamed from gerrit-common/src/main/java/com/google/gerrit/common/data/ParamertizedString.java)14
-rw-r--r--gerrit-common/src/test/java/com/google/gerrit/common/data/ParameterizedStringTest.java (renamed from gerrit-common/src/test/java/com/google/gerrit/common/data/ParamertizedStringTest.java)22
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java14
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapQuery.java6
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapRealm.java14
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/mail/FromAddressGeneratorProvider.java10
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/project/SectionMatcher.java6
8 files changed, 46 insertions, 46 deletions
diff --git a/gerrit-common/src/main/java/com/google/gerrit/common/data/GitwebLink.java b/gerrit-common/src/main/java/com/google/gerrit/common/data/GitwebLink.java
index 937bbd4b..0460bf2a 100644
--- a/gerrit-common/src/main/java/com/google/gerrit/common/data/GitwebLink.java
+++ b/gerrit-common/src/main/java/com/google/gerrit/common/data/GitwebLink.java
@@ -37,7 +37,7 @@ public class GitwebLink {
}
public String toRevision(final Project.NameKey project, final PatchSet ps) {
- ParamertizedString pattern = new ParamertizedString(type.getRevision());
+ ParameterizedString pattern = new ParameterizedString(type.getRevision());
final Map<String, String> p = new HashMap<String, String>();
p.put("project", URL.encodeQueryString(project.get()));
@@ -46,7 +46,7 @@ public class GitwebLink {
}
public String toProject(final Project.NameKey project) {
- ParamertizedString pattern = new ParamertizedString(type.getProject());
+ ParameterizedString pattern = new ParameterizedString(type.getProject());
final Map<String, String> p = new HashMap<String, String>();
p.put("project", URL.encodeQueryString(project.get()));
@@ -54,7 +54,7 @@ public class GitwebLink {
}
public String toBranch(final Branch.NameKey branch) {
- ParamertizedString pattern = new ParamertizedString(type.getBranch());
+ ParameterizedString pattern = new ParameterizedString(type.getBranch());
final Map<String, String> p = new HashMap<String, String>();
p.put("project", URL.encodeQueryString(branch.getParentKey().get()));
diff --git a/gerrit-common/src/main/java/com/google/gerrit/common/data/ParamertizedString.java b/gerrit-common/src/main/java/com/google/gerrit/common/data/ParameterizedString.java
index ae696b55..f9b0dade 100644
--- a/gerrit-common/src/main/java/com/google/gerrit/common/data/ParamertizedString.java
+++ b/gerrit-common/src/main/java/com/google/gerrit/common/data/ParameterizedString.java
@@ -21,10 +21,10 @@ import java.util.List;
import java.util.Map;
/** Performs replacements on strings such as <code>Hello ${user}</code>. */
-public class ParamertizedString {
+public class ParameterizedString {
/** Obtain a string which has no parameters and always produces the value. */
- public static ParamertizedString asis(final String constant) {
- return new ParamertizedString(new Constant(constant));
+ public static ParameterizedString asis(final String constant) {
+ return new ParameterizedString(new Constant(constant));
}
private final String pattern;
@@ -32,18 +32,18 @@ public class ParamertizedString {
private final List<Format> patternOps;
private final List<Parameter> parameters;
- protected ParamertizedString() {
+ protected ParameterizedString() {
this(new Constant(""));
}
- private ParamertizedString(final Constant c) {
+ private ParameterizedString(final Constant c) {
pattern = c.text;
rawPattern = c.text;
patternOps = Collections.<Format> singletonList(c);
parameters = Collections.emptyList();
}
- public ParamertizedString(final String pattern) {
+ public ParameterizedString(final String pattern) {
final StringBuilder raw = new StringBuilder();
final List<Parameter> prs = new ArrayList<Parameter>(4);
final List<Format> ops = new ArrayList<Format>(4);
@@ -152,7 +152,7 @@ public class ParamertizedString {
@Override
public String toString() {
- return ParamertizedString.this.replace(params);
+ return ParameterizedString.this.replace(params);
}
}
diff --git a/gerrit-common/src/test/java/com/google/gerrit/common/data/ParamertizedStringTest.java b/gerrit-common/src/test/java/com/google/gerrit/common/data/ParameterizedStringTest.java
index af8301fd..4fe3365d 100644
--- a/gerrit-common/src/test/java/com/google/gerrit/common/data/ParamertizedStringTest.java
+++ b/gerrit-common/src/test/java/com/google/gerrit/common/data/ParameterizedStringTest.java
@@ -19,9 +19,9 @@ import junit.framework.TestCase;
import java.util.HashMap;
import java.util.Map;
-public class ParamertizedStringTest extends TestCase {
+public class ParameterizedStringTest extends TestCase {
public void testEmptyString() {
- final ParamertizedString p = new ParamertizedString("");
+ final ParameterizedString p = new ParameterizedString("");
assertEquals("", p.getPattern());
assertEquals("", p.getRawPattern());
assertTrue(p.getParameterNames().isEmpty());
@@ -33,7 +33,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testAsis1() {
- final ParamertizedString p = ParamertizedString.asis("${bar}c");
+ final ParameterizedString p = ParameterizedString.asis("${bar}c");
assertEquals("${bar}c", p.getPattern());
assertEquals("${bar}c", p.getRawPattern());
assertTrue(p.getParameterNames().isEmpty());
@@ -46,7 +46,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplace1() {
- final ParamertizedString p = new ParamertizedString("${bar}c");
+ final ParameterizedString p = new ParameterizedString("${bar}c");
assertEquals("${bar}c", p.getPattern());
assertEquals("{0}c", p.getRawPattern());
assertEquals(1, p.getParameterNames().size());
@@ -61,7 +61,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplace2() {
- final ParamertizedString p = new ParamertizedString("a${bar}c");
+ final ParameterizedString p = new ParameterizedString("a${bar}c");
assertEquals("a${bar}c", p.getPattern());
assertEquals("a{0}c", p.getRawPattern());
assertEquals(1, p.getParameterNames().size());
@@ -76,7 +76,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplace3() {
- final ParamertizedString p = new ParamertizedString("a${bar}");
+ final ParameterizedString p = new ParameterizedString("a${bar}");
assertEquals("a${bar}", p.getPattern());
assertEquals("a{0}", p.getRawPattern());
assertEquals(1, p.getParameterNames().size());
@@ -91,7 +91,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplace4() {
- final ParamertizedString p = new ParamertizedString("a${bar}c");
+ final ParameterizedString p = new ParameterizedString("a${bar}c");
assertEquals("a${bar}c", p.getPattern());
assertEquals("a{0}c", p.getRawPattern());
assertEquals(1, p.getParameterNames().size());
@@ -105,7 +105,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplaceToLowerCase() {
- final ParamertizedString p = new ParamertizedString("${a.toLowerCase}");
+ final ParameterizedString p = new ParameterizedString("${a.toLowerCase}");
assertEquals(1, p.getParameterNames().size());
assertTrue(p.getParameterNames().contains("a"));
@@ -125,7 +125,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplaceToUpperCase() {
- final ParamertizedString p = new ParamertizedString("${a.toUpperCase}");
+ final ParameterizedString p = new ParameterizedString("${a.toUpperCase}");
assertEquals(1, p.getParameterNames().size());
assertTrue(p.getParameterNames().contains("a"));
@@ -145,7 +145,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testReplaceLocalName() {
- final ParamertizedString p = new ParamertizedString("${a.localPart}");
+ final ParameterizedString p = new ParameterizedString("${a.localPart}");
assertEquals(1, p.getParameterNames().size());
assertTrue(p.getParameterNames().contains("a"));
@@ -165,7 +165,7 @@ public class ParamertizedStringTest extends TestCase {
}
public void testUndefinedFunctionName() {
- ParamertizedString p = new ParamertizedString("${a.anUndefinedMethod}");
+ ParameterizedString p = new ParameterizedString("${a.anUndefinedMethod}");
assertEquals(1, p.getParameterNames().size());
assertTrue(p.getParameterNames().contains("a.anUndefinedMethod"));
}
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java
index 4a24b321..e11e9bb7 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java
@@ -14,7 +14,7 @@
package com.google.gerrit.server.auth.ldap;
-import com.google.gerrit.common.data.ParamertizedString;
+import com.google.gerrit.common.data.ParameterizedString;
import com.google.gerrit.reviewdb.AccountGroup;
import com.google.gerrit.server.account.AccountException;
import com.google.gerrit.server.account.GroupCache;
@@ -224,16 +224,16 @@ import javax.net.ssl.SSLSocketFactory;
class LdapSchema {
final LdapType type;
- final ParamertizedString accountFullName;
- final ParamertizedString accountEmailAddress;
- final ParamertizedString accountSshUserName;
+ final ParameterizedString accountFullName;
+ final ParameterizedString accountEmailAddress;
+ final ParameterizedString accountSshUserName;
final String accountMemberField;
final List<LdapQuery> accountQueryList;
boolean groupNeedsAccount;
final List<String> groupBases;
final SearchScope groupScope;
- final ParamertizedString groupPattern;
+ final ParameterizedString groupPattern;
final List<LdapQuery> groupMemberQueryList;
LdapSchema(final DirContext ctx) {
@@ -255,7 +255,7 @@ import javax.net.ssl.SSLSocketFactory;
for (String groupBase : groupBases) {
if (groupMemberPattern != null) {
final LdapQuery groupMemberQuery =
- new LdapQuery(groupBase, groupScope, new ParamertizedString(
+ new LdapQuery(groupBase, groupScope, new ParameterizedString(
groupMemberPattern), Collections.<String> emptySet());
if (groupMemberQuery.getParameters().isEmpty()) {
throw new IllegalArgumentException(
@@ -303,7 +303,7 @@ import javax.net.ssl.SSLSocketFactory;
for (String accountBase : LdapRealm.requiredList(config, "accountBase")) {
final LdapQuery accountQuery =
- new LdapQuery(accountBase, accountScope, new ParamertizedString(
+ new LdapQuery(accountBase, accountScope, new ParameterizedString(
accountPattern), accountAtts);
if (accountQuery.getParameters().isEmpty()) {
throw new IllegalArgumentException(
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapQuery.java b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapQuery.java
index 7d1e37d8..8a6dfeb3 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapQuery.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapQuery.java
@@ -14,7 +14,7 @@
package com.google.gerrit.server.auth.ldap;
-import com.google.gerrit.common.data.ParamertizedString;
+import com.google.gerrit.common.data.ParameterizedString;
import java.util.ArrayList;
import java.util.Collections;
@@ -38,11 +38,11 @@ class LdapQuery {
private final String base;
private final SearchScope searchScope;
- private final ParamertizedString pattern;
+ private final ParameterizedString pattern;
private final String[] returnAttributes;
LdapQuery(final String base, final SearchScope searchScope,
- final ParamertizedString pattern, final Set<String> returnAttributes) {
+ final ParameterizedString pattern, final Set<String> returnAttributes) {
this.base = base;
this.searchScope = searchScope;
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapRealm.java b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapRealm.java
index e008e6cc..e804ce64 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapRealm.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/LdapRealm.java
@@ -16,7 +16,7 @@ package com.google.gerrit.server.auth.ldap;
import static com.google.gerrit.reviewdb.AccountExternalId.SCHEME_GERRIT;
-import com.google.gerrit.common.data.ParamertizedString;
+import com.google.gerrit.common.data.ParameterizedString;
import com.google.gerrit.reviewdb.Account;
import com.google.gerrit.reviewdb.AccountExternalId;
import com.google.gerrit.reviewdb.AccountGroup;
@@ -148,14 +148,14 @@ class LdapRealm implements Realm {
return v;
}
- static ParamertizedString paramString(Config c, String n, String d) {
+ static ParameterizedString paramString(Config c, String n, String d) {
String expression = optdef(c, n, d);
if (expression == null) {
return null;
} else if (expression.contains("${")) {
- return new ParamertizedString(expression);
+ return new ParameterizedString(expression);
} else {
- return new ParamertizedString("${" + expression + "}");
+ return new ParameterizedString("${" + expression + "}");
}
}
@@ -164,7 +164,7 @@ class LdapRealm implements Realm {
return !readOnlyAccountFields.contains(field);
}
- private static String apply(ParamertizedString p, LdapQuery.Result m)
+ private static String apply(ParameterizedString p, LdapQuery.Result m)
throws NamingException {
if (p == null) {
return null;
@@ -273,8 +273,8 @@ class LdapRealm implements Realm {
final DirContext ctx = helper.open();
try {
final LdapSchema schema = helper.getSchema(ctx);
- final ParamertizedString filter =
- ParamertizedString.asis(schema.groupPattern
+ final ParameterizedString filter =
+ ParameterizedString.asis(schema.groupPattern
.replace(GROUPNAME, name).toString());
for (String groupBase : schema.groupBases) {
final LdapQuery query =
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/mail/FromAddressGeneratorProvider.java b/gerrit-server/src/main/java/com/google/gerrit/server/mail/FromAddressGeneratorProvider.java
index afcfccd3..05a419e8 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/mail/FromAddressGeneratorProvider.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/mail/FromAddressGeneratorProvider.java
@@ -14,7 +14,7 @@
package com.google.gerrit.server.mail;
-import com.google.gerrit.common.data.ParamertizedString;
+import com.google.gerrit.common.data.ParameterizedString;
import com.google.gerrit.reviewdb.Account;
import com.google.gerrit.server.GerritPersonIdent;
import com.google.gerrit.server.account.AccountCache;
@@ -41,7 +41,7 @@ public class FromAddressGeneratorProvider implements
final Address srvAddr = toAddress(myIdent);
if (from == null || "MIXED".equalsIgnoreCase(from)) {
- ParamertizedString name = new ParamertizedString("${user} (Code Review)");
+ ParameterizedString name = new ParameterizedString("${user} (Code Review)");
generator = new PatternGen(srvAddr, accountCache, name, srvAddr.email);
} else if ("USER".equalsIgnoreCase(from)) {
@@ -52,7 +52,7 @@ public class FromAddressGeneratorProvider implements
} else {
final Address a = Address.parse(from);
- final ParamertizedString name = a.name != null ? new ParamertizedString(a.name) : null;
+ final ParameterizedString name = a.name != null ? new ParameterizedString(a.name) : null;
if (name == null || name.getParameterNames().isEmpty()) {
generator = new ServerGen(a);
} else {
@@ -118,10 +118,10 @@ public class FromAddressGeneratorProvider implements
private final String senderEmail;
private final Address serverAddress;
private final AccountCache accountCache;
- private final ParamertizedString namePattern;
+ private final ParameterizedString namePattern;
PatternGen(final Address serverAddress, final AccountCache accountCache,
- final ParamertizedString namePattern, final String senderEmail) {
+ final ParameterizedString namePattern, final String senderEmail) {
this.senderEmail = senderEmail;
this.serverAddress = serverAddress;
this.accountCache = accountCache;
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/project/SectionMatcher.java b/gerrit-server/src/main/java/com/google/gerrit/server/project/SectionMatcher.java
index 9a937254..ce7530eb 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/project/SectionMatcher.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/project/SectionMatcher.java
@@ -17,7 +17,7 @@ package com.google.gerrit.server.project;
import static com.google.gerrit.server.project.RefControl.isRE;
import com.google.gerrit.common.data.AccessSection;
-import com.google.gerrit.common.data.ParamertizedString;
+import com.google.gerrit.common.data.ParameterizedString;
import dk.brics.automaton.Automaton;
@@ -106,12 +106,12 @@ abstract class SectionMatcher {
}
static class ExpandParameters extends SectionMatcher {
- private final ParamertizedString template;
+ private final ParameterizedString template;
private final String prefix;
ExpandParameters(String pattern, AccessSection section) {
super(section);
- template = new ParamertizedString(pattern);
+ template = new ParameterizedString(pattern);
if (isRE(pattern)) {
// Replace ${username} with ":USERNAME:" as : is not legal