[#12] ColumnWithoutAnnotation: columns suffixed _id should be typed…

… with `int`;

resources-tests/laravelInsight/annotation/ColumnWithoutAnnotationInspection.matchTypes.fixed.php

@@ -34,3 +34,13 @@ class Eloquent_IdString extends EloquentSimulation
 
 (new Eloquent_IdInt)->id;
 (new Eloquent_IdString)->id;
+
+/**
+ * @property int $id
+ * @property int $user_id
+ */
+class Eloquent_IdSuffix extends EloquentSimulation
+{
+}
+
+(new Eloquent_IdSuffix)->user_id;

resources-tests/laravelInsight/annotation/ColumnWithoutAnnotationInspection.matchTypes.php

@@ -22,3 +22,9 @@ class <weak_warning descr="@property $id was not annotated">Eloquent_IdString</w
 
 (new Eloquent_IdInt)-><weak_warning descr="@property $id was not annotated">id</weak_warning>;
 (new Eloquent_IdString)-><weak_warning descr="@property $id was not annotated">id</weak_warning>;
+
+class <weak_warning descr="@property $id was not annotated">Eloquent_IdSuffix</weak_warning> extends EloquentSimulation
+{
+}
+
+(new Eloquent_IdSuffix)-><weak_warning descr="@property $user_id was not annotated">user_id</weak_warning>;

src/laravelInsight/annotation/ColumnWithoutAnnotationInspection.java

@@ -84,6 +84,11 @@ static void validatePropertyAnnotation(
             final PsiElement issueReference,
             final String propertyName
         ) {
+            if (propertyName.endsWith("_id")) {
+                validatePropertyAnnotation(problemsHolder, phpClass, issueReference, propertyName, "int");
+                return;
+            }
+
             if (propertyName.endsWith("_at")) {
                 validatePropertyAnnotation(problemsHolder, phpClass, issueReference, propertyName, "\\Carbon\\Carbon");
                 return;