From c2c96ee96c095689c75d64705bf1442a4e615128 Mon Sep 17 00:00:00 2001
From: Ulf Gebhardt <ulf.gebhardt@webcraft-media.de>
Date: Mon, 10 Jul 2023 11:53:00 +0200
Subject: [PATCH 1/4] frontend rooms query, singleRoom gets an userid now, not
 a boolean

---
 webapp/components/Chat/Chat.vue | 97 +++++++++++++++++++++------------
 webapp/graphql/Rooms.js         | 17 ++++++
 webapp/layouts/default.vue      |  2 +-
 3 files changed, 81 insertions(+), 35 deletions(-)

diff --git a/webapp/components/Chat/Chat.vue b/webapp/components/Chat/Chat.vue
index c2b02c0af..6bbeb7c71 100644
--- a/webapp/components/Chat/Chat.vue
+++ b/webapp/components/Chat/Chat.vue
@@ -27,7 +27,7 @@
 </template>
 
 <script>
-// import { roomQuery } from '~/graphql/Rooms'
+import { roomQuery, createRoom } from '~/graphql/Rooms'
 import { messageQuery } from '~/graphql/Messages'
 
 export default {
@@ -36,16 +36,16 @@ export default {
     theme: {
       type: String,
     },
-    singleRoom: {
-      type: Boolean,
-      default: false,
+    singleRoomId: {
+      type: String,
+      default: null,
     },
   },
   data() {
     return {
       currentUserId: '1234',
       menuActions: [
-        {
+        /* {
           name: 'inviteUser',
           title: 'Invite User',
         },
@@ -56,7 +56,7 @@ export default {
         {
           name: 'deleteRoom',
           title: 'Delete Room',
-        },
+        }, */
       ],
       messageActions: [
         {
@@ -93,16 +93,16 @@ export default {
         CANCEL_SELECT_MESSAGE: 'Annuler Sélection',
       },
       roomActions: [
-        {
+        /* {
           name: 'archiveRoom',
           title: 'Archive Room',
-        },
-        { name: 'inviteUser', title: 'Invite User' },
-        { name: 'removeUser', title: 'Remove User' },
-        { name: 'deleteRoom', title: 'Delete Room' },
+        }, */
+        // { name: 'inviteUser', title: 'Invite User' },
+        // { name: 'removeUser', title: 'Remove User' },
+        // { name: 'deleteRoom', title: 'Delete Room' },
       ],
       rooms: [
-        {
+        /* {
           roomId: '1',
           roomName: 'John Snow',
           avatar: 'https://66.media.tumblr.com/avatar_c6a8eae4303e_512.pnj',
@@ -120,15 +120,35 @@ export default {
             { _id: '1234', username: 'Johnx Doe' },
             { _id: '43210', username: 'Max J. Mustermann' },
           ],
-        },
+        }, */
       ],
       messages: [],
       messagesLoaded: true,
       showDemoOptions: true,
       responsiveBreakpoint: 600,
+      singleRoom: !!this.singleRoomId || false
+    }
+  },
+  mounted() {
+    if(this.singleRoom){
+      this.$apollo
+        .mutate({
+          mutation: createRoom(),
+          variables: {
+            userId: this.singleRoomId
+          },
+        })
+        .then((res) => {
+          this.$apollo.queries.Rooms.refetch()
+        })
+        .catch(() => {
+          console.log(error)
+        })
+        .finally(() => {
+          // this.loading = false
+        })
     }
   },
-
   methods: {
     fetchMessages({ room, options = {} }) {
       // console.log(room, options)
@@ -212,26 +232,35 @@ export default {
       }, 2000)
     },
   },
-  //   apollo: {
-  //     Rooms: {
-  //       query() {
-  //         return roomQuery()
-  //       },
-  //       update({ Room }) {
-  //         console.log('Rooms', Room)
-  //         if (!Room) {
-  //           this.rooms = []
-  //           return
-  //         }
-  //         this.rooms = Room
-  //       },
-  //       error(error) {
-  //         this.rooms = []
-  //         this.$toast.error(error.message)
-  //       },
-  //       fetchPolicy: 'cache-and-network',
-  //     },
-  //   },
+  apollo: {
+    Rooms: {
+       query() {
+          return roomQuery()
+        },
+        update({ Room }) {
+          console.log('Rooms', Room)
+          if (!Room) {
+            this.rooms = []
+            return
+          }
+
+          // Backend result needs mapping of the following values
+          // room[i].users[j].name -> room[i].users[j].username
+          // room[i].users[j].avatar.url -> room[i].users[j].avatar
+          // also filter rooms for the single room
+          this.rooms = Room.map((r) => {
+            return {...r, users: r.users.map((u) => { return {...u, username: u.name, avatar: u.avatar?.url}})}
+          }).filter((r) => this.singleRoom ? r.users.filter((u) => u.id === this.singleRoomId ).length > 0 : true)
+          
+          console.log(this.rooms)
+        },
+       error(error) {
+          this.rooms = []
+           this.$toast.error(error.message)
+        },
+        fetchPolicy: 'cache-and-network',
+      },
+    },
 }
 </script>
 <style lang="scss">
diff --git a/webapp/graphql/Rooms.js b/webapp/graphql/Rooms.js
index e8f1d9f6f..0e6074a79 100644
--- a/webapp/graphql/Rooms.js
+++ b/webapp/graphql/Rooms.js
@@ -4,9 +4,26 @@ export const roomQuery = () => gql`
   query {
     Room {
       id
+      roomId
+      roomName
+      avatar
       users {
+        _id
         id
+        name
+        avatar {
+          url
+        }
       }
     }
   }
 `
+
+export const createRoom = () => gql`
+  mutation ($userId: ID!) {
+    CreateRoom(userId: $userId) {
+      id
+      roomId
+    }
+  }
+`
\ No newline at end of file
diff --git a/webapp/layouts/default.vue b/webapp/layouts/default.vue
index ae6c10b73..209649661 100644
--- a/webapp/layouts/default.vue
+++ b/webapp/layouts/default.vue
@@ -20,7 +20,7 @@
           x
         </ds-button>
       </ds-text>
-      <chat-module :singleRoom="true" />
+      <chat-module :singleRoomId="$store.getters['chat/showChat'].roomID"/>
     </div>
     >
   </div>

From 5f2cfdeab482ef886040808121070dcaa3745b75 Mon Sep 17 00:00:00 2001
From: Ulf Gebhardt <ulf.gebhardt@webcraft-media.de>
Date: Mon, 10 Jul 2023 11:53:20 +0200
Subject: [PATCH 2/4] backend room hacks

---
 backend/src/schema/resolvers/rooms.ts  | 4 ++++
 backend/src/schema/types/type/Room.gql | 1 +
 2 files changed, 5 insertions(+)

diff --git a/backend/src/schema/resolvers/rooms.ts b/backend/src/schema/resolvers/rooms.ts
index 2fe621d23..02309e172 100644
--- a/backend/src/schema/resolvers/rooms.ts
+++ b/backend/src/schema/resolvers/rooms.ts
@@ -12,7 +12,11 @@ export default {
       if (resolved) {
         resolved.forEach((room) => {
           if (room.users) {
+            // buggy, you must query the username for this to function correctly
             room.roomName = room.users.filter((user) => user.id !== context.user.id)[0].name
+            room.avatar =
+              room.users.filter((user) => user.id !== context.user.id)[0].avatar?.url ||
+              'default-avatar'
             room.users.forEach((user) => {
               user._id = user.id
             })
diff --git a/backend/src/schema/types/type/Room.gql b/backend/src/schema/types/type/Room.gql
index c90ebda3a..2ce6556f6 100644
--- a/backend/src/schema/types/type/Room.gql
+++ b/backend/src/schema/types/type/Room.gql
@@ -14,6 +14,7 @@ type Room {
 
   roomId: String! @cypher(statement: "RETURN this.id")
   roomName: String! ## @cypher(statement: "MATCH (this)<-[:CHATS_IN]-(user:User) WHERE NOT user.id = $cypherParams.currentUserId RETURN user[0].name")
+  avatar: String! ## @cypher match not own user in users array
 }
 
 type Mutation {

From 8859aa56a4fddd74f3598ba61d58a3562bed3876 Mon Sep 17 00:00:00 2001
From: Ulf Gebhardt <ulf.gebhardt@webcraft-media.de>
Date: Mon, 10 Jul 2023 12:00:11 +0200
Subject: [PATCH 3/4] lint fixes

---
 webapp/components/Chat/Chat.vue | 69 ++++++++++++++++++---------------
 webapp/graphql/Rooms.js         |  2 +-
 webapp/layouts/default.vue      |  2 +-
 3 files changed, 40 insertions(+), 33 deletions(-)

diff --git a/webapp/components/Chat/Chat.vue b/webapp/components/Chat/Chat.vue
index 6bbeb7c71..801b5e948 100644
--- a/webapp/components/Chat/Chat.vue
+++ b/webapp/components/Chat/Chat.vue
@@ -126,23 +126,23 @@ export default {
       messagesLoaded: true,
       showDemoOptions: true,
       responsiveBreakpoint: 600,
-      singleRoom: !!this.singleRoomId || false
+      singleRoom: !!this.singleRoomId || false,
     }
   },
   mounted() {
-    if(this.singleRoom){
+    if (this.singleRoom) {
       this.$apollo
         .mutate({
           mutation: createRoom(),
           variables: {
-            userId: this.singleRoomId
+            userId: this.singleRoomId,
           },
         })
-        .then((res) => {
+        .then(() => {
           this.$apollo.queries.Rooms.refetch()
         })
-        .catch(() => {
-          console.log(error)
+        .catch((error) => {
+          this.$toast.error(error)
         })
         .finally(() => {
           // this.loading = false
@@ -234,33 +234,40 @@ export default {
   },
   apollo: {
     Rooms: {
-       query() {
-          return roomQuery()
-        },
-        update({ Room }) {
-          console.log('Rooms', Room)
-          if (!Room) {
-            this.rooms = []
-            return
-          }
-
-          // Backend result needs mapping of the following values
-          // room[i].users[j].name -> room[i].users[j].username
-          // room[i].users[j].avatar.url -> room[i].users[j].avatar
-          // also filter rooms for the single room
-          this.rooms = Room.map((r) => {
-            return {...r, users: r.users.map((u) => { return {...u, username: u.name, avatar: u.avatar?.url}})}
-          }).filter((r) => this.singleRoom ? r.users.filter((u) => u.id === this.singleRoomId ).length > 0 : true)
-          
-          console.log(this.rooms)
-        },
-       error(error) {
-          this.rooms = []
-           this.$toast.error(error.message)
-        },
-        fetchPolicy: 'cache-and-network',
+      query() {
+        return roomQuery()
       },
+      update({ Room }) {
+        // console.log('Rooms', Room)
+        if (!Room) {
+          this.rooms = []
+          return
+        }
+
+        // Backend result needs mapping of the following values
+        // room[i].users[j].name -> room[i].users[j].username
+        // room[i].users[j].avatar.url -> room[i].users[j].avatar
+        // also filter rooms for the single room
+        this.rooms = Room.map((r) => {
+          return {
+            ...r,
+            users: r.users.map((u) => {
+              return { ...u, username: u.name, avatar: u.avatar?.url }
+            }),
+          }
+        }).filter((r) =>
+          this.singleRoom ? r.users.filter((u) => u.id === this.singleRoomId).length > 0 : true,
+        )
+
+        // console.log(this.rooms)
+      },
+      error(error) {
+        this.rooms = []
+        this.$toast.error(error.message)
+      },
+      fetchPolicy: 'cache-and-network',
     },
+  },
 }
 </script>
 <style lang="scss">
diff --git a/webapp/graphql/Rooms.js b/webapp/graphql/Rooms.js
index 0e6074a79..7bab25509 100644
--- a/webapp/graphql/Rooms.js
+++ b/webapp/graphql/Rooms.js
@@ -26,4 +26,4 @@ export const createRoom = () => gql`
       roomId
     }
   }
-`
\ No newline at end of file
+`
diff --git a/webapp/layouts/default.vue b/webapp/layouts/default.vue
index 209649661..8b6b80604 100644
--- a/webapp/layouts/default.vue
+++ b/webapp/layouts/default.vue
@@ -20,7 +20,7 @@
           x
         </ds-button>
       </ds-text>
-      <chat-module :singleRoomId="$store.getters['chat/showChat'].roomID"/>
+      <chat-module :singleRoomId="$store.getters['chat/showChat'].roomID" />
     </div>
     >
   </div>

From 88e20918044dd06eb04d9acb0c31a99af1bd4638 Mon Sep 17 00:00:00 2001
From: Ulf Gebhardt <ulf.gebhardt@webcraft-media.de>
Date: Mon, 10 Jul 2023 13:37:42 +0200
Subject: [PATCH 4/4] fix comments

---
 webapp/components/Chat/Chat.vue | 32 +++++++-------------------------
 1 file changed, 7 insertions(+), 25 deletions(-)

diff --git a/webapp/components/Chat/Chat.vue b/webapp/components/Chat/Chat.vue
index 801b5e948..9cd3e391a 100644
--- a/webapp/components/Chat/Chat.vue
+++ b/webapp/components/Chat/Chat.vue
@@ -93,35 +93,17 @@ export default {
         CANCEL_SELECT_MESSAGE: 'Annuler Sélection',
       },
       roomActions: [
-        /* {
+        /*
+        {
           name: 'archiveRoom',
           title: 'Archive Room',
-        }, */
-        // { name: 'inviteUser', title: 'Invite User' },
-        // { name: 'removeUser', title: 'Remove User' },
-        // { name: 'deleteRoom', title: 'Delete Room' },
-      ],
-      rooms: [
-        /* {
-          roomId: '1',
-          roomName: 'John Snow',
-          avatar: 'https://66.media.tumblr.com/avatar_c6a8eae4303e_512.pnj',
-          users: [
-            { _id: '1234', username: 'John Doe' },
-            { _id: '4321', username: 'John Snow' },
-          ],
         },
-        {
-          roomId: '2',
-          roomName: 'Max J. Mustermann',
-          avatar:
-            'https://64.media.tumblr.com/8889b6e26370f4e3837584c1c59721a6/f4f76ed6b0249d08-4b/s1280x1920/810e9e5fa724366d26c10c0fa22ba97dad8778d1.pnj',
-          users: [
-            { _id: '1234', username: 'Johnx Doe' },
-            { _id: '43210', username: 'Max J. Mustermann' },
-          ],
-        }, */
+        { name: 'inviteUser', title: 'Invite User' },
+        { name: 'removeUser', title: 'Remove User' },
+        { name: 'deleteRoom', title: 'Delete Room' },
+        */
       ],
+      rooms: [],
       messages: [],
       messagesLoaded: true,
       showDemoOptions: true,