From 4d13c9cada9a594195a268ce323ffed9603fbd22 Mon Sep 17 00:00:00 2001
From: drtshock <drtshock13@gmail.com>
Date: Thu, 15 Jan 2015 13:27:11 -0600
Subject: [PATCH] Change command syntax for radius claiming. Syntax was
 confusing and people wanted change. Resolves #215.

---
 src/main/java/com/massivecraft/factions/cmd/CmdClaim.java | 8 ++++----
 src/main/java/com/massivecraft/factions/cmd/FCommand.java | 6 +-----
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/main/java/com/massivecraft/factions/cmd/CmdClaim.java b/src/main/java/com/massivecraft/factions/cmd/CmdClaim.java
index e3c98f9f..5a7aeb0b 100644
--- a/src/main/java/com/massivecraft/factions/cmd/CmdClaim.java
+++ b/src/main/java/com/massivecraft/factions/cmd/CmdClaim.java
@@ -15,8 +15,8 @@ public class CmdClaim extends FCommand {
         this.aliases.add("claim");
 
         //this.requiredArgs.add("");
-        this.optionalArgs.put("faction", "your");
         this.optionalArgs.put("radius", "1");
+        this.optionalArgs.put("faction", "your");
 
         this.permission = Permission.CLAIM.node;
         this.disableOnLock = true;
@@ -30,8 +30,8 @@ public class CmdClaim extends FCommand {
     @Override
     public void perform() {
         // Read and validate input
-        final Faction forFaction = this.argAsFaction(0, myFaction);
-        int radius = this.argAsInt(1, 1);
+        int radius = this.argAsInt(0, 1); // Default to 1
+        final Faction forFaction = this.argAsFaction(1, myFaction); // Default to own
 
         if (radius < 1) {
             msg(TL.COMMAND_CLAIM_INVALIDRADIUS);
@@ -57,7 +57,7 @@ public class CmdClaim extends FCommand {
                     boolean success = fme.attemptClaim(forFaction, this.currentLocation(), true);
                     if (success) {
                         failCount = 0;
-                    } else if (!success && failCount++ >= limit) {
+                    } else if (failCount++ >= limit) {
                         this.stop();
                         return false;
                     }
diff --git a/src/main/java/com/massivecraft/factions/cmd/FCommand.java b/src/main/java/com/massivecraft/factions/cmd/FCommand.java
index d30b83b1..cd878c31 100644
--- a/src/main/java/com/massivecraft/factions/cmd/FCommand.java
+++ b/src/main/java/com/massivecraft/factions/cmd/FCommand.java
@@ -188,12 +188,8 @@ public abstract class FCommand extends MCommand<P> {
         Faction ret = def;
 
         if (name != null) {
-            Faction faction = null;
-
             // First we try an exact match
-            if (faction == null) {
-                faction = Factions.getInstance().getByTag(name); // Checks for faction name match.
-            }
+            Faction faction = Factions.getInstance().getByTag(name); // Checks for faction name match.
 
             // Next we match faction tags
             if (faction == null) {