Implemented the task - CD-418 and Bug fixes - CD-417.

Author: vasanthlmsace

lang/en/local_learningtools.php

@@ -246,4 +246,4 @@
 $string['strftimemonthnamedate'] = '%B %d, %Y';
 $string['subplugintype_ltool_plural'] = 'Learning Tools';
 $string['strbookmarked'] = "<i class='fa fa-bookmark'></i> Bookmarked";
-$string['strbookmark'] = "<i class='fa fa-bookmark'></i> Bookmark";
+$string['strbookmark'] = "<i class='fa fa-bookmark'></i> Bookmark";

ltool/bookmarks/amd/build/learningbookmarks.min.js

@@ -182,22 +182,22 @@ define(['core/str', 'core/ajax', 'core/notification', 'jquery'],
                 let bookmarkmarked = document.getElementById('bookmarks-marked');
                 if (response.bookmarksstatus) {
                     if (button) {
-                        button.addClass('active');
-                        button.addClass('btn-link');
-                        button.find('i').removeClass('fa-bookmark-o').addClass('fa-bookmark');
-                        String.get_string('strbookmarked', 'local_learningtools').then(function(langString) {
-                            button.html(langString);
+                        require(['mod_contentdesigner/elements'], function(Elements) {
+                            var chapterId = formData.itemid;
+                            if (chapterId) {
+                                Elements.refreshContent();
+                            }
                         });
                     } else {
                         bookmarkmarked.classList.add('marked');
                     }
                 } else {
                     if (button) {
-                        button.removeClass('active');
-                        button.removeClass('btn-link');
-                        button.find('i').removeClass('fa-bookmark').addClass('fa-bookmark-o');
-                        String.get_string('strbookmark', 'local_learningtools').then(function(langString) {
-                            button.html(langString);
+                        require(['mod_contentdesigner/elements'], function(Elements) {
+                            var chapterId = formData.itemid;
+                            if (chapterId) {
+                                Elements.refreshContent();
+                            }
                         });
                     } else {
                         bookmarkmarked.classList.remove('marked');

ltool/bookmarks/amd/build/learningbookmarks.min.js.map

@@ -33,6 +33,41 @@
  */
 class bookmarkstool_filter {
 
+    /**
+     * @var int
+     */
+    public $userid;
+
+    /**
+     * @var int
+     */
+    public $courseid;
+
+    /**
+     * @var int
+     */
+    public $child;
+
+    /**
+     * @var array
+     */
+    public $urlparams;
+
+    /**
+     * @var int
+     */
+    public $teacher;
+
+    /**
+     * @var string
+     */
+    public $baseurl;
+
+    /**
+     * @var string
+     */
+    public $pageurl;
+
     /**
      * Loads bookmarks tools info.
      * @param int $userid current userid

ltool/bookmarks/amd/src/learningbookmarks.js

@@ -26,6 +26,7 @@
 defined('MOODLE_INTERNAL') || die();
 
 require_once($CFG->dirroot. '/local/learningtools/lib.php');
+
 /**
  * Defines ltool bookmarks nodes for my profile navigation tree.
  *
@@ -119,6 +120,8 @@ function ltool_bookmarks_user_save_bookmarks($contextid, $data) {
     if ($itemtype == 'chapter') {
         $sql .= " AND itemtype = ? AND itemid = ?";
         $params = array_merge($params , [$itemtype, $itemid]);
+    } else {
+        $sql .= " AND itemtype = ''";
     }
     $bookrecord = $DB->get_record_sql($sql, $params);
 
@@ -167,6 +170,8 @@ function ltool_bookmarks_user_save_bookmarks($contextid, $data) {
         if ($itemtype == 'chapter') {
             $selectdelete .= " AND itemtype = ? AND itemid = ?";
             $deletedparams = array_merge($deletedparams, [$itemtype, $itemid]);
+        } else {
+            $selectdelete .= " AND itemtype = ''";
         }
         $DB->delete_records_select('ltool_bookmarks_data', $selectdelete, $deletedparams);
             // Add event to user delete the bookmark.
@@ -247,7 +252,8 @@ function ltool_bookmarks_check_page_bookmarks_exist($contextid, $pageurl, $useri
         FROM {ltool_bookmarks_data}
         WHERE " . $DB->sql_compare_text('pageurl', 255). " = " . $DB->sql_compare_text('?', 255) . "
         AND contextid = ?
-        AND userid = ?";
+        AND userid = ?
+        AND itemtype = ''";
     $params = array($pageurl, $contextid, $userid);
     if ($DB->record_exists_sql($sql, $params)) {
         $pagebookmarks = true;