Commit 2673f996 by Antti Tönkyrä

Remove commented core & fix logic

- biggestCard = card; card must be enabled in order to enter this if branch thus setEnabled is extraneous in this case
1 parent 4c7bb161
...@@ -144,24 +144,15 @@ public class CardTemplateBean implements CardTemplateBeanLocal { ...@@ -144,24 +144,15 @@ public class CardTemplateBean implements CardTemplateBeanLocal {
List<PrintedCard> myCards = printedcardfacade.getCards(user); List<PrintedCard> myCards = printedcardfacade.getCards(user);
PrintedCard biggestCard = null; PrintedCard biggestCard = null;
// PrintedCard smallestCard = null;
for (PrintedCard card : myCards) { for (PrintedCard card : myCards) {
// logger.info("Checking card {}", card);
if (card.getEnabled()) { if (card.getEnabled()) {
if (biggestCard == null || biggestCard.getTemplate().getPower() < card.getTemplate().getPower()) { if (biggestCard == null || biggestCard.getTemplate().getPower() < card.getTemplate().getPower()) {
// The biggest card should be the only one enabled. // The biggest card should be the only one enabled.
if (biggestCard != null) { if (biggestCard != null) {
biggestCard.setEnabled(false); biggestCard.setEnabled(false);
} }
biggestCard = card; biggestCard = card;
biggestCard.setEnabled(true);
} }
// if (smallestCard == null ||
// smallestCard.getTemplate().getPower() >
// card.getTemplate().getPower()) {
// smallestCard = card;
// }
} }
} }
......
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!