Merge pull request #4197 from Dominik-Kuzila/enhanc/validate-group-name

fix: validate group name

AUTHORS.md

@@ -9,6 +9,7 @@
 - Christian Kraus <[email protected]>
 - Christoph Wurst <[email protected]>
 - Daniel Kesselberg <[email protected]>
+- Dominik Kuzila <[email protected]>
 - Gary Kim <[email protected]>
 - Georg Ehrke <[email protected]>
 - Greta Doci <[email protected]>

src/components/AppNavigation/RootNavigation.vue

@@ -393,10 +393,14 @@ export default {
 
 			this.createGroupError = null
 			this.logger.debug('Created new local group', { groupName })
-			this.$store.dispatch('addGroup', groupName)
-			this.isNewGroupMenuOpen = false
 
-			emit('contacts:group:append', groupName)
+			try {
+				this.$store.dispatch('addGroup', groupName)
+				this.isNewGroupMenuOpen = false
+				emit('contacts:group:append', groupName)
+			} catch (error) {
+				showError(t('contacts', 'An error occurred while creating the group'))
+			}
 		},
 
 		// Ellipsis item toggles

src/store/groups.js

@@ -184,6 +184,9 @@ const actions = {
 	 * @param {string} groupName the name of the group
 	 */
 	addGroup(context, groupName) {
+		if (!groupName || groupName.trim() === '') {
+			throw new Error('Group name cannot be empty')
+		}
 		context.commit('addGroup', groupName)
 	},
 }