Update ktdart to 0.6.0

core/lib/src/networking/tmdb_api.dart

@@ -63,19 +63,15 @@ class TMDBApi {
   }
 
   KtList<Actor> _parseActorAvatars(List<Map<String, dynamic>> movieCast) {
-    final actorsWithAvatars = mutableListOf<Actor>();
-
-    movieCast.forEach((Map<String, dynamic> castMember) {
+    return listFrom(movieCast).map((castMember) {
       String pp = castMember['profile_path'];
       final profilePath =
           pp != null ? 'https://image.tmdb.org/t/p/w200$pp' : null;
 
-      actorsWithAvatars.add(Actor(
+      return Actor(
         name: castMember['name'],
         avatarUrl: profilePath,
-      ));
+      );
     });
-
-    return actorsWithAvatars;
   }
 }

core/lib/src/redux/actor/actor_reducer.dart

@@ -14,22 +14,19 @@ KtMap<String, Actor> actorReducer(KtMap<String, Actor> state, dynamic action) {
 
 KtMap<String, Actor> _updateActors(
     KtMap<String, Actor> state, ActorsUpdatedAction action) {
-  final actors = state.toMutableMap();
-  action.actors.forEach((actor) {
-    actors.putIfAbsent(actor.name, Actor(name: actor.name));
-  });
-  return actors.toMap();
+  final updated = action.actors.associateBy((it) => it.name);
+  return updated.plus(state);
 }
 
 KtMap<String, Actor> _updateActorAvatars(
     KtMap<String, Actor> state, ReceivedActorAvatarsAction action) {
-  final actorsWithAvatars = state.toMutableMap();
-  action.actors.forEach((actor) {
-    actorsWithAvatars[actor.name] = Actor(
-      name: actor.name,
-      avatarUrl: actor.avatarUrl,
-    );
-  });
-
-  return actorsWithAvatars.toMap();
+  final actors = state.toMutableMap();
+  for (final actor in action.actors.iter) {
+    final current = actors[actor.name];
+    if (current == null) {
+      continue;
+    }
+    actors.put(actor.name, actor);
+  }
+  return actors.toMap();
 }

core/pubspec.yaml

@@ -14,7 +14,7 @@ dependencies:
   http: ^0.12.0
   reselect: ^0.4.0
   key_value_store: ^1.0.0
-  kt_dart: ^0.5.0
+  kt_dart: ^0.6.0
 
 dev_dependencies:
   test: ^1.3.0

core/test/redux/actor_reducer_test.dart

@@ -11,7 +11,7 @@ void main() {
         'when called with ActorsUpdatedAction, should not modify existing actors',
         () {
       final state = AppState.initial().copyWith(
-        actorsByName: mapFrom<String, Actor>({
+        actorsByName: mapFrom({
           'Seth Ladd': Actor(
             name: 'Seth Ladd',
             avatarUrl: 'https://seths-avatar-url',
@@ -34,7 +34,7 @@ void main() {
 
       expect(
         reducedState.actorsByName,
-        mapFrom<String, Actor>({
+        mapFrom({
           'Seth Ladd': Actor(
             name: 'Seth Ladd',
             avatarUrl: 'https://seths-avatar-url',
@@ -55,7 +55,7 @@ void main() {
         'when called with ReceivedActorAvatarsAction, should add urls for actors',
         () {
       final state = AppState.initial().copyWith(
-        actorsByName: mapFrom<String, Actor>({
+        actorsByName: mapFrom({
           'Seth Ladd': Actor(name: 'Seth Ladd', avatarUrl: null),
           'Eric Seidel': Actor(name: 'Eric Seidel', avatarUrl: null),
         }),
@@ -71,7 +71,7 @@ void main() {
 
       expect(
         reducedState.actorsByName,
-        mapFrom<String, Actor>({
+        mapFrom({
           'Seth Ladd': Actor(
             name: 'Seth Ladd',
             avatarUrl: 'https://seths-avatar-url',

mobile/lib/ui/event_details/event_gallery_grid.dart

@@ -59,7 +59,7 @@ class _Grid extends StatelessWidget {
       ),
       children: event.galleryImages.map((image) {
         return _GalleryImage(image.location);
-      }).list,
+      }).asList(),
     );
   }
 }

mobile/lib/ui/showtimes/showtime_date_selector.dart

@@ -26,7 +26,7 @@ class ShowtimeDateSelector extends StatelessWidget {
           mainAxisAlignment: MainAxisAlignment.spaceAround,
           children: viewModel.dates.map((date) {
             return _DateSelectorItem(date, viewModel);
-          }).list,
+          }).asList(),
         ),
       ),
     );

web/lib/src/common/theater_selector/theater_selector_dropdown_menu_component.dart

@@ -1,5 +1,6 @@
 import 'package:angular/angular.dart';
 import 'package:core/core.dart';
+import 'package:kt_dart/collection.dart';
 import 'package:redux/redux.dart';
 import 'package:web/src/common/theater_selector/theater_dropdown_controller.dart';
 
@@ -18,7 +19,7 @@ class TheaterSelectorDropdownMenuComponent {
 
   TheaterListViewModel get _viewModel => TheaterListViewModel.fromStore(_store);
   Theater get selectedTheater => _viewModel.currentTheater;
-  List<Theater> get theaters => _viewModel.theaters.list;
+  KtList<Theater> get theaters => _viewModel.theaters;
 
   bool get focusTrapVisible => isOpen;
   bool isOpen = false;

web/lib/src/common/theater_selector/theater_selector_dropdown_menu_component.html

@@ -6,7 +6,7 @@
 <div class="menu" [class.opened]="isOpen" [style.background]="background">
   <div class="item"
        [class.selected]="theater == selectedTheater"
-       *ngFor="let theater of theaters"
+       *ngFor="let theater of theaters.asList()"
        (click)="onTheaterClicked(theater)">
     {{ theater.name }}
   </div>

web/lib/src/event_details/event_details_component.html

@@ -32,7 +32,7 @@ <h3>{{ messages.storyline }}</h3>
   <div class="section white top-shadow actor-section" *ngIf="event.actors.isNotEmpty()">
     <div class="centered-content">
       <h3>{{ messages.cast }}</h3>
-      <actor-scroller [actors]="event.actors.list"></actor-scroller>
+      <actor-scroller [actors]="event.actors.asList()"></actor-scroller>
     </div>
   </div>
 
@@ -41,7 +41,7 @@ <h3>{{ messages.cast }}</h3>
       <h3>{{ messages.gallery }}</h3>
 
       <div class="gallery">
-        <img *ngFor="let galleryImage of event.galleryImages.list" [src]="galleryImage.location"
+        <img *ngFor="let galleryImage of event.galleryImages.asList()" [src]="galleryImage.location"
              [alt]="'A still frame from the movie ' + event.title"/>
       </div>
     </div>

web/lib/src/events/events_page_component.html

@@ -9,7 +9,7 @@ <h3>{{ eventTypeTitle }}</h3>
                 [errorMessage]="messages.errorLoadingEvents"
                 (actionButtonClicked)="viewModel.refreshEvents()">
     <div class="grid-container">
-      <event-poster *ngFor="let event of viewModel.events.list"
+      <event-poster *ngFor="let event of viewModel.events.asList()"
                     [event]="event"
                     [isComingSoon]="isDisplayingComingSoonMovies"
                     (click)="openEventDetails(event)">

web/lib/src/showtimes/showtimes_page_component.html

@@ -4,15 +4,15 @@ <h3>{{ messages.showtimes }}</h3>
     <theater-selector></theater-selector>
   </div>
 
-  <date-selector [dates]="viewModel.dates.list"
+  <date-selector [dates]="viewModel.dates.asList()"
                  [selectedDate]="viewModel.selectedDate"
                  [newDateSelected]="viewModel.changeCurrentDate"></date-selector>
 
   <loading-view
       [status]="viewModel.status"
       [contentEmpty]="shows.isEmpty()"
       (actionButtonClicked)="viewModel.refreshShowtimes()">
-    <showtime-item *ngFor="let show of shows.list"
+    <showtime-item *ngFor="let show of shows.asList()"
                    [show]="show"
                    (click)="openShowDetails(show)"></showtime-item>
   </loading-view>