Merge branch 'eclipse-main' into maplibre-secure-map-tiles

Author: axeluhl

java/com.sap.sailing.domain/src/com/sap/sailing/domain/markpassingcalculation/MarkPassingCalculator.java

@@ -165,11 +165,7 @@ public MarkPassingCalculator(DynamicTrackedRace race, boolean doListen,
             if (listener != null) {
                 synchronized (MarkPassingCalculator.this) {
                     if (listenerThread == null) {
-                        listenerThread = createAndStartListenerThread();
-                        synchronized (listenerThread) {
-                            listenerThreadStarted = true;
-                            listenerThread.notifyAll();
-                        }
+                        createAndStartListenerThread();
                     }
                 }
             }
@@ -181,11 +177,21 @@ public MarkPassingCalculator(DynamicTrackedRace race, boolean doListen,
         }
     }
 
-    private Thread createAndStartListenerThread() {
-        final Thread result = new Thread(listen, "MarkPassingCalculator for race " + race.getRaceIdentifier());
-        result.setDaemon(true);
-        result.start();
-        return result;
+    /**
+     * Must be invoked while owning this object's monitor. Launches a thread running the {@link #listen} task, assigning
+     * the thread to the {@link #listenerThread} field and marking the thread as started while owning the thread's
+     * object monitor ({@code synchronized}), notifying all waiters of the {@link Thread} object, such as the
+     * {@link #waitUntilStopped(long)} method.
+     */
+    private void createAndStartListenerThread() {
+        assert Thread.holdsLock(this);
+        listenerThread = new Thread(listen, "MarkPassingCalculator for race " + race.getRaceIdentifier());
+        listenerThread.setDaemon(true);
+        listenerThread.start();
+        synchronized (listenerThread) {
+            listenerThreadStarted = true;
+            listenerThread.notifyAll();
+        }
     }
 
     /**
@@ -638,7 +644,7 @@ protected synchronized void enqueueUpdate(StorePositionUpdateStrategy update) {
         // the queue may have filled up while we were suspended
         if (!suspended) {
             if (listenerThread == null) {
-                listenerThread = createAndStartListenerThread();
+                createAndStartListenerThread();
             } else if (listenerThread.getState() == State.TERMINATED) {
                 logger.severe("Listener thread of MarkPassingCalculator (MPC) for race " + race.getRace().getName()
                         + " terminated but not null. Why are we still receiving updates? We must have been stopped before!");

java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/StringMessages_de.properties

@@ -395,7 +395,7 @@ windStatMax=Max
 fleet=Flotte
 boatClass=Bootsklasse
 setDelayToLive=Verzögerung zu ''Live'' setzen
-pleaseEnterNonEmptyVenue=Bitte geben sie einen Veranstaltungsort ein.
+pleaseEnterNonEmptyVenue=Bitte gib einen Veranstaltungsort ein.
 pleaseEnterStartAndEndDate=Bitte gib ein Start- und Endedatum an.
 startDateMustBeforeEndDate=Das Startdatum muß vor dem Endedatum liegen.
 eventWithThisNameAlreadyExists=Eine Veranstaltung mit diesem Namen existiert bereits.
@@ -1487,7 +1487,7 @@ selectCourseAreas=Regattabahnen auswählen
 resetCourseAreas=Zurücksetzen
 copyCompetitors=Teilnehmer kopieren
 smartphoneTracking=Smartphone Tracking
-titelOfChooseNameDialog= Wählen sie einen Namen
+titelOfChooseNameDialog= Wähle einen Namen
 ownPrefix=Eigener Bezeichner
 defaultName=Standard
 exampleTextForName=Namen werden in dieser Form erstellt:
@@ -1569,11 +1569,11 @@ addNewFix=Neuen Fix hinzufügen
 pleaseSelectOtherTimepoint=Bitte wähle einen anderen Zeitpunkt. Am gewählten Zeitpunkt ist bereits ein Fix.
 confirmMove=Bestätige Umpositionierung
 confirmNewFix=Bestätige neuen Fix
-selectAFixPositionBy=Wählen sie eine Fixposition, indem sie die Karte verschieben. Bestätigen oder Brechen sie danach mit den Knöpfen unten ab.
-pleaseSelectAMark=Bitte wählen sie einen Marker aus der Liste auf der linken Seite aus.
+selectAFixPositionBy=Wähle eine Fixposition, indem du die Karte verschiebst. Bestätige oder brich danach mit den Knöpfen unten ab.
+pleaseSelectAMark=Bitte wähle einen Marker aus der Liste auf der linken Seite aus.
 markFixes=Marker Fixe
 distanceFromAveragePosition=Entfernung vom Durchschnitt
-useATouchOptimizedUI=Benutzen sie eine auf Touchbildschirme optimierte Eingabe.
+useATouchOptimizedUI=Benutze eine auf Touchbildschirme optimierte Eingabe.
 deleteFix=Fix löschen
 theDeletionOfThisFix=The Löschung des Fixes is aktuell nicht möglich.
 loadingMarkFixes=Lade Marker Fixe...
@@ -2498,7 +2498,7 @@ goToSelfServicePortalDialogText=Self-Service Portal des Abonementanbieters in ei
 failedFetchingSelfServicePortalSession=Self-Service Portal konnte nicht geöffnet werden.
 oneAlwaysStaysOne=Wertung 1 bleibt 1, unabhängig vom Spalten-Faktor
 otherTieBreakingLeaderboard=Andere Rangliste zum Tie-Breaking
-helptextLinkingRaces=Um unverknüpfte Wettfahrten mit getrackten Rennen zu verknüpfen, müssen sie in der untenstehenden Tabelle eine Wettfahrt linksklicken und danach das entsprechende getrackte Rennen in der rechten Ansicht linksklicken.
+helptextLinkingRaces=Um unverknüpfte Wettfahrten mit getrackten Rennen zu verknüpfen, musst du in der untenstehenden Tabelle eine Wettfahrt linksklicken und danach das entsprechende getrackte Rennen in der rechten Ansicht linksklicken.
 scoringSchemeLowPointA82Only=Low Point System; Tie-Break gemäß RRS A8.2 (Letztes Rennen)
 scoringSchemeLowPointA82OnlyDescription=Low Point System; Tie-Break gemäß RRS A8.2; Falls ein Unentschieden nach dem Vergleich der Medaillen-Rennen verbleibt, vergleiche die Wertungen im letzten non-Medal Rennen, dann dem zweitletzten Rennen usw., bis ein Unterschied auftritt. Gestrichene Ergebnisse werden dabei ebenfalls berücksichtigt.
 scoringSchemeLowPointSystemFirstThreeWinsA82Only=Low Point System, Erster mit drei Siegen in Medaillen-Rennen gewinnt; Tie-Break nach A8.2 (Letztes Rennen)

java/com.sap.sailing.landscape.ui/src/com/sap/sailing/landscape/ui/client/i18n/StringMessages_de.properties

@@ -26,7 +26,7 @@ state=Zustand
 createdAt=Erstellt
 mongoEndpoints=MongoDB-Endpunkte
 machineImages=Amazon Maschinen-Abbilder (AMIs)
-doYouReallyWantToRemoveMachineImage=Möchten Sie wirklich das Machinen-Abbild {0} mit der ID {1} in der Region {2} löschen?
+doYouReallyWantToRemoveMachineImage=Möchtest du wirklich das Machinen-Abbild {0} mit der ID {1} in der Region {2} löschen?
 successfullyRemovedMachineImage=Machinen-Abbild {0} mit der ID {1} in der Region {2} erfolgreich gelöscht.
 successfullyUpgradedMachineImage=Machinen-Abbild {0} mit der ID {1} in der Region {2} erfolgreich aktualisiert zu {3}.
 startedImageUpgrade=Aktualisierung des Maschinen-Abbilds {0} mit der ID {1} in der region {2} gestartet. Der Vorgang wird einige Minuten dauern...

java/com.sap.sailing.selenium.test/META-INF/MANIFEST.MF

@@ -11,7 +11,7 @@ Require-Bundle: org.hamcrest;bundle-version="2.2.0",
  junit-platform-commons;bundle-version="1.11.3",
  junit-platform-engine;bundle-version="1.11.3",
  org.opentest4j;bundle-version="1.3.0",
- org.openqa.selenium.osgi;bundle-version="3.13.0",
+ org.openqa.selenium.osgi;bundle-version="4.13.0",
  com.sap.sse.mongodb;bundle-version="1.0.0",
  com.sap.sailing.domain.common,
  com.sap.sse.common,

java/com.sap.sailing.selenium.test/src/com/sap/sailing/selenium/core/SeleniumFieldDecorator.java

@@ -9,7 +9,7 @@
 
 import org.openqa.selenium.WebElement;
 import org.openqa.selenium.interactions.Locatable;
-import org.openqa.selenium.internal.WrapsElement;
+import org.openqa.selenium.WrapsElement;
 import org.openqa.selenium.support.pagefactory.ElementLocator;
 import org.openqa.selenium.support.pagefactory.ElementLocatorFactory;
 import org.openqa.selenium.support.pagefactory.FieldDecorator;

java/com.sap.sailing.selenium.test/src/com/sap/sailing/selenium/core/WindowManager.java

@@ -1,5 +1,6 @@
 package com.sap.sailing.selenium.core;
 
+import java.time.Duration;
 import java.util.HashSet;
 import java.util.Set;
 import java.util.function.BiConsumer;
@@ -80,7 +81,7 @@ public void withExtraWindow(WebDriver driver, BiConsumer<WebDriverWindow, WebDri
         // Open a new window using JavaScript
         ((JavascriptExecutor) driver).executeScript("window.open('about:blank','_blank');");
         // Wait until the new window appears
-        new WebDriverWait(driver, /* seconds */ 5).until(d -> d.getWindowHandles().size() > 1);
+        new WebDriverWait(driver, /* seconds */ Duration.ofSeconds(5)).until(d -> d.getWindowHandles().size() > 1);
         // Identify the new window handle
         String extraWindowHandle = driver.getWindowHandles().stream()
             .filter(handle -> !handle.equals(originalWindowHandle))

