Merge pull request #5086 from christianbeeznest/GH-4861-5

Course: Add resource_link_id in track_e_download and save download - refs #4861

public/main/inc/lib/document.lib.php

@@ -2227,6 +2227,7 @@ class DocumentManager
     public static function enough_space($file_size, $max_dir_space)
     {
         if ($max_dir_space) {
+            $max_dir_space = $max_dir_space * 1024 * 1024;
             $courseEntity = api_get_course_entity();
             $repo = Container::getDocumentRepository();
             $total = $repo->getFolderSize($courseEntity->getResourceNode(), $courseEntity);

public/main/inc/lib/events.lib.php

@@ -7,6 +7,7 @@ use Chamilo\CoreBundle\Entity\Course as CourseEntity;
 use Chamilo\CoreBundle\Entity\Session as SessionEntity;
 use Chamilo\CoreBundle\Entity\TrackEAttemptQualify;
 use Chamilo\CoreBundle\Entity\TrackEDefault;
+use Chamilo\CoreBundle\Entity\TrackEDownloads;
 use Chamilo\CoreBundle\Entity\TrackEExercise;
 use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Framework\Container;
@@ -272,24 +273,19 @@ class Event
             return false;
         }
 
-        $table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_DOWNLOADS);
+        $user = api_get_user_entity();
-        $documentUrl = Database::escape_string($documentUrl);
+        $course = api_get_course_entity();
+        $session = api_get_session_entity();
 
-        $reallyNow = api_get_utc_datetime();
+        $em = Database::getManager();
-        $userId = api_get_user_id();
-        $courseId = api_get_course_int_id();
-        $sessionId = api_get_session_id();
 
-        return Database::insert(
+        $repository = $em->getRepository(TrackEDownloads::class);
-            $table,
+
-            [
+        $resourceLinkId = $course->getFirstResourceLink()->getId();
-                'down_user_id' => $userId,
+
-                'c_id' => $courseId,
+        $downloadId = $repository->saveDownload($user->getId(), $resourceLinkId, $documentUrl);
-                'down_doc_path' => $documentUrl,
+
-                'down_date' => $reallyNow,
+        return $downloadId;
-                'session_id' => $sessionId,
-            ]
-        );
     }
 
     /**

src/CoreBundle/Controller/ResourceController.php

@@ -10,6 +10,8 @@ use Chamilo\CoreBundle\Entity\Course;
 use Chamilo\CoreBundle\Entity\ResourceLink;
 use Chamilo\CoreBundle\Entity\ResourceNode;
 use Chamilo\CoreBundle\Entity\Session;
+use Chamilo\CoreBundle\Entity\TrackEDownloads;
+use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Repository\ResourceWithLinkInterface;
 use Chamilo\CoreBundle\Security\Authorization\Voter\ResourceNodeVoter;
 use Chamilo\CoreBundle\Tool\ToolChain;
@@ -171,7 +173,7 @@ class ResourceController extends AbstractResourceController implements CourseCon
      * @return RedirectResponse|StreamedResponse
      */
     #[Route('/{tool}/{type}/{id}/download', name: 'chamilo_core_resource_download', methods: ['GET'])]
