diff options
author | Arthur Schiwon <blizzz@arthur-schiwon.de> | 2016-12-02 12:27:15 +0100 |
---|---|---|
committer | Bjoern Schiessle <bjoern@schiessle.org> | 2016-12-08 10:29:51 +0100 |
commit | 056e59af7e98a8526739a441e052b5af6d7f8320 (patch) | |
tree | 7eb10d6ec999b643de0f60532ab86d4e8f052e6f | |
parent | 39c2d9e6636b525a3166ce2826d7d282de169c93 (diff) | |
download | nextcloud-server-056e59af7e98a8526739a441e052b5af6d7f8320.tar.gz nextcloud-server-056e59af7e98a8526739a441e052b5af6d7f8320.zip |
fix some coding style and stricter comparisons, makes scrutinizer happier
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
-rw-r--r-- | core/js/sharedialogshareelistview.js | 6 | ||||
-rw-r--r-- | core/js/shareitemmodel.js | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/core/js/sharedialogshareelistview.js b/core/js/sharedialogshareelistview.js index 5bece6afc98..5f24b236464 100644 --- a/core/js/sharedialogshareelistview.js +++ b/core/js/sharedialogshareelistview.js @@ -305,7 +305,7 @@ $li.find('.popovermenu').replaceWith(this.popoverMenuTemplate(sharee)); var $edit = $li.parent().find('#canEdit-' + this.cid + '-' + sharee.shareWith); - if($edit.length == 1) { + if($edit.length === 1) { $edit.prop('checked', sharee.hasEditPermission); } } @@ -338,10 +338,10 @@ if (!this._template) { this._template = Handlebars.compile(TEMPLATE); } - var sharees = data['sharees']; + var sharees = data.sharees; if(_.isArray(sharees)) { for (var i = 0; i < sharees.length; i++) { - data['sharees'][i]['popoverMenu'] = this.popoverMenuTemplate(sharees[i]); + data.sharees[i].popoverMenu = this.popoverMenuTemplate(sharees[i]); } } return this._template(data); diff --git a/core/js/shareitemmodel.js b/core/js/shareitemmodel.js index 4a2f8f8bc8a..4a82630b967 100644 --- a/core/js/shareitemmodel.js +++ b/core/js/shareitemmodel.js @@ -404,7 +404,7 @@ } for(var i = 0; i < shares.length; i++) { var shareWith = shares[i]; - if(shareWith['id'] == shareId) { + if(shareWith.id === shareId) { return i; } } |