From d41323365e024b55b79377236998264e0d1da5a8 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 11:41:09 -0500
Subject: [PATCH 01/10] WIP fixing lti plugin - refs BT#13469

---
 plugin/ims_lti/Entity/ImsLtiTool.php | 20 ++++++++++++++++++++
 plugin/ims_lti/ImsLtiPlugin.php      |  7 +++++++
 plugin/ims_lti/configure.php         |  4 +++-
 plugin/ims_lti/form.php              | 10 +++++++---
 plugin/ims_lti/src/Form/FrmAdd.php   |  5 +++--
 plugin/ims_lti/src/Form/FrmEdit.php  |  7 ++++---
 plugin/ims_lti/view/start.tpl        |  2 +-
 7 files changed, 45 insertions(+), 10 deletions(-)

diff --git a/plugin/ims_lti/Entity/ImsLtiTool.php b/plugin/ims_lti/Entity/ImsLtiTool.php
index 12bce88a29..1d4e05f25c 100644
--- a/plugin/ims_lti/Entity/ImsLtiTool.php
+++ b/plugin/ims_lti/Entity/ImsLtiTool.php
@@ -249,6 +249,26 @@ class ImsLtiTool
         return $this->course === null;
     }
 
+    /**
+     * @param array $params
+     *
+     * @return null|string
+     */
+    public function encodeCustomParams(array $params)
+    {
+        if (empty($params)) {
+            return null;
+        }
+
+        $pairs = [];
+
+        foreach ($params as $key => $value) {
+            $pairs[] = "$key=$value";
+        }
+
+        return implode("\n", $pairs);
+    }
+
     /**
      * @return array
      */
diff --git a/plugin/ims_lti/ImsLtiPlugin.php b/plugin/ims_lti/ImsLtiPlugin.php
index 9c81379aac..b51cfe9578 100644
--- a/plugin/ims_lti/ImsLtiPlugin.php
+++ b/plugin/ims_lti/ImsLtiPlugin.php
@@ -394,6 +394,13 @@ class ImsLtiPlugin extends Plugin
                 !empty($contentItem['text']) ? $contentItem['text'] : null
             );
 
+        if (!empty($contentItem['custom'])) {
+            $newLtiTool
+                ->setCustomParams(
+                    $newLtiTool->encodeCustomParams($contentItem['custom'])
+                );
+        }
+
         $em->persist($newLtiTool);
         $em->flush();
 
