Merge pull request #7429 from christianbeeznest/GH-5151

Learnpath: Implement SCORM package replacement (LP update) - refs #5151

Author: christianbeeznest

assets/vue/components/lp/LpCategorySection.vue

@@ -22,6 +22,7 @@ const props = defineProps({
   buildDates: { type: Function, required: false },
   isSessionCategory: { type: Boolean, default: false },
 })
+
 const emit = defineEmits([
   "open",
   "edit",
@@ -33,6 +34,7 @@ const emit = defineEmits([
   "delete",
   "export-scorm",
   "export-pdf",
+  "update-scorm",
   "reorder",
   "toggle-auto-launch",
 ])
@@ -73,20 +75,24 @@ const goCat = (action, extraParams = {}) => {
   })
   window.location.assign(url)
 }
+
 const onCatEdit = () => goCat("add_lp_category")
 const onCatAddUsers = () => goCat("add_users_to_category")
+
 const onCatToggleVisibility = () => {
   const vis = props.category.visibility ?? props.category.visible
   const next = typeof vis === "number" ? (vis ? 0 : 1) : 1
   goCat("toggle_category_visibility", { new_status: next })
 }
+
 const onCatTogglePublish = () => {
   const pub = props.category.isPublished ?? props.category.published
   let next = 1
   if (typeof pub === "number") next = pub ? 0 : 1
   if (typeof pub === "string") next = pub === "v" ? 0 : 1
   goCat("toggle_category_publish", { new_status: next })
 }
