Ticket #4817 - Courses improvements.

unacms · Oct 25, 2024 · 80d4f85 · 80d4f85
1 parent 4cc2ba9
commit 80d4f85
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 6 deletions.
diff --git a/modules/base/files/classes/BxBaseModFilesModule.php b/modules/base/files/classes/BxBaseModFilesModule.php
@@ -24,12 +24,26 @@ public function serviceGetFile ($iContentId, $aParams = [])
         if(empty($aContentInfo) || !is_array($aContentInfo))
             return false;
 
-        $sStorage = !empty($aParams['storage']) ? $aParams['storage'] : $CNF['OBJECT_STORAGE'];
+        $sStorage = '';
+        if(!empty($aParams['storage']))
+            $sStorage = $aParams['storage'];
+        else if(!empty($CNF['OBJECT_STORAGE']))
+            $sStorage = $CNF['OBJECT_STORAGE'];
+        else 
+            return false;
+
         $oStorage = BxDolStorage::getObjectInstance($sStorage);
         if(!$oStorage)
             return false;
 
-        $sFieldFileId = !empty($aParams['field']) ? $aParams['field'] : $CNF['FIELD_FILE_ID'];
+        $sFieldFileId = '';
+        if(!empty($aParams['field']))
+            $sFieldFileId = $aParams['field'];
+        else if(!empty($CNF['FIELD_FILE_ID']))
+            $sFieldFileId = $CNF['FIELD_FILE_ID'];
+        else 
+            return false;
+
         return $oStorage->getFile($aContentInfo[$sFieldFileId]);
     }
 }

diff --git a/modules/boonex/courses/classes/BxCoursesTemplate.php b/modules/boonex/courses/classes/BxCoursesTemplate.php
@@ -584,10 +584,10 @@ protected function _entryNodeItems($iProfileId, $iContentId, $aNode, $iUsage)
                     if(!empty($aFileInfo) && is_array($aFileInfo)) {
                         $aTmplVarsShowSize['size'] = _t_format_size($aFileInfo['size']);
 
-                        $bTmplVarsShowDownload = !empty($aFileInfo['url_download']);
-                        $aTmplVarsShowDownload = [
-                            'link' => $aFileInfo['url_download']
-                        ];
+                        if(($bTmplVarsShowDownload = !empty($aFileInfo['url_download'])))
+                            $aTmplVarsShowDownload = [
+                                'link' => $aFileInfo['url_download']
+                            ];
                     }
                 }
 

diff --git a/modules/boonex/videos/classes/BxVideosModule.php b/modules/boonex/videos/classes/BxVideosModule.php
@@ -124,6 +124,18 @@ public function serviceGetVideo ($iContentId)
         return $this->getVideoData($aContentInfo);
     }
 
+    public function serviceGetFile ($iContentId, $aParams = []) 
+    {
+        $CNF = &$this->_oConfig->CNF;
+
+        if(!isset($aParams['storage']))
+            $aParams['storage'] = $CNF['OBJECT_STORAGE_VIDEOS'];
+        if(!isset($aParams['field']))
+            $aParams['field'] = $CNF['FIELD_VIDEO'];       
+
+        return parent::serviceGetFile($iContentId, $aParams);
+    }
+
     public function getVideoData($aContentInfo)
     {
         $CNF = &$this->_oConfig->CNF;