Do not use the calling context

[BorzdeG]

origin code:

public void testMethod() {
    testMethod2(testSub);
}

Sets the cursor to "testSub" in line 5 (the word is not highlighted) (screenshot)
calling the action "To Constant Style"

expect:

public void testMethod() {
    testMethod2(TEST_SUB);
}

actual (not only incorrect transformation, but also "broken" code):

public void testMethod() {
TEST_METHOD_(TEST_SUB_;
}

[krasa]

When nothing is selected, it uses the whole line.
Imho detecting what should be selected in various situations would be too hard.

[krasa]

maybe we could just reuse ctrl+w feature for it, but then why not just click once or twice on ctrl+w and then use the plugin action?

[BorzdeG]

Is it possible to somehow attached to Expressions similar to the action of the "Extract Variable"?

[krasa]

yeah, I could select any identifier (variable, field, method,...) or string literal (that thing between "...") under a caret in java files.

[BorzdeG]

Perhaps we are talking about different things. I'm talking about is: https://habrastorage.org/files/001/229/b81/001229b8161c4f7eb76b1f4d0a981a71.png

[krasa]

Oh yeah this, probably doable, but it seems unnecessary, if you are on a identifier or on a string, then it does not make sense to convert anything more than that.

[krasa]

I implemented smarter selections for java, properties files, and some generic support.
You can try it here: https://github.com/krasa/StringManipulation/releases/

[krasa]

it was just released.