funman300/Ferrous-Solitaire
1
1
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases 28 Wiki Activity

fix(engine): profile achievement count derived from ALL_ACHIEVEMENTS

Browse Source 
Hardcoded 18 in the profile summary line diverged from the actual count
of 19. Use ALL_ACHIEVEMENTS.len() so the count stays in sync when new
achievements are added.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
funman300
2026-05-12 17:54:59 -07:00
parent 6beb9f68ac
commit 46c3bf4bb2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
solitaire_engine/src/profile_plugin.rs
+2 -2
View File
@@ -8,7 +8,7 @@ use bevy::input::mouse::{MouseScrollUnit, MouseWheel};
use bevy::input::ButtonInput; use bevy::input::ButtonInput;
use bevy::prelude::*; use bevy::prelude::*;
use chrono::{Duration, Local, NaiveDate}; use chrono::{Duration, Local, NaiveDate};
use solitaire_core::achievement::achievement_by_id; use solitaire_core::achievement::{achievement_by_id, ALL_ACHIEVEMENTS};
use solitaire_data::SyncBackend; use solitaire_data::SyncBackend;
use crate::achievement_plugin::AchievementsResource; use crate::achievement_plugin::AchievementsResource;
@@ -323,7 +323,7 @@ fn spawn_profile_screen(
let records = &ar.0; let records = &ar.0;
let unlocked_count = records.iter().filter(|r| r.unlocked).count(); let unlocked_count = records.iter().filter(|r| r.unlocked).count();
body.spawn(( body.spawn((
Text::new(format!("{unlocked_count} / 18 unlocked")), Text::new(format!("{unlocked_count} / {} unlocked", ALL_ACHIEVEMENTS.len())),
font_row.clone(), font_row.clone(),
TextColor(ACCENT_PRIMARY), TextColor(ACCENT_PRIMARY),
)); ));