fix: sonarqube suggestion

Alternatives in regular expressions should be grouped when used with anchors
dhis2 · Jul 29, 2022 · 6f3c72c · 6f3c72c
1 parent ab3c2af
commit 6f3c72c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...ervice-dxf2/src/main/java/org/hisp/dhis/dxf2/events/event/csv/DefaultCsvEventService.java b/...ervice-dxf2/src/main/java/org/hisp/dhis/dxf2/events/event/csv/DefaultCsvEventService.java
@@ -62,7 +62,7 @@ public class DefaultCsvEventService
     private static final CsvSchema CSV_SCHEMA = CSV_MAPPER.schemaFor( CsvEventDataValue.class )
         .withLineSeparator( "\n" );
 
-    private static final Pattern TRIM_SINGLE_QUOTES = Pattern.compile( "^'|'$" );
+    private static final Pattern TRIM_SINGLE_QUOTES = Pattern.compile( "(?:^')|(?:'$)" );
 
     @Override
     public void writeEvents( OutputStream outputStream, List<Event> events, boolean withHeader )

diff --git a/...api/src/main/java/org/hisp/dhis/webapi/controller/tracker/csv/TrackerCsvEventService.java b/...api/src/main/java/org/hisp/dhis/webapi/controller/tracker/csv/TrackerCsvEventService.java
@@ -64,7 +64,7 @@ public class TrackerCsvEventService
 {
     private static final CsvMapper CSV_MAPPER = new CsvMapper().enable( CsvParser.Feature.WRAP_AS_ARRAY );
 
-    private static final Pattern TRIM_SINGLE_QUOTES = Pattern.compile( "^'|'$" );
+    private static final Pattern TRIM_SINGLE_QUOTES = Pattern.compile( "(?:^')|(?:'$)" );
 
     @Override
     public void writeEvents( OutputStream outputStream, List<Event> events, boolean withHeader )