Bug fix: if libphonenumber doesn't define any formatters for a country,

spit out +CC SUBSCRIBER instead of nothing. Was introduced in ce4e800, see #122
DrHyde · Mar 6, 2023 · 9277ad2 · 9277ad2
1 parent 1db1857
commit 9277ad2
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/MANIFEST b/MANIFEST
@@ -358,3 +358,4 @@ lib/Number/Phone/StubCountry/InternationalNetworks882.pm
 lib/Number/Phone/StubCountry/InternationalNetworks883.pm
 lib/Number/Phone/StubCountry/TelecomsForDisasterRelief.pm
 lib/Number/Phone/StubCountry/InternationalPremiumRate.pm
+t/country-test-bq.t
diff --git a/lib/Number/Phone/StubCountry.pm b/lib/Number/Phone/StubCountry.pm
@@ -92,7 +92,8 @@ sub format {
       return join(' ', '+'.$self->country_code(), @bits);
     }
   }
-  # return '+'.$self->country_code().' '.$number;
+  # if there's no formatters defined ...
+  return '+'.$self->country_code().' '.$number;
 }
 
 1;
diff --git a/t/country-test-bq.t b/t/country-test-bq.t
@@ -0,0 +1,12 @@
+use strict;
+use warnings;
+
+use Number::Phone;
+
+use Test::More;
+
+my $p = Number::Phone->new("+5997900000");
+
+is($p->format(), "+599 7900000", "Bonaire (BQ) number correctly formatted");
+
+done_testing();