You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@syncope.apache.org by il...@apache.org on 2020/05/19 14:27:16 UTC
[syncope] branch master updated: [SYNCOPE-1560] Possibly the last
tweaks
This is an automated email from the ASF dual-hosted git repository.
ilgrosso pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/syncope.git
The following commit(s) were added to refs/heads/master by this push:
new 8136257 [SYNCOPE-1560] Possibly the last tweaks
8136257 is described below
commit 813625713812587eb683205cc8e5d036ccda08ff
Author: Francesco Chicchiriccò <il...@apache.org>
AuthorDate: Tue May 19 16:05:45 2020 +0200
[SYNCOPE-1560] Possibly the last tweaks
---
.../syncope/client/console/wizards/CSVPullWizardBuilder.java | 9 ++++-----
.../client/console/wicket/markup/html/form/BinaryFieldPanel.java | 8 +++-----
.../client/enduser/markup/html/form/BinaryFieldPanel.java | 8 +++-----
.../org/apache/syncope/client/console/panels/ImportMetadata.java | 8 ++++----
4 files changed, 14 insertions(+), 19 deletions(-)
diff --git a/client/idm/console/src/main/java/org/apache/syncope/client/console/wizards/CSVPullWizardBuilder.java b/client/idm/console/src/main/java/org/apache/syncope/client/console/wizards/CSVPullWizardBuilder.java
index cc6ae54..3e21b1e 100644
--- a/client/idm/console/src/main/java/org/apache/syncope/client/console/wizards/CSVPullWizardBuilder.java
+++ b/client/idm/console/src/main/java/org/apache/syncope/client/console/wizards/CSVPullWizardBuilder.java
@@ -129,7 +129,6 @@ public class CSVPullWizardBuilder extends BaseAjaxWizardBuilder<CSVPullSpec> {
private static final long serialVersionUID = -4736870165235853919L;
public Details(final CSVPullSpec spec) {
- ListModel<FileUpload> fileUploadModel = new ListModel<>(new ArrayList<>());
FileInputConfig csvFile = new FileInputConfig().
showUpload(false).showRemove(false).showPreview(false).
browseClass("btn btn-success").browseIcon("<i class=\"fas fa-folder-open\"></i> ");
@@ -137,16 +136,16 @@ public class CSVPullWizardBuilder extends BaseAjaxWizardBuilder<CSVPullSpec> {
if (!Locale.ENGLISH.getLanguage().equals(language)) {
csvFile.withLocale(language);
}
- BootstrapFileInputField csvUpload = new BootstrapFileInputField("csvUpload", fileUploadModel, csvFile);
+ BootstrapFileInputField csvUpload =
+ new BootstrapFileInputField("csvUpload", new ListModel<>(new ArrayList<>()), csvFile);
csvUpload.add(new AjaxFormSubmitBehavior(Constants.ON_CHANGE) {
private static final long serialVersionUID = 5538299138211283825L;
@Override
protected void onSubmit(final AjaxRequestTarget target) {
- if (!fileUploadModel.getObject().isEmpty()) {
- FileUpload uploadedFile = fileUploadModel.getObject().get(0);
-
+ FileUpload uploadedFile = csvUpload.getFileUpload();
+ if (uploadedFile != null) {
if (maxUploadSize != null && uploadedFile.getSize() > maxUploadSize.bytes()) {
SyncopeConsoleSession.get().error(getString("tooLargeFile").
replace("${maxUploadSizeB}", String.valueOf(maxUploadSize.bytes())).
diff --git a/client/idrepo/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/BinaryFieldPanel.java b/client/idrepo/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/BinaryFieldPanel.java
index 58e2ef6..6219134 100644
--- a/client/idrepo/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/BinaryFieldPanel.java
+++ b/client/idrepo/console/src/main/java/org/apache/syncope/client/console/wicket/markup/html/form/BinaryFieldPanel.java
@@ -179,7 +179,6 @@ public class BinaryFieldPanel extends BaseBinaryFieldPanel {
downloadLink.setOutputMarkupId(true);
uploadForm.add(downloadLink);
- ListModel<FileUpload> fileUploadModel = new ListModel<>(new ArrayList<>());
FileInputConfig config = new FileInputConfig().
showUpload(false).showRemove(false).showPreview(false).
browseClass("btn btn-success").browseIcon("<i class=\"fas fa-folder-open\"></i> ");
@@ -187,16 +186,15 @@ public class BinaryFieldPanel extends BaseBinaryFieldPanel {
if (!Locale.ENGLISH.getLanguage().equals(language)) {
config.withLocale(language);
}
- fileUpload = new BootstrapFileInputField("fileUpload", fileUploadModel, config);
+ fileUpload = new BootstrapFileInputField("fileUpload", new ListModel<>(new ArrayList<>()), config);
fileUpload.add(new AjaxFormSubmitBehavior(Constants.ON_CHANGE) {
private static final long serialVersionUID = -1107858522700306810L;
@Override
protected void onSubmit(final AjaxRequestTarget target) {
- if (!fileUploadModel.getObject().isEmpty()) {
- FileUpload uploaded = fileUploadModel.getObject().get(0);
-
+ FileUpload uploaded = fileUpload.getFileUpload();
+ if (uploaded != null) {
if (maxUploadSize != null && uploaded.getSize() > maxUploadSize.bytes()) {
// SYNCOPE-1213 manage directly max upload file size (if set in properties file)
SyncopeConsoleSession.get().error(getString("tooLargeFile").
diff --git a/client/idrepo/enduser/src/main/java/org/apache/syncope/client/enduser/markup/html/form/BinaryFieldPanel.java b/client/idrepo/enduser/src/main/java/org/apache/syncope/client/enduser/markup/html/form/BinaryFieldPanel.java
index 1a752cb..7de1299 100644
--- a/client/idrepo/enduser/src/main/java/org/apache/syncope/client/enduser/markup/html/form/BinaryFieldPanel.java
+++ b/client/idrepo/enduser/src/main/java/org/apache/syncope/client/enduser/markup/html/form/BinaryFieldPanel.java
@@ -177,7 +177,6 @@ public class BinaryFieldPanel extends BaseBinaryFieldPanel {
downloadLink.setOutputMarkupId(true);
uploadForm.add(downloadLink);
- ListModel<FileUpload> fileUploadModel = new ListModel<>(new ArrayList<>());
FileInputConfig config = new FileInputConfig().
showUpload(false).showRemove(false).showPreview(false).
browseClass("btn btn-success").browseIcon("<i class=\"fas fa-folder-open\"></i> ");
@@ -186,7 +185,7 @@ public class BinaryFieldPanel extends BaseBinaryFieldPanel {
config.withLocale(language);
}
- fileUpload = new BootstrapFileInputField("fileUpload", fileUploadModel, config);
+ fileUpload = new BootstrapFileInputField("fileUpload", new ListModel<>(new ArrayList<>()), config);
fileUpload.setOutputMarkupId(true);
fileUpload.add(new AjaxFormSubmitBehavior(Constants.ON_CHANGE) {
@@ -195,9 +194,8 @@ public class BinaryFieldPanel extends BaseBinaryFieldPanel {
@Override
protected void onSubmit(final AjaxRequestTarget target) {
- if (!fileUploadModel.getObject().isEmpty()) {
- FileUpload uploaded = fileUploadModel.getObject().get(0);
-
+ FileUpload uploaded = fileUpload.getFileUpload();
+ if (uploaded != null) {
if (maxUploadSize != null && uploaded.getSize() > maxUploadSize.bytes()) {
// SYNCOPE-1213 manage directly max upload file size (if set in properties file)
SyncopeEnduserSession.get().error(getString("tooLargeFile").
diff --git a/ext/saml2sp/client-console/src/main/java/org/apache/syncope/client/console/panels/ImportMetadata.java b/ext/saml2sp/client-console/src/main/java/org/apache/syncope/client/console/panels/ImportMetadata.java
index 37bee93..449795f 100644
--- a/ext/saml2sp/client-console/src/main/java/org/apache/syncope/client/console/panels/ImportMetadata.java
+++ b/ext/saml2sp/client-console/src/main/java/org/apache/syncope/client/console/panels/ImportMetadata.java
@@ -45,7 +45,6 @@ public class ImportMetadata extends TogglePanel<Serializable> {
Form<?> form = new Form<>("metadataForm");
addInnerObject(form);
- ListModel<FileUpload> fileUploadModel = new ListModel<>(new ArrayList<>());
FileInputConfig config = new FileInputConfig().
showUpload(false).showRemove(false).showPreview(false).
browseClass("btn btn-success").browseIcon("<i class=\"fas fa-folder-open\"></i> ");
@@ -53,7 +52,8 @@ public class ImportMetadata extends TogglePanel<Serializable> {
if (!Locale.ENGLISH.getLanguage().equals(language)) {
config.withLocale(language);
}
- BootstrapFileInputField fileUpload = new BootstrapFileInputField("fileUpload", fileUploadModel, config);
+ BootstrapFileInputField fileUpload =
+ new BootstrapFileInputField("fileUpload", new ListModel<>(new ArrayList<>()), config);
form.add(fileUpload.setOutputMarkupId(true));
form.add(new AjaxSubmitLink("doUpload", form) {
@@ -62,8 +62,8 @@ public class ImportMetadata extends TogglePanel<Serializable> {
@Override
protected void onSubmit(final AjaxRequestTarget target) {
- if (!fileUploadModel.getObject().isEmpty()) {
- FileUpload uploaded = fileUploadModel.getObject().get(0);
+ FileUpload uploaded = fileUpload.getFileUpload();
+ if (uploaded != null) {
try {
SAML2IdPsRestClient.importIdPs(uploaded.getInputStream());