Social: Enhancements and Fixes for Social Network Features - refs BT#21101

assets/css/app.scss

@@ -619,6 +619,16 @@ form .field {
   display: none;
 }
 
+.sub-item-indent {
+  padding-left: 20px;
+}
+
+.sub-item-indent.active {
+  background-color: #e0e0e0;
+  color: #333;
+  font-weight: bold;
+}
+
 //@import 'primevue-md-light-indigo/theme.css';
 //@import '~primevue/resources/primevue.min.css';
 //@import '~primeflex/primeflex.css';

assets/vue/components/message/MessageLayout.vue

@@ -2,7 +2,6 @@
   <div class="message-layout flex">
     <div class="sidebar">
       <UserProfileCard />
-      <SocialSideMenu />
     </div>
     <div class="content flex-grow">
       <router-view></router-view>
@@ -11,7 +10,6 @@
 </template>
 <script setup>
 import UserProfileCard from "../social/UserProfileCard.vue"
-import SocialSideMenu from "../social/SocialSideMenu.vue"
 import { onMounted, provide } from "vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
 import { useSecurityStore } from "../../store/securityStore"

assets/vue/components/personalfile/Layout.vue

@@ -2,7 +2,6 @@
   <div class="flex flex-col md:flex-row gap-4">
     <div class="md:basis-1/3 lg:basis-1/4 2xl:basis-1/6 flex flex-col">
       <UserProfileCard />
-      <SocialSideMenu />
     </div>
     <div class="md:basis-2/3 lg:basis-3/4 2xl:basis-5/6">
       <h2 v-t="'My files'" class="mr-auto" />
@@ -13,7 +12,6 @@
 </template>
 <script setup>
 import UserProfileCard from "../social/UserProfileCard.vue"
-import SocialSideMenu from "../social/SocialSideMenu.vue"
 import { onMounted, provide } from "vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
 

assets/vue/components/social/MyGroupsCard.vue

