From f709bd869a3676b273d70b51c0ed2ef97ab1e89a Mon Sep 17 00:00:00 2001
From: Dmitry Jemerov <yole@jetbrains.com>
Date: Thu, 31 Aug 2017 19:01:27 +0200
Subject: [PATCH] Accept invalid patterns in naming convention settings

 #KT-19926 Fixed
---
 .../inspections/NamingConventionInspections.kt     | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/idea/src/org/jetbrains/kotlin/idea/inspections/NamingConventionInspections.kt b/idea/src/org/jetbrains/kotlin/idea/inspections/NamingConventionInspections.kt
index c0fd252cccd..e783ab86811 100644
--- a/idea/src/org/jetbrains/kotlin/idea/inspections/NamingConventionInspections.kt
+++ b/idea/src/org/jetbrains/kotlin/idea/inspections/NamingConventionInspections.kt
@@ -34,21 +34,27 @@ import org.jetbrains.kotlin.psi.*
 import org.jetbrains.kotlin.psi.psiUtil.containingClassOrObject
 import org.jetbrains.kotlin.psi.psiUtil.visibilityModifierType
 import java.awt.BorderLayout
+import java.util.regex.PatternSyntaxException
 import javax.swing.JPanel
 
 abstract class NamingConventionInspection(private val entityName: String,
                                           defaultNamePattern: String) : AbstractKotlinInspection() {
-    protected var nameRegex: Regex = defaultNamePattern.toRegex()
+    protected var nameRegex: Regex? = defaultNamePattern.toRegex()
     var namePattern: String = defaultNamePattern
         set(value) {
             field = value
-            nameRegex = value.toRegex()
+            nameRegex = try {
+                value.toRegex()
+            }
+            catch(e: PatternSyntaxException) {
+                null
+            }
         }
 
     protected fun verifyName(element: PsiNameIdentifierOwner, holder: ProblemsHolder) {
         val name = element.name
         val nameIdentifier = element.nameIdentifier
-        if (name != null && nameIdentifier != null && !nameRegex.matches(name)) {
+        if (name != null && nameIdentifier != null && nameRegex?.matches(name) == false) {
             holder.registerProblem(element.nameIdentifier!!,
                                    "$entityName name <code>#ref</code> doesn't match regex '$namePattern' #loc",
                                    RenameIdentifierFix())
@@ -140,7 +146,7 @@ class PackageNameInspection : NamingConventionInspection("Package", "[a-z][A-Za-
         return object : KtVisitorVoid() {
             override fun visitPackageDirective(directive: KtPackageDirective) {
                 val qualifiedName = directive.qualifiedName
-                if (qualifiedName.isNotEmpty() && !nameRegex.matches(qualifiedName)) {
+                if (qualifiedName.isNotEmpty() && nameRegex?.matches(qualifiedName) == false) {
                     holder.registerProblem(directive.packageNameExpression!!,
                                            "Package name <code>#ref</code> doesn't match regex '$namePattern' #loc",
                                            RenamePackageFix())