Report "Make <visibility>" for visibility fixes #KT-19274 Fixed

(instead of "Add <visibility> modifier")
This commit is contained in:
Mikhail Glukhikh
2017-07-31 15:18:53 +03:00
parent 29dba9edb3
commit 1ed1751b63
11 changed files with 11 additions and 11 deletions
@@ -46,7 +46,7 @@ open class AddModifierFix(
override fun getText(): String { override fun getText(): String {
val element = element ?: return "" val element = element ?: return ""
if (modifier in modalityModifiers) { if (modifier in modalityModifiers || modifier in KtTokens.VISIBILITY_MODIFIERS) {
return "Make ${getElementName(element)} ${modifier.value}" return "Make ${getElementName(element)} ${modifier.value}"
} }
return "Add '${modifier.value}' modifier" return "Add '${modifier.value}' modifier"
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A(<caret>internal val a: String = "") { class A(<caret>internal val a: String = "") {
fun foo() { fun foo() {
a a
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A(private val a: String = "") { class A(private val a: String = "") {
fun foo() { fun foo() {
a a
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A { class A {
<caret>internal val a = "" <caret>internal val a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A { class A {
private val a = "" private val a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A { class A {
val <caret>a = "" val <caret>a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A { class A {
private val a = "" private val a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
open class A { open class A {
<caret>protected val a = "" <caret>protected val a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
open class A { open class A {
private val a = "" private val a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A { class A {
<caret>public val a = "" <caret>public val a = ""
@@ -1,4 +1,4 @@
// "Add 'private' modifier" "true" // "Make 'a' private" "true"
class A { class A {
private val a = "" private val a = ""