@@ -69,14 +69,18 @@ function search() {
 
 async function fetchGroups(userId) {
   try {
-    const response = await axios.get( `/social-network/groups/${userId}`)
-    groups.value = response.data.groups
-    goToUrl.value = response.data.groups.find(group => group.go_to)?.go_to || ''
+    const response = await axios.get(`/social-network/groups/${userId}`)
+    if (response.data) {
+      groups.value = response.data.items
+      goToUrl.value = response.data.go_to
+    }
   } catch (error) {
-    console.error('Error fetching groups:', error)
+    groups.value = []
+    goToUrl.value = ''
   }
 }
 
+
 watchEffect(() => {
   if (user.value && user.value.id) {
     fetchGroups(user.value.id)

assets/vue/components/user/Layout.vue

@@ -1,10 +1,9 @@
 <template>
-  <div class="flex flex-col md:flex-row gap-4">
-    <div class="md:basis-1/3 lg:basis-1/4 2xl:basis-1/6 flex flex-col">
+  <div class="flex flex-col md:flex-row gap-4 w-full">
+    <div class="flex flex-col w-full md:w-1/5">
       <UserProfileCard />
-      <SocialSideMenu />
     </div>
-    <div class="md:basis-2/3 lg:basis-1/2 2xl:basis-4/6">
+    <div class="flex-grow w-full md:w-4/5">
       <router-view />
     </div>
   </div>
@@ -12,8 +11,6 @@
 
 <script setup>
 import UserProfileCard from "../social/UserProfileCard.vue"
-import SocialSideMenu from "../social/SocialSideMenu.vue"
-
 import { onMounted, provide } from "vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
 

assets/vue/components/usergroup/Layout.vue

@@ -3,7 +3,6 @@
     <div class="md:basis-1/3 lg:basis-1/4 2xl:basis-1/6 flex flex-col">
       <UserProfileCard v-if="!isLoading && !isGroup" />
       <GroupInfoCard v-if="!isLoading && isGroup" />
-      <SocialSideMenu v-if="!isLoading && !isGroup" />
       <SocialGroupMenu v-if="!isLoading && isGroup" />
     </div>
     <div class="md:basis-2/3 lg:basis-3/4 2xl:basis-5/6">
@@ -13,7 +12,6 @@
 </template>
 <script setup>
 import UserProfileCard from "../social/UserProfileCard.vue"
-import SocialSideMenu from "../social/SocialSideMenu.vue"
 import { provide } from "vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
 import SocialGroupMenu from "../social/SocialGroupMenu.vue"

assets/vue/components/userreluser/Layout.vue

@@ -2,7 +2,6 @@
   <div class="flex flex-col md:flex-row gap-4">
     <div class="md:basis-1/3 lg:basis-1/4 2xl:basis-1/6 flex flex-col">
       <UserProfileCard />
-      <SocialSideMenu />
     </div>
     <div class="md:basis-2/3 lg:basis-3/4 2xl:basis-5/6">
       <router-view></router-view>
@@ -11,7 +10,6 @@
 </template>
 <script setup>
 import UserProfileCard from "../social/UserProfileCard.vue"
-import SocialSideMenu from "../social/SocialSideMenu.vue"
 import { onMounted, provide } from "vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
 

assets/vue/composables/sidebarMenu.js

@@ -1,19 +1,31 @@
 import { useI18n } from "vue-i18n"
-import { computed } from "vue"
+import { computed, ref } from "vue"
 import { useSecurityStore } from "../store/securityStore"
 import { usePlatformConfig } from "../store/platformConfig"
 import { useEnrolledStore } from "../store/enrolledStore"
-import { useRouter } from "vue-router"
+import { useRoute, useRouter } from "vue-router"
+import { useSocialMenuItems } from "./useSocialMenuItems"
 
 export function useSidebarMenu() {
   const { t } = useI18n()
   const router = useRouter()
+  const route = useRoute()
   const securityStore = useSecurityStore()
   const platformConfigStore = usePlatformConfig()
   const enrolledStore = useEnrolledStore()
+  const { items: socialItems } = useSocialMenuItems();
   const showTabsSetting = platformConfigStore.getSetting("platform.show_tabs")
   const showCatalogue = platformConfigStore.getSetting("platform.catalog_show_courses_sessions")
 
+  const isActive = (item) => {
+    if (item.route) {
+      return route.path === item.route || (item.route.name && route.name === item.route.name)
+    } else if (item.items) {
+      return item.items.some(subItem => isActive(subItem))
+    }
+    return false
+  };
+
   const menuItemsBeforeMyCourse = computed(() => {
     const items = []
 
@@ -118,11 +130,29 @@ export function useSidebarMenu() {
     }
 
     if (showTabsSetting.indexOf("social") > -1) {
+      const styledSocialItems = socialItems.value.map(item => {
+        const newItem = {
+          ...item,
+          class: `sub-item-indent${isActive(item) ? ' active' : ''}`
+        };
+
+        if (newItem.isLink && newItem.route) {
+          newItem.command = () => window.location.href = newItem.route
+        } else if (newItem.route) {
+          newItem.command = () => router.push(newItem.route)
+        } else if (newItem.link) {
+          newItem.command = () => window.location.href = newItem.link
+        }
+
+        return newItem
+      });
+
       items.push({
         icon: "mdi mdi-sitemap-outline",
         label: t("Social network"),
-        command: () => router.push({ name: "SocialWall" }),
-      })
+        items: styledSocialItems,
+        expanded: isActive({ items: styledSocialItems })
+      });
     }
 
     if (platformConfigStore.plugins?.bbb?.show_global_conference_link) {

assets/vue/views/account/Home.vue

@@ -2,38 +2,18 @@
   <div class="flex flex-col md:flex-row gap-4">
     <div class="md:basis-1/3 lg:basis-1/4 2xl:basis-1/6 flex flex-col">
       <UserProfileCard />
-      <SocialSideMenu />
     </div>
     <div class="md:basis-2/3 lg:basis-3/4 2xl:basis-5/6">
-      <div id="account-home">
-        <div class="flex mb-4">
-          <Avatar
-            :image="user.illustrationUrl + '?w=80&h=80&fit=crop'"
-            class="flex-none mr-2"
-            shape="circle"
-            size="large"
-          />
-          <div class="flex-1">
-            <p class="text-body-1">
-              {{ user.fullName }}
-            </p>
-            <p class="text-caption">
-              {{ user.username }}
-            </p>
-          </div>
-        </div>
-      </div>
+      <SocialWall :hidePostForm="true" />
     </div>
   </div>
 </template>
 
 <script setup>
 import { onMounted, provide } from "vue"
-
-import Avatar from "primevue/avatar"
-import SocialSideMenu from "../../components/social/SocialSideMenu.vue";
 import UserProfileCard from "../../components/social/UserProfileCard.vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
+import SocialWall from "../social/SocialWall.vue"
 
 
 const { user, isCurrentUser, groupInfo, isGroup, loadUser } = useSocialInfo();

assets/vue/views/message/MessageList.vue

@@ -179,7 +179,6 @@ import { MESSAGE_STATUS_DELETED, MESSAGE_TYPE_INBOX } from "../../components/mes
 import { GET_USER_MESSAGE_TAGS } from "../../graphql/queries/MessageTag"
 import { useNotification } from "../../composables/notification"
 import { useMessageRelUserStore } from "../../store/messageRelUserStore"
-import SocialSideMenu from "../../components/social/SocialSideMenu.vue"
 import { useSecurityStore } from "../../store/securityStore"
 
 const route = useRoute()

assets/vue/views/social/SocialLayout.vue

@@ -1,17 +1,14 @@
 <template>
   <div v-if="!isLoadingPage && hasPermission" class="flex flex-wrap md:flex-nowrap gap-4">
-        <div class="flex flex-col w-full md:w-1/4 lg:w-1/6">
+    <div class="flex flex-col w-full md:w-1/5">
       <UserProfileCard />
-          <SocialSideMenu />
-        </div>
-        <div class="flex-grow w-full md:basis-1/2 lg:basis-2/3">
-          <component :is="currentComponent" />
-        </div>
-        <div class="flex flex-col w-full md:w-1/4 lg:w-1/6" v-if="!isSearchPage">
       <MyGroupsCard v-if="!hideSocialGroupBlock" />
       <MyFriendsCard />
       <MySkillsCard />
     </div>
+    <div class="flex-grow w-full md:w-4/5">
+      <component :is="currentComponent" />
+    </div>
   </div>
   <div v-if="!isLoadingPage && !hasPermission">
     <div class="flex flex-wrap md:flex-nowrap gap-4">
@@ -21,17 +18,15 @@
 </template>
 
 <script setup>
-import { onMounted, provide, computed, readonly, ref, watch, watchEffect } from "vue"
+import { onMounted, provide, computed, ref } from "vue"
 import { useRoute } from "vue-router"
 import SocialWall from "./SocialWall.vue"
 import SocialSearch from "./SocialSearch.vue"
 import UserProfileCard from "../../components/social/UserProfileCard.vue"
-import SocialSideMenu from "../../components/social/SocialSideMenu.vue"
 import MyGroupsCard from "../../components/social/MyGroupsCard.vue"
 import MyFriendsCard from "../../components/social/MyFriendsCard.vue"
 import MySkillsCard from "../../components/social/MySkillsCard.vue"
 import { useSocialInfo } from "../../composables/useSocialInfo"
-import { useSocialStore } from "../../store/socialStore"
 import { useI18n } from "vue-i18n"
 import { useSecurityStore } from "../../store/securityStore"
 import { usePlatformConfig } from "../../store/platformConfig"
@@ -41,26 +36,20 @@ const hideSocialGroupBlock =  "true" === platformConfigStore.getSetting("social.
 const route = useRoute()
 const { t } = useI18n()
 const securityStore = useSecurityStore()
-const socialStore = useSocialStore()
 const hasPermission = ref(false)
 const isLoadingPage = ref(true)
-const { user, isCurrentUser, groupInfo, isGroup, loadUser } = useSocialInfo()
+const { user, isCurrentUser, groupInfo, isGroup } = useSocialInfo()
 
 provide("social-user", user)
 provide("is-current-user", isCurrentUser)
 provide("group-info", groupInfo)
 provide("is-group", isGroup)
 
-const profileUserId = computed(() => route.query.id)
-
 onMounted(async () => {
-  if (profileUserId.value) {
-    await socialStore.checkUserRelation(securityStore.user.id, profileUserId.value)
-    hasPermission.value = socialStore.isProfileVisible
-  } else {
+  isLoadingPage.value = false
+  if (securityStore.user.id) {
     hasPermission.value = true
   }
-  isLoadingPage.value = false
 })
 
 const isSearchPage = computed(() => route.path.includes('/social/search'))

assets/vue/views/social/SocialWall.vue

@@ -1,6 +1,6 @@
 <template>
   <div>
-    <SocialWallPostForm @post-created="refreshPosts" class="mb-6" />
+    <SocialWallPostForm v-if="!hidePostForm"  @post-created="refreshPosts" class="mb-6" />
     <SocialWallPostList ref="postListRef" class="mb-6" />
   </div>
 </template>
@@ -10,6 +10,13 @@ import { ref } from 'vue';
 import SocialWallPostForm from "../../components/social/SocialWallPostForm.vue"
 import SocialWallPostList from "../../components/social/SocialWallPostList.vue"
 
+const props = defineProps({
+  hidePostForm: {
+    type: Boolean,
+    default: false
+  }
+});
+
 const postListRef = ref(null);
 
 function refreshPosts() {

assets/vue/views/userreluser/Invitations.vue

@@ -27,8 +27,7 @@
 </template>
 
 <script setup>
-import axios from 'axios'
-import { inject, onMounted, ref, watchEffect } from "vue"
+import { inject, onMounted, ref } from "vue"
 import InvitationList from "../../components/userreluser/InvitationList.vue"
 import BaseButton from "../../components/basecomponents/BaseButton.vue"
 import { useRouter } from "vue-router"
@@ -38,12 +37,9 @@ const receivedInvitations = ref([])
 const sentInvitations = ref([])
 const pendingInvitations = ref([])
 const router = useRouter()
-
 const user = inject('social-user')
-const isCurrentUser = inject('is-current-user')
-
 
-watchEffect(() => {
+onMounted(() => {
   if (user.value && user.value.id) {
     fetchInvitations(user.value.id)
   }

src/CoreBundle/Controller/SocialController.php

@@ -312,25 +312,26 @@ class SocialController extends AbstractController
     ): JsonResponse {
         $baseUrl = $requestStack->getCurrentRequest()->getBaseUrl();
         $cid = (int) $settingsManager->getSetting('forum.global_forums_course_id');
-        $groupsArray = [];
-        $threadsArray = [];
+        $items = [];
+        $goToUrl = '';
+
         if (!empty($cid)) {
             $threads = $forumThreadRepository->getThreadsBySubscriptions($userId, $cid);
             foreach ($threads as $thread) {
                 $threadId = $thread->getIid();
                 $forumId = (int) $thread->getForum()->getIid();
-                $threadsArray[] = [
+                $items[] = [
                     'id' => $threadId,
                     'name' => $thread->getTitle(),
                     'description' => '',
                     'url' => $baseUrl.'/main/forum/viewthread.php?cid='.$cid.'&sid=0&gid=0&forum='.$forumId.'&thread='.$threadId,
-                    'go_to' => $baseUrl.'/main/forum/index.php?cid='.$cid.'&sid=0&gid=0',
                 ];
             }
+            $goToUrl = $baseUrl.'/main/forum/index.php?cid='.$cid.'&sid=0&gid=0';
         } else {
             $groups = $usergroupRepository->getGroupsByUser($userId);
             foreach ($groups as $group) {
-                $groupsArray[] = [
+                $items[] = [
                     'id' => $group->getId(),
                     'name' => $group->getTitle(),
                     'description' => $group->getDescription(),
@@ -339,11 +340,10 @@ class SocialController extends AbstractController
             }
         }
 
-        if (!empty($threadsArray)) {
-            return $this->json(['groups' => $threadsArray]);
-        }
-
-        return $this->json(['groups' => $groupsArray]);
+        return $this->json([
+            'items' => $items,
+            'go_to' => $goToUrl
+        ]);
     }
 
     #[Route('/group/{groupId}/discussion/{discussionId}/messages', name: 'chamilo_core_social_group_discussion_messages')]