Skip to content

Commit 870bbbd

Browse files
committed
Fix checkstyle issues
1 parent 9a21233 commit 870bbbd

File tree

15 files changed

+818
-643
lines changed

15 files changed

+818
-643
lines changed

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azuretools/ijidea/actions/AzureSignInAction.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,9 @@ public static void onAzureSignIn(Project project) {
140140

141141
public static boolean doSignIn(AuthMethodManager authMethodManager, Project project) throws Exception {
142142
boolean isSignIn = authMethodManager.isSignedIn();
143-
if (isSignIn) return true;
143+
if (isSignIn) {
144+
return true;
145+
}
144146
SignInWindow w = SignInWindow.go(authMethodManager.getAuthMethodDetails(), project);
145147
if (w != null) {
146148
AuthMethodDetails authMethodDetailsUpdated = w.getAuthMethodDetails();

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azuretools/ijidea/ui/NotificationWindow.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,6 @@ public void deliver(String subject, String message) {
4040
public void run() {
4141
JPanel panel = new JPanel();
4242
Messages.showMessageDialog(panel, message, subject, AllIcons.General.Information);
43-
// JOptionPane.showMessageDialog(panel,
44-
// message,
45-
// subject,
46-
// JOptionPane.INFORMATION_MESSAGE);
4743
}
4844
});
4945

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azuretools/ijidea/ui/SignInWindow.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,8 @@ public void run() {
308308
}
309309

310310
authFileTextField.setText(path);
311-
PluginUtil.displayInfoDialog("Authentication File Created", String.format("Your credentials have been exported to %s, please keep the authentication file safe", path));
311+
PluginUtil.displayInfoDialog("Authentication File Created", String.format(
312+
"Your credentials have been exported to %s, please keep the authentication file safe", path));
312313
} catch (Exception ex) {
313314
ex.printStackTrace();
314315
//LOGGER.error("doCreateServicePrincipal", ex);

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azuretools/ijidea/ui/SrvPriSettingsDialog.java

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -56,17 +56,16 @@ public List<SubscriptionDetail> getSubscriptionDetails() {
5656
}
5757

5858
DefaultTableModel model = new DefaultTableModel() {
59-
final Class[] columnClass = new Class[] {
60-
Boolean.class, String.class, String.class
59+
final Class[] columnClass = new Class[]{
60+
Boolean.class, String.class, String.class
6161
};
6262
@Override
6363
public boolean isCellEditable(int row, int col) {
6464
return (col == 0);
6565
}
6666

6767
@Override
68-
public Class<?> getColumnClass(int columnIndex)
69-
{
68+
public Class<?> getColumnClass(int columnIndex) {
7069
return columnClass[columnIndex];
7170
}
7271
};
@@ -142,8 +141,10 @@ protected void doOKAction() {
142141
int rc = model.getRowCount();
143142
int unselectedCount = 0;
144143
for (int ri = 0; ri < rc; ++ri) {
145-
boolean selected = (boolean)model.getValueAt(ri, 0);
146-
if (!selected) unselectedCount++;
144+
boolean selected = (boolean) model.getValueAt(ri, 0);
145+
if (!selected) {
146+
unselectedCount++;
147+
}
147148
}
148149

149150
if (unselectedCount == rc) {
@@ -153,7 +154,7 @@ protected void doOKAction() {
153154
}
154155

155156
for (int ri = 0; ri < rc; ++ri) {
156-
boolean selected = (boolean)model.getValueAt(ri, 0);
157+
boolean selected = (boolean) model.getValueAt(ri, 0);
157158
this.sdl.get(ri).setSelected(selected);
158159
}
159160

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azuretools/ijidea/ui/SubscriptionsDialog.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -122,10 +122,10 @@ private void refreshSubscriptions() {
122122
subscriptionManager.setSubscriptionDetails(sdl);
123123

124124
}, (ex) -> {
125-
ex.printStackTrace();
126-
//LOGGER.error("refreshSubscriptions", ex);
127-
ErrorWindow.show(project, ex.getMessage(), "Refresh Subscriptions Error");
128-
});
125+
ex.printStackTrace();
126+
//LOGGER.error("refreshSubscriptions", ex);
127+
ErrorWindow.show(project, ex.getMessage(), "Refresh Subscriptions Error");
128+
});
129129
}
130130

131131
private void setSubscriptions() {
@@ -229,7 +229,7 @@ protected String getDimensionServiceKey() {
229229

230230
private class SubscriptionTableModel extends DefaultTableModel {
231231
final Class[] columnClass = new Class[]{
232-
Boolean.class, String.class, String.class
232+
Boolean.class, String.class, String.class
233233
};
234234

235235
@Override

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azuretools/ijidea/ui/WarSelectDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ protected Action[] createActions() {
8787

8888
@Override
8989
protected void doOKAction() {
90-
DefaultTableModel tableModel = (DefaultTableModel)table.getModel();
90+
DefaultTableModel tableModel = (DefaultTableModel) table.getModel();
9191
int i = table.getSelectedRow();
9292
if (i < 0) {
9393
DefaultLoader.getUIHelper().showMessageDialog(contentPane, "Please select an artifact", "Select Artifact "

0 commit comments

Comments
 (0)