LTI Allow edit/configure tool with deep linking - refs BT#13469

plugin/ims_lti/configure.php

@@ -62,7 +62,7 @@ switch ($action) {
                     empty($formValues['custom_params']) ? null : $formValues['custom_params']
                 )
                 ->setCourse($course)
-                ->setActiveDeepLinking(false)
+                ->setActiveDeepLinking(!empty($formValues['deep_linking']))
                 ->setPrivacy(
                     !empty($formValues['share_name']),
                     !empty($formValues['share_email']),
@@ -85,7 +85,9 @@ switch ($action) {
             $em->persist($tool);
             $em->flush();
 
+            if (!$tool->isActiveDeepLinking()) {
                 $plugin->addCourseTool($course, $tool);
+            }
 
             Display::addFlash(
                 Display::return_message($plugin->get_lang('ToolAdded'), 'success')

plugin/ims_lti/lang/english.php

@@ -35,3 +35,4 @@ $strings['ShareLauncherEmail'] = 'Share launcher\'s email';
 $strings['ShareLauncherPicture'] = 'Share launcher\'s picture';
 $strings['NoTool'] = 'Tool not exists';
 $strings['ToolAddedOnCourseX'] = 'Tool addeed on course <strong>%s</strong>.';
+$strings['SupportDeppLinkingHelp'] = 'Contact your Tool Provider to verify if Deep Linking support is mandatory';

plugin/ims_lti/lang/french.php

@@ -30,3 +30,9 @@ $strings['ScoreNotSet'] = 'Score non défini';
 $strings['ScoreForXUserIsYScore'] = 'Score pour %s est %s';
 $strings['ToolsAdded'] = 'Outils ajoutés';
 $strings['ToolEdited'] = 'Outil édité';
+$strings['ShareLauncherName'] = 'Share launcher\'s name';
+$strings['ShareLauncherEmail'] = 'Share launcher\'s email';
+$strings['ShareLauncherPicture'] = 'Share launcher\'s picture';
+$strings['NoTool'] = 'Tool not exists';
+$strings['ToolAddedOnCourseX'] = 'Tool addeed on course <strong>%s</strong>.';
+$strings['SupportDeppLinkingHelp'] = 'Contact your Tool Provider to verify if Deep Linking support is mandatory';

plugin/ims_lti/lang/spanish.php

@@ -30,3 +30,9 @@ $strings['ScoreNotSet'] = 'Puntuación no establecida';
 $strings['ScoreForXUserIsYScore'] = 'Puntuación para %s es %s';
 $strings['ToolsAdded'] = 'Herramientas agregadas';
 $strings['ToolEdited'] = 'Herramienta editada';
+$strings['ShareLauncherName'] = 'Enviar el nombre del usuario';
+$strings['ShareLauncherEmail'] = 'Enviar el correo electrónico del usuario';
+$strings['ShareLauncherPicture'] = 'Enviar la foto del usuario';
+$strings['NoTool'] = 'La herramienta no existe';
+$strings['ToolAddedOnCourseX'] = 'Herramienta agregada en el curso <strong>%s</strong>.';
+$strings['SupportDeppLinkingHelp'] = 'Contacte a su Proveedor de Herramienta para verificar si el soporte a Deep Linking es obligatorio';

plugin/ims_lti/src/Form/FrmAdd.php

@@ -58,7 +58,11 @@ class FrmAdd extends FormValidator
         if (null === $this->baseTool ||
             ($this->baseTool && !$this->baseTool->isActiveDeepLinking())
         ) {
-            $this->addCheckBox('deep_linking', null, $plugin->get_lang('SupportDeepLinking'));
+            $this->addCheckBox(
+                'deep_linking',
+                [null, $plugin->get_lang('SupportDeppLinkingHelp'), null],
+                $plugin->get_lang('SupportDeepLinking')
+            );
         }
 
         $this->addHtml('</div>');

plugin/ims_lti/src/Form/FrmEdit.php

@@ -75,7 +75,11 @@ class FrmEdit extends FormValidator
         if (null === $parent ||
             (null !== $parent && !$parent->isActiveDeepLinking())
         ) {
-            $this->addCheckBox('deep_linking', null, $plugin->get_lang('SupportDeepLinking'));
+            $this->addCheckBox(
+                'deep_linking',
+                [null, $plugin->get_lang('SupportDeppLinkingHelp'), null],
+                $plugin->get_lang('SupportDeepLinking')
+            );
         }
 
         $this->addHtml('</div>');

plugin/ims_lti/view/add.tpl

@@ -2,31 +2,55 @@
     {% if global_tools|length or added_tools|length %}
         <div class="col-sm-3">
             {% if added_tools|length %}
-                <h2>{{ 'ToolsAdded'|get_plugin_lang('ImsLtiPlugin') }}</h2>
-                <ul class="nav nav-pills nav-stacked">
+                <div class="panel panel-default">
+                    <div class="panel-heading">
+                        <h2 class="panel-title">{{ 'ToolsAdded'|get_plugin_lang('ImsLtiPlugin') }}</h2>
+                    </div>
+                    <ul class="list-group">
                         {% for tool in added_tools %}
-                        <li class="{{ type == tool.id ? 'active' : '' }}">
+                            <li class="list-group-item {{ type == tool.id ? 'active' : '' }}">
+                                <div class="pull-right">
+                                    {% if tool.isActiveDeepLinking %}
+                                        <a href="{{ _p.web_plugin }}ims_lti/start.php?id={{ tool.id }}&{{ _p.web_cid_query }}">
+                                            {{ 'settings.png'|img(22, 'Configure'|get_lang) }}
+                                        </a>
+                                    {% endif %}
+
                                     <a href="{{ _p.web_plugin }}ims_lti/configure.php?action=edit&id={{ tool.id }}&{{ _p.web_cid_query }}">
-                                {{ tool.name }}
+                                        {{ 'edit.png'|img(22, 'Edit'|get_lang) }}
                                     </a>
+                                </div>
+                                {{ tool.name }}
                             </li>
                         {% endfor %}
                     </ul>
+                </div>
             {% endif %}
 
             {% if global_tools|length %}
-                <h2>{{ 'AvailableTools'|get_plugin_lang('ImsLtiPlugin') }}</h2>
-                <ul class="nav nav-pills nav-stacked">
+                <div class="panel panel-default">
+                    <div class="panel-heading">
+                        <h2 class="panel-title">{{ 'AvailableTools'|get_plugin_lang('ImsLtiPlugin') }}</h2>
+                    </div>
+                    <ul class="list-group">
                         {% for tool in global_tools %}
-                        <li class="{{ type == tool.id ? 'active' : '' }}">
+                            <li class="list-group-item {{ type == tool.id ? 'active' : '' }}">
+                                <div class="pull-right">
                                     {% if tool.isActiveDeepLinking %}
-                                <a href="{{ _p.web_plugin }}ims_lti/start.php?id={{ tool.id }}&{{ _p.web_cid_query }}">{{ tool.name }}</a>
+                                        <a href="{{ _p.web_plugin }}ims_lti/start.php?id={{ tool.id }}&{{ _p.web_cid_query }}">
+                                            {{ 'settings.png'|img(22, 'Configure'|get_lang) }}
+                                        </a>
                                     {% else %}
-                                <a href="{{ _p.web_self }}?type={{ tool.id }}&{{ _p.web_cid_query }}">{{ tool.name }}</a>
+                                        <a href="{{ _p.web_self }}?type={{ tool.id }}&{{ _p.web_cid_query }}">
+                                            {{ 'add.png'|img(22, 'Add'|get_lang) }}
+                                        </a>
                                     {% endif %}
+                                </div>
+                                {{ tool.name }}
                             </li>
                         {% endfor %}
                     </ul>
+                </div>
             {% endif %}
         </div>
     {% endif %}