Merge pull request #294 from czqoocavatsim/JoshuaBranch

Add Controller to HomePage
czqoocavatsim · Aug 16, 2024 · c4a3b38 · c4a3b38
2 parents 8f2b870 + 22edca1
commit c4a3b38
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/app/Http/Controllers/Training/InstructingController.php b/app/Http/Controllers/Training/InstructingController.php
@@ -4,6 +4,7 @@
 
 use App\Http\Controllers\Controller;
 use App\Models\Roster\RosterMember;
+use App\Models\News\HomeNewControllerCert;
 use App\Models\Training\Instructing\Instructors\Instructor;
 use App\Models\Training\Instructing\Links\InstructorStudentAssignment;
 use App\Models\Training\Instructing\Links\StudentStatusLabelLink;
@@ -479,8 +480,16 @@ public function certifyStudent($cid)
         $link->save();
 
         // Unassign Instructor from Student
-        $instructor_id = InstructorStudentAssignment::where('student_id', $student->id)->firstOrFail();
-        $instructor_id->delete();
+        $instructor_link = InstructorStudentAssignment::where('student_id', $cid);
+        $instructor_link->delete();
+
+        // Create new certification (for home page)
+        $controller_cert = new HomeNewControllerCert([
+            'controller_id' => $cid,
+            'user_id' => $cid,
+            'timestamp' => Carbon::now(),
+        ]);
+        $controller_cert->save();
 
         // Update Thread Tag to match site
         $discord = new DiscordClient();

diff --git a/app/Models/News/HomeNewControllerCert.php b/app/Models/News/HomeNewControllerCert.php
@@ -28,6 +28,10 @@
  */
 class HomeNewControllerCert extends Model
 {
+    protected $fillable = [
+        'controller_id', 'user_id', 'timestamp',
+    ];
+
     public function controller()
     {
         return $this->belongsTo(User::class, 'controller_id');