Not makes more sense to me as Unless (Unless Controls Permanent Fewer than 4) something along those lines

Mage.Sets/src/mage/sets/magic2010/DragonskullSummit.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.BlackManaAbility;
@@ -61,7 +62,7 @@ public class DragonskullSummit extends CardImpl<DragonskullSummit> {
 		super(ownerId, 223, "Dragonskull Summit", Rarity.RARE, new CardType[]{CardType.LAND}, null);
 		this.expansionSetCode = "M10";
 
-		Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0);
+		Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0));
 		String abilityText = "tap it unless you control a " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new BlackManaAbility());

Mage.Sets/src/mage/sets/magic2010/DrownedCatacomb.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.BlackManaAbility;
@@ -61,7 +62,7 @@ public class DrownedCatacomb extends CardImpl<DrownedCatacomb> {
 		super(ownerId, 224, "Drowned Catacomb", Rarity.RARE, new CardType[]{CardType.LAND}, null);
 		this.expansionSetCode = "M10";
 
-		Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0);
+		Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0));
 		String abilityText = "tap it unless you control a " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new BlackManaAbility());

Mage.Sets/src/mage/sets/magic2010/GlacialFortress.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.BlueManaAbility;
@@ -61,7 +62,7 @@ public class GlacialFortress extends CardImpl<GlacialFortress> {
 		super(ownerId, 226, "Glacial Fortress", Rarity.RARE, new CardType[]{CardType.LAND}, null);
 		this.expansionSetCode = "M10";
 
-		Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0);
+		Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0));
 		String abilityText = "tap it unless you control a " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new BlueManaAbility());

Mage.Sets/src/mage/sets/magic2010/RootboundCrag.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.GreenManaAbility;
@@ -61,7 +62,7 @@ public class RootboundCrag extends CardImpl<RootboundCrag> {
 		super(ownerId, 227, "Rootbound Crag", Rarity.RARE, new CardType[]{CardType.LAND}, null);
 		this.expansionSetCode = "M10";
 
-		Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0);
+		Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0));
 		String abilityText = "tap it unless you control a " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new RedManaAbility());

Mage.Sets/src/mage/sets/magic2010/SunpetalGrove.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.GreenManaAbility;
@@ -61,7 +62,7 @@ public class SunpetalGrove extends CardImpl<SunpetalGrove> {
 		super(ownerId, 228, "Sunpetal Grove", Rarity.RARE, new CardType[]{CardType.LAND}, null);
 		this.expansionSetCode = "M10";
 
-		Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0);
+		Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.MORE_THAN, 0));
 		String abilityText = "tap it unless you control a " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new GreenManaAbility());

Mage.Sets/src/mage/sets/scarsofmirrodin/BlackcleaveCliffs.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.BlackManaAbility;
@@ -59,7 +60,7 @@ public class BlackcleaveCliffs extends CardImpl<BlackcleaveCliffs> {
         super(ownerId, 224, "Blackcleave Cliffs", Rarity.RARE, new CardType[]{CardType.LAND}, null);
         this.expansionSetCode = "SOM";
 
-		Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 3);
+		Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 4));
 		String abilityText = "tap it unless you control fewer than 3 " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new BlackManaAbility());

Mage.Sets/src/mage/sets/scarsofmirrodin/CopperlineGorge.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.GreenManaAbility;
@@ -59,7 +60,7 @@ public class CopperlineGorge extends CardImpl<CopperlineGorge> {
         super(ownerId, 225, "Copperline Gorge", Rarity.RARE, new CardType[]{CardType.LAND}, null);
         this.expansionSetCode = "SOM";
 
-        Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 3);
+        Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 4));
 		String abilityText = "tap it unless you control fewer than 3 " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new RedManaAbility());

Mage.Sets/src/mage/sets/scarsofmirrodin/DarkslickShores.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.BlackManaAbility;
@@ -59,7 +60,7 @@ public class DarkslickShores extends CardImpl<DarkslickShores> {
         super(ownerId, 226, "Darkslick Shores", Rarity.RARE, new CardType[]{CardType.LAND}, null);
         this.expansionSetCode = "SOM";
 
-        Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 3);
+        Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 4));
 		String abilityText = "tap it unless you control fewer than 3 " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new BlueManaAbility());

