SONARJAVA-6098 Update rule S1128's description (#5482)

Author: aurelien-coet-sonarsource

sonar-java-plugin/src/main/resources/org/sonar/l10n/java/rules/java/S1128.html

@@ -2,10 +2,10 @@ <h2>Why is this an issue?</h2>
 <p>Unnecessary imports refer to importing types that are not used or referenced anywhere in the code.</p>
 <p>Although they don’t affect the runtime behavior of the application after compilation, removing them will:</p>
 <ul>
-  <li> Improve the readability and maintainability of the code. </li>
-  <li> Help avoid potential naming conflicts. </li>
-  <li> Improve the build time, as the compiler has fewer lines to read and fewer types to resolve. </li>
-  <li> Reduce the number of items the code editor will show for auto-completion, thereby showing fewer irrelevant suggestions. </li>
+  <li>Improve the readability and maintainability of the code.</li>
+  <li>Help avoid potential naming conflicts.</li>
+  <li>Improve the build time, as the compiler has fewer lines to read and fewer types to resolve.</li>
+  <li>Reduce the number of items the code editor will show for auto-completion, thereby showing fewer irrelevant suggestions.</li>
 </ul>
 <h3>Exceptions</h3>
 <p>Imports for types mentioned in Javadocs are ignored.</p>
@@ -17,13 +17,19 @@ <h4>Noncompliant code example</h4>
 <pre data-diff-id="1" data-diff-type="noncompliant">
 package myapp.helpers;
 
+import module java.logging; // Module imports are a Java 25 feature
+import module java.logging; // Noncompliant - module is imported twice
+
 import java.io.IOException;
 import java.nio.file.*;
 import java.nio.file.*;     // Noncompliant - package is imported twice
 import java.lang.Runnable;  // Noncompliant - java.lang is imported by default
 
 public class FileHelper {
+    private static final Logger LOGGER = Logger.getLogger("FileLogger");
+
     public static String readFirstLine(String filePath) throws IOException {
+        LOGGER.log(Level.INFO, "Reading first line from: " + filePath);
         return Files.readAllLines(Paths.get(filePath)).get(0);
     }
 }
@@ -32,23 +38,28 @@ <h4>Compliant solution</h4>
 <pre data-diff-id="1" data-diff-type="compliant">
 package myapp.helpers;
 
+import module java.logging;
+
 import java.io.IOException;
 import java.nio.file.*;
 
 public class FileHelper {
+    private static final Logger LOGGER = Logger.getLogger("FileLogger");
+
     public static String readFirstLine(String filePath) throws IOException {
+        LOGGER.log(Level.INFO, "Reading first line from: " + filePath);
         return Files.readAllLines(Paths.get(filePath)).get(0);
     }
 }
 </pre>
 <h2>Resources</h2>
 <h3>Documentation</h3>
 <ul>
-  <li> <a href="https://docs.oracle.com/javase/tutorial/java/package/usepkgs.html">Java packages</a> </li>
+  <li><a href="https://docs.oracle.com/javase/tutorial/java/package/usepkgs.html">Java packages</a></li>
 </ul>
 <h3>Related rules</h3>
 <ul>
-  <li> {rule:java:S1144} - Unused "private" methods should be removed </li>
-  <li> {rule:java:S1481} - Unused local variables should be removed </li>
+  <li>{rule:java:S1144} - Unused "private" methods should be removed</li>
+  <li>{rule:java:S1481} - Unused local variables should be removed</li>
 </ul>