Added failsafes for retrieving remote apps on settings->apps page, in case categories or app data are not found

1 files changed, 25 insertions, 19 deletions

diff --git a/settings/apps.php b/settings/apps.php
index 1481f6fc4ad..a1c1bf6aa53 100644
--- a/settings/apps.php
+++ b/settings/apps.php
@@ -31,10 +31,6 @@ OC_App::setActiveNavigationEntry( "core_apps" );
 
 $installedApps = OC_App::getAllApps();
 
-$remoteApps = OC_App::getAppstoreApps();
-
-//$remoteApps = array();
-
 //TODO which apps do we want to blacklist and how do we integrate blacklisting with the multi apps folder feature?
 
 $blacklist = array('files');//we dont want to show configuration for these
@@ -90,29 +86,39 @@ foreach ( $installedApps as $app ) {
 	}
 }
 
-// Remove duplicates
-foreach ( $appList as $app ) {
+$remoteApps = OC_App::getAppstoreApps();
 
-	foreach ( $remoteApps AS $key => $remote ) {
-	
-		if ( 
-		$app['name'] == $remote['name']
-		// To set duplicate detection to use OCS ID instead of string name, 
-		// enable this code, remove the line of code above, 
-		// and add <ocs_id>[ID]</ocs_id> to info.xml of each 3rd party app: 
-		// OR $app['ocs_id'] == $remote['ocs_id']
-		) {
+if ( $remoteApps ) {
+
+	// Remove duplicates
+	foreach ( $appList as $app ) {
+
+		foreach ( $remoteApps AS $key => $remote ) {
+		
+			if ( 
+			$app['name'] == $remote['name']
+			// To set duplicate detection to use OCS ID instead of string name, 
+			// enable this code, remove the line of code above, 
+			// and add <ocs_id>[ID]</ocs_id> to info.xml of each 3rd party app: 
+			// OR $app['ocs_id'] == $remote['ocs_id']
+			) {
+				
+				unset( $remoteApps[$key]);
 			
-			unset( $remoteApps[$key]);
+			}
 		
 		}
-	
+		
 	}
+
+	$combinedApps = array_merge( $appList, $remoteApps );
+
+} else {
+
+	$combinedApps = $appList;
 	
 }
 
-$combinedApps = array_merge( $appList, $remoteApps );
-
 function app_sort( $a, $b ) {
 
 	if ($a['active'] != $b['active']) {