From d6f3f876c39ad66a630d26507de3c04227196052 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lukas=20Br=C3=BCbach?= Date: Fri, 13 Jun 2025 01:57:58 +0200 Subject: [PATCH] Debug messages. --- .../card_group_display_widget.cpp | 17 +++++++++++++++-- .../flat_card_group_display_widget.cpp | 1 + .../overlapped_card_group_display_widget.cpp | 1 + .../cards/deck_card_zone_display_widget.cpp | 17 ++++++++++++++++- .../visual_deck_editor_widget.cpp | 2 ++ 5 files changed, 35 insertions(+), 3 deletions(-) diff --git a/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/card_group_display_widget.cpp b/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/card_group_display_widget.cpp index 7af47f79f..b6a4264bc 100644 --- a/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/card_group_display_widget.cpp +++ b/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/card_group_display_widget.cpp @@ -27,7 +27,12 @@ CardGroupDisplayWidget::CardGroupDisplayWidget(QWidget *parent, banner = new BannerWidget(this, cardGroupCategory, Qt::Orientation::Vertical, bannerOpacity); layout->addWidget(banner); - CardGroupDisplayWidget::updateCardDisplays(); + qInfo() << "Tracked index (default CardGroupDisplayWidget): " << cardGroupCategory << " (" << zoneName << ")" + << "_trackedIndex.isValid()=" << _trackedIndex.isValid() + << "row=" << _trackedIndex.row() + << "column=" << _trackedIndex.column() + << "parent(row)=" << _trackedIndex.parent().row() + << "parent(column)=" << _trackedIndex.parent().column(); connect(deckListModel, &QAbstractItemModel::rowsInserted, this, &CardGroupDisplayWidget::onCardAddition); connect(deckListModel, &QAbstractItemModel::rowsRemoved, this, &CardGroupDisplayWidget::onCardRemoval); @@ -58,7 +63,7 @@ QWidget* CardGroupDisplayWidget::constructWidgetForIndex(int rowIndex) void CardGroupDisplayWidget::updateCardDisplays() { - qInfo() << "Constructing Card Display Widgets for Card Group Display widget"; + qInfo() << "Constructing Card Display Widgets for Card Group Display widget for zone: " << cardGroupCategory; qInfo() << deckListModel->data(trackedIndex.sibling(trackedIndex.row(), 1), Qt::EditRole).toString() << " has " << deckListModel->rowCount(trackedIndex.sibling(trackedIndex.row(), 0)) << " entries."; for (int i = 0; i < deckListModel->rowCount(trackedIndex); ++i) { @@ -69,6 +74,13 @@ void CardGroupDisplayWidget::updateCardDisplays() void CardGroupDisplayWidget::onCardAddition(const QModelIndex &parent, int first, int last) { if (parent == trackedIndex) { + qInfo() << "Tracked index (default CardGroupDisplayWidget): " << cardGroupCategory << " (" << zoneName << ")" + << "_trackedIndex.isValid()=" << trackedIndex.isValid() + << "row=" << trackedIndex.row() + << "column=" << trackedIndex.column() + << "parent(row)=" << trackedIndex.parent().row() + << "parent(column)=" << trackedIndex.parent().column(); + qInfo() << "Card addition to " << cardGroupCategory; for (int i = first; i <= last; i++) { insertIntoLayout(constructWidgetForIndex(i), i); } @@ -81,6 +93,7 @@ void CardGroupDisplayWidget::onCardRemoval(const QModelIndex &parent, int first, Q_UNUSED(first); Q_UNUSED(last); if (parent == trackedIndex) { + qInfo() << "Card removal from " << cardGroupCategory; for (const QPersistentModelIndex &idx : indexToWidgetMap.keys()) { if (!idx.isValid()) { removeFromLayout(indexToWidgetMap.value(idx)); diff --git a/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/flat_card_group_display_widget.cpp b/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/flat_card_group_display_widget.cpp index ee8c9d2e5..a3f0fe392 100644 --- a/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/flat_card_group_display_widget.cpp +++ b/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/flat_card_group_display_widget.cpp @@ -31,6 +31,7 @@ FlatCardGroupDisplayWidget::FlatCardGroupDisplayWidget(QWidget *parent, banner->setBuddy(flowWidget); layout->addWidget(flowWidget); + qInfo() << "Initialized a flat CardGroupDisplayWidget"; FlatCardGroupDisplayWidget::updateCardDisplays(); // connect(deckListModel, &DeckListModel::dataChanged, this, &FlatCardGroupDisplayWidget::updateCardDisplays); } diff --git a/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/overlapped_card_group_display_widget.cpp b/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/overlapped_card_group_display_widget.cpp index fbae212d1..4c0687a37 100644 --- a/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/overlapped_card_group_display_widget.cpp +++ b/cockatrice/src/client/ui/widgets/cards/card_group_display_widgets/overlapped_card_group_display_widget.cpp @@ -30,6 +30,7 @@ OverlappedCardGroupDisplayWidget::OverlappedCardGroupDisplayWidget(QWidget *pare banner->setBuddy(overlapWidget); layout->addWidget(overlapWidget); + qInfo() << "Initialized an overlap CardGroupDisplayWidget"; OverlappedCardGroupDisplayWidget::updateCardDisplays(); connect(cardSizeWidget->getSlider(), &QSlider::valueChanged, this, diff --git a/cockatrice/src/client/ui/widgets/cards/deck_card_zone_display_widget.cpp b/cockatrice/src/client/ui/widgets/cards/deck_card_zone_display_widget.cpp index 70c8cd304..d165982f4 100644 --- a/cockatrice/src/client/ui/widgets/cards/deck_card_zone_display_widget.cpp +++ b/cockatrice/src/client/ui/widgets/cards/deck_card_zone_display_widget.cpp @@ -39,12 +39,27 @@ DeckCardZoneDisplayWidget::DeckCardZoneDisplayWidget(QWidget *parent, // connect(deckListModel, &DeckListModel::dataChanged, this, &DeckCardZoneDisplayWidget::decklistDataChanged); connect(deckListModel, &QAbstractItemModel::rowsInserted, this, &DeckCardZoneDisplayWidget::onCategoryAddition); connect(deckListModel, &QAbstractItemModel::rowsRemoved, this, &DeckCardZoneDisplayWidget::onCategoryRemoval); + + qInfo() << "Tracked index:" << zoneName << "_trackedIndex.isValid()=" << _trackedIndex.isValid() + << "row=" << _trackedIndex.row() << "column=" << _trackedIndex.column() + << "parent(row)=" << _trackedIndex.parent().row() << "parent(column)=" << _trackedIndex.parent().column() + << "parent(valid)=" << _trackedIndex.parent().isValid(); } + qInfo() << "Cleaning up invalid card group " << displayWidget->cardGroupCategory << " from " << displayWidget->zoneName; + qInfo() << "Cleaned."; + qInfo() << categoryName << " is already contained in the index to widget map"; + qInfo() << categoryName << " needs a widget"; + qInfo() << "It's getting an overlap widget."; + qInfo() << "Getting a flat widget for christmas"; + qInfo() << "Constructing Group Display Widgets for Zone Display widget"; + qInfo() << deckListModel->data(trackedIndex.sibling(trackedIndex.row(), 1), Qt::EditRole).toString() << " has " + << deckListModel->rowCount(trackedIndex.sibling(trackedIndex.row(), 0)) << " entries."; void DeckCardZoneDisplayWidget::onCategoryAddition(const QModelIndex &parent, int first, int last) { if (parent == trackedIndex) { - qInfo() << deckListModel->data(trackedIndex.sibling(trackedIndex.row(), 1), Qt::EditRole).toString() << " zone thinks it has a new category"; + qInfo() << deckListModel->data(trackedIndex.sibling(trackedIndex.row(), 1), Qt::EditRole).toString() + << " zone thinks it has a new category"; for (int i = first; i <= last; i++) { QPersistentModelIndex index = QPersistentModelIndex(deckListModel->index(i, 0, trackedIndex)); diff --git a/cockatrice/src/client/ui/widgets/visual_deck_editor/visual_deck_editor_widget.cpp b/cockatrice/src/client/ui/widgets/visual_deck_editor/visual_deck_editor_widget.cpp index 333055c8d..b73cdf028 100644 --- a/cockatrice/src/client/ui/widgets/visual_deck_editor/visual_deck_editor_widget.cpp +++ b/cockatrice/src/client/ui/widgets/visual_deck_editor/visual_deck_editor_widget.cpp @@ -198,6 +198,8 @@ void VisualDeckEditorWidget::retranslateUi() displayTypeButton->setToolTip( tr("Change how cards are displayed within zones (i.e. overlapped or fully visible.)")); } + qInfo() << "Cleaning up invalid card zone " << displayWidget->zoneName; + qInfo() << "Cleaned."; void VisualDeckEditorWidget::onCardAddition(const QModelIndex &parent, int first, int last) {