[WINDUPRULE-1037] Improves EAP 8 JSP rule, clarifying it is about Fac…

…es removing support for JSP, and make it potential (#1030) (#1034) Co-authored-by: Phil Cattanach <31246010+PhilipCattanach@users.noreply.github.com> (cherry picked from commit 99fefd4) Co-authored-by: Eduardo Martins <emartins@redhat.com>
windup · Oct 24, 2023 · 012af62 · 012af62
1 parent bee4ed9
commit 012af62
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 8 deletions.
diff --git a/rules/rules-reviewed/eap8/eap6/eap8-faces.windup.xml b/rules/rules-reviewed/eap8/eap6/eap8-faces.windup.xml
@@ -178,14 +178,24 @@
         </rule>
         <rule id="eap8-faces-00009">
             <when>
-                <or>
-                    <filecontent filename="{*}.jsp" pattern="&lt;jsp"/>
-                    <filecontent filename="{*}.jsp" pattern="&lt;%"/>
-                </or>
+                <and>
+                    <or>
+                        <filecontent filename="{*}.jsp" pattern="&lt;jsp"/>
+                        <filecontent filename="{*}.jsp" pattern="&lt;%"/>
+                    </or>
+                    <or>
+                        <project>
+                            <artifact groupId="org.jboss.spec.javax.faces" artifactId="{*}"/>
+                        </project>
+                        <project>
+                            <artifact groupId="com.sun.faces" artifactId="{*}"/>
+                        </project>
+                    </or>
+                </and>
             </when>
             <perform>
-                <hint title="JSP support has been removed in Jakarta EE 10" effort="5" category-id="mandatory">
-                    <message>JSP support has been removed in Jakarta EE 10. Facelets will remain as the only default view language, and views can also now be created solely using java.</message>
+                <hint title="Jakarta Faces support for JSP has been removed in Jakarta EE 10" effort="5" category-id="potential">
+                    <message>As of Jakarta EE 10, Jakarta Faces does no longer support JSP as a faces view technology. JSP usage within Jakarta Faces must be removed/substituted. Facelets will remain as the only default view language, and views can also now be created solely using java.</message>
                     <link title="Jakarta EE Faces 4.0 Facelets"
                           href="https://jakarta.ee/specifications/faces/4.0/jakarta-faces-4.0.html#a5476"/>
                     <link title="Support for creating views in Java"

diff --git a/rules/rules-reviewed/eap8/eap6/tests/eap8-faces.windup.test.xml b/rules/rules-reviewed/eap8/eap6/tests/eap8-faces.windup.test.xml
@@ -70,8 +70,8 @@
             <rule id="eap8-faces-00009-test">
                 <when>
                     <not>
-                        <iterable-filter size="1">
-                            <hint-exists message="JSP support has been removed in Jakarta EE 10"/>
+                        <iterable-filter size="2">
+                            <hint-exists message="As of Jakarta EE 10, Jakarta Faces does no longer support JSP as a faces view technology. JSP usage within Jakarta Faces must be removed/substituted. Facelets will remain as the only default view language, and views can also now be created solely using java."/>
                         </iterable-filter>
                     </not>
                 </when>