From 82ae6b56d91138a52b718fd221bd97a4748de01a Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Thu, 25 Mar 2021 02:44:30 -0500
Subject: [PATCH] TopLinks: Reset cid on admin page + improve messages - refs
 BT#18474

---
 plugin/toplinks/admin.php        | 19 +++++++++++++++++--
 plugin/toplinks/src/LinkForm.php |  6 ------
 2 files changed, 17 insertions(+), 8 deletions(-)

diff --git a/plugin/toplinks/admin.php b/plugin/toplinks/admin.php
index f77f33b2c1..8f481f9c0f 100644
--- a/plugin/toplinks/admin.php
+++ b/plugin/toplinks/admin.php
@@ -5,6 +5,8 @@
 use Chamilo\PluginBundle\Entity\TopLinks\TopLink;
 use Chamilo\PluginBundle\TopLinks\Form\LinkForm as TopLinkForm;
 
+$cidReset = true;
+
 require_once __DIR__.'/../../main/inc/global.inc.php';
 
 api_protect_admin_script();
@@ -16,6 +18,7 @@ $pageBaseUrl = api_get_self();
 $em = Database::getManager();
 $linkRepo = $em->getRepository(TopLink::class);
 
+$pageTitle = $plugin->get_title();
 $pageActions = Display::url(
     Display::return_icon('back.png', get_lang('Back'), [], ICON_SIZE_MEDIUM),
     $pageBaseUrl
@@ -83,9 +86,18 @@ switch ($httpRequest->query->getAlpha('action', 'list')) {
             }
         );
 
-        $pageContent = $table->return_table();
-        break;
+        if ($table->total_number_of_items) {
+            $pageContent = $table->return_table();
+        } else {
+            $pageContent = Display::return_message(
+                get_lang('NoData'),
+                'info'
+            );
+        }
+    break;
     case 'add':
+        $pageTitle = get_lang('LinkAdd');
+
         $form = new TopLinkForm();
         $form->createElements();
 
@@ -112,6 +124,8 @@ switch ($httpRequest->query->getAlpha('action', 'list')) {
         $pageContent = $form->returnForm();
         break;
     case 'edit':
+        $pageTitle = get_lang('LinkMod');
+
         $link = $em->find(TopLink::class, $httpRequest->query->getInt('link'));
 
         if (null === $link) {
@@ -173,6 +187,7 @@ switch ($httpRequest->query->getAlpha('action', 'list')) {
 }
 
 $view = new Template($plugin->get_title());
+$view->assign('header', $pageTitle);
 $view->assign('actions', Display::toolbarAction('xapi_actions', [$pageActions]));
 $view->assign('content', $pageContent);
 $view->display_one_col_template();
diff --git a/plugin/toplinks/src/LinkForm.php b/plugin/toplinks/src/LinkForm.php
index bf694d301d..6396725dab 100644
--- a/plugin/toplinks/src/LinkForm.php
+++ b/plugin/toplinks/src/LinkForm.php
@@ -58,12 +58,6 @@ class LinkForm extends FormValidator
 
     public function createElements()
     {
-        if ($this->link) {
-            $this->addHeader(get_lang('LinkMod'));
-        } else {
-            $this->addHeader(get_lang('LinkAdd'));
-        }
-
         $this->addText('title', get_lang('LinkName'));
         $this->addUrl('url', 'URL');
         $this->addRule('url', get_lang('GiveURL'), 'url');