Comma replace messing up with javascript in Reports

UniversityOfAlberta / GrandForum

GRAND Forum

GNU General Public License v2.0

2 stars 2 forks source link

Comma replace messing up with javascript in Reports #97

Closed rjynn closed 8 years ago

rjynn commented 8 years ago

The recent changes with "str_replace(",", ",", $string)" breaks javascript that is in the reports. Example from FEC.xml: <script type='text/javascript'> var intervalId = setInterval(function(){ (CODE) }, 100); </script> The comma is replaced.

Finalfantasykid commented 8 years ago

This should be fixed as of bc3be1d3a114c0d6155da519517f547561105aa2

Could you verify?

rjynn commented 8 years ago

fixed :)