fix: always initialize course info after fetching (#3907)

lib/pangea/activity_sessions/activity_session_chat/activity_finished_status_message.dart

@@ -56,11 +56,6 @@ class ActivityFinishedStatusMessage extends StatelessWidget {
       courseParent!.coursePlan!.uuid,
     );
 
-    if (coursePlan == null) {
-      throw L10n.of(context).noCourseFound;
-    }
-
-    await coursePlan.init();
     final activityId = controller.room.activityPlan!.activityId;
     final topicId = coursePlan.topicID(activityId);
     if (topicId == null) {

lib/pangea/course_plans/course_plan_builder.dart

@@ -61,14 +61,10 @@ class CoursePlanController extends State<CoursePlanBuilder> {
       });
 
       course = await CoursePlansRepo.get(widget.courseId!);
-      if (course == null) {
-        widget.onNotFound?.call();
-      } else {
-        await course!.init();
-      }
 
       widget.onLoaded?.call(course!);
     } catch (e) {
+      widget.onNotFound?.call();
       error = e;
     } finally {
       setState(() {

lib/pangea/course_plans/course_plans_repo.dart

@@ -91,7 +91,7 @@ class CoursePlansRepo {
     await _courseStorage.write(_searchKey(filter), jsonList);
   }
 
-  static Future<CoursePlanModel?> get(String id) async {
+  static Future<CoursePlanModel> get(String id) async {
     final cached = _getCached(id);
     if (cached != null) {
       return cached;
@@ -113,6 +113,7 @@ class CoursePlansRepo {
 
       final coursePlan = cmsCoursePlan.toCoursePlanModel();
       await _setCached(coursePlan);
+      await coursePlan.init();
       completer.complete(coursePlan);
       return coursePlan;
     } catch (e) {
@@ -194,6 +195,8 @@ class CoursePlansRepo {
       coursePlans,
     );
 
+    final futures = coursePlans.map((c) => c.init());
+    await Future.wait(futures);
     return coursePlans;
   }
 }