From ac2135560fcac13b08744388e24c498433ba5297 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Fri, 11 Nov 2016 14:11:19 +0100 Subject: [PATCH 1/2] Fix single author with details Signed-off-by: Joas Schilling --- settings/js/apps.js | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/settings/js/apps.js b/settings/js/apps.js index 654756af53..451becc67a 100644 --- a/settings/js/apps.js +++ b/settings/js/apps.js @@ -189,6 +189,8 @@ OC.Settings.Apps = OC.Settings.Apps || { } }); app.author = authors.join(', '); + } else if (typeof app.author !== 'string') { + app.author = app.author['@value']; } var html = template(app); @@ -539,8 +541,8 @@ OC.Settings.Apps = OC.Settings.Apps || { // Author Name apps = apps.concat(_.filter(OC.Settings.Apps.State.apps, function (app) { + var authors = []; if (_.isArray(app.author)) { - var authors = []; _.each(app.author, function (author) { if (typeof author === 'string') { authors.push(author); @@ -555,6 +557,15 @@ OC.Settings.Apps = OC.Settings.Apps || { } }); return OC.Settings.Apps._search(authors.join(' '), query); + } else if (typeof app.author !== 'string') { + authors.push(app.author['@value']); + if (!_.isUndefined(app.author['@attributes']['homepage'])) { + authors.push(app.author['@attributes']['homepage']); + } + if (!_.isUndefined(app.author['@attributes']['mail'])) { + authors.push(app.author['@attributes']['mail']); + } + return OC.Settings.Apps._search(authors.join(' '), query); } return OC.Settings.Apps._search(app.author, query); })); From 896be27e7232cae77f8e0ec25e1fec9f548c9a9a Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Sun, 13 Nov 2016 14:28:32 +0100 Subject: [PATCH 2/2] Fixes javascript tests for apps.js Signed-off-by: Roeland Jago Douma --- settings/tests/js/appsSpec.js | 31 +++++++++++++++++++++---------- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/settings/tests/js/appsSpec.js b/settings/tests/js/appsSpec.js index d2ca1fb5c8..aa785a6768 100644 --- a/settings/tests/js/appsSpec.js +++ b/settings/tests/js/appsSpec.js @@ -185,26 +185,32 @@ describe('OC.Settings.Apps tests', function() { { id: 'foo', name: 'Foo app', - level: 0 + level: 0, + author: 'foo' }, { id: 'alpha', name: 'Alpha app', - level: 300 + level: 300, + author: ['alpha', 'beta'] }, { id: 'nolevel', - name: 'No level' + name: 'No level', + author: 'bar' }, { id: 'zork', name: 'Some famous adventure game', - level: 200 + level: 200, + author: 'baz' + }, { id: 'delta', name: 'Mathematical symbol', - level: 200 + level: 200, + author: 'foobar' } ] }) @@ -217,26 +223,31 @@ describe('OC.Settings.Apps tests', function() { 'foo': { id: 'foo', name: 'Foo app', - level: 0 + level: 0, + author: 'foo' }, 'alpha': { id: 'alpha', name: 'Alpha app', - level: 300 + level: 300, + author: ['alpha', 'beta'] }, 'nolevel': { id: 'nolevel', - name: 'No level' + name: 'No level', + author: 'bar' }, 'zork': { id: 'zork', name: 'Some famous adventure game', - level: 200 + level: 200, + author: 'baz', }, 'delta': { id: 'delta', name: 'Mathematical symbol', - level: 200 + level: 200, + author: 'foobar' } }); });