Merge pull request #689 from TheRestartProject/RES-1944_group_locatio…

…n_not_persisted

RES-1944 group location not persisted

lang/en/events.php

@@ -157,7 +157,7 @@
   'image_delete_error' => 'Sorry, but the image can\'t be deleted.',
   'delete_permission' => 'You do not have permission to delete this event.',
   'delete_success' => 'Event has been deleted.',
-  'geocode_failed' => 'The address you entered could not be found. Please try a more general address.',
+  'geocode_failed' => 'Location not found. If you are unable to find the location of your event, please try a more general location (such as village/town), or a specific street address, rather than a building name.',
   'create_failed' => 'Event could <strong>not</strong> be created. Please look at the reported errors, correct them, and try again.',
   'edit_succeeded' => 'Event updated!',
   'edit_failed' => 'Event could not be edited.',

lang/en/groups.php

@@ -56,7 +56,7 @@
   'edit_group' => 'Edit group',
   'edit_group_save_changes' => 'Save changes',
   'edit_group_text' => 'Go ahead and change or improve your group\'s profile.',
-  'groups_location_small' => 'I.e the place where the fixing happens!',
+  'groups_location_small' => 'The venue or area where your group operates',
   'group_image' => 'Group image',
   'tag-1' => 'Tag 1',
   'tag-2' => 'Tag 2',
@@ -144,7 +144,7 @@
   'nearest_groups' => 'These are the groups that are within 50 km of :location',
   'nearest_groups_change' => '(change)',
   'invitation_pending' => 'You have an invitation to this group.  Please click <a href=":accept">here</a> if you would like to join.',
-  'geocode_failed' => 'The address you entered could not be found. Please try a more general address.',
+  'geocode_failed' => 'Location not found. If you are unable to find the location of your group, please try a more general location (such as village/town), or a specific street address, rather than a building name.',
   'discourse_title' => 'This is a discussion group for anyone who follows :group.
 
 Find the group\'s main page here: :link.

lang/fr-BE/events.php

@@ -154,7 +154,7 @@
   'image_delete_error' => 'Désolé, mais l\'image ne peut pas être supprimée.',
   'delete_permission' => 'Vous n\'avez pas la permission de supprimer cet événement',
   'delete_success' => 'L\'événement a été supprimé.',
-  'geocode_failed' => 'L\'adresse que vous avez encodée n\'a pas pu être trouvée. Veuillez essayer une adresse plus générale.',
+  'geocode_failed' => 'Lieu non trouvé. Si vous ne parvenez pas à trouver le lieu où se trouve votre événement, essayez d\'indiquer un lieu plus général (tel qu\'un village ou une ville) ou une adresse spécifique, plutôt qu\'un nom de bâtiment.',
   'create_failed' => 'L\'événement <strong>n\'a pas pu</strong> être créé. Veuillez regarder les erreurs, les corriger, et essayer à nouveau.',
   'edit_succeeded' => 'L\'édition du l\'événement Café a réussi',
   'edit_failed' => 'L\'événement n\'a pas pu être édité.',

lang/fr-BE/groups.php

@@ -48,7 +48,7 @@
   'edit_group' => 'Editer Repair Café',
   'edit_group_save_changes' => 'Sauvegarder',
   'edit_group_text' => 'Allez-y! Changez ou améliorez le profil de votre Repair Café',
-  'groups_location_small' => 'Ex: l\'endroit où les réparations ont lieu!',
+  'groups_location_small' => 'Le lieu ou la région où votre Repair Café opère',
   'group_image' => 'Image du Repair Café',
   'tag-1' => 'Tag 1',
   'tag-2' => 'Tag 2',
@@ -142,7 +142,7 @@
   'delete_group_confirm' => 'Veuillez confirmer que vous voulez supprimer :name',
   'delete_succeeded' => 'Repair Café <strong>:name</strong> a été supprimé.',
   'duplicate' => 'Ce nom (:name) existe déjà.  Si c\'est le vôtre, veuillez vous rendre dans la page Repair Cafés, utiliser le menu et l\'éditer.',
-  'geocode_failed' => 'L\'adresse que vous avez encodée n\'a pas pu être trouvée. Veuillez essayer une adresse plus générale.',
+  'geocode_failed' => 'Lieu non trouvé. Si vous ne parvenez pas à trouver le lieu où se trouve votre Repair Café, essayez d\'indiquer un lieu plus général (tel qu\'un village ou une ville) ou une adresse spécifique, plutôt qu\'un nom de bâtiment.',
   'discourse_title' => 'Il s\'agit d\'un groupe de discussion pour tous ceux qui suivent le Repair Café :group.
 
 Vous trouverez la page principale du Repair Café ici : :link.

lang/fr/events.php

@@ -154,7 +154,7 @@
   'image_delete_error' => 'Désolé, mais l\'image ne peut pas être supprimée.',
   'delete_permission' => 'Vous n\'avez pas la permission de supprimer cet événement',
   'delete_success' => 'L\'événement a été supprimé.',
