fix internal scoregrabby function

src/Profile.cpp

@@ -2408,17 +2408,16 @@ void Profile::GetScoresByKey(vector<SongID>& songids, vector<StepsID>& stepsids,
 	}
 }
 
-// new function that uses the chartkey indexed map, wait wtf is going on here -mina
 vector<HighScore> Profile::GetScoresByKey(RString ck) {
-	auto& songids = SONGMAN->SongIDsByChartkey[ck];
-	auto& stepsids = SONGMAN->StepsIDsByChartkey[ck];
 	vector<HighScore> o;
+	if (!HighScoresByChartKey.count(ck))
+		return o;
 
-	for (int i = 0; i < songids.size(); ++i) {
-		auto& hsfas = m_SongHighScores[songids[i]].m_StepsHighScores;
-		vector<HighScore>& scores = hsfas[stepsids[i]].hsl.vHighScores;
-		for (int ii = 0; ii < scores.size(); ++ii) {			
-			o.emplace_back(scores[ii]);
+	auto &hsrm = HighScoresByChartKey.at(ck);
+	FOREACHM(float, vector<HighScore>, hsrm, zz) {
+		auto &hsv = zz->second;
+		for (size_t ii = 0; ii < hsv.size(); ii++) {
+			o.emplace_back(hsv[ii]);
 		}
 	}
 

src/Profile.h

@@ -270,7 +270,7 @@ class Profile
 	void AddToFavorites(RString ck) { FavoritedCharts.emplace_back(ck); }
 	void RemoveFromFavorites(RString ck);
 
-	// doesn't work properly yet - mina
+	// uses the keyed scoremap -mina
 	vector<HighScore> GetScoresByKey(RString ck);
 
 	// Vector for now, we can make this more efficient later