+
 const onCatDelete = () => {
   // Do not allow deletion if category is not empty
   if (localList.value.length > 0) {
@@ -103,11 +109,14 @@ const onCatDelete = () => {
 
 const isOpen = ref(true)
 const storageKey = computed(() => `lpCatOpen:${props.category?.iid || props.title}`)
+
 onMounted(() => {
   const saved = localStorage.getItem(storageKey.value)
   if (saved !== null) isOpen.value = saved === "1"
 })
+
 watch(isOpen, (v) => localStorage.setItem(storageKey.value, v ? "1" : "0"))
+
 const panelId = computed(() => `cat-panel-${props.category?.iid || props.title}`)
 const toggleOpen = () => {
   if (localList.value.length) isOpen.value = !isOpen.value
@@ -207,6 +216,7 @@ const toggleOpen = () => {
               ></i>
             </span>
           </template>
+
           <template #menu>
             <div
               class="absolute right-0 top-full mt-2 w-60 bg-white border border-gray-25 rounded-xl shadow-xl p-1 z-50"
@@ -217,26 +227,32 @@ const toggleOpen = () => {
               >
                 {{ t("Edit category") }}
               </button>
+
               <button
                 class="w-full text-left px-3 py-2 rounded hover:bg-gray-15"
                 @click="onCatAddUsers"
               >
                 {{ t("Subscribe users to category") }}
               </button>
+
               <div class="my-1 h-px bg-gray-15"></div>
+
               <button
                 class="w-full text-left px-3 py-2 rounded hover:bg-gray-15"
                 @click="onCatToggleVisibility"
               >
                 {{ t("Toggle visibility") }}
               </button>
+
               <button
                 class="w-full text-left px-3 py-2 rounded hover:bg-gray-15"
                 @click="onCatTogglePublish"
               >
                 {{ t("Publish / Hide") }}
               </button>
+
               <div class="my-1 h-px bg-gray-15"></div>
+
               <button
                 class="w-full text-left px-3 py-2 rounded hover:bg-gray-15 text-danger"
                 @click="onCatDelete"
@@ -309,17 +325,18 @@ const toggleOpen = () => {
             :lp="element"
             :ringDash="ringDash"
             :ringValue="ringValue"
-            @build="$emit('build', element)"
-            @delete="$emit('delete', element)"
-            @edit="$emit('edit', element)"
-            @open="$emit('open', element)"
-            @report="$emit('report', element)"
-            @settings="$emit('settings', element)"
-            @toggle-auto-launch="$emit('toggle-auto-launch', element)"
-            @toggle-visible="$emit('toggle-visible', element)"
-            @toggle-publish="$emit('toggle-publish', element)"
-            @export-scorm="$emit('export-scorm', element)"
-            @export-pdf="$emit('export-pdf', element)"
+            @build="emit('build', element)"
+            @delete="emit('delete', element)"
+            @edit="emit('edit', element)"
+            @open="emit('open', element)"
+            @report="emit('report', element)"
+            @settings="emit('settings', element)"
+            @toggle-auto-launch="emit('toggle-auto-launch', element)"
+            @toggle-visible="emit('toggle-visible', element)"
+            @toggle-publish="emit('toggle-publish', element)"
+            @export-scorm="emit('export-scorm', element)"
+            @export-pdf="emit('export-pdf', element)"
+            @update-scorm="emit('update-scorm', element)"
           />
         </template>
       </Draggable>

assets/vue/components/lp/LpRowItem.vue

@@ -28,8 +28,18 @@ const emit = defineEmits([
   "export-scorm",
   "export-pdf",
   "toggle-auto-launch",
+  "update-scorm",
 ])
 
+const lpType = computed(() => {
+  const v = props.lp?.lpType ?? props.lp?.lp_type ?? props.lp?.type ?? props.lp?.lpTypeId ?? props.lp?.lp_type_id ?? 0
+
+  return Number(v) || 0
+})
+
+// Only SCORM packages (type = 2 in Chamilo legacy)
+const canUpdateScorm = computed(() => props.canEdit && lpType.value === 2)
+
 const dateText = computed(() => {
   const v = props.buildDates ? props.buildDates(props.lp) : ""
   return typeof v === "string" ? v.trim() : ""
@@ -212,6 +222,13 @@ const progressTextClass = computed(() => {
             >
               {{ t("Export as SCORM") }}
             </button>
+            <button
+              v-if="canUpdateScorm"
+              class="w-full text-left px-3 py-2 rounded hover:bg-gray-15 md:hidden"
+              @click="emit('update-scorm', lp)"
+            >
+              {{ t("Update SCORM") }}
+            </button>
             <button
               class="w-full text-left px-3 py-2 rounded hover:bg-gray-15 md:hidden"
               @click="emit('settings', lp)"
@@ -272,6 +289,15 @@ const progressTextClass = computed(() => {
           >
             <i class="mdi mdi-archive-arrow-down text-xl" />
           </button>
+          <button
+            v-if="canUpdateScorm"
+            :aria-label="t('Update SCORM')"
+            :title="t('Update SCORM')"
+            class="row-start-1 col-start-5 opacity-70 hover:opacity-100 hidden md:block"
+            @click="emit('update-scorm', lp)"
+          >
+            <i class="mdi mdi-upload text-xl" />
+          </button>
           <button
             v-if="canExportPdf"
             :aria-label="t('Export to PDF')"
@@ -333,6 +359,13 @@ const progressTextClass = computed(() => {
                 >
                   {{ t("Edit learnpath") }}
                 </button>
+                <button
+                  v-if="canUpdateScorm"
+                  class="w-full text-left px-3 py-2 rounded hover:bg-gray-15"
+                  @click="emit('update-scorm', lp)"
+                >
+                  {{ t("Update SCORM") }}
+                </button>
                 <button
                   class="w-full text-left px-3 py-2 rounded hover:bg-gray-15 text-danger"
                   @click="emit('delete', lp)"

assets/vue/services/lpService.js

@@ -85,6 +85,22 @@ const buildLegacyActionUrl = (arg1, arg2, arg3 = {}) => {
   return `/main/lp/lp_controller.php?${search.toString()}`
 }
 
+/** Build URL for updating/uploading SCORM package for an existing LP. */
+const buildLegacyUploadUrl = (lpId, { cid, sid, node, gid = 0, gradebook = 0, origin = "" } = {}) => {
+  return buildLegacyActionUrl(lpId, "upload", {
+    cid,
+    sid,
+    node,
+    gid,
+    gradebook,
+    origin,
+    params: {
+      // force teacher mode context
+      isStudentView: "false",
+    },
+  })
+}
+
 /** Navigates immediately to a legacy controller action. */
 const goLegacyAction = (lpId, action, opts = {}) => {
   window.location.href =
@@ -102,9 +118,7 @@ const goLegacyAction = (lpId, action, opts = {}) => {
  * @returns {Promise<Object[]>}
  */
 const getLpCategories = async (searchParams) => {
-  // API Platform resource for CLpCategory (GET collection)
   const { items } = await baseService.getCollection("/api/learning_path_categories/", searchParams)
-
   return items
 }
 
@@ -113,6 +127,7 @@ export default {
   getLearningPath,
   buildLegacyViewUrl,
   buildLegacyActionUrl,
+  buildLegacyUploadUrl,
   goLegacyAction,
   getLpCategories,
 }