java/com.sap.sailing.selenium.test/src/com/sap/sailing/selenium/pages/PageObject.java

@@ -504,12 +504,12 @@ protected void waitForAjaxRequestsExecuted(String category, int numberOfCalls, i
     }
 
     protected void waitForElement(String seleniumId) {
-        WebDriverWait webDriverWait = new WebDriverWait(driver, DEFAULT_LOOKUP_TIMEOUT);
+        WebDriverWait webDriverWait = new WebDriverWait(driver, Duration.ofSeconds(DEFAULT_LOOKUP_TIMEOUT));
         webDriverWait.until(ExpectedConditions.presenceOfElementLocated(new BySeleniumId(seleniumId)));
     }
 
     protected void waitUntil(Function<WebDriver, Boolean> predicate) {
-        WebDriverWait webDriverWait = new WebDriverWait(driver, DEFAULT_LOOKUP_TIMEOUT);
+        WebDriverWait webDriverWait = new WebDriverWait(driver, Duration.ofSeconds(DEFAULT_LOOKUP_TIMEOUT));
         webDriverWait.until(predicate);
     }
 
@@ -518,7 +518,7 @@ protected void waitUntil(BooleanSupplier supplier) {
     }
 
     protected void waitUntilAlertIsPresent() {
-        WebDriverWait webDriverWait = new WebDriverWait(driver, DEFAULT_LOOKUP_TIMEOUT);
+        WebDriverWait webDriverWait = new WebDriverWait(driver, Duration.ofSeconds(DEFAULT_LOOKUP_TIMEOUT));
         webDriverWait.until(ExpectedConditions.alertIsPresent());
     }
 
