2024-August-09 - 2.7.14854

Fix issue 12823 (EditPanel#sortUnitsToRemove) illegal argument except…

2024-August-09 - 2.7.14853

Fix issue 12826 (UnitSeparator#categorize:213 - java.util.ConcurrentM…

2024-August-05 - 2.7.14852

Fix issues 12777 (#12819)

Rename settings windows button from "Reset" to "Reset to Saved"

SettingsWindow.java

2024-August-04 - 2.7.14851

battleCalculator #1 (#12816)

issue 12688 (battle calc on own territory shows own player as attacker and defender)
Desired:
1. Attacker: Current player always, except when only allied units present (then next enemy)
2. Defender: As enemy to attacker with most units (fallback: whomever is at war and has the next turn)

AttackerAndDefenderSelectorTest.java
- simplification of method getAttackerAndDefender()
- new methods getBestAttacker() and getBestAttacker()
- removed now unused methods getOpponentWithPriorityList, playersAtWarWith, neutralPlayersTowards, getEnemyWithMostUnits

UnitCollection.java
- rename method getPlayersByUnitCount to getPlayersSortedByUnitCount

BattleCalculatorDialog.java
- extract common logic from method addAttackers and addDefenders to new method adjustBattleCalculatorPanel
- extract static attribute modifications from instance method dispose into new static method disposeInstance (especially pack also for adding attacker)

BattleCalculatorPanel.java
- introduce new methods setAttackerWithUnits and setDefenderWithUnits to limit attacker/defender change to own instance
- make setter of attribute attacker/defender private
- rename methods hasAttackingUnitsAdded and hasDefendingUnitsAdded to hasAttackingUnits and hasDefendingUnits
- rename attribute attackerUnitsTotalHitpoints to attackerUnitsTotalHitPoints

2024-August-04 - 2.7.14850

Sort battles cleanup (#12814)

* sort_Battles #1

TripleAFrame.java
- rename variable territoryDetails to territoryDetailsPanel
- rename variable actionButtons to actionButtonsPanel
- rename variable datacopy to gameDataCopy
- fix typos e.g. savegame or gamedata

Rename class ActionButtons to ActionButtonsPanel:
- ForumPosterComponent.java
- ActionPanel.java

BattleCalculatorDialog.java
- renamed Getter TripleAFrame.java.TerritoryPanel

TripleAPlayer.java
- fix typos savegame and gamedata
- rename variable battlesite to battleTerritory

Replace typos 'savegame' with 'save game'
- ActionButtonsPanel.java
- GameSequence.java
- HeadlessGameServer.java
- IEmailSender.java
- NodeBbForumPoster.java
- ServerLauncher.java
- TripleAFrame.java
- TripleAPlayer.java
- UpdateChecks.java

* sort_Battles #2

BattleTracker.java
- rename variables (e.g. territory collections) with same name as member attribute
- replace Map loop over keys with over entrySet
- rework methods getPendingBattles to consistently use CollectionUtils.getMatches instead of stream()
- rework methods getPendingBattle to consistently use stream() instead of loops
- make method getPendingBattleSites(bombing) private and usages more clear with new methods getPendingBattleSitesWithBombing() and getPendingBattleSitesWithoutBombing()
- replace lambdas with method
- typo 's instead of s

replace with new method BattleTracker.getPendingBattleSitesWithoutBombing()
- BattleDelegate.java
- MoveDelegateTest.java
- MustFightBattleTest.java
- RevisedTest.java
- WW2V3Year41Test.java
- WW2V3Year42Test.java

* sort_Battles #3
Keep BattleListing for passing to other methods and let method/variable names indicate BattleListing

ActionButtonsPanel.java
- let method changeToBattle use BattleListing instead of battles map

BattleTracker.java
- rename method getPendingBattleSites to getBattleListingFromPendingBattles

BattlePanel.java
- rename attribute battles to battleListing
- remove isBomb variable in method addBattleActions and subsequent methods as value can be determined by other parameter (BattleType)

IBattleDelegate.java
- rename method getBattles to getBattleListing

TripleAFrame.java
- let method getBattle use BattleListing instead of battles map

Affected by renaming method (getBattles to getBattleListing; getPendingBattleSites to getBattleListingFromPendingBattles)
- AbstractAi.java
- AirThatCantLandUtilTest.java
- BattleDelegate.java
- ProSimulateTurnUtils.java
- TripleAPlayer.java
- WW2V3Year41Test.java

* sort_Battles #4
Move BattleListing map building from BattleTracker into BattleListing

BattleTracker
- move logic of method getBattleListingFromPendingBattles() to constructor of BattleListing

BattleListing.java
- rename attribute battles to battlesMap
- change constructor to include logic of method BattleTracker.getBattleListingFromPendingBattles()
- extract new method getBattlesWith(predicate)

Subsequent changes for Getter on BattleListing.battlesMap
- AbstractAi.java
- AirThatCantLandUtilTest.java
- ProSimulateTurnUtils.java
- WW2V3Year41Test.java

* sort_Battles #5
fix RemoteActionCodeTest due to renaming IBattleDelegate.getBattles to getBattleListing
- required-op-codes.csv

2024-July-30 - 2.7.14848

Lobby websocket - fix disconnect - increase ping frequency (#12806)

Lobby was upgraded recently to be on a newer version of the
server platform. Seemingly the websocket timeout has dropped
to 30 seconds. The previous keep-alive of 45 seconds was
not enough.

Changing the value to 10 seconds allows for 3 failed tries before
a disconnect.

2024-July-30 - 2.7.14846

Publish feign-common as lib dependency (#12803)

2024-July-29 - 2.7.14845

Publish more dependencies (#12802)

Need more (all) of the transitive dependencies now that we no longer
publish a shadow jar

2024-July-29 - 2.7.14841

Jar publishing: also publish 'domain-data' jar (#12798)

Now that we no longer publish a shadow jar of 'http-client/lobby',
we need to provide dependencies as well.

2024-July-29 - 2.7.14840

Lobby-client Jar build config: properly update gradle to publish jar …