ambidextrous/gemp-lotr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Performance - reduced number of db queries if draft with the same id starts again (ie. after server restart if one was in progress)

Browse Source
This commit is contained in:
jakub.salavec
2025-04-17 12:35:08 +02:00
parent 50f8af651f
commit 46f98fd144
2 changed files with 11 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
gemp-lotr/gemp-lotr-server/src/main/java/com/gempukku/lotro/collection/CollectionsManager.java
View File

@@ -135,6 +135,16 @@ public class CollectionsManager {
}
}
public void removeFromPlayerCollection(String player, String collectionType, CardCollection cardCollection, String reason) throws SQLException, IOException {
_readWriteLock.writeLock().lock();
try {
removeFromPlayerCollection(_playerDAO.getPlayer(player), collectionType, cardCollection, reason);
} finally {
_readWriteLock.writeLock().unlock();
}
}
private void removeFromPlayerCollection(Player player, String collectionType, CardCollection cardCollection, String reason) {
if (collectionType.contains("+"))
throw new IllegalArgumentException("Invalid collection type: " + collectionType);
@@ -308,10 +318,6 @@ public class CollectionsManager {
}
}
public boolean sellCardInPlayerCollection(String player, CollectionType collectionType, String blueprintId, int currency) throws SQLException, IOException {
return sellCardInPlayerCollection(_playerDAO.getPlayer(player), collectionType, blueprintId, currency);
}
public boolean sellCardInPlayerCollection(Player player, CollectionType collectionType, String blueprintId, int currency) throws SQLException, IOException {
_readWriteLock.writeLock().lock();
try {

6
gemp-lotr/gemp-lotr-server/src/main/java/com/gempukku/lotro/draft3/TableDraftClassic.java
View File

@@ -164,11 +164,7 @@ public class TableDraftClassic implements TableDraft{
if (draftPlayer instanceof DraftBot) {
botCollections.remove(draftPlayer);
} else {
for (CardCollection.Item item : collection.getAll()) {
for (int i = 0; i < item.getCount(); i++) {
collectionsManager.sellCardInPlayerCollection(draftPlayer.getName(), collectionType, item.getBlueprintId(), 0);
}
}
collectionsManager.removeFromPlayerCollection(draftPlayer.getName(), collectionType.getCode(), collection, "Duplicate tournament collection");
}
}
} catch (SQLException | IOException ignore) {