Fixes issue with partial matches.

lib/global_phone/format.rb

@@ -10,7 +10,10 @@ class Format < Record
 
     def match(national_string, match_leading_digits = true)
       return false if match_leading_digits && leading_digits && national_string !~ leading_digits
-      national_string =~ pattern
+      if match = pattern.match(national_string)
+        return match[0].length == national_string.length
+      end
+      false
     end
 
     def format_replacement_string(type)

lib/global_phone/number.rb

@@ -54,7 +54,11 @@ def international_format
     end
 
     def valid?
-      !!(format && national_string =~ national_pattern)
+      return unless !!format
+      if match = national_pattern.match(national_string)
+        return match[0] == national_string
+      end
+      false
     end
 
     def inspect

test/number_test.rb

@@ -56,5 +56,26 @@ class NumberTest < TestCase
       number = context.parse("(312) 555-1212")
       assert_equal "+1 312-555-1212", number.international_format
     end
+
+    test "partial matches do not validate" do
+      # Guatemala has eight digit phone numbers, but the number 5555
+      # 1234 123 still got through the matcher.
+      # 
+      # The regex is /^[2-7]\d{7}|1[89]\d{9}$/, but with =~ matching a
+      # partial match is still possible.
+      # 
+      # I think this might actually be a broken regex, but changing
+      # the code seems better than mucking about with all the regex's
+      # from google.
+      number = context.parse("5555 1234 123", :gt)
+      assert !number || !number.valid?
+    end
+
+    test "national format with or in regex still works with correct number" do
+      number = context.parse("5555 2134", :gt)
+      assert number.valid?
+    end
+
+
   end
 end