lgfa29 / basiclti4moodle

Automatically exported from code.google.com/p/basiclti4moodle
0 stars 0 forks source link

resource_link_description is not plain text #26

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create a link to a test LTI Tool Provider (e.g. 
http://www.imsglobal.org/developers/LTI/test/v1p1/tool.php)
2. Add a description to the link (e.g. "A description")
3. Execute a launch
4. Inspect the value of the resource_link_description parameter

What is the expected output?

"A description"

What do you see instead?

"<p>A description</p>"

What version of the product are you using?

basiclti4moodle-2012-03-15.zip with Moodle 2.2

On what operating system?

Windows 7

Please provide any additional information below.

The IMS spec requires this parameter to be plain text.  It can be fixed by 
changing line 120 of locallib.php to use the strip_tage function:

  "resource_link_description" => strip_tags($instance->intro),

Original issue reported on code.google.com by stephen.vickers@oscelot.org on 15 Jun 2012 at 10:58

GoogleCodeExporter commented 9 years ago
I'll handle this one.

Original comment by ngalanis@gmail.com on 2 Jul 2012 at 9:26

GoogleCodeExporter commented 9 years ago

Original comment by ngalanis@gmail.com on 2 Jul 2012 at 10:00