Fixed complete app-breakdown if no favorites are set

Signed-off-by: fnuesse <felix.nuesse@t-online.de>

Hid three-dot menu if no favorites are selected

Signed-off-by: fnuesse <felix.nuesse@t-online.de>

apps/files/lib/Controller/ViewController.php

@@ -161,6 +161,7 @@ class ViewController extends Controller {
 
 		$user = $this->userSession->getUser()->getUID();
 
+		//Load QuickAccess-Defaults
 		$sorting=$this->config->getUserValue($user,$this->appName,'quickaccess_sorting_strategy','date');
 		$reverseListSetting=$this->config->getUserValue($user,$this->appName,'quickaccess_reverse_list','false');
 		if($this->config->getUserValue($user,$this->appName,'show_Quick_Access',true)){
@@ -169,6 +170,20 @@ class ViewController extends Controller {
 			$expanded='false';
 		}
 
+
+		//Get Favorite-Folder
+		$tagger=\OC::$server->getTagManager();
+
+		$helper= new \OCA\Files\Activity\Helper($tagger);
+
+		try {
+			$favElements = $helper->getFavoriteFilePaths($this->userSession->getUser()->getUID());
+		} catch (\RuntimeException $e) {
+			$favElements['folders'] = null;
+		}
+
+		$FavoritesFolderCount=sizeof($favElements['folders']);
+
 		\OCA\Files\App::getNavigationManager()->add(
 			[
 				'id' => 'favorites',
@@ -178,16 +193,14 @@ class ViewController extends Controller {
 				'quickaccessSortingStrategy' => $sorting,
 				'quickaccessSortingReverse' => $reverseListSetting,
 				'order' => 5,
-				'name' => $this->l10n->t('Favorites')
+				'name' => $this->l10n->t('Favorites'),
+				//If there are zero elements, add ul end tag directly.
+				'favoritescount' => $FavoritesFolderCount
 			]
 		);
 
-		$tagger=\OC::$server->getTagManager();
-
-		$helper= new \OCA\Files\Activity\Helper($tagger);
-		$favElements = $helper->getFavoriteFilePaths($this->userSession->getUser()->getUID());
 
-		$FavoritesFolderCount=sizeof($favElements['folders']);
+		//Add Favorite-folder as menuentries, if there are any
 		if($FavoritesFolderCount>0){
 
 			$NavBarPositionPosition=6;

apps/files/templates/appnavigation.php

@@ -14,7 +14,7 @@
 				   class="nav-icon-<?php p($item['icon'] !== '' ? $item['icon'] : $item['id']) ?> svg"><?php p($item['name']);?></a>
 				<?php if($item['id']=='favorites'){?>
 
-				<div class="app-navigation-entry-utils">
+				<div class="app-navigation-entry-utils" <?php if($item['favoritescount']==0){ ?> style="display: none"<?php } ?>>
 					<ul>
 						<li class="app-navigation-entry-utils-menu-button svg"><button id="button-<?php p($item['id']) ?>"></button></li>
 					</ul>
@@ -47,8 +47,10 @@
 						</li>
 					</ul>
 				</div>
-				<ul id="quickaccess-list"  <?php if($item['enableQuickaccess']=='false'){ ?> style="display: none"<?php } ?>>
+					<ul id="quickaccess-list"  <?php if($item['enableQuickaccess']=='false'){ ?> style="display: none"<?php } ?>>
+					<?php /*This fixes the styleerrors if no favorites are set*/ if($item['favoritescount']==0){?></ul><?php } ?>
 				<?php } ?>
+
 				<?php if($item['quickaccesselement']=='last'){?>
 				</ul>
 				<?php } ?>