Skip to content

Commit f02e89d

Browse files
committed
Fix file selection widget in data import plugin
1 parent 8145a00 commit f02e89d

File tree

3 files changed

+10
-11
lines changed

3 files changed

+10
-11
lines changed

opal-gwt-client/src/main/java/org/obiba/opal/web/gwt/app/client/magma/importdata/DataImportView.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
*/
1010
package org.obiba.opal.web.gwt.app.client.magma.importdata;
1111

12+
import com.google.gwt.user.client.ui.*;
1213
import org.obiba.opal.web.gwt.app.client.i18n.Translations;
1314
import org.obiba.opal.web.gwt.app.client.i18n.TranslationsUtils;
1415
import org.obiba.opal.web.gwt.app.client.js.JsArrays;
@@ -40,11 +41,6 @@
4041
import com.google.gwt.event.dom.client.ClickHandler;
4142
import com.google.gwt.uibinder.client.UiBinder;
4243
import com.google.gwt.uibinder.client.UiField;
43-
import com.google.gwt.user.client.ui.FlowPanel;
44-
import com.google.gwt.user.client.ui.IsWidget;
45-
import com.google.gwt.user.client.ui.Panel;
46-
import com.google.gwt.user.client.ui.SimplePanel;
47-
import com.google.gwt.user.client.ui.Widget;
4844
import com.google.inject.Inject;
4945
import com.google.web.bindery.event.shared.EventBus;
5046
import com.google.web.bindery.event.shared.HandlerRegistration;
@@ -411,9 +407,13 @@ public void updateHelp() {
411407
private void updateHelpPanelsVisibility() {
412408
hideHelpPanels();
413409
PluginPackageDto foundPluginPackage = getPluginPackage(formatChooser.getSelectedValue());
410+
helpOpalDatasourcePluginPanel.clear();
414411
if (foundPluginPackage != null) {
412+
Label helpLabel = new Label();
413+
helpLabel.getElement().setInnerHTML(foundPluginPackage.getDescription());
414+
helpLabel.addStyleName("help-block");
415+
helpOpalDatasourcePluginPanel.add(helpLabel);
415416
helpOpalDatasourcePluginPanel.setVisible(true);
416-
helpOpalDatasourcePluginPanel.getElement().getFirstChildElement().setInnerHTML(foundPluginPackage.getDescription());
417417
} else {
418418
switch(ImportFormat.valueOf(getSelectedFormat())) {
419419
case CSV:
@@ -453,8 +453,10 @@ private void hideHelpPanels() {
453453
helpJDBC.setVisible(false);
454454
helpOpalRest.setVisible(false);
455455
helpRSASHaven.setVisible(false);
456+
helpRXPTHaven.setVisible(false);
456457
helpRSPSSHaven.setVisible(false);
457458
helpRStataHaven.setVisible(false);
459+
helpRDSHaven.setVisible(false);
458460
helpHealthCanada.setVisible(false);
459461
helpGeonamesPostalCodes.setVisible(false);
460462
helpOpalDatasourcePluginPanel.setVisible(false);

opal-gwt-client/src/main/java/org/obiba/opal/web/gwt/app/client/magma/importdata/DataImportView.ui.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,6 @@
118118
</g:FlowPanel>
119119

120120
<g:FlowPanel addStyleNames="small-top-margin" ui:field="helpOpalDatasourcePluginPanel" visible="false">
121-
<g:Label styleName="help-block">
122-
123-
</g:Label>
124121
</g:FlowPanel>
125122

126123
</g:FlowPanel>

opal-gwt-client/src/main/java/org/obiba/opal/web/gwt/app/client/ui/FileSelection.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ private void addEventHandlers(final EventBus eventBus) {
6767
browseButton.addClickHandler(new ClickHandler() {
6868
@Override
6969
public void onClick(ClickEvent event) {
70-
fireEvent(new FileSelectionRequestEvent(FileSelection.this, format));
70+
eventBus.fireEvent(new FileSelectionRequestEvent(FileSelection.this, format));
7171
}
7272
});
7373

@@ -76,7 +76,7 @@ public void onClick(ClickEvent event) {
7676
public void onFileSelection(FileSelectionEvent event) {
7777
if(FileSelection.this.equals(event.getSource())) {
7878
file.setValue(event.getSelectedFile().getSelectionPath());
79-
fireEvent(new FileSelectionUpdatedEvent(FileSelection.this));
79+
eventBus.fireEvent(new FileSelectionUpdatedEvent(FileSelection.this));
8080
}
8181
}
8282
});

0 commit comments

Comments
 (0)