summaryrefslogtreecommitdiffstats
path: root/core
diff options
context:
space:
mode:
authorArthur Schiwon <blizzz@owncloud.com>2015-08-24 23:20:01 +0200
committerVincent Petry <pvince81@owncloud.com>2015-09-16 07:23:26 +0200
commitfdb95613e9b133feed7b7dbad22dc289a54ef094 (patch)
tree1a56906ef86da1b219d4fdafe3fd324d535b2327 /core
parentf709022559d434612f995ecce56a65b034590f6b (diff)
downloadnextcloud-server-fdb95613e9b133feed7b7dbad22dc289a54ef094.tar.gz
nextcloud-server-fdb95613e9b133feed7b7dbad22dc289a54ef094.zip
simplification, and throwing where throwing is needed
Diffstat (limited to 'core')
-rw-r--r--core/js/sharedialogexpirationview.js5
-rw-r--r--core/js/sharedialoglinkshareview.js8
-rw-r--r--core/js/sharedialogresharerinfoview.js5
-rw-r--r--core/js/sharedialogview.js2
4 files changed, 8 insertions, 12 deletions
diff --git a/core/js/sharedialogexpirationview.js b/core/js/sharedialogexpirationview.js
index e752c66bf35..4c628f5498a 100644
--- a/core/js/sharedialogexpirationview.js
+++ b/core/js/sharedialogexpirationview.js
@@ -48,7 +48,7 @@
if(!_.isUndefined(options.configModel)) {
this.configModel = options.configModel;
} else {
- console.warn('missing OC.Share.ShareConfigModel');
+ throw 'missing OC.Share.ShareConfigModel';
}
},
@@ -64,8 +64,7 @@
}
var expirationTemplate = this.template();
- this.$el.empty();
- this.$el.append(expirationTemplate({
+ this.$el.html(expirationTemplate({
setExpirationLabel: t('core', 'Set expiration date'),
expirationLabel: t('core', 'Expiration'),
expirationDatePlaceholder: t('core', 'Expiration date'),
diff --git a/core/js/sharedialoglinkshareview.js b/core/js/sharedialoglinkshareview.js
index ca6413761b0..ff22b629dc4 100644
--- a/core/js/sharedialoglinkshareview.js
+++ b/core/js/sharedialoglinkshareview.js
@@ -85,7 +85,7 @@
if(!_.isUndefined(options.configModel)) {
this.configModel = options.configModel;
} else {
- console.warn('missing OC.Share.ShareConfigModel');
+ throw 'missing OC.Share.ShareConfigModel';
}
},
@@ -96,8 +96,7 @@
|| !this.showLink
|| !this.configModel.isShareWithLinkAllowed())
{
- this.$el.empty();
- this.$el.append(linkShareTemplate({
+ this.$el.html(linkShareTemplate({
shareAllowed: false,
noSharingPlaceholder: t('core', 'Resharing is not allowed')
}));
@@ -114,8 +113,7 @@
publicUploadChecked = 'checked="checked"';
}
- this.$el.empty();
- this.$el.append(linkShareTemplate({
+ this.$el.html(linkShareTemplate({
shareAllowed: true,
linkShareLabel: t('core', 'Share link'),
urlLabel: t('core', 'Link'),
diff --git a/core/js/sharedialogresharerinfoview.js b/core/js/sharedialogresharerinfoview.js
index 8970d857fc3..63df25b4ed8 100644
--- a/core/js/sharedialogresharerinfoview.js
+++ b/core/js/sharedialogresharerinfoview.js
@@ -58,7 +58,7 @@
if(!_.isUndefined(options.configModel)) {
this.configModel = options.configModel;
} else {
- console.warn('missing OC.Share.ShareConfigModel');
+ throw 'missing OC.Share.ShareConfigModel';
}
},
@@ -90,8 +90,7 @@
);
}
- this.$el.empty();
- this.$el.append(reshareTemplate({
+ this.$el.html(reshareTemplate({
avatarEnabled: this.configModel.areAvatarsEnabled(),
sharedByText: sharedByText
}));
diff --git a/core/js/sharedialogview.js b/core/js/sharedialogview.js
index 63c3473252f..e07a5cee49a 100644
--- a/core/js/sharedialogview.js
+++ b/core/js/sharedialogview.js
@@ -81,7 +81,7 @@
if(!_.isUndefined(options.configModel)) {
this.configModel = options.configModel;
} else {
- console.warn('missing OC.Share.ShareConfigModel');
+ throw 'missing OC.Share.ShareConfigModel';
}
var subViewOptions = {