silverstripe / silverstripe-dms

Adds a Document Management System to Silverstripe
BSD 3-Clause "New" or "Revised" License
40 stars 52 forks source link

broken download link #14

Closed dacar closed 11 years ago

dacar commented 11 years ago

Hi, with SS3.1 there seem to be a problem with the downloadlinks. Here is an example: <a title="1 - Leitfaden der Grundschule" href="/dmsdocument/1&quot; class=&quot;data:{size:'40 KB'} documentLink pdf">hier</a>

Changing line 948 in DMSDocument.php from $linkText = $document->getLink()."\" class=\"$size documentLink $extension"; to $linkText = $document->getLink(); fixed it.

cam-findlay commented 11 years ago

perhaps it is just missing an escape character in that class attribute?

cam-findlay commented 11 years ago

Had a quick chat to Julian (the creator of the module) this issue may be related to SS3.1 shortcode handling.