Optimization: Store user ID into variable

To avoid calling the same function twice
pH7Software · Jul 26, 2020 · 8a746d6 · 8a746d6
1 parent e56b25d
commit 8a746d6
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/_protected/framework/Mvc/Controller/Controller.class.php b/_protected/framework/Mvc/Controller/Controller.class.php
@@ -265,13 +265,15 @@ private function setModsStatusTplVars()
 
     private function setUserNotifications()
     {
+        $iMemberId = $this->session->get('member_id');
+
         $aNotificationCounter = [
-            'count_unread_mail' => MailCoreModel::countUnreadMsg($this->session->get('member_id')),
-            'count_pen_friend_request' => FriendCoreModel::getPending($this->session->get('member_id'))
+            'count_unread_mail' => MailCoreModel::countUnreadMsg($iMemberId),
+            'count_pen_friend_request' => FriendCoreModel::getPending($iMemberId)
         ];
         $this->view->assigns($aNotificationCounter);
 
-        unset($aNotificationCounter);
+        unset($aNotificationCounter, $iMemberId);
     }
 
     /**