Mage.Sets/src/mage/sets/scarsofmirrodin/RazorvergeThicket.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.GreenManaAbility;
@@ -59,7 +60,7 @@ public class RazorvergeThicket extends CardImpl<RazorvergeThicket> {
         super(ownerId, 228, "Razorverge Thicket", Rarity.RARE, new CardType[]{CardType.LAND}, null);
         this.expansionSetCode = "SOM";
 
-        Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 3);
+        Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 4));
 		String abilityText = "tap it unless you control fewer than 3 " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new GreenManaAbility());

Mage.Sets/src/mage/sets/scarsofmirrodin/SeachromeCoast.java

@@ -34,6 +34,7 @@ import mage.Constants.Rarity;
 import mage.abilities.common.EntersBattlefieldAbility;
 import mage.abilities.condition.Condition;
 import mage.abilities.condition.common.ControlsPermanent;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.TapSourceEffect;
 import mage.abilities.mana.BlueManaAbility;
@@ -59,7 +60,7 @@ public class SeachromeCoast extends CardImpl<SeachromeCoast> {
         super(ownerId, 229, "Seachrome Coast", Rarity.RARE, new CardType[]{CardType.LAND}, null);
         this.expansionSetCode = "SOM";
 
-        Condition controls = new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 3);
+        Condition controls = new Unless(new ControlsPermanent(filter, ControlsPermanent.CountType.FEWER_THAN, 4));
 		String abilityText = "tap it unless you control fewer than 3 " + filter.getMessage();
 		this.addAbility(new EntersBattlefieldAbility(new ConditionalOneShotEffect(new TapSourceEffect(), controls, abilityText), abilityText));
 		this.addAbility(new WhiteManaAbility());

Mage.Sets/src/mage/sets/zendikar/VampireLacerator.java

@@ -34,7 +34,7 @@ import mage.Constants.Rarity;
 import mage.Constants.Zone;
 import mage.MageInt;
 import mage.abilities.common.SimpleTriggeredAbility;
-import mage.abilities.condition.common.Not;
+import mage.abilities.condition.common.Unless;
 import mage.abilities.condition.common.TenOrLessLife;
 import mage.abilities.decorator.ConditionalOneShotEffect;
 import mage.abilities.effects.common.LoseLifeSourceEffect;
@@ -79,7 +79,7 @@ class VampireLaceratorTriggeredAbility extends SimpleTriggeredAbility {
 		super(Zone.BATTLEFIELD, EventType.UPKEEP_STEP_PRE,
 				new ConditionalOneShotEffect(
 					new LoseLifeSourceEffect(1),
-					new Not( new TenOrLessLife(AN_OPPONENT)),
+					new Unless( new TenOrLessLife(AN_OPPONENT)),
 					"At the beginning of your upkeep, you lose 1 "
 					+ "life unless an opponent has 10 or less life."));
 	}

Mage/src/mage/abilities/condition/common/ControlsPermanent.java

@@ -97,10 +97,10 @@ public class ControlsPermanent implements Condition {
 
 		switch ( this.type ) {
 			case FEWER_THAN:
-				conditionApplies = game.getBattlefield().countAll(filter, source.getControllerId()) > this.count;
+				conditionApplies = game.getBattlefield().countAll(filter, source.getControllerId()) < this.count;
 				break;
 			case MORE_THAN:
-				conditionApplies = game.getBattlefield().countAll(filter, source.getControllerId()) < this.count;
+				conditionApplies = game.getBattlefield().countAll(filter, source.getControllerId()) > this.count;
 				break;
 			case EQUAL_TO:
 				conditionApplies = game.getBattlefield().countAll(filter, source.getControllerId()) == this.count;

Mage/src/mage/abilities/condition/common/Unless.java

@@ -16,11 +16,11 @@ import mage.game.Game;
  * 
  * @author maurer.it_at_gmail.com
  */
-public class Not implements Condition {
+public class Unless implements Condition {
 
 	private Condition condition;
 
-	public Not ( Condition condition ) {
+	public Unless ( Condition condition ) {
 		this.condition = condition;
 	}