-    public function downloadAction(Request $request)
+    public function downloadAction(Request $request, EntityManagerInterface $entityManager)
     {
         $id = $request->get('id');
         $resourceNode = $this->getResourceNodeRepository()->findOneBy(['uuid' => $id]);
@@ -190,6 +192,14 @@ class ResourceController extends AbstractResourceController implements CourseCon
 
         // If resource node has a file just download it. Don't download the children.
         if ($resourceNode->hasResourceFile()) {
+
+            /** @var ?User $user */
+            $user = $this->getUser();
+            $resourceLinkId = $resourceNode->getResourceLinks()->first()->getId();
+            $url = $resourceNode->getResourceFile()->getOriginalName();
+            $downloadRepository = $entityManager->getRepository(TrackEDownloads::class);
+            $downloadId = $downloadRepository->saveDownload($user->getId(), $resourceLinkId, $url);
+
             // Redirect to download single file.
             return $this->processFile($request, $resourceNode, 'download');
         }

src/CoreBundle/Entity/TrackEDownloads.php

@@ -6,6 +6,7 @@ declare(strict_types=1);
 
 namespace Chamilo\CoreBundle\Entity;
 
+use Chamilo\CoreBundle\Repository\TrackEDownloadsRepository;
 use DateTime;
 use Doctrine\ORM\Mapping as ORM;
 
@@ -14,9 +15,7 @@ use Doctrine\ORM\Mapping as ORM;
  */
 #[ORM\Table(name: 'track_e_downloads')]
 #[ORM\Index(name: 'idx_ted_user_id', columns: ['down_user_id'])]
-#[ORM\Index(name: 'idx_ted_c_id', columns: ['c_id'])]
+#[ORM\Entity(repositoryClass: TrackEDownloadsRepository::class)]
-#[ORM\Index(name: 'session_id', columns: ['session_id'])]
-#[ORM\Entity]
 class TrackEDownloads
 {
     #[ORM\Column(name: 'down_id', type: 'integer')]
@@ -30,132 +29,99 @@ class TrackEDownloads
     #[ORM\Column(name: 'down_date', type: 'datetime', nullable: false)]
     protected DateTime $downDate;
 
-    #[ORM\Column(name: 'c_id', type: 'integer', nullable: false)]
-    protected int $cId;
-
     #[ORM\Column(name: 'down_doc_path', type: 'string', length: 255, nullable: false)]
     protected string $downDocPath;
 
-    #[ORM\Column(name: 'session_id', type: 'integer', nullable: false)]
+    #[ORM\ManyToOne(targetEntity: ResourceLink::class, cascade: ['persist', 'remove'])]
-    protected int $sessionId;
+    #[ORM\JoinColumn(name: 'resource_link_id', referencedColumnName: 'id', onDelete: 'SET NULL', nullable: true)]
+    protected ?ResourceLink $resourceLink = null;
 
     /**
-     * Set downUserId.
+     * Set downDocPath.
      *
      * @return TrackEDownloads
      */
-    public function setDownUserId(int $downUserId)
+    public function setDownDocPath(string $downDocPath)
     {
-        $this->downUserId = $downUserId;
+        $this->downDocPath = $downDocPath;
 
         return $this;
     }
 
     /**
-     * Get downUserId.
+     * Get downDocPath.
-     *
-     * @return int
-     */
-    public function getDownUserId()
-    {
-        return $this->downUserId;
-    }
-
-    /**
-     * Set downDate.
      *
-     * @return TrackEDownloads
+     * @return string
      */
-    public function setDownDate(DateTime $downDate)
+    public function getDownDocPath()
     {
-        $this->downDate = $downDate;
+        return $this->downDocPath;
-
-        return $this;
     }
 
     /**
-     * Get downDate.
+     * Get downId.
      *
-     * @return DateTime
+     * @return int
      */
-    public function getDownDate()
+    public function getDownId()
     {
-        return $this->downDate;
+        return $this->downId;
     }
 
     /**
-     * Set cId.
+     * Set downUserId.
      *
      * @return TrackEDownloads
      */
-    public function setCId(int $cId)
+    public function setDownUserId(int $downUserId)
     {
-        $this->cId = $cId;
+        $this->downUserId = $downUserId;
 
         return $this;
     }
 
     /**
-     * Get cId.
+     * Get downUserId.
      *
      * @return int
      */
-    public function getCId()
+    public function getDownUserId()
     {
-        return $this->cId;
+        return $this->downUserId;
     }
 
     /**
-     * Set downDocPath.
+     * Set downDate.
      *
      * @return TrackEDownloads
      */
-    public function setDownDocPath(string $downDocPath)
+    public function setDownDate(DateTime $downDate)
     {
-        $this->downDocPath = $downDocPath;
+        $this->downDate = $downDate;
 
         return $this;
     }
 
     /**
-     * Get downDocPath.
+     * Get downDate.
      *
-     * @return string
+     * @return DateTime
      */
-    public function getDownDocPath()
+    public function getDownDate()
     {
-        return $this->downDocPath;
+        return $this->downDate;
     }
 
     /**
-     * Set sessionId.
+     * Set resourceLink.
      *
+     * @param ResourceLink|null $resourceLink
      * @return TrackEDownloads
      */
-    public function setSessionId(int $sessionId)
+    public function setResourceLink(?ResourceLink $resourceLink): self
     {
-        $this->sessionId = $sessionId;
+        $this->resourceLink = $resourceLink;
 
         return $this;
     }
-
-    /**
-     * Get sessionId.
-     *
-     * @return int
-     */
-    public function getSessionId()
-    {
-        return $this->sessionId;
-    }
-
-    /**
-     * Get downId.
-     *
-     * @return int
-     */
-    public function getDownId()
-    {
-        return $this->downId;
-    }
 }

src/CoreBundle/Framework/Container.php

@@ -35,6 +35,7 @@ use Chamilo\CoreBundle\Repository\SkillRepository;
 use Chamilo\CoreBundle\Repository\SocialPostRepository;
 use Chamilo\CoreBundle\Repository\SysAnnouncementRepository;
 use Chamilo\CoreBundle\Repository\TagRepository;
+use Chamilo\CoreBundle\Repository\TrackEDownloadsRepository;
 use Chamilo\CoreBundle\Repository\TrackEExerciseRepository;
 use Chamilo\CoreBundle\Repository\TrackELoginRecordRepository;
 use Chamilo\CoreBundle\Serializer\UserToJsonNormalizer;
@@ -574,6 +575,11 @@ class Container
         return self::$container->get(TrackEExerciseRepository::class);
     }
 