-  'geocode_failed' => 'L\'adresse que vous avez encodée n\'a pas pu être trouvée. Veuillez essayer une adresse plus générale.',
+  'geocode_failed' => 'Lieu non trouvé. Si vous ne parvenez pas à trouver le lieu où se trouve votre événement, essayez d\'indiquer un lieu plus général (tel qu\'un village ou une ville) ou une adresse spécifique, plutôt qu\'un nom de bâtiment.',
   'create_failed' => 'L\'événement <strong>n\'a pas pu</strong> être créé. Veuillez regarder les erreurs, les corriger, et essayer à nouveau.',
   'edit_succeeded' => 'L\'édition du l\'événement Café a réussi',
   'edit_failed' => 'L\'événement n\'a pas pu être édité.',

lang/fr/groups.php

@@ -48,7 +48,7 @@
   'edit_group' => 'Editer Repair Café',
   'edit_group_save_changes' => 'Sauvegarder',
   'edit_group_text' => 'Allez-y! Changez ou améliorez le profil de votre Repair Café',
-  'groups_location_small' => 'Ex: l\'endroit où les réparations ont lieu!',
+  'groups_location_small' => 'Le lieu ou la région où votre Repair Café opère',
   'group_image' => 'Image du Repair Café',
   'tag-1' => 'Tag 1',
   'tag-2' => 'Tag 2',
@@ -142,7 +142,7 @@
   'delete_group_confirm' => 'Veuillez confirmer que vous voulez supprimer :name',
   'delete_succeeded' => 'Repair Café <strong>:name</strong> a été supprimé.',
   'duplicate' => 'Ce nom (:name) existe déjà.  Si c\'est le vôtre, veuillez vous rendre dans la page Repair Cafés, utiliser le menu et l\'éditer.',
-  'geocode_failed' => 'L\'adresse que vous avez encodée n\'a pas pu être trouvée. Veuillez essayer une adresse plus générale.',
+  'geocode_failed' => 'Lieu non trouvé. Si vous ne parvenez pas à trouver le lieu où se trouve votre Repair Café, essayez d\'indiquer un lieu plus général (tel qu\'un village ou une ville) ou une adresse spécifique, plutôt qu\'un nom de bâtiment.',
   'discourse_title' => 'Il s\'agit d\'un groupe de discussion pour tous ceux qui suivent le Repair Café :group.
 
 Vous trouverez la page principale du Repair Café ici : :link.

lang/no/groups.php

@@ -46,7 +46,7 @@
   'edit_group' => 'Edit group',
   'edit_group_save_changes' => 'Save changes',
   'edit_group_text' => 'Go ahead and change or improve your group\'s profile.',
-  'groups_location_small' => 'I.e the place where the fixing happens!',
+  'groups_location_small' => 'The venue or area where your group operates',
   'group_image' => 'Group image',
   'tag-1' => 'Tag 1',
   'tag-2' => 'Tag 2',

resources/js/components/GroupLocation.vue

@@ -8,6 +8,7 @@
           classname="form-control group-location"
           :placeholder="__('groups.groups_location_placeholder')"
           @placechanged="placeChanged"
+          @change="resetValues"
           aria-describedby="locationHelpBlock"
           types="geocode"
           ref="autocomplete"
@@ -106,12 +107,21 @@ export default {
     },
   },
   methods: {
-    placeChanged(addressData, placeResultData) {
+    async placeChanged(addressData, placeResultData) {
+      // nextTick which means the change event will get processed before we emit our new values.
+      await this.$nextTick()
       this.currentValue = placeResultData.formatted_address
       this.$emit('update:value', this.currentValue)
       this.$emit('update:lat', addressData.latitude)
       this.$emit('update:lng', addressData.longitude)
     },
+    resetValues() {
+      // This means that if the input changes, we will assume it's invalid unless we subsequently (because of
+      // the nextTick above) get a valid placeChanged event.
+      this.$emit('update:value', null)
+      this.$emit('update:lat', null)
+      this.$emit('update:lng', null)
+    }
   }
 }
 </script>

resources/js/components/VenueAddress.vue

@@ -9,6 +9,7 @@
             classname="form-control"
             :placeholder="__('events.field_venue_placeholder')"
             @placechanged="placeChanged"
+            @change="resetValues"
             aria-describedby="locationHelpBlock"
             types="geocode"
             ref="autocomplete"
@@ -164,12 +165,21 @@ export default {
     clearTimeout(this.timer)
   },
   methods: {
-    placeChanged(addressData, placeResultData) {
+    async placeChanged(addressData, placeResultData) {
+      // nextTick which means the change event will get processed before we emit our new values.
+      await this.$nextTick()
       this.currentValue = placeResultData.formatted_address
       this.$emit('update:value', this.currentValue)
       this.$emit('update:lat', addressData.latitude)
       this.$emit('update:lng', addressData.longitude)
     },
+    resetValues() {
+      // This means that if the input changes, we will assume it's invalid unless we subsequently (because of
+      // the nextTick above) get a valid placeChanged event.
+      this.$emit('update:value', null)
+      this.$emit('update:lat', null)
+      this.$emit('update:lng', null)
+    },
     useGroup() {
       this.$refs.autocomplete.update(this.groupLocation)
       this.$emit('update:value', this.groupLocation)