diff --git a/plugin/ims_lti/configure.php b/plugin/ims_lti/configure.php
index b562f0d8bf..f9eb22f986 100644
--- a/plugin/ims_lti/configure.php
+++ b/plugin/ims_lti/configure.php
@@ -126,7 +126,9 @@ switch ($action) {
 
             $tool
                 ->setName($formValues['name'])
-                ->setDescription($formValues['description'])
+                ->setDescription(
+                    empty($formValues['description']) ? null : $formValues['description']
+                )
                 ->setActiveDeepLinking(
                     !empty($formValues['deep_linking'])
                 )
diff --git a/plugin/ims_lti/form.php b/plugin/ims_lti/form.php
index 247efc620e..43cf8a3d8c 100644
--- a/plugin/ims_lti/form.php
+++ b/plugin/ims_lti/form.php
@@ -82,7 +82,7 @@ $params['roles'] = ImsLtiPlugin::getUserRoles($user);
 if ($tool->isSharingName()) {
     $params['lis_person_name_given'] = $user->getFirstname();
     $params['lis_person_name_family'] = $user->getLastname();
-    $params['lis_person_name_full'] = $user->getCompleteName();
+    $params['lis_person_name_full'] = $user->getFirstname().' '.$user->getLastname();
 }
 
 if ($tool->isSharingEmail()) {
@@ -90,7 +90,11 @@ if ($tool->isSharingEmail()) {
 }
 
 if (api_is_allowed_to_edit(false, true)) {
-    $params['role_scope_mentor'] = ImsLtiPlugin::getRoleScopeMentor($course, $session);
+    $scopeMentor = ImsLtiPlugin::getRoleScopeMentor($course, $session);
+
+    if (!empty($scopeMentor)) {
+        $params['role_scope_mentor'] = $scopeMentor;
+    }
 }
 
 $params['context_id'] = $course->getId();
@@ -132,7 +136,7 @@ $result = $oauth->sign(array(
       encType="application/x-www-form-urlencoded">
     <?php
     foreach ($result["parameters"] as $key => $values) { //Dump parameters
-        echo '<input type="hidden" name="'.$key.'" value="'.$values.'" />';
+        echo '<input type="hidden" name="'.$key.'" value="'.htmlspecialchars($values).'" />'.PHP_EOL;
     }
     ?>
     <button type="submit">
diff --git a/plugin/ims_lti/src/Form/FrmAdd.php b/plugin/ims_lti/src/Form/FrmAdd.php
index 2bfb512712..db0886a13f 100644
--- a/plugin/ims_lti/src/Form/FrmAdd.php
+++ b/plugin/ims_lti/src/Form/FrmAdd.php
@@ -42,8 +42,7 @@ class FrmAdd extends FormValidator
         $this->addTextarea('description', get_lang('Description'));
 
         if (null === $this->baseTool) {
-            $this->addElement('url', 'launch_url', $plugin->get_lang('LaunchUrl'));
-            $this->addRule('launch_url', get_lang('Required'), 'required');
+            $this->addUrl('launch_url', $plugin->get_lang('LaunchUrl'), true);
             $this->addText('consumer_key', $plugin->get_lang('ConsumerKey'));
             $this->addText('shared_secret', $plugin->get_lang('SharedSecret'));
         }
@@ -74,6 +73,8 @@ class FrmAdd extends FormValidator
         $this->addHtml('</div>');
         $this->addButtonCreate($plugin->get_lang('AddExternalTool'));
         $this->applyFilter('__ALL__', 'Security::remove_XSS');
+        $this->applyFilter('__ALL__', 'htmlspecialchars_decode');
+        $this->applyFilter('__ALL__', 'trim');
     }
 
     public function setDefaultValues()
diff --git a/plugin/ims_lti/src/Form/FrmEdit.php b/plugin/ims_lti/src/Form/FrmEdit.php
index b323b222aa..14a29e4e01 100644
--- a/plugin/ims_lti/src/Form/FrmEdit.php
+++ b/plugin/ims_lti/src/Form/FrmEdit.php
@@ -59,8 +59,7 @@ class FrmEdit extends FormValidator
         $this->addTextarea('description', get_lang('Description'));
 
         if (null === $parent) {
-            $this->addElement('url', 'launch_url', $plugin->get_lang('LaunchUrl'));
-            $this->addRule('launch_url', get_lang('Required'), 'required');
+            $this->addUrl('launch_url', $plugin->get_lang('LaunchUrl'), true);
             $this->addText('consumer_key', $plugin->get_lang('ConsumerKey'));
             $this->addText('shared_secret', $plugin->get_lang('SharedSecret'));
         }
@@ -89,10 +88,12 @@ class FrmEdit extends FormValidator
         $this->addCheckBox('share_email', null, $plugin->get_lang('ShareLauncherEmail'));
         $this->addCheckBox('share_picture', null, $plugin->get_lang('ShareLauncherPicture'));
         $this->addHtml('</div>');
-        $this->addButtonCreate($plugin->get_lang('EditExternalTool'));
+        $this->addButtonUpdate($plugin->get_lang('EditExternalTool'));
         $this->addHidden('id', $this->tool->getId());
         $this->addHidden('action', 'edit');
         $this->applyFilter('__ALL__', 'Security::remove_XSS');
+        $this->applyFilter('__ALL__', 'htmlspecialchars_decode');
+        $this->applyFilter('__ALL__', 'trim');
     }
 
     public function setDefaultValues()
diff --git a/plugin/ims_lti/view/start.tpl b/plugin/ims_lti/view/start.tpl
index 7434fcdd1d..a8c13a39fa 100644
--- a/plugin/ims_lti/view/start.tpl
+++ b/plugin/ims_lti/view/start.tpl
@@ -1,5 +1,5 @@
 {%  if tool.description %}
-    <p class="lead">{{ tool.description }}</p>
+    <p class="lead">{{ tool.description|nl2br }}</p>
 {%  endif %}
 <div class="embed-responsive embed-responsive-4by3">
     <iframe src="{{ launch_url }}" class="plugin-ims-lti-iframe"></iframe>

From efdd5ded9b4015fc66169a2c8f3bf487a3085470 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 13:43:04 -0500
Subject: [PATCH 02/10] WIP LTI fix replace service request - refs BT#13469

---
 plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php b/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php
index 685e7b410b..f6b27cab5f 100644
--- a/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php
+++ b/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php
@@ -18,7 +18,7 @@ class ImsLtiServiceReplaceRequest extends ImsLtiServiceRequest
     {
         parent::__construct($xml);
 
-        $this->responseType = ImsLtiServiceResponse::TYPE_DELETE;
+        $this->responseType = ImsLtiServiceResponse::TYPE_REPLACE;
         $this->xmlRequest = $this->xmlRequest->replaceResultRequest;
     }
 

From 2527e87f87a67ae75ae171a674256e3bda2b2f51 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 14:19:32 -0500
Subject: [PATCH 03/10] WIP LTI adding unsupported service response - refs
 BT#13469

---
 .../src/ImsLtiServiceRequestFactory.php       |  8 ++++-
 .../src/ImsLtiServiceResponseFactory.php      |  2 ++
 .../src/ImsLtiServiceResponseStatus.php       |  2 +-
 .../src/ImsLtiServiceUnsupportedRequest.php   | 31 +++++++++++++++++++
 .../src/ImsLtiServiceUnsupportedResponse.php  | 28 +++++++++++++++++
 5 files changed, 69 insertions(+), 2 deletions(-)
 create mode 100644 plugin/ims_lti/src/ImsLtiServiceUnsupportedRequest.php
 create mode 100644 plugin/ims_lti/src/ImsLtiServiceUnsupportedResponse.php

diff --git a/plugin/ims_lti/src/ImsLtiServiceRequestFactory.php b/plugin/ims_lti/src/ImsLtiServiceRequestFactory.php
index 8c8dd4967a..0756e96aab 100644
--- a/plugin/ims_lti/src/ImsLtiServiceRequestFactory.php
+++ b/plugin/ims_lti/src/ImsLtiServiceRequestFactory.php
@@ -16,13 +16,19 @@ class ImsLtiServiceRequestFactory
         $bodyChildren = $xml->imsx_POXBody->children();
 
         if (!empty($bodyChildren)) {
-            switch ($bodyChildren->getName()) {
+            $name = $bodyChildren->getName();
+
+            switch ($name) {
                 case 'replaceResultRequest':
                     return new ImsLtiServiceReplaceRequest($xml);
                 case 'readResultRequest':
                     return new ImsLtiServiceReadRequest($xml);
                 case 'deleteResultRequest':
                     return new ImsLtiServiceDeleteRequest($xml);
+                default:
+                    $name = str_replace(['ResultRequest', 'Request'], '', $name);
+
+                    return new ImsLtiServiceUnsupportedRequest($xml, $name);
             }
         }
 
diff --git a/plugin/ims_lti/src/ImsLtiServiceResponseFactory.php b/plugin/ims_lti/src/ImsLtiServiceResponseFactory.php
index 9cac5508f0..cb8b0ae516 100644
--- a/plugin/ims_lti/src/ImsLtiServiceResponseFactory.php
+++ b/plugin/ims_lti/src/ImsLtiServiceResponseFactory.php
@@ -22,6 +22,8 @@ class ImsLtiServiceResponseFactory
                 return new ImsLtiServiceReadResponse($statusInfo, $bodyParam);
             case ImsLtiServiceResponse::TYPE_DELETE:
                 return new ImsLtiServiceDeleteResponse($statusInfo, $bodyParam);
+            default:
+                return new ImsLtiServiceUnsupportedResponse($statusInfo, $type);
         }
 
         return null;
diff --git a/plugin/ims_lti/src/ImsLtiServiceResponseStatus.php b/plugin/ims_lti/src/ImsLtiServiceResponseStatus.php
index 5f06c2bf91..4fb535d488 100644
--- a/plugin/ims_lti/src/ImsLtiServiceResponseStatus.php
+++ b/plugin/ims_lti/src/ImsLtiServiceResponseStatus.php
@@ -13,7 +13,7 @@ class ImsLtiServiceResponseStatus
     const CODEMAJOR_SUCCESS = 'success';
     const CODEMAJOR_PROCESSING = 'processing';
     const CODEMAJOR_FAILURE = 'failure';
-    const CODEMAJOR_UNSUPPORTED = 'supported';
+    const CODEMAJOR_UNSUPPORTED = 'unsupported';
 
     /**
      * @var string
diff --git a/plugin/ims_lti/src/ImsLtiServiceUnsupportedRequest.php b/plugin/ims_lti/src/ImsLtiServiceUnsupportedRequest.php
new file mode 100644
index 0000000000..28f79f4d6e
--- /dev/null
+++ b/plugin/ims_lti/src/ImsLtiServiceUnsupportedRequest.php
@@ -0,0 +1,31 @@
+<?php
+/* For licensing terms, see /license.txt */
+
+/**
+ * Class ImsLtiServiceUnsupportedRequest.
+ */
+class ImsLtiServiceUnsupportedRequest extends ImsLtiServiceRequest
+{
+    /**
+     * ImsLtiDeleteServiceRequest constructor.
+     *
+     * @param SimpleXMLElement $xml
+     * @param string           $name
+     */
+    public function __construct(SimpleXMLElement $xml, $name)
+    {
+        parent::__construct($xml);
+
+        $this->responseType = $name;
+    }
+
+    protected function processBody()
+    {
+        $this->statusInfo
+            ->setSeverity(ImsLtiServiceResponseStatus::SEVERITY_STATUS)
+            ->setCodeMajor(ImsLtiServiceResponseStatus::CODEMAJOR_UNSUPPORTED)
+            ->setDescription(
+                $this->responseType.' is not supported'
+            );
+    }
+}
diff --git a/plugin/ims_lti/src/ImsLtiServiceUnsupportedResponse.php b/plugin/ims_lti/src/ImsLtiServiceUnsupportedResponse.php
new file mode 100644
index 0000000000..1d45f583cb
--- /dev/null
+++ b/plugin/ims_lti/src/ImsLtiServiceUnsupportedResponse.php
@@ -0,0 +1,28 @@
+<?php
+/* For licensing terms, see /license.txt */
+
+/**
+ * Class ImsLtiServiceUnsupportedResponse.
+ */
+class ImsLtiServiceUnsupportedResponse extends ImsLtiServiceResponse
+{
+    /**
+     * ImsLtiServiceUnsupportedResponse constructor.
+     *
+     * @param ImsLtiServiceResponseStatus $statusInfo
+     * @param string                      $type
+     */
+    public function __construct(ImsLtiServiceResponseStatus $statusInfo, $type)
+    {
+        $statusInfo->setOperationRefIdentifier($type);
+
+        parent::__construct($statusInfo);
+    }
+
+    /**
+     * @param SimpleXMLElement $xmlBody
+     */
+    protected function generateBody(SimpleXMLElement $xmlBody)
+    {
+    }
+}

From c4211136e1fad4ace7e7303520d5bb110ad60a55 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 14:47:15 -0500
Subject: [PATCH 04/10] LTI fix read result service to allow zero value - refs
 BT#13469

---
 plugin/ims_lti/src/ImsLtiServiceReadRequest.php | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/plugin/ims_lti/src/ImsLtiServiceReadRequest.php b/plugin/ims_lti/src/ImsLtiServiceReadRequest.php
index d58881ab99..6f22ca545b 100644
--- a/plugin/ims_lti/src/ImsLtiServiceReadRequest.php
+++ b/plugin/ims_lti/src/ImsLtiServiceReadRequest.php
@@ -51,16 +51,17 @@ class ImsLtiServiceReadRequest extends ImsLtiServiceRequest
         if (!empty($results)) {
             /** @var Result $result */
             $result = $results[0];
+            $ltiScore = 0;
 
             if (!empty($result->get_score())) {
                 $ltiScore = $result->get_score() / $evaluation->getMax();
-
-                $responseDescription = sprintf(
-                    get_plugin_lang('ScoreForXUserIsYScore', 'ImsLtiPlugin'),
-                    $user->getId(),
-                    $ltiScore
-                );
             }
+
+            $responseDescription = sprintf(
+                get_plugin_lang('ScoreForXUserIsYScore', 'ImsLtiPlugin'),
+                $user->getId(),
+                $ltiScore
+            );
         }
 
         $this->statusInfo

From 787eceee8c2621b80d5b0a0e27f66c2aa424aa52 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 15:23:44 -0500
Subject: [PATCH 05/10] LTI reject non-numeric value in replace service - refs
 BT#13469

---
 plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php b/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php
index f6b27cab5f..67569ccd93 100644
--- a/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php
+++ b/plugin/ims_lti/src/ImsLtiServiceReplaceRequest.php
@@ -26,7 +26,17 @@ class ImsLtiServiceReplaceRequest extends ImsLtiServiceRequest
     {
         $resultRecord = $this->xmlRequest->resultRecord;
         $sourcedId = (string) $resultRecord->sourcedGUID->sourcedId;
-        $resultScore = (float) $resultRecord->result->resultScore->textString;
+        $resultScore = (string) $resultRecord->result->resultScore->textString;
+
+        if (!is_numeric($resultScore)) {
+            $this->statusInfo
+                ->setSeverity(ImsLtiServiceResponseStatus::SEVERITY_ERROR)
+                ->setCodeMajor(ImsLtiServiceResponseStatus::CODEMAJOR_FAILURE);
+
+            return;
+        }
+
+        $resultScore = (float) $resultScore;
 
         if (0 > $resultScore || 1 < $resultScore) {
             $this->statusInfo

From 8628fa6fceed2fb1fa180ff83052980225bcd266 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 15:42:47 -0500
Subject: [PATCH 06/10] LTI fix role_score_mentor param - refs BT#13469

---
 plugin/ims_lti/ImsLtiPlugin.php | 32 +++++++++++++++++---------------
 plugin/ims_lti/form.php         |  4 ++--
 2 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/plugin/ims_lti/ImsLtiPlugin.php b/plugin/ims_lti/ImsLtiPlugin.php
index b51cfe9578..1f85736fe5 100644
--- a/plugin/ims_lti/ImsLtiPlugin.php
+++ b/plugin/ims_lti/ImsLtiPlugin.php
@@ -298,18 +298,22 @@ class ImsLtiPlugin extends Plugin
      */
     public static function getUserRoles(User $user)
     {
+        if (DRH === $user->getStatus()) {
+            return 'urn:lti:role:ims/lis/Mentor';
+        }
+
         if ($user->getStatus() === INVITEE) {
-            return 'Learner/GuestLearner,Learner';
+            return 'Learner,urn:lti:role:ims/lis/Learner/GuestLearner';
         }
 
         if (!api_is_allowed_to_edit(false, true)) {
-            return 'Learner,Learner/Learner';
+            return 'Learner';
         }
 
         $roles = ['Instructor'];
 
         if (api_is_platform_admin_by_id($user->getId())) {
-            $roles[] = 'Administrator/SystemAdministrator';
+            $roles[] = 'urn:lti:role:ims/lis/Administrator';
         }
 
         return implode(',', $roles);
@@ -331,24 +335,22 @@ class ImsLtiPlugin extends Plugin
     }
 
     /**
-     * @param Course       $course
-     * @param Session|null $session
+     * @param User $currentUser
      *
      * @return string
      */
-    public static function getRoleScopeMentor(Course $course, Session $session = null)
+    public static function getRoleScopeMentor(User $currentUser)
     {
-        $scope = [];
-
-        if ($session) {
-            $students = $session->getUserCourseSubscriptionsByStatus($course, Session::STUDENT);
-        } else {
-            $students = $course->getStudents();
+        if (DRH !== $currentUser->getStatus()) {
+            return '';
         }
 
-        /** @var SessionRelCourseRelUser|CourseRelUser $subscription */
-        foreach ($students as $subscription) {
-            $scope[] = self::generateToolUserId($subscription->getUser()->getId());
+        $followedUsers = UserManager::get_users_followed_by_drh($currentUser->getId());
+
+        $scope = [];
+
+        foreach ($followedUsers as $userInfo) {
+            $scope[] = self::generateToolUserId($userInfo['user_id']);
         }
 
         return implode(',', $scope);
diff --git a/plugin/ims_lti/form.php b/plugin/ims_lti/form.php
index 43cf8a3d8c..ded7e2a42b 100644
--- a/plugin/ims_lti/form.php
+++ b/plugin/ims_lti/form.php
@@ -89,8 +89,8 @@ if ($tool->isSharingEmail()) {
     $params['lis_person_contact_email_primary'] = $user->getEmail();
 }
 
-if (api_is_allowed_to_edit(false, true)) {
-    $scopeMentor = ImsLtiPlugin::getRoleScopeMentor($course, $session);
+if (DRH === $user->getStatus()) {
+    $scopeMentor = ImsLtiPlugin::getRoleScopeMentor($user);
 
     if (!empty($scopeMentor)) {
         $params['role_scope_mentor'] = $scopeMentor;

From 01be29c192735645ad3acf1c6f3fd6e1c57b3231 Mon Sep 17 00:00:00 2001
From: Yannick Warnier <ywarnier@beeznest.org>
Date: Wed, 14 Nov 2018 16:42:14 -0500
Subject: [PATCH 07/10] Add twitter:title, description, image and image:alt
 meta tags to headers

---
 main/inc/lib/template.lib.php | 24 +++++++++++++++++++-----
 1 file changed, 19 insertions(+), 5 deletions(-)

diff --git a/main/inc/lib/template.lib.php b/main/inc/lib/template.lib.php
index ddafc20605..d2a87662b7 100755
--- a/main/inc/lib/template.lib.php
+++ b/main/inc/lib/template.lib.php
@@ -1852,20 +1852,25 @@ class Template
                     $course = api_get_course_entity($courseId);
                     // @TODO: support right-to-left in title
                     $socialMeta .= '<meta property="og:title" content="'.$course->getTitle().' - '.$metaTitle.'" />'."\n";
+                    $socialMeta .= '<meta property="twitter:title" content="'.$course->getTitle().' - '.$metaTitle.'" />'."\n";
                     $socialMeta .= '<meta property="og:url" content="'.api_get_course_url($course->getCode()).'" />'."\n";
 
                     $metaDescription = api_get_setting('meta_description');
                     if (!empty($course->getDescription())) {
                         $socialMeta .= '<meta property="og:description" content="'.strip_tags($course->getDescription()).'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:description" content="'.strip_tags($course->getDescription()).'" />'."\n";
                     } elseif (!empty($metaDescription)) {
                         $socialMeta .= '<meta property="og:description" content="'.$metaDescription.'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:description" content="'.$metaDescription.'" />'."\n";
                     }
 
                     $picture = CourseManager::getPicturePath($course, true);
                     if (!empty($picture)) {
                         $socialMeta .= '<meta property="og:image" content="'.$picture.'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:image" content="'.$picture.'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:image:alt" content="'.$course->getTitle().' - '.$metaTitle.'" />'."\n";
                     } else {
-                        $socialMeta .= $this->getMetaPortalImagePath();
+                        $socialMeta .= $this->getMetaPortalImagePath($metaTitle);
                     }
                 } elseif ($sessionId !== 0) {
                     // If we are on a session "about" screen, publish info about the session
@@ -1873,6 +1878,7 @@ class Template
                     $session = $em->find('ChamiloCoreBundle:Session', $sessionId);
 
                     $socialMeta .= '<meta property="og:title" content="'.$session->getName().' - '.$metaTitle.'" />'."\n";
+                    $socialMeta .= '<meta property="twitter:title" content="'.$session->getName().' - '.$metaTitle.'" />'."\n";
                     $socialMeta .= '<meta property="og:url" content="'.api_get_path(WEB_PATH)."session/{$session->getId()}/about/".'" />'."\n";
 
                     $sessionValues = new ExtraFieldValue('session');
@@ -1882,19 +1888,23 @@ class Template
                     if (!empty($sessionImage) && is_file($sessionImageSysPath)) {
                         $sessionImagePath = api_get_path(WEB_UPLOAD_PATH).$sessionImage;
                         $socialMeta .= '<meta property="og:image" content="'.$sessionImagePath.'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:image" content="'.$sessionImagePath.'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:image:alt" content="'.$session->getName().' - '.$metaTitle.'" />'."\n";
                     } else {
-                        $socialMeta .= $this->getMetaPortalImagePath();
+                        $socialMeta .= $this->getMetaPortalImagePath($metaTitle);
                     }
                 } else {
                     // Otherwise (not a course nor a session, nor a user, nor a badge), publish portal info
                     $socialMeta .= '<meta property="og:title" content="'.$metaTitle.'" />'."\n";
+                    $socialMeta .= '<meta property="twitter:title" content="'.$metaTitle.'" />'."\n";
                     $socialMeta .= '<meta property="og:url" content="'.api_get_path(WEB_PATH).'" />'."\n";
 
                     $metaDescription = api_get_setting('meta_description');
                     if (!empty($metaDescription)) {
                         $socialMeta .= '<meta property="og:description" content="'.$metaDescription.'" />'."\n";
+                        $socialMeta .= '<meta property="twitter:description" content="'.$metaDescription.'" />'."\n";
                     }
-                    $socialMeta .= $this->getMetaPortalImagePath();
+                    $socialMeta .= $this->getMetaPortalImagePath($metaTitle);
                 }
             }
         }
@@ -1906,10 +1916,10 @@ class Template
 
     /**
      * Get platform meta image tag (check meta_image_path setting, then use the logo).
-     *
+     * @param string $imageAlt The alt attribute for the image
      * @return string The meta image HTML tag, or empty
      */
-    private function getMetaPortalImagePath()
+    private function getMetaPortalImagePath($imageAlt = '')
     {
         // Load portal meta image if defined
         $metaImage = api_get_setting('meta_image_path');
@@ -1919,11 +1929,15 @@ class Template
         if (!empty($metaImage)) {
             if (is_file($metaImageSysPath)) {
                 $portalImageMeta = '<meta property="og:image" content="'.$metaImageWebPath.'" />'."\n";
+                $portalImageMeta .= '<meta property="twitter:image" content="'.$metaImageWebPath.'" />'."\n";
+                $portalImageMeta .= '<meta property="twitter:image:alt" content="'.$imageAlt.'" />'."\n";
             }
         } else {
             $logo = ChamiloApi::getPlatformLogoPath($this->theme);
             if (!empty($logo)) {
                 $portalImageMeta = '<meta property="og:image" content="'.$logo.'" />'."\n";
+                $portalImageMeta .= '<meta property="twitter:image" content="'.$logo.'" />'."\n";
+                $portalImageMeta .= '<meta property="twitter:image:alt" content="'.$imageAlt.'" />'."\n";
             }
         }
 

From 51a51a1e48dd6fe9bc291a67c0d36d85666c4aef Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Wed, 14 Nov 2018 17:03:07 -0500
Subject: [PATCH 08/10] Fix tool introduction in gradebook - refs BT#14835

---
 main/gradebook/index.php | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/main/gradebook/index.php b/main/gradebook/index.php
index 4ae86f2b86..04feeeb673 100755
--- a/main/gradebook/index.php
+++ b/main/gradebook/index.php
@@ -772,6 +772,12 @@ if (!api_is_allowed_to_edit(null, true)) {
 if (isset($first_time) && $first_time == 1 && api_is_allowed_to_edit(null, true)) {
     echo '<meta http-equiv="refresh" content="0;url='.api_get_self().'?'.api_get_cidreq().'" />';
 } else {
+    // Tool introduction
+    Display::display_introduction_section(
+        TOOL_GRADEBOOK,
+        ['ToolbarSet' => 'AssessmentsIntroduction']
+    );
+
     if (!empty($actionsLeft)) {
         echo $toolbar = Display::toolbarAction(
             'gradebook-student-actions',
@@ -780,11 +786,6 @@ if (isset($first_time) && $first_time == 1 && api_is_allowed_to_edit(null, true)
     }
 
     if (api_is_allowed_to_edit(null, true)) {
-        // Tool introduction
-        Display::display_introduction_section(
-            TOOL_GRADEBOOK,
-            ['ToolbarSet' => 'AssessmentsIntroduction']
-        );
 
         if (((empty($selectCat)) || (isset($_GET['cidReq']) && $_GET['cidReq'] !== '')) ||
             (isset($_GET['isStudentView']) && $_GET['isStudentView'] == 'false')

From c5df881babf1b363ce6bfe7aaf8b9072e82ee354 Mon Sep 17 00:00:00 2001
From: Yannick Warnier <ywarnier@beeznest.org>
Date: Wed, 14 Nov 2018 17:48:09 -0500
Subject: [PATCH 09/10] Behat: Add waitVeryLongForThePageToBeLoaded() to wait 6
 seconds instead of 3

---
 tests/behat/features/bootstrap/FeatureContext.php | 10 +++++++++-
 tests/behat/features/toolExercise.feature         |  4 ++--
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/tests/behat/features/bootstrap/FeatureContext.php b/tests/behat/features/bootstrap/FeatureContext.php
index f891fb5788..e0bd97c381 100644
--- a/tests/behat/features/bootstrap/FeatureContext.php
+++ b/tests/behat/features/bootstrap/FeatureContext.php
@@ -384,10 +384,18 @@ class FeatureContext extends MinkContext
      */
     public function waitForThePageToBeLoaded()
     {
-        //$this->getSession()->wait(10000, "document.readyState === 'complete'");
         $this->getSession()->wait(3000);
     }
 
+    /**
+     * @When /^wait very long for the page to be loaded$/
+     */
+    public function waitVeryLongForThePageToBeLoaded()
+    {
+        //$this->getSession()->wait(10000, "document.readyState === 'complete'");
+        $this->getSession()->wait(6000);
+    }
+
     /**
      * @When /^I check the "([^"]*)" radio button$/
      */
diff --git a/tests/behat/features/toolExercise.feature b/tests/behat/features/toolExercise.feature
index a9184a225c..73e94d3b06 100644
--- a/tests/behat/features/toolExercise.feature
+++ b/tests/behat/features/toolExercise.feature
@@ -384,7 +384,7 @@ Feature: Exercise tool
       | none          | 80 / 60        | 133.33%        |
       | Total         | 190 / 190      | 100%           |
 
-  Scenario: Teacher see exercise results by categories
+  Scenario: Teacher looks at exercise results by categories
     Given I am on "/user_portal.php"
     And I am on course "TEMP" homepage in session "Session Exercise"
     Then I should see "TEMP (Session Exercise)"
@@ -392,7 +392,7 @@ Feature: Exercise tool
     And I follow "Exercise for Behat test"
     And I follow "Results and feedback"
     Then I should see "Learner score"
-    And wait for the page to be loaded
+    And wait very long for the page to be loaded
     And I follow "Grade activity"
     Then I should see "Score for the test: 190 / 190"
     And I should see the table "#category_results":

From 70038185b49094e8db18beeb28dc354cbf9d0894 Mon Sep 17 00:00:00 2001
From: Yannick Warnier <ywarnier@beeznest.org>
Date: Wed, 14 Nov 2018 23:18:39 -0500
Subject: [PATCH 10/10] Update language terms

---
 main/lang/english/trad4all.inc.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/main/lang/english/trad4all.inc.php b/main/lang/english/trad4all.inc.php
index a9a2d41b04..a8fdbfa9bb 100644
--- a/main/lang/english/trad4all.inc.php
+++ b/main/lang/english/trad4all.inc.php
@@ -7582,7 +7582,7 @@ $SkillProfile = "Skill profile";
 $AchievedSkills = "Achieved skills";
 $BusinessCard = "Business card";
 $BadgeDetails = "Badge details";
-$TheUserXNotYetAchievedTheSkillX = "The user %s not yet achieved the skill \"%s\"";
+$TheUserXNotYetAchievedTheSkillX = "The user %s has not achieved the skill \"%s\" yet";
 $IssuedBadgeInformation = "Issued badge information";
 $RecipientDetails = "Recipient details";
 $SkillAcquiredAt = "Skill acquired at";