Add option to skip non-translatable strings

Author: AndreyPavlenko

src/main/java/com/airsaid/localization/config/SettingsComponent.form

@@ -141,7 +141,7 @@
           </component>
         </children>
       </grid>
-      <grid id="cad41" layout-manager="GridLayoutManager" row-count="1" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+      <grid id="cad41" layout-manager="GridLayoutManager" row-count="2" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
         <margin top="0" left="0" bottom="0" right="0"/>
         <constraints>
           <grid row="2" column="0" row-span="1" col-span="2" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
@@ -182,6 +182,14 @@
               <grid row="0" column="2" row-span="1" col-span="1" vsize-policy="1" hsize-policy="6" anchor="0" fill="1" indent="0" use-parent-layout="false"/>
             </constraints>
           </hspacer>
+          <component id="29b24" class="javax.swing.JCheckBox" binding="skipNonTranslatableCheckBox" default-binding="true">
+            <constraints>
+              <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value="Skip non-translatable"/>
+            </properties>
+          </component>
         </children>
       </grid>
       <vspacer id="1a75f">

src/main/java/com/airsaid/localization/config/SettingsComponent.java

@@ -58,6 +58,7 @@ public class SettingsComponent {
   private JBCheckBox enableCacheCheckBox;
   private ComboBox<String> maxCacheSizeComboBox;
   private ComboBox<String> translationIntervalComboBox;
+  private JCheckBox skipNonTranslatableCheckBox;
 
   public SettingsComponent() {
     initTranslatorComponents();
@@ -206,4 +207,12 @@ public int getTranslationInterval() {
   public void setTranslationInterval(int intervalTime) {
     translationIntervalComboBox.setSelectedItem(String.valueOf(intervalTime));
   }
+
+  public boolean isSkipNonTranslatable() {
+    return skipNonTranslatableCheckBox.isSelected();
+  }
+
+  public void setSkipNonTranslatable(boolean isSkipNonTranslatable) {
+    skipNonTranslatableCheckBox.setSelected(isSkipNonTranslatable);
+  }
 }

src/main/java/com/airsaid/localization/config/SettingsConfigurable.java

@@ -18,6 +18,7 @@
 package com.airsaid.localization.config;
 
 import com.airsaid.localization.constant.Constants;
+import com.airsaid.localization.services.AndroidValuesService;
 import com.airsaid.localization.translate.AbstractTranslator;
 import com.airsaid.localization.translate.services.TranslatorService;
 import com.intellij.openapi.diagnostic.Logger;
@@ -63,6 +64,7 @@ private void initComponents() {
     settingsComponent.setEnableCache(settingsState.isEnableCache());
     settingsComponent.setMaxCacheSize(settingsState.getMaxCacheSize());
     settingsComponent.setTranslationInterval(settingsState.getTranslationInterval());
+    settingsComponent.setSkipNonTranslatable(settingsState.isSkipNonTranslatable());
   }
 
   @Override
@@ -75,6 +77,7 @@ public boolean isModified() {
     isChanged |= settingsState.isEnableCache() == settingsComponent.isEnableCache();
     isChanged |= settingsState.getMaxCacheSize() == settingsComponent.getMaxCacheSize();
     isChanged |= settingsState.getTranslationInterval() == settingsComponent.getTranslationInterval();
+    isChanged |= settingsState.isSkipNonTranslatable() == settingsComponent.isSkipNonTranslatable();
     LOG.info("isModified: " + isChanged);
     return isChanged;
   }
@@ -103,12 +106,15 @@ public void apply() throws ConfigurationException {
     settingsState.setEnableCache(settingsComponent.isEnableCache());
     settingsState.setMaxCacheSize(settingsComponent.getMaxCacheSize());
     settingsState.setTranslationInterval(settingsComponent.getTranslationInterval());
+    settingsState.setSkipNonTranslatable(settingsComponent.isSkipNonTranslatable());
 
     TranslatorService translatorService = TranslatorService.getInstance();
     translatorService.setSelectedTranslator(selectedTranslator);
     translatorService.setEnableCache(settingsComponent.isEnableCache());
     translatorService.setMaxCacheSize(settingsComponent.getMaxCacheSize());
     translatorService.setTranslationInterval(settingsComponent.getTranslationInterval());
+
+    AndroidValuesService.getInstance().setSkipNonTranslatable(settingsComponent.isSkipNonTranslatable());
   }
 
   @Override
@@ -122,6 +128,7 @@ public void reset() {
     settingsComponent.setEnableCache(settingsState.isEnableCache());
     settingsComponent.setMaxCacheSize(settingsState.getMaxCacheSize());
     settingsComponent.setTranslationInterval(settingsState.getTranslationInterval());
+    settingsComponent.setSkipNonTranslatable(settingsState.isSkipNonTranslatable());
   }
 
   @Override

src/main/java/com/airsaid/localization/config/SettingsState.java

@@ -17,6 +17,7 @@
 
 package com.airsaid.localization.config;
 
+import com.airsaid.localization.services.AndroidValuesService;
 import com.airsaid.localization.translate.AbstractTranslator;
 import com.airsaid.localization.translate.services.TranslatorService;
 import com.airsaid.localization.utils.SecureStorage;
@@ -71,6 +72,8 @@ public void initSetting() {
       translatorService.setMaxCacheSize(getMaxCacheSize());
       translatorService.setTranslationInterval(getTranslationInterval());
     }
+
+    AndroidValuesService.getInstance().setSkipNonTranslatable(isSkipNonTranslatable());
   }
 
   public AbstractTranslator getSelectedTranslator() {
@@ -129,6 +132,14 @@ public void setTranslationInterval(int intervalTime) {
     state.translationInterval = intervalTime;
   }
 
+  public boolean isSkipNonTranslatable() {
+    return state.isSkipNonTranslatable;
+  }
+
+  public void setSkipNonTranslatable(boolean isSkipNonTranslatable) {
+    state.isSkipNonTranslatable = isSkipNonTranslatable;
+  }
+
   @Override
   public @Nullable SettingsState.State getState() {
     return state;
@@ -145,5 +156,6 @@ static class State {
     public boolean isEnableCache = true;
     public int maxCacheSize = 500;
     public int translationInterval = 2; // 2 second
+    public boolean isSkipNonTranslatable;
   }
 }

src/main/java/com/airsaid/localization/services/AndroidValuesService.java

@@ -42,6 +42,7 @@
 import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 import java.util.regex.Pattern;
 
@@ -57,6 +58,8 @@ public final class AndroidValuesService {
 
   private static final Pattern STRINGS_FILE_NAME_PATTERN = Pattern.compile(".+\\.xml");
 
+  private boolean isSkipNonTranslatable;
+
   /**
    * Returns the {@link AndroidValuesService} object instance.
    *
@@ -96,18 +99,45 @@ public List<PsiElement> loadValues(@NotNull PsiFile valueFile) {
     });
   }
 
+  public boolean isSkipNonTranslatable() {
+    return isSkipNonTranslatable;
+  }
+
+  public void setSkipNonTranslatable(boolean isSkipNonTranslatable) {
+    this.isSkipNonTranslatable = isSkipNonTranslatable;
+  }
+
   private List<PsiElement> parseValuesXml(@NotNull PsiFile valueFile) {
-    final List<PsiElement> values = new ArrayList<>();
     final XmlFile xmlFile = (XmlFile) valueFile;
 
     final XmlDocument document = xmlFile.getDocument();
-    if (document == null) return values;
+    if (document == null) return Collections.emptyList();
 
     final XmlTag rootTag = document.getRootTag();
-    if (rootTag == null) return values;
+    if (rootTag == null) return Collections.emptyList();
 
     PsiElement[] subTags = rootTag.getChildren();
-    values.addAll(Arrays.asList(subTags));
+
+    if (!isSkipNonTranslatable()) {
+      return Arrays.asList(subTags);
+    }
+
+    List<PsiElement> values = new ArrayList<>(subTags.length);
+    boolean skipNext = false;
+
+    for (PsiElement e : subTags) {
+      if (skipNext) {
+        skipNext = false;
+        if (!(e instanceof XmlTag)) {
+          continue;
+        }
+      }
+      if ((e instanceof XmlTag) && !isTranslatable((XmlTag) e)) {
+        skipNext = true;
+      } else {
+        values.add(e);
+      }
+    }
 
     return values;
   }