diff options
| author | Bananeweizen <bananeweizen@gmx.de> | 2011-11-12 19:54:28 +0100 |
|---|---|---|
| committer | Bananeweizen <bananeweizen@gmx.de> | 2011-11-12 19:54:28 +0100 |
| commit | 864ef4e54bc39333b8912d3a1c469a113b8976cc (patch) | |
| tree | a552b4882529b3d4e687a08bce21796abaa2dd51 | |
| parent | 9b43b1ed8f21577de6b020cdf03246cc74254681 (diff) | |
| download | cgeo-864ef4e54bc39333b8912d3a1c469a113b8976cc.zip cgeo-864ef4e54bc39333b8912d3a1c469a113b8976cc.tar.gz cgeo-864ef4e54bc39333b8912d3a1c469a113b8976cc.tar.bz2 | |
don't accept geocodes starting with special character
| -rw-r--r-- | main/src/cgeo/geocaching/connector/ConnectorFactory.java | 2 | ||||
| -rw-r--r-- | tests/src/cgeo/geocaching/connector/ConnectorFactoryTest.java | 1 |
2 files changed, 2 insertions, 1 deletions
diff --git a/main/src/cgeo/geocaching/connector/ConnectorFactory.java b/main/src/cgeo/geocaching/connector/ConnectorFactory.java index 9405023..39e6dee 100644 --- a/main/src/cgeo/geocaching/connector/ConnectorFactory.java +++ b/main/src/cgeo/geocaching/connector/ConnectorFactory.java @@ -60,6 +60,6 @@ public final class ConnectorFactory { } private static boolean isInvalidGeocode(final String geocode) { - return StringUtils.isBlank(geocode); + return StringUtils.isBlank(geocode) || !Character.isLetterOrDigit(geocode.charAt(0)); } } diff --git a/tests/src/cgeo/geocaching/connector/ConnectorFactoryTest.java b/tests/src/cgeo/geocaching/connector/ConnectorFactoryTest.java index 5d719c7..043a6b1 100644 --- a/tests/src/cgeo/geocaching/connector/ConnectorFactoryTest.java +++ b/tests/src/cgeo/geocaching/connector/ConnectorFactoryTest.java @@ -15,6 +15,7 @@ public class ConnectorFactoryTest extends AbstractResourceInstrumentationTestCas assertFalse(ConnectorFactory.canHandle("")); assertTrue(ConnectorFactory.canHandle("GC12345")); assertTrue(ConnectorFactory.canHandle("some string")); // using unknown connector + assertFalse(ConnectorFactory.canHandle("[/start with special char")); } public void testGetConnectorCgCache() { |