Replace StringUtils.isNumeric() with custom method

Author: Airsaid

src/main/kotlin/com/airsaid/localization/extensions/StringExtensions.kt

@@ -0,0 +1,46 @@
+/*
+ * Copyright 2021 Airsaid. https://github.com/airsaid
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ */
+
+package com.airsaid.localization.extensions
+
+private val NON_TRANSLATABLE_REGEX = Regex("^[\\d\\p{P}\\p{S}\\s]+$")
+
+/**
+ * Checks if this string contains only numbers, symbols, punctuation, and whitespace.
+ * This is used to determine if text needs translation - strings with only these characters
+ * typically don't require translation.
+ *
+ * @return `true` if this string is not null, not empty, and contains only digits, punctuation,
+ *         symbols, and whitespace. `false` if this string is null, empty, or contains any
+ *         letters that would need translation.
+ *
+ * @sample
+ * ```
+ * "123".hasNoTranslatableText()      // returns true
+ * "12.3".hasNoTranslatableText()     // returns true
+ * "-123".hasNoTranslatableText()     // returns true
+ * "123!@#".hasNoTranslatableText()   // returns true
+ * "12 34".hasNoTranslatableText()    // returns true
+ * "abc".hasNoTranslatableText()      // returns false
+ * "123abc".hasNoTranslatableText()   // returns false
+ * null.hasNoTranslatableText()       // returns true
+ * ```
+ */
+fun String?.hasNoTranslatableText(): Boolean {
+  if (this.isNullOrEmpty()) return true
+  return this.matches(NON_TRANSLATABLE_REGEX)
+}

src/main/kotlin/com/airsaid/localization/translate/services/TranslatorService.kt

@@ -17,17 +17,16 @@
 
 package com.airsaid.localization.translate.services
 
+import com.airsaid.localization.extensions.hasNoTranslatableText
 import com.airsaid.localization.translate.AbstractTranslator
 import com.airsaid.localization.translate.interceptors.EscapeCharactersInterceptor
 import com.airsaid.localization.translate.lang.Lang
 import com.intellij.openapi.application.ApplicationManager
 import com.intellij.openapi.components.Service
 import com.intellij.openapi.components.service
 import com.intellij.openapi.diagnostic.Logger
-import org.apache.commons.lang.StringUtils
 import java.util.*
 import java.util.function.Consumer
-import kotlin.jvm.Volatile
 
 /**
  * @author airsaid
@@ -115,8 +114,8 @@ class TranslatorService {
       }
     }
 
-    // Arabic numbers skip translation
-    if (StringUtils.isNumeric(text)) {
+    // Skip translation for text containing only numbers, symbols, and punctuation
+    if (text.hasNoTranslatableText()) {
       return text
     }
 

src/test/kotlin/com/airsaid/localization/extensions/StringExtensionsTest.kt

@@ -0,0 +1,53 @@
+package com.airsaid.localization.extensions
+
+import org.junit.jupiter.api.Assertions.assertFalse
+import org.junit.jupiter.api.Assertions.assertTrue
+import org.junit.jupiter.api.Test
+
+class StringExtensionsTest {
+
+  @Test
+  fun hasNoTranslatableText() {
+    // Numbers only - should return true
+    assertTrue("123".hasNoTranslatableText())
+    assertTrue("0".hasNoTranslatableText())
+    assertTrue("999999".hasNoTranslatableText())
+
+    // Numbers with symbols/punctuation - should return true
+    assertTrue("12.3".hasNoTranslatableText())
+    assertTrue("-123".hasNoTranslatableText())
+    assertTrue("+123".hasNoTranslatableText())
+    assertTrue("123!@#".hasNoTranslatableText())
+    assertTrue("12,34".hasNoTranslatableText())
+    assertTrue("$100".hasNoTranslatableText())
+    assertTrue("50%".hasNoTranslatableText())
+    assertTrue("(123)".hasNoTranslatableText())
+
+    // Numbers with whitespace - should return true
+    assertTrue("12 3".hasNoTranslatableText())
+    assertTrue("12\n3".hasNoTranslatableText())
+    assertTrue("12\t3".hasNoTranslatableText())
+    assertTrue(" 123 ".hasNoTranslatableText())
+
+    // Symbols/punctuation only - should return true
+    assertTrue("!@#".hasNoTranslatableText())
+    assertTrue("...".hasNoTranslatableText())
+    assertTrue("***".hasNoTranslatableText())
+    assertTrue("???".hasNoTranslatableText())
+
+    // Text with letters - should return false (needs translation)
+    assertFalse("abc".hasNoTranslatableText())
+    assertFalse("123abc".hasNoTranslatableText())
+    assertFalse("abc123".hasNoTranslatableText())
+    assertFalse("Hello".hasNoTranslatableText())
+    assertFalse("Hello123".hasNoTranslatableText())
+    assertFalse("123Hello".hasNoTranslatableText())
+    assertFalse("中文".hasNoTranslatableText())
+    assertFalse("测试123".hasNoTranslatableText())
+
+    // Empty/null - should return true
+    assertTrue("".hasNoTranslatableText())
+    val nullString: String? = null
+    assertTrue(nullString.hasNoTranslatableText())
+  }
+}