@eeplatform/basic-edu 1.3.11 → 1.3.12

package/dist/index.mjs

@@ -1864,9 +1864,11 @@ var schemaEnrollment = Joi3.object({
   division: Joi3.string().required(),
   // Changed from hex to string to match frontend
   divisionName: Joi3.string().optional().allow("", null),
-  school: Joi3.string().required(),
+  schoolId: Joi3.string().required(),
   // Changed from hex to string to match frontend
-  schoolName: Joi3.string().optional().allow("", null),
+  school: Joi3.string().hex().required(),
+  // Added to match frontend
+  schoolName: Joi3.string().required(),
   // Added to match frontend
   schoolYear: Joi3.string().required(),
   gradeLevel: Joi3.string().valid(...gradeLevels).required(),
@@ -1937,6 +1939,7 @@ function MEnrollment(value) {
     division: value.division,
     divisionName: value.divisionName ?? "",
     school: value.school,
+    schoolId: value.schoolId,
     schoolName: value.schoolName ?? "",
     // Added to match frontend
     schoolYear: value.schoolYear ?? "",
@@ -2078,7 +2081,11 @@ function useEnrollmentRepo() {
       status
     };
     if (school) {
-      query.school = school;
+      try {
+        query.school = new ObjectId4(school);
+      } catch (error) {
+        throw new BadRequestError5("Invalid school ID.");
+      }
     }
     if (schoolYear) {
       query.schoolYear = schoolYear;
@@ -2125,41 +2132,13 @@ function useEnrollmentRepo() {
         });
         return cached;
       }
-      const items = await collection.aggregate([
-        { $match: query },
-        {
-          $lookup: {
-            from: "admin.schools",
-            localField: "school",
-            foreignField: "_id",
-            // School collection still uses ObjectId, we'll need to convert during lookup
-            as: "schoolInfo"
-          }
-        },
-        {
-          $lookup: {
-            from: "admin.regions",
-            localField: "region",
-            foreignField: "_id",
-            // Region collection still uses ObjectId, we'll need to convert during lookup
-            as: "regionInfo"
-          }
-        },
-        {
-          $lookup: {
-            from: "admin.divisions",
-            localField: "division",
-            // Changed from sdo to division to match frontend
-            foreignField: "_id",
-            // Division collection still uses ObjectId, we'll need to convert during lookup
-            as: "divisionInfo"
-            // Changed from sdoInfo to divisionInfo
-          }
-        },
-        { $sort: sort },
-        { $skip: page * limit },
-        { $limit: limit }
-      ]).toArray();
+      const pipeline = [{ $match: query }];
+      if (Object.keys(sort).length) {
+        pipeline.push({ $sort: sort });
+      }
+      pipeline.push({ $skip: page * limit });
+      pipeline.push({ $limit: limit });
+      const items = await collection.aggregate(pipeline).toArray();
       const length = await collection.countDocuments(query);
       const data = paginate2(items, page, limit, length);
       setCache(cacheKey, data, 600).then(() => {
@@ -2478,7 +2457,7 @@ function useEnrollmentService() {
         placeOfBirth: value.learnerInfo.placeOfBirth,
         gradeLevel: value.gradeLevel,
         schoolYear: value.schoolYear,
-        school: value.school
+        school: value.schoolId
       });
       if (applicantInfo.accepted) {
         throw new BadRequestError6(