#362 Mismatch between documentation and code

closed-fixed
core (125)
5
2008-05-03
2007-12-21
Dan Pascu
No

The s.escape.common doc says it will escape single and double quotes, backslashes and 0 chars. However the double quote is not escaped.

Discussion

  • Logged In: YES
    user_id=337916
    Originator: NO

    Documentation fixed for 1.3 branch and trunk.

    Cheers,

    Henning

     
    • assigned_to: nobody --> henningw
    • status: open --> closed-fixed
     
  • Dan Pascu
    Dan Pascu
    2008-03-07

    • status: closed-fixed --> open-fixed
     
  • Dan Pascu
    Dan Pascu
    2008-03-07

    Logged In: YES
    user_id=1296758
    Originator: YES

    I opened this ticket for the issue to be fixed in the code (I consider the docs to be correct in this case), not to remove the reference to double quotes from the docs.
    s.escape.common should escape double quotes too, as it was initially written in the docs.

     
  • Dan Pascu
    Dan Pascu
    2008-03-07

    • assigned_to: henningw --> nobody
    • status: open-fixed --> open
     
  • Logged In: YES
    user_id=1246013
    Originator: NO

    Escaping double quotes now in code.

     
    • assigned_to: nobody --> miconda
    • status: open --> closed-fixed