From a981af4e6fd0047bc3e96468db48dd7820dac808 Mon Sep 17 00:00:00 2001 From: Vasil Dimov Date: Mon, 6 Mar 2023 12:49:39 +0100 Subject: gui: use the stored CSubNet entry when unbanning The previous code visualized the `CSubNet` object as string, then parsed that string back to `CSubNet`. This is sub-optimal given that the original `CSubNet` object can be used directly instead. This avoids calling `LookupSubNet()` from the GUI. Co-authored-by: furszy --- src/qt/bantablemodel.cpp | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/qt/bantablemodel.cpp') diff --git a/src/qt/bantablemodel.cpp b/src/qt/bantablemodel.cpp index 4f57cd4457..d68ca1e2ea 100644 --- a/src/qt/bantablemodel.cpp +++ b/src/qt/bantablemodel.cpp @@ -178,3 +178,9 @@ bool BanTableModel::shouldShow() { return priv->size() > 0; } + +bool BanTableModel::unban(const QModelIndex& index) +{ + CCombinedBan* ban{static_cast(index.internalPointer())}; + return ban != nullptr && m_node.unban(ban->subnet); +} -- cgit v1.2.3