refactor dashboard modals for multi manage (#8522)

ILIAS-eLearning · Dec 3, 2024 · bc5adc1 · bc5adc1
1 parent 8c0e784
commit bc5adc1
Show file tree

Hide file tree

Showing 10 changed files with 161 additions and 349 deletions.
diff --git a/Modules/LearningSequence/classes/class.ilDashboardLearningSequenceGUI.php b/Modules/LearningSequence/classes/class.ilDashboardLearningSequenceGUI.php
@@ -21,10 +21,6 @@
 use ILIAS\UI\Component\Symbol\Icon\Standard;
 use ILIAS\Services\Dashboard\Block\BlockDTO;
 
-/**
- * @ilCtrl_IsCalledBy ilDashboardLearningSequenceGUI: ilColumnGUI
- * @ilCtrl_Calls      ilDashboardLearningSequenceGUI: ilCommonActionDispatcherGUI
- */
 class ilDashboardLearningSequenceGUI extends ilDashboardBlockGUI
 {
     protected function getIcon(string $title): Standard
@@ -111,23 +107,18 @@ public function getBlockType(): string
         return 'pdlern';
     }
 
-    public function confirmedRemoveObject(): void
+    public function confirmedRemove(array $ids): void
     {
-        $refIds = (array) ($this->http->request()->getParsedBody()['ref_id'] ?? []);
-        if ($refIds === []) {
-            $this->ctrl->redirect($this, 'manage');
-        }
-
-        foreach ($refIds as $ref_id) {
-            if ($this->access->checkAccess('leave', '', (int) $ref_id)) {
-                if (ilObject::_lookupType((int) $ref_id, true) === 'lso') {
-                    $lso = ilObjLearningSequence::getInstanceByRefId((int) $ref_id);
+        foreach ($ids as $ref_id) {
+            if ($this->access->checkAccess('leave', '', $ref_id)) {
+                if (ilObject::_lookupType($ref_id, true) === 'lso') {
+                    $lso = ilObjLearningSequence::getInstanceByRefId($ref_id);
                     if ($lso instanceof ilObjLearningSequence) {
                         $lso->getLSRoles()->leave($this->user->getId());
                     }
                 }
 
-                ilForumNotification::checkForumsExistsDelete((int) $ref_id, $this->user->getId());
+                ilForumNotification::checkForumsExistsDelete($ref_id, $this->user->getId());
             }
         }
 

diff --git a/Modules/StudyProgramme/classes/class.ilStudyProgrammeDashboardViewGUI.php b/Modules/StudyProgramme/classes/class.ilStudyProgrammeDashboardViewGUI.php
@@ -65,7 +65,7 @@ public function initData(): void
             ) {
                 $properties[$this->lng->txt('prg_dash_label_valid')] = $row->getExpiryDate() ?: $row->getValidity();
 
-                if($cert_link = $this->maybeGetCertificateLink($this->user->getId(), $prg->getId(), $prg->getRefId())) {
+                if ($cert_link = $this->maybeGetCertificateLink($this->user->getId(), $prg->getId(), $prg->getRefId())) {
                     $properties[$this->lng->txt('certificate')] = $cert_link;
                 }
             } else {
@@ -112,10 +112,6 @@ public function getBlockType(): string
         return 'pdprg';
     }
 
-    public function confirmedRemoveObject(): void
-    {
-    }
-
     public function removeMultipleEnabled(): bool
     {
         return false;