KT-5896 Select Word should always have step for selecting the content of string literal

#KT-5896 Fixed

idea/src/META-INF/plugin.xml

@@ -321,6 +321,7 @@
     <extendWordSelectionHandler implementation="org.jetbrains.jet.plugin.editor.wordSelection.KotlinDocCommentSelectioner"/>
     <extendWordSelectionHandler implementation="org.jetbrains.jet.plugin.editor.wordSelection.KotlinDeclarationSelectioner"/>
     <extendWordSelectionHandler implementation="org.jetbrains.jet.plugin.editor.wordSelection.KotlinListSelectioner"/>
+    <extendWordSelectionHandler implementation="org.jetbrains.jet.plugin.editor.wordSelection.KotlinStringLiteralSelectioner"/>
     <basicWordSelectionFilter implementation="org.jetbrains.jet.plugin.editor.wordSelection.KotlinWordSelectionFilter"/>
 
     <typedHandler implementation="org.jetbrains.jet.plugin.editor.KotlinTypedHandler"/>

idea/src/org/jetbrains/jet/plugin/editor/wordSelection/KotlinStringLiteralSelectioner.kt

@@ -0,0 +1,35 @@
+/*
+ * Copyright 2010-2014 JetBrains s.r.o.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.jetbrains.jet.plugin.editor.wordSelection
+
+import com.intellij.codeInsight.editorActions.ExtendWordSelectionHandlerBase
+import com.intellij.psi.PsiElement
+import com.intellij.openapi.editor.Editor
+import com.intellij.openapi.util.TextRange
+import org.jetbrains.jet.lang.psi.JetStringTemplateExpression
+
+public class KotlinStringLiteralSelectioner : ExtendWordSelectionHandlerBase() {
+    override fun canSelect(e: PsiElement) = e is JetStringTemplateExpression
+
+    override fun select(e: PsiElement, editorText: CharSequence, cursorOffset: Int, editor: Editor): List<TextRange>? {
+        val entries = (e as JetStringTemplateExpression).getEntries()
+        if (entries.isEmpty()) return null
+        val start = entries.first().getTextRange().getStartOffset()
+        val end = entries.last().getTextRange().getEndOffset()
+        return listOf(TextRange(start, end))
+    }
+}

idea/src/org/jetbrains/jet/plugin/editor/wordSelection/KotlinWordSelectionFilter.kt

@@ -18,10 +18,23 @@ package org.jetbrains.jet.plugin.editor.wordSelection
 
 import com.intellij.openapi.util.Condition
 import com.intellij.psi.PsiElement
-import org.jetbrains.jet.lang.psi.JetBlockExpression
+import org.jetbrains.jet.JetNodeTypes.*
 import org.jetbrains.jet.lang.psi.JetContainerNode
+import org.jetbrains.jet.lang.psi.JetElement
 
 public class KotlinWordSelectionFilter : Condition<PsiElement>{
-    override fun value(e: PsiElement)
-            = !KotlinListSelectioner.canSelect(e) && e !is JetBlockExpression && e !is JetContainerNode
+    override fun value(e: PsiElement): Boolean {
+
+        if (KotlinListSelectioner.canSelect(e)) return false
+        if (e is JetContainerNode) return false
+
+        val parent = e.getParent()
+        if (parent !is JetElement) return true
+        if (parent.getFirstChild().getNextSibling() == null) return false // skip nodes with the same range as their parent
+
+        return when (e.getNode().getElementType()) {
+            BLOCK, LITERAL_STRING_TEMPLATE_ENTRY -> false
+            else -> true
+        }
+    }
 }

idea/testData/wordSelection/SimpleStringLiteral/0.kt

@@ -0,0 +1 @@
+val v = "aaa <caret>bbb"

idea/testData/wordSelection/SimpleStringLiteral/1.kt

@@ -0,0 +1 @@
+val v = "aaa <selection><caret>bbb</selection>"

idea/testData/wordSelection/SimpleStringLiteral/2.kt

@@ -0,0 +1 @@
+val v = "<selection>aaa <caret>bbb</selection>"

idea/testData/wordSelection/SimpleStringLiteral/3.kt

@@ -0,0 +1 @@
+val v = <selection>"aaa <caret>bbb"</selection>

idea/testData/wordSelection/TemplateStringLiteral1/0.kt

@@ -0,0 +1 @@
+val v = "aaa $x <caret>bbb"

idea/testData/wordSelection/TemplateStringLiteral1/1.kt

@@ -0,0 +1 @@
+val v = "aaa $x <selection><caret>bbb</selection>"

idea/testData/wordSelection/TemplateStringLiteral1/2.kt

@@ -0,0 +1 @@
+val v = "<selection>aaa $x <caret>bbb</selection>"

idea/testData/wordSelection/TemplateStringLiteral1/3.kt

@@ -0,0 +1 @@
+val v = <selection>"aaa $x <caret>bbb"</selection>

idea/testData/wordSelection/TemplateStringLiteral2/0.kt

@@ -0,0 +1 @@
+val v = "aaa $<caret>x bbb"

idea/testData/wordSelection/TemplateStringLiteral2/1.kt

@@ -0,0 +1 @@
+val v = "aaa $<selection><caret>x</selection> bbb"

idea/testData/wordSelection/TemplateStringLiteral2/2.kt

@@ -0,0 +1 @@
+val v = "aaa <selection>$<caret>x</selection> bbb"

idea/testData/wordSelection/TemplateStringLiteral2/3.kt

@@ -0,0 +1 @@
+val v = "<selection>aaa $<caret>x bbb</selection>"

idea/testData/wordSelection/TemplateStringLiteral2/4.kt

@@ -0,0 +1 @@
+val v = <selection>"aaa $<caret>x bbb"</selection>

idea/testData/wordSelection/TemplateStringLiteral3/0.kt

@@ -0,0 +1 @@
+val v = "aaa ${<caret>x + y} bbb"

idea/testData/wordSelection/TemplateStringLiteral3/1.kt

@@ -0,0 +1 @@
+val v = "aaa ${<selection><caret>x</selection> + y} bbb"

idea/testData/wordSelection/TemplateStringLiteral3/2.kt

@@ -0,0 +1 @@
+val v = "aaa ${<selection><caret>x + y</selection>} bbb"

idea/testData/wordSelection/TemplateStringLiteral3/3.kt

@@ -0,0 +1 @@
+val v = "aaa <selection>${<caret>x + y}</selection> bbb"

idea/testData/wordSelection/TemplateStringLiteral3/4.kt

@@ -0,0 +1 @@
+val v = "<selection>aaa ${<caret>x + y} bbb</selection>"

idea/testData/wordSelection/TemplateStringLiteral3/5.kt

@@ -0,0 +1 @@
+val v = <selection>"aaa ${<caret>x + y} bbb"</selection>

idea/tests/org/jetbrains/jet/plugin/WordSelectionTest.java

@@ -49,6 +49,12 @@ public class WordSelectionTest extends JetLightCodeInsightFixtureTestCase {
 
     public void testCommentForStatements() { doTest(); }
 
+    public void testSimpleStringLiteral() { doTest(); }
+
+    public void testTemplateStringLiteral1() { doTest(); }
+    public void testTemplateStringLiteral2() { doTest(); }
+    public void testTemplateStringLiteral3() { doTest(); }
+
     private void doTest() {
         String dirName = getTestName(false);