Merge pull request #98 from benhurott/fix/#96-celphone-mask-change-error

fix(#96): cel-phone mask change issue

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Fixed
 
 -   [#95](https://github.com/benhurott/react-native-masked-text/issues/95): Number value cause errors into only-numbers mask (thanks to [dfsilva](https://github.com/dfsilva) for reporting)
+-   [#96](https://github.com/benhurott/react-native-masked-text/issues/96): cel-phone mask is not changing properly. (thanks to [jgfidelis](https://github.com/jgfidelis) for reporting)
 
 ## [1.9.1] - 2018-09-09
 

Makefile

@@ -0,0 +1,5 @@
+copy-to-test:
+	rm -rf ../rn-test/react-native-masked-text
+	npm run clean
+	npm run build
+	cp -R dist ../rn-test/react-native-masked-text

__tests__/mask/cel-phone.mask.test.js

@@ -148,10 +148,18 @@ test('getMask for 99888877776 returns (99) 99999-9999', () => {
     expect(received).toBe(expected)
 })
 
-test('getMask for 999998888 withoud DDD returns 99999-9999', () => {
+test('getMask for 999998888 without DDD returns 99999-9999', () => {
     var mask = new CelPhoneMask()
     var expected = '99999-9999'
     var received = mask.getMask('999998888', { withDDD: false })
 
     expect(received).toBe(expected)
 })
+
+test('getValue for (12) 1234-56789 must return (12) 12345-6789', () => {
+    var mask = new CelPhoneMask()
+    var expected = '(12) 12345-6789'
+    var received = mask.getValue('(12) 1234-56789')
+
+    expect(received).toBe(expected)
+})

lib/masks/cel-phone.mask.js

@@ -14,8 +14,9 @@ export default class CelPhoneMask extends BaseMask {
     }
 
     getValue(value, settings) {
-        let mask = this.getMask(value, settings)
-        return CustomMask.shared.getValue(value, { mask })
+        let cleanedValue = super.removeNotNumbers(value)
+        let mask = this.getMask(cleanedValue, settings)
+        return CustomMask.shared.getValue(cleanedValue, { mask })
     }
 
     getRawValue(maskedValue, settings) {