Retrieve user status only once when displaying page

We don't need to inject it in the template if the data is already
available in the initial state. This is also a bit cleaner.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Author: CarlSchwan

apps/user_status/src/menu.js

@@ -27,6 +27,7 @@ import { getRequestToken } from '@nextcloud/auth'
 import UserStatus from './UserStatus'
 import store from './store'
 import Avatar from '@nextcloud/vue/dist/Components/Avatar'
+import { loadState } from '@nextcloud/initial-state'
 
 // eslint-disable-next-line camelcase
 __webpack_nonce__ = btoa(getRequestToken())
@@ -35,11 +36,12 @@ Vue.prototype.t = t
 Vue.prototype.$t = t
 
 const avatarDiv = document.getElementById('avatardiv-menu')
+const userStatusData = loadState('user_status', 'status')
 const propsData = {
 	preloadedUserStatus: {
-		message: avatarDiv.dataset.userstatus_message,
-		icon: avatarDiv.dataset.userstatus_icon,
-		status: avatarDiv.dataset.userstatus,
+		message: userStatusData.message,
+		icon: userStatusData.icon,
+		status: userStatusData.status
 	},
 	user: avatarDiv.dataset.user,
 	displayName: avatarDiv.dataset.displayname,

core/templates/layout.user.php

@@ -153,11 +153,7 @@
 			} ?>"
 							 data-user="<?php p($_['user_uid']); ?>"
 							 data-displayname="<?php p($_['user_displayname']); ?>"
-			<?php if ($_['userStatus'] !== false) { ?>
-				data-userstatus="<?php p($_['userStatus']->getStatus()); ?>"
-				data-userstatus_message="<?php p($_['userStatus']->getMessage()); ?>"
-				data-userstatus_icon="<?php p($_['userStatus']->getIcon()); ?>"
-			<?php }
+			<?php
 			if ($_['userAvatarSet']) {
 				$avatar32 = $getUserAvatar(32); ?> data-avatar="<?php p($avatar32); ?>"
 			<?php

dist/user_status-menu.js

@@ -140,17 +140,6 @@ public function __construct($renderAs, $appId = '') {
 			} else {
 				$this->assign('userAvatarSet', true);
 				$this->assign('userAvatarVersion', $this->config->getUserValue(\OC_User::getUser(), 'avatar', 'version', 0));
-				if (\OC::$server->get(IAppManager::class)->isEnabledForUser('user_status')) {
-					$userStatusManager = \OC::$server->get(IUserStatusManager::class);
-					$userStatuses = $userStatusManager->getUserStatuses([$user->getUID()]);
-					if (array_key_exists($user->getUID(), $userStatuses)) {
-						$this->assign('userStatus', $userStatuses[$user->getUID()]);
-					} else {
-						$this->assign('userStatus', false);
-					}
-				} else {
-					$this->assign('userStatus', false);
-				}
 			}
 
 			// check if app menu icons should be inverted