From 3643eb2a997e2d95fe514c249faf620869168e02 Mon Sep 17 00:00:00 2001 From: Chris Santillan Date: Tue, 23 Apr 2024 12:13:37 -0500 Subject: [PATCH] fix 224: implement `getProviderServices` to get appropriate service from `provider_platform_id` in request --- .../v1/Actions/StoreCanvasCoursesController.php | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/app/Http/Controllers/v1/Actions/StoreCanvasCoursesController.php b/app/Http/Controllers/v1/Actions/StoreCanvasCoursesController.php index 16098e617..b2bfa181f 100644 --- a/app/Http/Controllers/v1/Actions/StoreCanvasCoursesController.php +++ b/app/Http/Controllers/v1/Actions/StoreCanvasCoursesController.php @@ -8,19 +8,16 @@ use App\Http\Requests\AdminRequest; use App\Http\Resources\CourseResource; use App\Models\Course; -use App\Services\CanvasServices; +use App\Models\ProviderPlatform; class StoreCanvasCoursesController extends Controller { public function __invoke(AdminRequest $request) { - try { - $canvasService = CanvasServices::byProviderId($request['provider_platform_id']); - } catch (\Exception) { - return response()->json(['message' => 'Provider not found'], 404); - } + $provider = ProviderPlatform::findOrFail($request['provider_platform_id']); + $canvas = $provider->getProviderServices(); // list all the courses in the account - $canvasCourses = $canvasService->listCourses(); + $canvasCourses = $canvas->listCourses(); $courseCollection = collect(); foreach ($canvasCourses as $course) { if (isset($course['public_description'])) {