diff --git a/settings/js/apps.js b/settings/js/apps.js index 6da8c395ecb..3326886951f 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -487,22 +487,22 @@ OC.Settings.Apps = OC.Settings.Apps || { container.children('li[data-id]').each(function (index, el) { var id = $(el).data('id'); // remove all apps that are not in the correct order - if ((navEntries[index] && navEntries[index].id !== $(el).data('id'))) { + if (!navEntries[index] || (navEntries[index] && navEntries[index].id !== $(el).data('id'))) { $(el).remove(); $('#appmenu li[data-id='+id+']').remove(); } }); - var previousEntry; + var previousEntry = {}; // add enabled apps to #navigation and #appmenu for (var i = 0; i < navEntries.length; i++) { var entry = navEntries[i]; if (container.children('li[data-id="' + entry.id + '"]').length === 0) { var li = $('
'); li.attr('data-id', entry.id); - var img = ''; var a = $('').attr('href', entry.href); var filename = $(''); var loading = $('').css('display', 'none'); @@ -526,9 +526,7 @@ OC.Settings.Apps = OC.Settings.Apps || { } } - if ($('#appmenu ul').children('li[data-id="' + entry.id + '"]').length === 0) { - // add apps to #appmenu until it is full - if ($('#appmenu li').not('.hidden').length < 8) { + if ($('#appmenu').children('li[data-id="' + entry.id + '"]').length === 0) { var li = $(''); li.attr('data-id', entry.id); var img = '