fix: fixed empty builds errors

orteney · Mar 2, 2024 · 0fdfc96 · 0fdfc96
1 parent ce41952
commit 0fdfc96
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 16 deletions.
diff --git a/lib/presentation/champion_pick/bloc/champion_pick_bloc.dart b/lib/presentation/champion_pick/bloc/champion_pick_bloc.dart
@@ -117,7 +117,7 @@ class ChampionPickBloc extends Bloc<ChampionPickEvent, ChampionPickState> with E
       role: builds.role,
       builds: builds.builds,
       selectedBuildIndex: 0,
-      selectedPerkStyle: PerkStyle.fromId(builds.builds[0].runes.primaryPath),
+      selectedPerkStyle: builds.builds.isEmpty ? null : PerkStyle.fromId(builds.builds[0].runes.primaryPath),
       runesImages: _getPerksImages(builds.builds),
       itemImages: _getItemImages(builds.builds),
       summonerSpellImages: _getSummonerSpellImages(builds.builds),
@@ -143,7 +143,7 @@ class ChampionPickBloc extends Bloc<ChampionPickEvent, ChampionPickState> with E
       role: builds.role,
       builds: builds.builds,
       selectedBuildIndex: 0,
-      selectedPerkStyle: PerkStyle.fromId(builds.builds[0].runes.primaryPath),
+      selectedPerkStyle: builds.builds.isEmpty ? null : PerkStyle.fromId(builds.builds[0].runes.primaryPath),
       runesImages: _getPerksImages(builds.builds),
       itemImages: _getItemImages(builds.builds),
       summonerSpellImages: _getSummonerSpellImages(builds.builds),

diff --git a/lib/presentation/champion_pick/bloc/champion_pick_state.dart b/lib/presentation/champion_pick/bloc/champion_pick_state.dart
@@ -12,7 +12,7 @@ class ActiveChampionPickState extends ChampionPickState with EquatableMixin {
   final Role? role;
   final List<BuildInfo> builds;
   final int selectedBuildIndex;
-  final PerkStyle selectedPerkStyle;
+  final PerkStyle? selectedPerkStyle;
   final Map<int, LcuImage> runesImages;
   final Map<int, LcuImage> itemImages;
   final Map<int, LcuImage> summonerSpellImages;

diff --git a/lib/presentation/champion_pick/champion_pick_page.dart b/lib/presentation/champion_pick/champion_pick_page.dart
@@ -123,7 +123,7 @@ class _ActiveChampionPickWidget extends StatelessWidget {
                           child: _BuildTab(
                             keyPerkIcon: state.runesImages[build.keystoneId]!,
                             championBuild: build,
-                            color: index == state.selectedBuildIndex ? state.selectedPerkStyle.color : null,
+                            color: index == state.selectedBuildIndex ? state.selectedPerkStyle?.color : null,
                             onTap: () =>
                                 context.read<ChampionPickBloc>().add(TapAvailableBuildTabChampionPickEvent(index)),
                           ),
@@ -132,18 +132,19 @@ class _ActiveChampionPickWidget extends StatelessWidget {
                     ),
                   ),
                   const SizedBox(height: 8),
-                  LayoutBuilder(
-                    builder: (context, constraints) {
-                      return BuildDetails(
-                        singleColumn: constraints.maxWidth < 720,
-                        championBuild: state.builds[state.selectedBuildIndex],
-                        runesImages: state.runesImages,
-                        itemImages: state.itemImages,
-                        summonerSpellImages: state.summonerSpellImages,
-                        color: state.selectedPerkStyle.color,
-                      );
-                    },
-                  ),
+                  if (state.builds.isNotEmpty)
+                    LayoutBuilder(
+                      builder: (context, constraints) {
+                        return BuildDetails(
+                          singleColumn: constraints.maxWidth < 720,
+                          championBuild: state.builds[state.selectedBuildIndex],
+                          runesImages: state.runesImages,
+                          itemImages: state.itemImages,
+                          summonerSpellImages: state.summonerSpellImages,
+                          color: state.selectedPerkStyle?.color ?? Colors.transparent,
+                        );
+                      },
+                    ),
                 ]
               ],
             ),