Show alert on account home page if home location is not set

app/assets/javascripts/index/home.js

@@ -12,17 +12,26 @@ OSM.Home = function (map) {
     map.setSidebarOverlaid(true);
     clearMarker();
 
-    OSM.router.withoutMoveListener(function () {
+    if (OSM.home) {
-      map.setView(OSM.home, 15, { reset: true });
+      OSM.router.withoutMoveListener(function () {
-    });
+        map.setView(OSM.home, 15, { reset: true });
-    marker = L.marker(OSM.home, {
+      });
-      icon: OSM.getUserIcon(),
+      marker = L.marker(OSM.home, {
-      title: I18n.t("javascripts.home.marker_title")
+        icon: OSM.getUserIcon(),
-    }).addTo(map);
+        title: I18n.t("javascripts.home.marker_title")
+      }).addTo(map);
+    } else {
+      $("#browse_status").html(
+        $("<div class='m-2 alert alert-warning'>").text(
+          I18n.t("javascripts.home.not_set")
+        )
+      );
+    }
   };
 
   page.unload = function () {
     clearMarker();
+    $("#browse_status").empty();
   };
 
   return page;

config/locales/en.yml

@@ -3318,6 +3318,7 @@ en:
       centre_map: Centre map here
     home:
       marker_title: My home location
+      not_set: Home location is not set for your account
   redactions:
     edit:
       heading: "Edit Redaction"

test/system/account_home_test.rb

@@ -45,4 +45,13 @@ class AccountHomeTest < ApplicationSystemTestCase
     click_on "test user"
     assert_no_link "Go to Home Location"
   end
+
+  test "account home page shows a warning when visited by users without home location" do
+    user = create(:user, :display_name => "test user")
+    sign_in_as(user)
+
+    visit account_home_path
+    assert_no_selector "img.leaflet-marker-icon"
+    assert_text "Home location is not set"
+  end
 end