Extraction Engine: Set default visibility to 'public'

This commit is contained in:
Alexey Sedunov
2015-10-21 11:25:34 +03:00
committed by Alexey Sedunov
parent 3fd6864461
commit 7e9222f70a
@@ -35,6 +35,7 @@ import org.jetbrains.kotlin.idea.core.refactoring.JetRefactoringUtilKt;
import org.jetbrains.kotlin.idea.refactoring.JetRefactoringBundle;
import org.jetbrains.kotlin.idea.refactoring.introduce.extractionEngine.*;
import org.jetbrains.kotlin.idea.util.IdeDescriptorRenderers;
import org.jetbrains.kotlin.lexer.KtTokens;
import org.jetbrains.kotlin.types.KotlinType;
import javax.swing.*;
@@ -98,7 +99,7 @@ public class KotlinExtractFunctionDialog extends DialogWrapper {
if (!isVisibilitySectionAvailable()) return "";
String value = (String) visibilityBox.getSelectedItem();
return "internal".equals(value) ? "" : value;
return KtTokens.PUBLIC_KEYWORD.getValue().equals(value) ? "" : value;
}
private boolean checkNames() {
@@ -179,7 +180,11 @@ public class KotlinExtractFunctionDialog extends DialogWrapper {
boolean enableVisibility = isVisibilitySectionAvailable();
visibilityBox.setEnabled(enableVisibility);
if (enableVisibility) {
visibilityBox.setSelectedItem(extractableCodeDescriptor.getVisibility());
String defaultVisibility = extractableCodeDescriptor.getVisibility();
if (defaultVisibility.isEmpty()) {
defaultVisibility = KtTokens.PUBLIC_KEYWORD.getValue();
}
visibilityBox.setSelectedItem(defaultVisibility);
}
visibilityBox.addItemListener(
new ItemListener() {