fix: syllabus being detected and replaced in file change process

Author: Ishfaq-code

assets/js/Components/ReviewFilesPage.js

@@ -521,13 +521,18 @@ useEffect(() => {
     }
   }
 
-  const extractUrl = (url) => {
+  const extractUrl = (url, contentType) => {
   if(!url) return ''
 
   const idx = url.indexOf('courses/');
   if (idx !== -1) {
     // slice from "courses/" onward and strip any leading slashes (defensive)
-    return url.slice(idx).replace(/^\/+/, '');
+    let slicedUrl = url.slice(idx).replace(/^\/+/, '');
+    if(contentType == "syllabus"){
+      const parts = slicedUrl.split("/")
+      slicedUrl = `${parts[0]}/${parts[1]}?include[]=syllabus_body`
+    }
+    return slicedUrl
   }
 
   // if no "courses/" found, remove leading slashes and return the remainder
@@ -614,7 +619,7 @@ useEffect(() => {
   }
 
   const createContentItemPostOptions = (fullPageHtml, contentUrl, contentId, contentType, sectionIds) => {
-      const contentItemOption = {
+    const contentItemOption = {
         fullPageHtml: fullPageHtml,
         contentUrl: contentUrl,
         contentId: contentId,
@@ -646,7 +651,7 @@ useEffect(() => {
         if(reference.contentItemBody){
           newFullPageHtml = replaceFileInHtml(reference.contentItemBody, file.lmsFileId, newFile.metadata.url)
         }
-        postContentItemOptions.push(createContentItemPostOptions(newFullPageHtml, extractUrl(reference.contentItemUrl), reference.contentItemId, reference.contentType, reference.sectionIds))
+        postContentItemOptions.push(createContentItemPostOptions(newFullPageHtml, extractUrl(reference.contentItemUrl, reference.contentType), reference.contentItemId, reference.contentType, reference.sectionIds))
       })
     }
     return postContentItemOptions
@@ -836,6 +841,7 @@ const getSectionPostOptions = (newFile, sectionReferences) => {
     const postContentItemOptions = getContentPostItems(activeFile.replacement, activeFile, contentReferences)
     const postSectionOptions = getSectionPostOptions(activeFile, sectionReferences)
 
+
     if((postContentItemOptions && postContentItemOptions.length > 0) ||  (postSectionOptions && postSectionOptions.length > 0)){
         const responseStatus = await updateAndScanContent(postContentItemOptions, postSectionOptions, activeFile.id)
         if(responseStatus && responseStatus[0]?.type == "error"){

src/Lms/Canvas/CanvasApi.php

@@ -217,6 +217,7 @@ public function apiFilePost(string $url, array $options, string $filepath, strin
     public function apiPut($url, $options)
     {
         $lmsResponse = new LmsResponse();
+        $output = new ConsoleOutput();
         if(!isset($options['headers'])) {
             $options['headers'] = [];
         }
@@ -285,6 +286,10 @@ public function apiPutBatch(array $paths, array $options){
             $type = "";
             $lmsId = "";
 
+            if (preg_match('#^courses/(\d+)\?include\[\]=syllabus_body$#', $paths[$i], $matches)) {
+                $type = "syllabus";
+            }
+
             if (preg_match('#/(\w+)/([^/]+)$#', $paths[$i], $matches)) {
                 $type = $matches[1]; 
                 $type = preg_replace('/s$/', '', $type);
@@ -301,6 +306,10 @@ public function apiPutBatch(array $paths, array $options){
             if ($type == 'discussion_topic' && isset($normalizedContent->is_announcement) && $normalizedContent->is_announcement) {
                 $type = 'announcement';
             }
+
+            if ($type == 'syllabus'){
+                $lmsId = $normalizedContent->id;
+            }
             $response = [
                 'content' => $normalizedContent,
                 'id' => $lmsId,

src/Lms/Canvas/CanvasLms.php

@@ -644,7 +644,11 @@ public function postContentItemNoIssue($contentOptions, $sectionOptions)
                 if($contentItem){
                     $normalizedContent = [];
                     if($response['status'] == 200){
-                        $normalizedContent = $this->normalizeLmsContent($contentItem->getCourse(), $response['type'], json_decode(json_encode($response['content']), true));
+                        $lmsContentNew = json_decode(json_encode($response['content']), true);
+                        if ($response['type'] == 'syllabus') {
+                            $lmsContentNew['syllabus_body'] = $option['fullPageHtml'];
+                        }
+                        $normalizedContent = $this->normalizeLmsContent($contentItem->getCourse(), $response['type'], $lmsContentNew);
                         $contentItem->update($normalizedContent);
                         $this->entityManager->flush();
                     }
@@ -900,6 +904,13 @@ protected function createLmsPostOptions(ContentItem $contentItem)
     protected function createLmsPostOptionsWithHtml($type, $fullPageHtml){
         $options = [];
         switch($type){
+            case('syllabus'):
+                $options = [
+                    'course' => [
+                        'syllabus_body' => $fullPageHtml,
+                    ],
+                ];
+                break;
             case('page'):
                 $options = [
                         'wiki_page' => [