Просмотр исходного кода

Merge pull request #2132 from visuddhinanda/laravel

🐛 $request->get('course') -> $courseId
visuddhinanda 1 год назад
Родитель
Сommit
1058360dc8
2 измененных файлов с 3 добавлено и 2 удалено
  1. 2 1
      app/Http/Api/CourseApi.php
  2. 1 1
      app/Http/Controllers/TagMapController.php

+ 2 - 1
app/Http/Api/CourseApi.php

@@ -22,7 +22,8 @@ class CourseApi{
 
     public static function role($courseId,$userUid){
         $role = CourseMember::where('course_id',$courseId)
-                            ->where('user_id')
+                            ->where('user_id',$userUid)
+                            ->where('is_current',true)
                             ->value('role');
         return $role;
     }

+ 1 - 1
app/Http/Controllers/TagMapController.php

@@ -58,7 +58,7 @@ class TagMapController extends Controller
             return true;
         }
         if(!empty($courseId)){
-            $role = CourseApi::role($request->get('course'),$userId);
+            $role = CourseApi::role($courseId,$userId);
             if(!empty($role) && $role !== 'student'){
                 return true;
             }