Remove check for setting name to null or undefined

- this was making things more likely to fail from the frontend, we would
need to consider doing a db manipulation for users from the old alpha
who have user.name as null.
- it only protects against someone who bypasses our UI and sends a
message directly to the backend, but if they can do that we have bigger
problems.

backend/src/middleware/validation/validationMiddleware.js

@@ -129,10 +129,9 @@ export const validateNotifyUsers = async (label, reason) => {
 
 const validateUpdateUser = async (resolve, root, params, context, info) => {
   const { name } = params
-  if (typeof name === 'string' && name.trim().length > USERNAME_MIN_LENGTH)
-    return resolve(root, params, context, info)
-  if (typeof name !== 'string' || name.trim().length < USERNAME_MIN_LENGTH)
+  if (typeof name === 'string' && name.trim().length < USERNAME_MIN_LENGTH)
     throw new UserInputError(`Username must be at least ${USERNAME_MIN_LENGTH} character long!`)
+  return resolve(root, params, context, info)
 }
 
 export default {

webapp/components/Embed/EmbedComponent.vue

@@ -132,7 +132,6 @@ export default {
           mutation: updateUserMutation(),
           variables: {
             id: this.currentUser.id,
-            name: this.currentUser.name,
             allowEmbedIframes,
           },
           update: (store, { data: { UpdateUser } }) => {

webapp/components/LocaleSwitch/LocaleSwitch.vue

@@ -90,7 +90,6 @@ export default {
           mutation: updateUserMutation(),
           variables: {
             id: this.currentUser.id,
-            name: this.currentUser.name,
             locale: this.$i18n.locale(),
           },
           update: (store, { data: { UpdateUser } }) => {

webapp/components/Upload/index.vue

@@ -66,7 +66,6 @@ export default {
           variables: {
             avatarUpload,
             id: this.user.id,
-            name: this.user.name,
           },
         })
         .then(() => {

webapp/pages/settings/embeds.vue

@@ -72,7 +72,6 @@ export default {
           mutation: updateUserMutation(),
           variables: {
             id: this.currentUser.id,
-            name: this.currentUser.name,
             allowEmbedIframes: !this.disabled,
           },
           update: (store, { data: { UpdateUser } }) => {

webapp/pages/settings/privacy.vue

@@ -39,7 +39,6 @@ export default {
           mutation: updateUserMutation(),
           variables: {
             id: this.currentUser.id,
-            name: this.currentUser.name,
             showShoutsPublicly: this.shoutsAllowed,
           },
           update: (_, { data: { UpdateUser } }) => {

webapp/pages/terms-and-conditions-confirm.vue

@@ -70,7 +70,6 @@ export default {
           mutation: updateUserMutation(),
           variables: {
             id: this.currentUser.id,
-            name: this.currentUser.name,
             termsAndConditionsAgreedVersion: VERSION,
           },
           update: (store, { data: { UpdateUser } }) => {