Fixed the sonar issues

Signed-off-by: Gaurav <gaurav.sahay@in.bosch.com>
This commit is contained in:
Gaurav
2016-08-19 11:17:18 +02:00
parent b80323caf9
commit b2c1e78c72
3 changed files with 44 additions and 12 deletions

View File

@@ -43,14 +43,47 @@ public final class SPUIWindowDecorator {
* window id * window id
* @param type * @param type
* window type * window type
* @param content
* content in the window
* @param cancelButtonClickListener
* cancel button
* @param helpLink
* help link
* @param layout
* layout in the window
* @param i18n
* i18n for internationalization
* @return Window * @return Window
*
*/ */
public static CommonDialogWindow getWindow(final String caption, final String id, final String type, public static CommonDialogWindow getWindow(final String caption, final String id, final String type,
final Component content, final ClickListener cancelButtonClickListener, final String helpLink, final Component content, final ClickListener cancelButtonClickListener, final String helpLink,
final AbstractLayout layout, final I18N i18n) { final AbstractLayout layout, final I18N i18n) {
return getWindow(caption, id, type, content, null, cancelButtonClickListener, helpLink, layout, i18n); return getWindow(caption, id, type, content, null, cancelButtonClickListener, helpLink, layout, i18n);
} }
/**
* @param caption
* window caption
* @param id
* window id
* @param type
* window type
* @param content
* content in the window
* @param saveButtonClickListener
* save button
* @param cancelButtonClickListener
* cancel button
* @param helpLink
* help link
* @param layout
* layout in the window
* @param i18n
* i18n for internationalization
* @return window
*/
public static CommonDialogWindow getWindow(final String caption, final String id, final String type, public static CommonDialogWindow getWindow(final String caption, final String id, final String type,
final Component content, final ClickListener saveButtonClickListener, final Component content, final ClickListener saveButtonClickListener,
final ClickListener cancelButtonClickListener, final String helpLink, final AbstractLayout layout, final ClickListener cancelButtonClickListener, final String helpLink, final AbstractLayout layout,

View File

@@ -183,6 +183,13 @@ public class TargetAddUpdateWindowLayout extends CustomComponent {
return window; return window;
} }
/**
* Returns Target Update window based on the selected Entity Id in the
* target table.
*
* @param entityId
* @return window
*/
public Window getWindow(final String entityId) { public Window getWindow(final String entityId) {
populateValuesOfTarget(entityId); populateValuesOfTarget(entityId);
getWindow(); getWindow();

View File

@@ -166,10 +166,10 @@ public class AddUpdateRolloutWindowLayout extends GridLayout {
public CommonDialogWindow getWindow() { public CommonDialogWindow getWindow() {
resetComponents(); resetComponents();
final CommonDialogWindow window = SPUIWindowDecorator.getWindow(i18n.get("caption.configure.rollout"), null, final CommonDialogWindow commonDialogWindow = SPUIWindowDecorator.getWindow(
SPUIDefinitions.CREATE_UPDATE_WINDOW, this, null, i18n.get("caption.configure.rollout"), null, SPUIDefinitions.CREATE_UPDATE_WINDOW, this, null,
uiProperties.getLinks().getDocumentation().getRolloutView(), this, i18n); uiProperties.getLinks().getDocumentation().getRolloutView(), this, i18n);
window.setSaveDialogCloseListener(new SaveDialogCloseListener() { commonDialogWindow.setSaveDialogCloseListener(new SaveDialogCloseListener() {
@Override @Override
public void saveOrUpdate() { public void saveOrUpdate() {
@@ -189,7 +189,7 @@ public class AddUpdateRolloutWindowLayout extends GridLayout {
return duplicateCheck(); return duplicateCheck();
} }
}); });
return window; return commonDialogWindow;
} }
/** /**
@@ -513,14 +513,6 @@ public class AddUpdateRolloutWindowLayout extends GridLayout {
return errorThresoldPercent; return errorThresoldPercent;
} }
private boolean validateFields() {
if (!noOfGroups.isValid() || !errorThreshold.isValid() || !triggerThreshold.isValid()) {
uiNotification.displayValidationError(i18n.get("message.correct.invalid.value"));
return false;
}
return true;
}
private boolean duplicateCheck() { private boolean duplicateCheck() {
if (rolloutManagement.findRolloutByName(getRolloutName()) != null) { if (rolloutManagement.findRolloutByName(getRolloutName()) != null) {
uiNotification.displayValidationError( uiNotification.displayValidationError(