From 8f588efafbbd973ecfa90afe5559d3af58c22b24 Mon Sep 17 00:00:00 2001
From: jddeep <jddeep003@gmail.com>
Date: Wed, 10 Apr 2019 07:41:57 +0530
Subject: [PATCH] Fix: Refactored PlanetsLayer

---
 .../com/google/android/stardroid/layers/PlanetsLayer.java  | 7 +++----
 app/src/main/res/values/strings.xml                        | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/app/src/main/java/com/google/android/stardroid/layers/PlanetsLayer.java b/app/src/main/java/com/google/android/stardroid/layers/PlanetsLayer.java
index 08bfe22f3..95af19946 100644
--- a/app/src/main/java/com/google/android/stardroid/layers/PlanetsLayer.java
+++ b/app/src/main/java/com/google/android/stardroid/layers/PlanetsLayer.java
@@ -48,9 +48,8 @@ protected void initializeAstroSources(ArrayList<AstronomicalSource> sources) {
       sources.add(new PlanetSource(planet, getResources(), model, preferences));
     }
   }
-
-  // TODO(brent): Remove this.
-  @Override
+  // If the preference Id is needed. There is no super method and no need
+  // to override.
   public String getPreferenceId() {
     return "source_provider.3";
   }
@@ -63,6 +62,6 @@ public int getLayerDepthOrder() {
 
   @Override
   protected int getLayerNameId() {
-    return R.string.show_planets_pref;  // TODO(johntaylor): rename the string id.
+    return R.string.solar_system;
   }
 }
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 2be35f6c7..24a4cc333 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -34,7 +34,7 @@
     <string name="show_stars_pref">Stars</string>
     <string name="show_constellations_pref">Constellations</string>
     <string name="show_messier_objects_pref" translation_description="The celestial objects catalogued by Charles Messier">Messier Objects</string>
-    <string name="show_planets_pref">Solar System</string>
+    <string name="solar_system">Solar System</string>
     <string name="show_grid_pref">RA/Dec Grid</string>
     <string name="show_horizon_pref">Horizon</string>
     <string name="show_sky_gradient" translation_description="This is a shading of the color of the sky used for effect">Sky Gradient</string>