MDL-52229 tool_lp: Handle hidden templates/frameworks

moodle · Apr 18, 2016 · 964afa9 · 964afa9
1 parent ed549db
commit 964afa9
Show file tree

Hide file tree

Showing 25 changed files with 226 additions and 170 deletions.
diff --git a/admin/tool/lp/amd/build/competencyactions.min.js b/admin/tool/lp/amd/build/competencyactions.min.js
diff --git a/admin/tool/lp/amd/build/competencypicker.min.js b/admin/tool/lp/amd/build/competencypicker.min.js
diff --git a/admin/tool/lp/amd/src/competencyactions.js b/admin/tool/lp/amd/src/competencyactions.js
@@ -376,7 +376,6 @@ define(['jquery',
             idnumber: relatedTarget.idnumber,
             description: relatedTarget.description,
             descriptionformat: relatedTarget.descriptionformat,
-            visible: relatedTarget.visible,
             ruletype: config.ruletype,
             ruleoutcome: config.ruleoutcome,
             ruleconfig: config.ruleconfig

diff --git a/admin/tool/lp/amd/src/competencypicker.js b/admin/tool/lp/amd/src/competencypicker.js
@@ -80,6 +80,8 @@ define(['jquery',
     Picker.prototype._singleFramework = false;
     /** @type {Boolean} Do we allow multi select? */
     Picker.prototype._multiSelect = true;
+    /** @type {Boolean} Do we allow to display hidden framework? */
+    Picker.prototype._onlyVisible = true;
 
     /**
      * Hook to executed after the view is rendered.
@@ -324,7 +326,8 @@ define(['jquery',
                 { methodname: 'tool_lp_list_competency_frameworks', args: {
                     sort: 'shortname',
                     context: { contextid: self._pageContextId },
-                    includes: self._pageContextIncludes
+                    includes: self._pageContextIncludes,
+                    onlyvisible: self._onlyVisible
                 }}
             ])[0];
         }