Fix for InterpolatorType crash

Summary:
We're currently getting a redbox in Turkish when we try to convert the string 'easeInEaseOut' to an InterpolatorType. This is because I use toLowerCase() to compare the string without setting a locale; in Turkish, the capital letter 'I' doesn't convert to 'i' when you lowercase it, but rather to 'ı' (http://www.i18nguy.com/unicode/turkish-i18n.html).

Passing in a locale param to `toLowerCase()` fixes it. Also updating the test.

Differential Revision: D10315474

fbshipit-source-id: 54be3ff1d3f91cb2ec765ff705ac364b976b8c6f
This commit is contained in:
Emily Janzer
2018-10-11 23:18:32 +02:00
committed by Mike Grabowski
parent 69a51da3a1
commit 300ba7a87e
2 changed files with 10 additions and 1 deletions
@@ -5,6 +5,8 @@
package com.facebook.react.uimanager.layoutanimation;
import java.util.Locale;
/**
* Enum representing the different interpolators that can be used in layout animation configuration.
*/
@@ -16,7 +18,7 @@ package com.facebook.react.uimanager.layoutanimation;
SPRING;
public static InterpolatorType fromString(String name) {
switch (name.toLowerCase()) {
switch (name.toLowerCase(Locale.US)) {
case "linear":
return LINEAR;
case "easein":
@@ -8,6 +8,7 @@
package com.facebook.react.uimanager.layoutanimation;
import com.facebook.react.uimanager.layoutanimation.InterpolatorType;
import java.util.Locale;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.robolectric.RobolectricTestRunner;
@@ -32,6 +33,12 @@ public class InterpolatorTypeTest {
assertThat(InterpolatorType.fromString("easeineaseout")).isEqualTo(InterpolatorType.EASE_IN_EASE_OUT);
}
@Test
public void testLocales() {
Locale.setDefault(Locale.forLanguageTag("tr-TR"));
assertThat(InterpolatorType.fromString("easeInEaseOut")).isEqualTo(InterpolatorType.EASE_IN_EASE_OUT);
}
@Test(expected = IllegalArgumentException.class)
public void testInvalidInterpolatorTypes() throws IllegalArgumentException {
InterpolatorType.fromString("ease_in_ease_out");