Collect checked entry as the first step, fixes #1700

OtterBrowser · Jan 21, 2022 · a3d65b6 · a3d65b6
1 parent df36372
commit a3d65b6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/ui/ContentFiltersViewWidget.cpp b/src/ui/ContentFiltersViewWidget.cpp
@@ -1,6 +1,6 @@
 /**************************************************************************
 * Otter Browser: Web browser controlled by the user, not vice-versa.
-* Copyright (C) 2013 - 2020 Michal Dutkiewicz aka Emdek <[email protected]>
+* Copyright (C) 2013 - 2022 Michal Dutkiewicz aka Emdek <[email protected]>
 * Copyright (C) 2014 - 2017 Jan Bajer aka bajasoft <[email protected]>
 *
 * This program is free software: you can redistribute it and/or modify
@@ -852,6 +852,11 @@ void ContentFiltersViewWidget::save()
 		{
 			const QModelIndex entryIndex(getIndex(j, 0, categoryIndex));
 
+			if (entryIndex.data(Qt::CheckStateRole).toInt() == Qt::Checked)
+			{
+				profiles.append(entryIndex.data(NameRole).toString());
+			}
+
 			if (!entryIndex.data(IsModifiedRole).toBool())
 			{
 				continue;
@@ -907,11 +912,6 @@ void ContentFiltersViewWidget::save()
 			}
 
 			m_model->setData(entryIndex, false, IsModifiedRole);
-
-			if (entryIndex.data(Qt::CheckStateRole).toBool())
-			{
-				profiles.append(profileSummary.name);
-			}
 		}
 	}
 

diff --git a/src/ui/ContentFiltersViewWidget.h b/src/ui/ContentFiltersViewWidget.h
@@ -1,6 +1,6 @@
 /**************************************************************************
 * Otter Browser: Web browser controlled by the user, not vice-versa.
-* Copyright (C) 2013 - 2020 Michal Dutkiewicz aka Emdek <[email protected]>
+* Copyright (C) 2013 - 2022 Michal Dutkiewicz aka Emdek <[email protected]>
 * Copyright (C) 2014 - 2017 Jan Bajer aka bajasoft <[email protected]>
 *
 * This program is free software: you can redistribute it and/or modify