Merge pull request #13496 from RocketChat/regression/geolocation

Regression: Message box geolocation was throwing error
RocketChat · Feb 15, 2019 · 8a2d7e5 · 8a2d7e5
2 parents 1a46bb0 + 0e6ff28
commit 8a2d7e5
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/packages/rocketchat-ui-message/client/startup/messageBoxActions.js b/packages/rocketchat-ui-message/client/startup/messageBoxActions.js
@@ -8,7 +8,6 @@ import { messageBox, modal } from 'meteor/rocketchat:ui-utils';
 import { fileUpload } from 'meteor/rocketchat:ui';
 import { settings } from 'meteor/rocketchat:settings';
 import { t } from 'meteor/rocketchat:utils';
-import { geolocation } from '../messageBox';
 
 messageBox.actions.add('Create_new', 'Video_message', {
 	id: 'video-message',
@@ -63,6 +62,8 @@ messageBox.actions.add('Add_files_from', 'Computer', {
 	},
 });
 
+const geolocation = new ReactiveVar(false);
+
 messageBox.actions.add('Share', 'My_location', {
 	id: 'share-location',
 	icon: 'map-pin',
@@ -97,10 +98,8 @@ messageBox.actions.add('Share', 'My_location', {
 });
 
 Meteor.startup(() => {
-	RocketChat.Geolocation = new ReactiveVar(false);
-
-	const handleGeolocation = (position) => RocketChat.Geolocation.set(position);
-	const handleGeolocationError = () => RocketChat.Geolocation.set(false);
+	const handleGeolocation = (position) => geolocation.set(position);
+	const handleGeolocationError = () => geolocation.set(false);
 
 	Tracker.autorun(() => {
 		const isMapViewEnabled = RocketChat.settings.get('MapView_Enabled') === true;
@@ -110,7 +109,7 @@ Meteor.startup(() => {
 			isMapViewEnabled && isGeolocationWatchSupported && (googleMapsApiKey && googleMapsApiKey.length);
 
 		if (!canGetGeolocation) {
-			RocketChat.Geolocation.set(false);
+			geolocation.set(false);
 			return;
 		}