From bcabf11a318aa345c6280d68b88b51ebeefbba23 Mon Sep 17 00:00:00 2001 From: Anatoliy Tomilov Date: Mon, 14 Dec 2020 15:50:26 +0500 Subject: [PATCH] [bookmarks][looonely] Do not force visibility to false for categories that does not have visible constituents. MAPSME-15349 --- map/bookmark_manager.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/map/bookmark_manager.cpp b/map/bookmark_manager.cpp index 517b33bd82..1a4f52bdca 100644 --- a/map/bookmark_manager.cpp +++ b/map/bookmark_manager.cpp @@ -4131,19 +4131,14 @@ void BookmarkManager::MarksChangesTracker::InferVisibility(BookmarkCategory * co visibility.emplace(compilationId); } auto const groupId = group->GetID(); - bool hasUserMarksOrDanglingBookmarks = false; for (kml::MarkId const userMark : m_bmManager->GetUserMarkIds(groupId)) { if (!m_bmManager->IsBookmark(userMark)) - { - hasUserMarksOrDanglingBookmarks = true; continue; - } Bookmark * const bookmark = m_bmManager->GetBookmarkForEdit(userMark); bool isVisible = false; if (bookmark->GetCompilations().empty()) { - hasUserMarksOrDanglingBookmarks = true; // Bookmarks that not belong to any compilation have to be visible. // They can be hidden only by changing parental BookmarkCategory visibility to false. isVisible = true; @@ -4161,11 +4156,6 @@ void BookmarkManager::MarksChangesTracker::InferVisibility(BookmarkCategory * co } bookmark->SetIsVisible(isVisible); } - if (visibility.empty() && m_bmManager->GetTrackIds(groupId).empty() && - !hasUserMarksOrDanglingBookmarks) - { - group->SetIsVisible(false); - } } void BookmarkManager::MarksChangesTracker::OnAttachBookmark(kml::MarkId markId,