Merge pull request #4 from anas-elgarhy/fix-forget-playerid-issue

💙 Fix forget edition id issue after close the ide 🥰

src/main/java/com/anas/intellij/plugins/ayah/settings/AyahSettingsState.java

@@ -34,12 +34,7 @@ public class AyahSettingsState implements PersistentStateComponent<AyahSettingsS
         basmalhOnStart = new BasmalhOnStart();
         intervalTimeBetweenNotifications = 30; // 30 minutes
         autoPlayAudio = false;
-        try {
+        edition = new SelectedEdition();
-            edition = new SelectedEdition(Edition
-                    .getEditions(EditionFormat.AUDIO)[0].getIdentifier(), 0);
-        } catch (final IOException e) {
-            edition = null;
-        }
     }
 
 

src/main/java/com/anas/intellij/plugins/ayah/settings/BasmalhOnStart.java

@@ -11,20 +11,13 @@ import java.io.IOException;
  */
 public class BasmalhOnStart {
     private boolean isActive;
-    private boolean isNotificationActive;
     private boolean isSoundActive;
     private SelectedEdition edition;
 
     public BasmalhOnStart() {
         isActive = true;
-        isNotificationActive = true;
         isSoundActive = false;
-        try {
+        edition = new SelectedEdition();
-            edition = new SelectedEdition(Edition
-                    .getEditions(EditionFormat.AUDIO)[0].getIdentifier(), 0);
-        } catch (final IOException e) {
-            edition = null;
-        }
     }
 
     public boolean isActive() {
@@ -35,14 +28,6 @@ public class BasmalhOnStart {
         isActive = active;
     }
 
-    public boolean isNotificationActive() {
-        return isNotificationActive;
-    }
-
-    public void setNotificationActive(final boolean notificationActive) {
-        isNotificationActive = notificationActive;
-    }
-
     public boolean isSoundActive() {
         return isSoundActive;
     }

src/main/java/com/anas/intellij/plugins/ayah/settings/ReadableEdition.java

@@ -7,7 +7,7 @@ import com.anas.alqurancloudapi.edition.Edition;
  * @date: 8/20/22
  */
 public class ReadableEdition {
-    private final Edition edition;
+    private Edition edition;
 
     public ReadableEdition(final Edition edition) {
         this.edition = edition;

src/main/java/com/anas/intellij/plugins/ayah/settings/SelectedEdition.java

@@ -1,14 +1,27 @@
 package com.anas.intellij.plugins.ayah.settings;
 
 import com.anas.alqurancloudapi.edition.Edition;
+import com.anas.alqurancloudapi.edition.EditionFormat;
+
+import java.io.IOException;
 
 /**
  * @author: <a href="https://github.com/anas-elgarhy">Anas Elgarhy</a>
  * @date: 8/21/22
  */
 public class SelectedEdition {
-    private final String editionIdentifier;
+    private String editionIdentifier;
-    private final int index;
+    private int index;
+
+    // For XML serialization
+    public SelectedEdition() {
+        try {
+            editionIdentifier = Edition.getEditions(EditionFormat.AUDIO)[0].getIdentifier();
+        } catch (final IOException e) {
+            editionIdentifier = "ar.abdulbasitmurattal";
+        }
+        index = 0;
+    }
 
     public SelectedEdition(final String editionIdentifier, final int index) {
         this.editionIdentifier = editionIdentifier;
@@ -19,7 +32,17 @@ public class SelectedEdition {
         return editionIdentifier;
     }
 
+    // For XML serialization
+    public void setEditionIdentifier(final String editionIdentifier) {
+        this.editionIdentifier = editionIdentifier;
+    }
+
     public int getIndex() {
         return index;
     }
+
+    // For XML serialization
+    public void setIndex(final int index) {
+        this.index = index;
+    }
 }

src/main/java/com/anas/intellij/plugins/ayah/settings/SettingsComponent.java

@@ -86,12 +86,14 @@ public class SettingsComponent {
         notificationsAudioCheckBox.setSelected(settings.isAutoPlayAudio());
         basmalhPlayerIdComboBox.setEnabled(settings.getBasmalhOnStart().isActive());
 
-        if (settings.getBasmalhOnStart().getEdition() != null) {
+        if (basmalhPlayerIdComboBox.getItemCount() <= 0) {
-            basmalhPlayerIdComboBox.setSelectedIndex(settings.getBasmalhOnStart().getEdition().getIndex());
+            basmalhPlayerIdComboBox.addItem(new ReadableEdition(settings.getBasmalhOnStart()
-        }
+                    .getEdition().getEditionIdentifier()));
-        if (settings.getEdition() != null) {
+            ayahPlayerIdComboBox.addItem(new ReadableEdition(settings.getEdition().getEditionIdentifier()));
-            ayahPlayerIdComboBox.setSelectedIndex(settings.getEdition().getIndex());
         }
+
+        basmalhPlayerIdComboBox.setSelectedIndex(settings.getBasmalhOnStart().getEdition().getIndex());
+        ayahPlayerIdComboBox.setSelectedIndex(settings.getEdition().getIndex());
     }
 
     private void addListeners() {
@@ -147,7 +149,6 @@ public class SettingsComponent {
         b.setSoundActive(autoPlayBasmalhCheckBox.isSelected());
         b.setEdition(new SelectedEdition(((ReadableEdition) Objects.requireNonNull(
                 basmalhPlayerIdComboBox.getSelectedItem())).getEdition().getIdentifier(), basmalhPlayerIdComboBox.getSelectedIndex()));
-        b.setNotificationActive(notificationsAudioCheckBox.isSelected());
         return b;
     }