+    public static function getTrackEDownloadsRepository(): TrackEDownloadsRepository
+    {
+        return self::$container->get(TrackEDownloadsRepository::class);
+    }
+
     public static function getWikiRepository(): CWikiRepository
     {
         return self::$container->get(CWikiRepository::class);

src/CoreBundle/Migrations/Schema/V200/Version20240114215900.php

@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Chamilo\CoreBundle\Migrations\Schema\V200;
+
+use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
+use Doctrine\DBAL\Schema\Schema;
+
+final class Version20240114215900 extends AbstractMigrationChamilo
+{
+    public function getDescription(): string
+    {
+        return 'Modify track_e_downloads table structure';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('DROP INDEX idx_ted_c_id ON track_e_downloads;');
+        $this->addSql('DROP INDEX down_session_id ON track_e_downloads;');
+        $this->addSql('ALTER TABLE track_e_downloads ADD resource_link_id INT DEFAULT NULL, DROP c_id, DROP down_session_id;');
+        $this->addSql('ALTER TABLE track_e_downloads ADD CONSTRAINT FK_EEDF4DA6F004E599 FOREIGN KEY (resource_link_id) REFERENCES resource_link (id) ON DELETE SET NULL;');
+        $this->addSql('CREATE INDEX IDX_EEDF4DA6F004E599 ON track_e_downloads (resource_link_id);');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE track_e_downloads DROP FOREIGN KEY FK_EEDF4DA6F004E599;');
+        $this->addSql('DROP INDEX IDX_EEDF4DA6F004E599 ON track_e_downloads;');
+        $this->addSql('ALTER TABLE track_e_downloads DROP resource_link_id;');
+        $this->addSql('ALTER TABLE track_e_downloads ADD c_id INT NOT NULL;');
+        $this->addSql('ALTER TABLE track_e_downloads ADD down_session_id INT DEFAULT NULL;');
+        $this->addSql('CREATE INDEX idx_ted_c_id ON track_e_downloads (c_id);');
+        $this->addSql('CREATE INDEX down_session_id ON track_e_downloads (down_session_id);');
+    }
+}

src/CoreBundle/Repository/TrackEDownloadsRepository.php

@@ -0,0 +1,39 @@
+<?php
+
+declare(strict_types=1);
+
+/* For licensing terms, see /license.txt */
+
+namespace Chamilo\CoreBundle\Repository;
+
+use Chamilo\CoreBundle\Entity\ResourceLink;
+use Chamilo\CoreBundle\Entity\TrackEDownloads;
+use DateTime;
+use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
+use Doctrine\Persistence\ManagerRegistry;
+
+class TrackEDownloadsRepository extends ServiceEntityRepository
+{
+    public function __construct(ManagerRegistry $registry)
+    {
+        parent::__construct($registry, TrackEDownloads::class);
+    }
+
+    public function saveDownload(int $userId, int $resourceLinkId, string $documentUrl)
+    {
+        $download = new TrackEDownloads();
+        $download->setDownDocPath($documentUrl);
+        $download->setDownUserId($userId);
+        $download->setDownDate(new DateTime());
+
+        $resourceLink =  $this->_em->getRepository(ResourceLink::class)->find($resourceLinkId);
+        if ($resourceLink) {
+            $download->setResourceLink($resourceLink);
+        }
+
+        $this->_em->persist($download);
+        $this->_em->flush();
+
+        return $download->getDownId();
+    }
+}