collapse equal range

Author: xepozz

src/main/kotlin/com/github/xepozz/crontab/ide/inspections/CrontabInspectionUtil.kt

@@ -31,4 +31,28 @@ object CrontabInspectionUtil {
             }
         )
     }
+
+    fun registerSimplifyRange(holder: ProblemsHolder, element: CrontabTimeRange) {
+        val interval = "${element.first}"
+
+        holder.registerProblem(
+            element,
+            "Invalid time range. One of intervals can be eliminated.",
+            object : CrontabScheduleQuickFix() {
+                override fun getName() = "Replace with \"$interval\""
+
+                override fun applyFix(project: Project, descriptor: ProblemDescriptor) {
+                    val psiElement = descriptor.psiElement as? CrontabTimeRange ?: return
+                    if (psiElement.isWritable) {
+                        psiElement.replace(
+                            CrontabElementFactory.createCrontabTimeExact(
+                                project,
+                                psiElement.first,
+                            )
+                        )
+                    }
+                }
+            }
+        )
+    }
 }

src/main/kotlin/com/github/xepozz/crontab/ide/inspections/CrontabScheduleTimeRangeInspection.kt

@@ -19,6 +19,9 @@ class CrontabScheduleTimeRangeInspection : LocalInspectionTool() {
                 if (element.first > element.second) {
                     CrontabInspectionUtil.registerSwapRange(holder, element)
                 }
+                if (element.first == element.second) {
+                    CrontabInspectionUtil.registerSimplifyRange(holder, element)
+                }
                 super.visitTimeRange(element)
             }
         }

src/main/kotlin/com/github/xepozz/crontab/language/psi/CrontabElementFactory.kt

@@ -11,6 +11,11 @@ object CrontabElementFactory {
 
         return PsiTreeUtil.findChildOfType(file, CrontabTimeRange::class.java) as CrontabTimeRange
     }
+    fun createCrontabTimeExact(project: Project, value: Int): CrontabTimeExact {
+        val file = createFile(project, "$value")
+
+        return PsiTreeUtil.findChildOfType(file, CrontabTimeExact::class.java) as CrontabTimeExact
+    }
 
     fun createFile(project: Project, text: String): CrontabFile {
         val name = "dummy.crontab"

src/main/kotlin/com/github/xepozz/crontab/language/psi/impl/CrontabImplUtil.kt

@@ -4,14 +4,11 @@ import com.github.xepozz.crontab.language.psi.CrontabTimeRange
 
 object CrontabImplUtil {
     @JvmStatic
-    fun getFirst(element: CrontabTimeRange) = element.node.firstChildNode.text.toInt()
+    fun getFirst(element: CrontabTimeRange) = element.node.firstChildNode.text.toIntOrNull() ?: 0
 
     @JvmStatic
-    fun getSecond(element: CrontabTimeRange): Int = element.node.lastChildNode.text.toInt()
+    fun getSecond(element: CrontabTimeRange): Int = element.node.lastChildNode.text.toIntOrNull() ?: 0
 
     @JvmStatic
-    fun getIntRange(element: CrontabTimeRange): IntRange = IntRange(
-        element.node.lastChildNode.text.toInt(),
-        element.node.firstChildNode.text.toInt(),
-    )
+    fun getIntRange(element: CrontabTimeRange): IntRange = IntRange(element.first, element.second)
 }