Changeset 270 for trunk/trax/vendor

Show
Ignore:
Timestamp:
10/31/06 19:52:18 (6 years ago)
Author:
john
Message:

escape_javacript fix from grabmail

Files:
1 modified

Legend:

Unmodified
Added
Removed
  • trunk/trax/vendor/trax/action_view/helpers/javascript_helper.php

    r229 r270  
    578578    # Escape carrier returns and single and double quotes for JavaScript segments. 
    579579    function escape_javascript($javascript) { 
    580         return preg_replace('/\r\n|\n|\r/', "\\n", 
    581                preg_replace_callback('/["\']/', create_function('$m', 'return "\\{$m}";'), 
    582                (!is_null($javascript) ? $javascript : ''))); 
     580        $escape = array( 
     581            "\r\n"  => '\n', 
     582            "\r"    => '\n', 
     583            "\n"    => '\n', 
     584            '"'     => '\"', 
     585            "'"     => "\\'" 
     586        ); 
     587        return str_replace(array_keys($escape), array_values($escape), $javascript);  
     588        #return preg_replace('/\r\n|\n|\r/', "\\n", 
     589        #       preg_replace_callback('/["\']/', create_function('$m', 'return "\\{$m}";'), 
     590        #       (!is_null($javascript) ? $javascript : ''))); 
    583591    } 
    584592