opt:favorite global data structure adjust

xiaoyifang · Feb 19, 2025 · 42161e2 · 42161e2
1 parent 7f9a23f
commit 42161e2
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/common/globalbroadcaster.hh b/src/common/globalbroadcaster.hh
@@ -39,6 +39,7 @@ public:
   QString translateLineText{};
   //hold the dictionary id;
   QSet< QString > collapsedDicts;
+  //key hold the headword,  value holds the folders.
   QMap< QString, QSet< QString > > folderFavoritesMap;
   QMap< unsigned, QString > groupFolderMap;
   PronounceEngine pronounce_engine;

diff --git a/src/ui/favoritespanewidget.cc b/src/ui/favoritespanewidget.cc
@@ -274,12 +274,12 @@ void FavoritesPaneWidget::clearAllItems()
 void FavoritesPaneWidget::addHeadword( QString const & path, QString const & headword )
 {
   m_favoritesModel->addNewHeadword( path, headword );
-  GlobalBroadcaster::instance()->folderFavoritesMap[ path ].insert( headword );
+  GlobalBroadcaster::instance()->folderFavoritesMap[ headword ].insert( path );
 }
 
 bool FavoritesPaneWidget::removeHeadword( QString const & path, QString const & headword )
 {
-  GlobalBroadcaster::instance()->folderFavoritesMap[ path ].remove( headword );
+  GlobalBroadcaster::instance()->folderFavoritesMap[ headword ].remove( path );
 
   return m_favoritesModel->removeHeadword( path, headword );
 }
@@ -761,7 +761,7 @@ void FavoritesModel::addFolder( TreeItem * parent, QDomNode & node )
       }
       parent->appendChild( new TreeItem( word, parent, TreeItem::Word ) );
 
-      GlobalBroadcaster::instance()->folderFavoritesMap[ parent->data().toString() ].insert( word );
+      GlobalBroadcaster::instance()->folderFavoritesMap[ word ].insert( parent->data().toString() );
     }
   }
   dirty = true;

diff --git a/src/ui/scanpopup.cc b/src/ui/scanpopup.cc
@@ -1231,7 +1231,7 @@ bool ScanPopup::isWordPresentedInFavorites( QString const & word, unsigned group
 {
   QString folder = GlobalBroadcaster::instance()->groupFolderMap[ groupId ];
 
-  return GlobalBroadcaster::instance()->folderFavoritesMap[ folder ].contains( word );
+  return GlobalBroadcaster::instance()->folderFavoritesMap[ word ].contains( folder );
 }
 
 #ifdef HAVE_X11