add local inspection and ability to swap incorrect range

Author: xepozz

src/main/gen/com/github/xepozz/crontab/language/psi/CrontabTimeRange.java

@@ -0,0 +1,9 @@
+package com.github.xepozz.crontab.ide.inspections
+
+import com.intellij.codeInspection.LocalQuickFix
+
+abstract class CrontabScheduleQuickFix : LocalQuickFix {
+    override fun getFamilyName() = this.name
+
+    abstract override fun getName(): String
+}

src/main/gen/com/github/xepozz/crontab/language/psi/impl/CrontabTimeRangeImpl.java

@@ -0,0 +1,34 @@
+package com.github.xepozz.crontab.ide.inspections
+
+import com.github.xepozz.crontab.language.psi.CrontabElementFactory
+import com.github.xepozz.crontab.language.psi.CrontabTimeRange
+import com.intellij.codeInspection.ProblemDescriptor
+import com.intellij.codeInspection.ProblemsHolder
+import com.intellij.openapi.project.Project
+
+object CrontabInspectionUtil {
+    fun registerSwapRange(holder: ProblemsHolder, element: CrontabTimeRange) {
+        val interval = "${element.second}-${element.first}"
+
+        holder.registerProblem(
+            element,
+            "Invalid time range. First interval must be greater than last.",
+            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.createCrontabTimeRange(
+                                project,
+                                psiElement.second,
+                                psiElement.first,
+                            )
+                        )
+                    }
+                }
+            }
+        )
+    }
+}

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

@@ -0,0 +1,27 @@
+package com.github.xepozz.crontab.ide.inspections
+
+import com.github.xepozz.crontab.language.CrontabFile
+import com.github.xepozz.crontab.language.psi.CrontabTimeRange
+import com.github.xepozz.crontab.language.psi.CrontabVisitor
+import com.intellij.codeInspection.LocalInspectionTool
+import com.intellij.codeInspection.ProblemsHolder
+import com.intellij.psi.PsiElementVisitor
+import com.intellij.psi.PsiFile
+
+class CrontabScheduleTimeRangeInspection : LocalInspectionTool() {
+    override fun getStaticDescription(): String = "Crontab schedule mistakes"
+
+    override fun isAvailableForFile(file: PsiFile): Boolean = file is CrontabFile
+
+    override fun buildVisitor(holder: ProblemsHolder, isOnTheFly: Boolean): PsiElementVisitor {
+        return object : CrontabVisitor() {
+            override fun visitTimeRange(element: CrontabTimeRange) {
+                if (element.first > element.second) {
+                    CrontabInspectionUtil.registerSwapRange(holder, element)
+                }
+                super.visitTimeRange(element)
+            }
+        }
+    }
+
+}

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

@@ -11,6 +11,8 @@
   elementTypeHolderClass="com.github.xepozz.crontab.language.psi.CrontabTypes"
   elementTypeClass="com.github.xepozz.crontab.language.psi.CrontabElementType"
   tokenTypeClass="com.github.xepozz.crontab.language.psi.CrontabTokenType"
+
+  psiImplUtilClass="com.github.xepozz.crontab.language.psi.impl.CrontabImplUtil"
 }
 
 crontabFile ::= item_*
@@ -35,6 +37,8 @@ TIME_PERIODIC ::= (STAR SLASH NUMBER)
 TIME_RANGE ::= NUMBER HYPHEN NUMBER
 {
     pin=2
+    extends="com.github.xepozz.crontab.language.psi.impl.CrontabTimeRangeBaseImpl"
+    methods=[getFirst getSecond getIntRange]
 }
 TIME_RANGE_STEP ::= NUMBER SLASH NUMBER
 {

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

@@ -0,0 +1,20 @@
+package com.github.xepozz.crontab.language.psi
+
+import com.github.xepozz.crontab.language.CrontabFile
+import com.github.xepozz.crontab.language.CrontabFileType
+import com.intellij.openapi.project.Project
+import com.intellij.psi.util.PsiTreeUtil
+
+object CrontabElementFactory {
+    fun createCrontabTimeRange(project: Project, first: Int, last: Int): CrontabTimeRange {
+        val file = createFile(project, "$first-$last")
+
+        return PsiTreeUtil.findChildOfType(file, CrontabTimeRange::class.java) as CrontabTimeRange
+    }
+
+    fun createFile(project: Project, text: String): CrontabFile {
+        val name = "dummy.crontab"
+        return com.intellij.psi.PsiFileFactory.getInstance(project)
+            .createFileFromText(name, CrontabFileType.INSTANCE, text) as CrontabFile
+    }
+}

src/main/kotlin/com/github/xepozz/crontab/language/parser/Crontab.bnf

@@ -0,0 +1,17 @@
+package com.github.xepozz.crontab.language.psi.impl
+
+import com.github.xepozz.crontab.language.psi.CrontabTimeRange
+
+object CrontabImplUtil {
+    @JvmStatic
+    fun getFirst(element: CrontabTimeRange) = element.node.firstChildNode.text.toInt()
+
+    @JvmStatic
+    fun getSecond(element: CrontabTimeRange): Int = element.node.lastChildNode.text.toInt()
+
+    @JvmStatic
+    fun getIntRange(element: CrontabTimeRange): IntRange = IntRange(
+        element.node.lastChildNode.text.toInt(),
+        element.node.firstChildNode.text.toInt(),
+    )
+}

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

@@ -0,0 +1,17 @@
+package com.github.xepozz.crontab.language.psi.impl
+
+import com.github.xepozz.crontab.language.psi.CrontabTimeRange
+import com.github.xepozz.crontab.language.psi.CrontabTypes
+import com.intellij.extapi.psi.ASTWrapperPsiElement
+import com.intellij.lang.ASTNode
+
+abstract class CrontabTimeRangeBaseImpl : ASTWrapperPsiElement, CrontabTimeRange {
+    constructor(node: ASTNode) : super(node)
+
+    override fun getText(): String {
+        val keyNode = this.node.findChildByType(CrontabTypes.CONTENT)
+
+        return keyNode?.text ?: ""
+    }
+
+}

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

@@ -36,6 +36,10 @@
         <lang.commenter
                 language="Crontab"
                 implementationClass="com.github.xepozz.crontab.ide.CrontabCommenter"/>
+        <localInspection
+                language="Crontab" enabledByDefault="true" level="ERROR"
+                groupName="Crontab schedule" displayName="Crontab schedule"
+                implementationClass="com.github.xepozz.crontab.ide.inspections.CrontabScheduleTimeRangeInspection"/>
 
         <intentionAction>
             <language>Crontab</language>