Minor: Format code - refs BT#21114

src/CoreBundle/Entity/TrackEAttempt.php

@@ -1,9 +1,9 @@
 <?php
 
-declare(strict_types=1);
-
 /* For licensing terms, see /license.txt */
 
+declare(strict_types=1);
+
 namespace Chamilo\CoreBundle\Entity;
 
 use ApiPlatform\Doctrine\Orm\Filter\SearchFilter;
@@ -22,61 +22,102 @@ use Symfony\Component\Validator\Constraints as Assert;
 /**
  * Questions per quiz user attempts.
  */
-#[ApiResource(operations: [new Get(security: 'is_granted("VIEW", object)'), new GetCollection(security: 'is_granted("ROLE_USER")')], security: 'is_granted("ROLE_USER")', normalizationContext: ['groups' => ['track_e_attempt:read']])]
+#[ApiResource(
+    operations: [
+        new Get(security: 'is_granted("VIEW", object)'),
+        new GetCollection(security: 'is_granted("ROLE_USER")'),
+    ],
+    normalizationContext: [
+        'groups' => ['track_e_attempt:read'],
+    ],
+    security: 'is_granted("ROLE_USER")'
+)]
 #[ORM\Table(name: 'track_e_attempt')]
-#[ORM\Index(name: 'exe_id', columns: ['exe_id'])]
+#[ORM\Index(columns: ['exe_id'], name: 'exe_id')]
-#[ORM\Index(name: 'user_id', columns: ['user_id'])]
+#[ORM\Index(columns: ['user_id'], name: 'user_id')]
-#[ORM\Index(name: 'question_id', columns: ['question_id'])]
+#[ORM\Index(columns: ['question_id'], name: 'question_id')]
-#[ORM\Index(name: 'idx_track_e_attempt_tms', columns: ['tms'])]
+#[ORM\Index(columns: ['tms'], name: 'idx_track_e_attempt_tms')]
 #[ORM\Entity]