@@ -614,7 +614,7 @@ protected WebElement goToTab(WebElement tabPanel, String tabName, final String i
         }
         final String expression = expressionFormat.format(new Object[] {tabName});
         WebElement tab = tabPanel.findElement(By.xpath(expression));
-        WebDriverWait waitForTab = new WebDriverWait(driver, 20); // here, wait time is 20 seconds
+        WebDriverWait waitForTab = new WebDriverWait(driver, Duration.ofSeconds(20)); // here, wait time is 20 seconds
         waitForTab.until(ExpectedConditions.visibilityOf(tab)); // this will wait for tab to be visible for 20 seconds
         tab.click();
         return waitForWebElement(tabPanel, id);      
@@ -676,7 +676,7 @@ protected void waitForAlertContainingMessageAndAccept(int timeoutInSeconds, Stri
      * Waits for an alert box to appear. If no alert shows up, an Exception is thrown.
      */
     protected Alert waitForAlert(int timeoutInSeconds) {
-        return new WebDriverWait(driver, timeoutInSeconds).until(ExpectedConditions.alertIsPresent());
+        return new WebDriverWait(driver, Duration.ofSeconds(timeoutInSeconds)).until(ExpectedConditions.alertIsPresent());
     }
 
     protected void waitForAlertAndAccept(String expectedMessage) {
@@ -688,7 +688,7 @@ protected void waitForAlertAndAccept(String expectedMessage) {
      * Exception is thrown.
      */
     protected void waitForAlertAndAccept(int timeoutInSeconds, String expectedMessageRegexp) {
-        final Alert expectedAlert = new WebDriverWait(driver, timeoutInSeconds).until(ExpectedConditions.alertIsPresent());
+        final Alert expectedAlert = new WebDriverWait(driver, Duration.ofSeconds(timeoutInSeconds)).until(ExpectedConditions.alertIsPresent());
         assertTrue(expectedAlert.getText().matches(expectedMessageRegexp));
         expectedAlert.accept();
     }
@@ -711,7 +711,7 @@ protected void waitForNotificationAndDismiss(String expectedNotificationMessage)
      * Waits for an notification to appear and dismisses the notification by clicking on it. If no notification shows up, an Exception is thrown.
      */
     protected void waitForNotificationAndDismiss(int timeoutInSeconds, String expectedNotificationMessage) {
-        WebDriverWait wait = new WebDriverWait(driver, timeoutInSeconds);
+        WebDriverWait wait = new WebDriverWait(driver, Duration.ofSeconds(timeoutInSeconds));
         wait.until(new Function<WebDriver, Boolean>() {
             @Override
             public Boolean apply(WebDriver t) {

java/com.sap.sailing.selenium.test/src/com/sap/sailing/selenium/pages/adminconsole/leaderboard/LeaderboardDetailsPanelPO.java

@@ -1,5 +1,6 @@
 package com.sap.sailing.selenium.pages.adminconsole.leaderboard;
 
+import java.time.Duration;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
@@ -172,7 +173,7 @@ public void refreshRaceLog(RaceDescriptor race) {
     }
 
     public void linkRace(RaceDescriptor race, TrackedRaceDescriptor tracking) {
-        WebDriverWait webDriverWait = new WebDriverWait(driver, DEFAULT_LOOKUP_TIMEOUT);
+        WebDriverWait webDriverWait = new WebDriverWait(driver, Duration.ofSeconds(DEFAULT_LOOKUP_TIMEOUT));
         DataEntryPO raceRow = webDriverWait.until(new Function<WebDriver, DataEntryPO>() {
             @Override
             public DataEntryPO apply(WebDriver t) {

java/com.sap.sailing.selenium.test/src/com/sap/sailing/selenium/pages/adminconsole/regatta/RegattaStructureManagementPanelPO.java

@@ -1,5 +1,6 @@
 package com.sap.sailing.selenium.pages.adminconsole.regatta;
 
+import java.time.Duration;
 import java.util.function.Function;
 
 import org.openqa.selenium.WebDriver;
@@ -92,7 +93,7 @@ public RegattaDetailsCompositePO getRegattaDetails(RegattaDescriptor regatta) {
     }
 
     public RegattaDetailsCompositePO getRegattaDetails() {
-        return new WebDriverWait(driver, DEFAULT_WAIT_TIMEOUT_SECONDS).until(new Function<WebDriver, RegattaDetailsCompositePO>() {
+        return new WebDriverWait(driver, Duration.ofSeconds(DEFAULT_WAIT_TIMEOUT_SECONDS)).until(new Function<WebDriver, RegattaDetailsCompositePO>() {
             @Override
             public RegattaDetailsCompositePO apply(WebDriver t) {
                 if (regattaDetails.isDisplayed()) {

java/com.sap.sailing.selenium.test/src/com/sap/sailing/selenium/pages/adminconsole/usergroups/UserGroupRoleDefinitionPanelPO.java

@@ -1,7 +1,7 @@
 package com.sap.sailing.selenium.pages.adminconsole.usergroups;
 
 import org.openqa.selenium.By.ByName;
-import org.openqa.selenium.ElementNotSelectableException;
+import org.openqa.selenium.ElementNotInteractableException;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.WebElement;
 
@@ -71,15 +71,15 @@ public void enterNewRoleName(String rolename) {
 
     public void clickAddButtonOrThrow() {
         if (!addRoleButton.isEnabled()) {
-            throw new ElementNotSelectableException("Add Button was disabled");
+            throw new ElementNotInteractableException("Add Button was disabled");
         } else {
             addRoleButton.click();
         }
     }
 
     public void clickAddButtonAndExpectPermissionError() {
         if (!addRoleButton.isEnabled()) {
-            throw new ElementNotSelectableException("Add Button was disabled");
+            throw new ElementNotInteractableException("Add Button was disabled");
         } else {
             addRoleButton.click();
         }