From 6bc46e45987da1f0a5c439bd559e575ea0bc6471 Mon Sep 17 00:00:00 2001
From: Jonas Kvinge <jonas@jkvinge.net>
Date: Sat, 20 Aug 2022 16:47:09 +0200
Subject: [PATCH] CollectionModel: Make `separate_albums_by_grouping` optional

---
 src/collection/collectionfilterwidget.cpp | 2 +-
 src/collection/collectionmodel.h          | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/collection/collectionfilterwidget.cpp b/src/collection/collectionfilterwidget.cpp
index cc7cb6dd5..3731572eb 100644
--- a/src/collection/collectionfilterwidget.cpp
+++ b/src/collection/collectionfilterwidget.cpp
@@ -407,7 +407,7 @@ void CollectionFilterWidget::GroupByClicked(QAction *action) {
   }
 
   CollectionModel::Grouping g = action->property("group_by").value<CollectionModel::Grouping>();
-  model_->SetGroupBy(g, false);
+  model_->SetGroupBy(g);
 
 }
 
diff --git a/src/collection/collectionmodel.h b/src/collection/collectionmodel.h
index 3220a82fd..f640171fb 100644
--- a/src/collection/collectionmodel.h
+++ b/src/collection/collectionmodel.h
@@ -192,7 +192,7 @@ class CollectionModel : public SimpleTreeModel<CollectionItem> {
   void ExpandAll(CollectionItem *item = nullptr) const;
 
   const CollectionModel::Grouping GetGroupBy() const { return group_by_; }
-  void SetGroupBy(const CollectionModel::Grouping g, const std::optional<bool> separate_albums_by_grouping);
+  void SetGroupBy(const CollectionModel::Grouping g, const std::optional<bool> separate_albums_by_grouping = std::optional<bool>());
 
   static QString ContainerKey(const GroupBy group_by, const bool separate_albums_by_grouping, const Song &song);