-#[ApiFilter(filterClass: SearchFilter::class, properties: ['user' => 'exact', 'questionId' => 'exact', 'answer' => 'exact', 'marks' => 'exact'])]
+#[ApiFilter(
+    filterClass: SearchFilter::class,
+    properties: [
+        'user' => 'exact',
+        'questionId' => 'exact',
+        'answer' => 'exact',
+        'marks' => 'exact',
+    ]
+)]
 class TrackEAttempt
 {
     use UserTrait;
+
     #[ORM\Column(name: 'id', type: 'integer')]
     #[ORM\Id]
     #[ORM\GeneratedValue(strategy: 'IDENTITY')]
     protected ?int $id = null;
+
     #[Assert\NotNull]
-    #[ORM\ManyToOne(targetEntity: \Chamilo\CoreBundle\Entity\TrackEExercise::class, inversedBy: 'attempts')]
+    #[ORM\ManyToOne(targetEntity: TrackEExercise::class, inversedBy: 'attempts')]
     #[ORM\JoinColumn(name: 'exe_id', referencedColumnName: 'exe_id', nullable: false, onDelete: 'CASCADE')]
     protected TrackEExercise $trackExercise;
+
     #[Assert\NotNull]
     #[Groups(['track_e_attempt:read'])]
-    #[ORM\ManyToOne(targetEntity: \Chamilo\CoreBundle\Entity\User::class, inversedBy: 'trackEAttempts')]
+    #[ORM\ManyToOne(targetEntity: User::class, inversedBy: 'trackEAttempts')]
     #[ORM\JoinColumn(name: 'user_id', referencedColumnName: 'id', onDelete: 'CASCADE')]
     protected User $user;
+
     #[Assert\NotBlank]
     #[Groups(['track_e_attempt:read'])]
     #[ORM\Column(name: 'question_id', type: 'integer', nullable: false)]
     protected ?int $questionId = null;
+
     #[Groups(['track_e_attempt:read'])]
     #[ORM\Column(name: 'answer', type: 'text', nullable: false)]
     protected string $answer;
+
     #[ORM\Column(name: 'teacher_comment', type: 'text', nullable: false)]
     protected string $teacherComment;
+
     #[Groups(['track_e_attempt:read'])]
     #[ORM\Column(name: 'marks', type: 'float', precision: 6, scale: 2, nullable: false)]
     protected float $marks;
+
     #[ORM\Column(name: 'position', type: 'integer', nullable: true)]
     protected ?int $position = null;
+
     #[Assert\NotNull]
     #[ORM\Column(name: 'tms', type: 'datetime', nullable: false)]
     protected DateTime $tms;
+
     #[ORM\Column(name: 'filename', type: 'string', length: 255, nullable: true)]
     protected ?string $filename = null;
+
     #[ORM\Column(name: 'seconds_spent', type: 'integer')]
     protected int $secondsSpent;
+
     /**
-     * @var Collection|AttemptFile[]
+     * @var Collection<int, AttemptFile>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\AttemptFile::class, mappedBy: 'attempt', cascade: ['persist'], orphanRemoval: true)]
+    #[ORM\OneToMany(
+        mappedBy: 'attempt',
+        targetEntity: AttemptFile::class,
+        cascade: ['persist'],
+        orphanRemoval: true
+    )]
     protected Collection $attemptFiles;
+
     /**
-     * @var Collection|AttemptFeedback[]
+     * @var Collection<int, AttemptFeedback>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\AttemptFeedback::class, mappedBy: 'attempt', cascade: ['persist'], orphanRemoval: true)]
+    #[ORM\OneToMany(
+        mappedBy: 'attempt',
+        targetEntity: AttemptFeedback::class,
+        cascade: ['persist'],
+        orphanRemoval: true
+    )]
     protected Collection $attemptFeedbacks;
+
     public function __construct()
     {
         $this->attemptFiles = new ArrayCollection();
@@ -84,187 +125,168 @@ class TrackEAttempt
         $this->teacherComment = '';
         $this->secondsSpent = 0;
     }
+
+    public function getQuestionId(): ?int
+    {
+        return $this->questionId;
+    }
+
     public function setQuestionId(int $questionId): self
     {
         $this->questionId = $questionId;
 
         return $this;
     }
-    /**
+
-     * Get questionId.
+    public function getAnswer(): string
-     *
-     * @return int
-     */
-    public function getQuestionId()
     {
-        return $this->questionId;
+        return $this->answer;
     }
+
     public function setAnswer(string $answer): self
     {
         $this->answer = $answer;
 
         return $this;
     }
-    /**
+
-     * Get answer.
+    public function getTeacherComment(): string
-     *
-     * @return string
-     */
-    public function getAnswer()
     {
-        return $this->answer;
+        return $this->teacherComment;
     }
+
     public function setTeacherComment(string $teacherComment): self
     {
         $this->teacherComment = $teacherComment;
 
         return $this;
     }
-    /**
+
-     * Get teacherComment.
+    public function getMarks(): float
-     *
-     * @return string
-     */
-    public function getTeacherComment()
     {
-        return $this->teacherComment;
+        return $this->marks;
     }
+
     public function setMarks(float $marks): self
     {
         $this->marks = $marks;
 
         return $this;
     }
-    /**
+
-     * Get marks.
+    public function getPosition(): ?int
-     *
-     * @return float
-     */
-    public function getMarks()
     {
-        return $this->marks;
+        return $this->position;
     }
+
     public function setPosition(int $position): self
     {
         $this->position = $position;
 
         return $this;
     }
-    /**
+
-     * Get position.
+    public function getTms(): DateTime
-     *
-     * @return int
-     */
-    public function getPosition()
     {
-        return $this->position;
+        return $this->tms;
     }
+
     public function setTms(DateTime $tms): self
     {
         $this->tms = $tms;
 
         return $this;
     }
-    /**
+
-     * Get tms.
+    public function getFilename(): ?string
-     *
-     * @return DateTime
-     */
-    public function getTms()
     {
-        return $this->tms;
+        return $this->filename;
     }
-    /**
+
-     * Set filename.
+    public function setFilename(string $filename): static
-     *
-     * @return TrackEAttempt
-     */
-    public function setFilename(string $filename)
     {
         $this->filename = $filename;
 
         return $this;
     }
-    /**
+
-     * Get filename.
+    public function getId(): ?int
-     *
-     * @return string
-     */
-    public function getFilename()
-    {
-        return $this->filename;
-    }
-    /**
-     * Get id.
-     *
-     * @return int
-     */
-    public function getId()
     {
         return $this->id;
     }
+
     public function getUser(): User
     {
         return $this->user;
     }
+
     public function setUser(User $user): self
     {
         $this->user = $user;
 
         return $this;
     }
+
     public function getTrackEExercise(): TrackEExercise
     {
         return $this->trackExercise;
     }
+
     public function setTrackEExercise(TrackEExercise $trackExercise): self
     {
         $this->trackExercise = $trackExercise;
 
         return $this;
     }
+
     public function getSecondsSpent(): int
     {
         return $this->secondsSpent;
     }
+
     public function setSecondsSpent(int $secondsSpent): self
     {
         $this->secondsSpent = $secondsSpent;
 
         return $this;
     }
+
     /**
-     * @return AttemptFile[]|Collection
+     * @return Collection<int, AttemptFile>
      */
-    public function getAttemptFiles(): array|Collection
+    public function getAttemptFiles(): Collection
     {
         return $this->attemptFiles;
     }
+
     /**
-     * @param AttemptFile[]|Collection $attemptFiles
+     * @param Collection<int, AttemptFile> $attemptFiles
      */
-    public function setAttemptFiles(array|Collection $attemptFiles): self
+    public function setAttemptFiles(Collection $attemptFiles): self
     {
         $this->attemptFiles = $attemptFiles;
 
         return $this;
     }
+
     /**
-     * @return AttemptFeedback[]|Collection
+     * @return Collection<int, AttemptFeedback>
      */
-    public function getAttemptFeedbacks(): array|Collection
+    public function getAttemptFeedbacks(): Collection
     {
         return $this->attemptFeedbacks;
     }
+
     /**
-     * @param AttemptFeedback[]|Collection $attemptFeedbacks
+     * @param Collection<int, AttemptFeedback> $attemptFeedbacks
      */
-    public function setAttemptFeedbacks(array|Collection $attemptFeedbacks): self
+    public function setAttemptFeedbacks(Collection $attemptFeedbacks): self
     {
         $this->attemptFeedbacks = $attemptFeedbacks;
 
         return $this;
     }
+
     public function addAttemptFeedback(AttemptFeedback $attemptFeedback): self
     {
         if (!$this->attemptFeedbacks->contains($attemptFeedback)) {
@@ -274,6 +296,7 @@ class TrackEAttempt
 
         return $this;
     }
+
     public function addAttemptFile(AttemptFile $attemptFile): self
     {
         if (!$this->attemptFiles->contains($attemptFile)) {

src/CoreBundle/Entity/TrackEAttemptRecording.php

@@ -1,22 +1,19 @@
 <?php
 
-declare(strict_types=1);
-
 /* For licensing terms, see /license.txt */
 
+declare(strict_types=1);
+
 namespace Chamilo\CoreBundle\Entity;
 
 use DateTime;
 use Doctrine\ORM\Mapping as ORM;
 use Gedmo\Mapping\Annotation as Gedmo;
 
-/**
- * TrackEAttemptRecording.
- */
 #[ORM\Table(name: 'track_e_attempt_recording')]
-#[ORM\Index(name: 'exe_id', columns: ['exe_id'])]
+#[ORM\Index(columns: ['exe_id'], name: 'exe_id')]
-#[ORM\Index(name: 'question_id', columns: ['question_id'])]
+#[ORM\Index(columns: ['question_id'], name: 'question_id')]
-#[ORM\Index(name: 'session_id', columns: ['session_id'])]
+#[ORM\Index(columns: ['session_id'], name: 'session_id')]
 #[ORM\Entity]
 class TrackEAttemptRecording
 {
@@ -58,48 +55,33 @@ class TrackEAttemptRecording
         $this->author = 0;
     }
 
-    /**
+    public function getExeId(): int
-     * Set exeId.
+    {
-     *
+        return $this->exeId;
-     * @return TrackEAttemptRecording
+    }
-     */
+
-    public function setExeId(int $exeId)
+    public function setExeId(int $exeId): static
     {
         $this->exeId = $exeId;
 
         return $this;
     }
 
-    /**
+    public function getQuestionId(): int
-     * Get exeId.
-     *
-     * @return int
-     */
-    public function getExeId()
     {
-        return $this->exeId;
+        return $this->questionId;
     }
 
-    /**
+    public function setQuestionId(int $questionId): static
-     * Set questionId.
-     *
-     * @return TrackEAttemptRecording
-     */
-    public function setQuestionId(int $questionId)
     {
         $this->questionId = $questionId;
 
         return $this;
     }
 
-    /**
+    public function getMarks(): int
-     * Get questionId.
-     *
-     * @return int
-     */
-    public function getQuestionId()
     {
-        return $this->questionId;
+        return $this->marks;
     }
 
     public function setMarks(int $marks): self
@@ -109,14 +91,9 @@ class TrackEAttemptRecording
         return $this;
     }
 
-    /**
+    public function getInsertDate(): DateTime
-     * Get marks.
-     *
-     * @return int
-     */
-    public function getMarks()
     {
-        return $this->marks;
+        return $this->insertDate;
     }
 
     public function setInsertDate(DateTime $insertDate): self
@@ -126,36 +103,21 @@ class TrackEAttemptRecording
         return $this;
     }
 
-    /**
+    public function getAuthor(): int
-     * Get insertDate.
-     *
-     * @return DateTime
-     */
-    public function getInsertDate()
     {
-        return $this->insertDate;
+        return $this->author;
     }
 
-    /**
+    public function setAuthor(int $author): static
-     * Set author.
-     *
-     * @return TrackEAttemptRecording
-     */
-    public function setAuthor(int $author)
     {
         $this->author = $author;
 
         return $this;
     }
 
-    /**
+    public function getTeacherComment(): string
-     * Get author.
-     *
-     * @return int
-     */
-    public function getAuthor()
     {
-        return $this->author;
+        return $this->teacherComment;
     }
 
     public function setTeacherComment(string $teacherComment): self
@@ -165,44 +127,19 @@ class TrackEAttemptRecording
         return $this;
     }
 
-    /**
+    public function getSessionId(): int
-     * Get teacherComment.
-     *
-     * @return string
-     */
-    public function getTeacherComment()
     {
-        return $this->teacherComment;
+        return $this->sessionId;
     }
 
-    /**
+    public function setSessionId(int $sessionId): static
-     * Set sessionId.
-     *
-     * @return TrackEAttemptRecording
-     */
-    public function setSessionId(int $sessionId)
     {
         $this->sessionId = $sessionId;
 
         return $this;
     }
 
-    /**
+    public function getId(): ?int
-     * Get sessionId.
-     *
-     * @return int
-     */
-    public function getSessionId()
-    {
-        return $this->sessionId;
-    }
-
-    /**
-     * Get id.
-     *
-     * @return int
-     */
-    public function getId()
     {
         return $this->id;
     }