|Summary:||ASTERISK-08809: [patch] agi dumphtml has incorrect closing tag|
|Reporter:||Sean Bright (seanbright)||Labels:|
|Date Opened:||2007-02-14 13:09:45.000-0600||Date Closed:||2007-06-30 09:19:59|
|Environment:||Attachments:||( 0) res_agi.c.patch|
( 1) res_agi.c.updated.patch
|Description:||Simple fix to close the opening <TH> tag with </TH> instead of </TD>. Patch attached.|
|Comments:||By: Clod Patry (junky) 2007-02-14 13:53:23.000-0600|
I also noted a small bug with final HTML,
will provide another patch tonight.
By: Sean Bright (seanbright) 2007-02-14 14:51:43.000-0600
res_agi.c.updated.patch supersedes res_agi.c.patch
It adds escaping to the <, >, &, and " characters in the AGI output so that they appear in the generated HTML. I don't know if this was the best way to do this, so please let me know if I should change it.
By: Sean Bright (seanbright) 2007-02-14 14:57:57.000-0600
Sorry, in the 'agi dumphtml' output, not the AGI output.
By: Sean Bright (seanbright) 2007-02-15 17:59:02.000-0600
Think you'll get this applied sometime soon?
By: Olle Johansson (oej) 2007-02-16 05:47:23.000-0600
Changed the character in 1.2 and 1.4, which is released code. Gotta be a bit careful with changes there.
Committed the larger change to svn trunk.
By: Olle Johansson (oej) 2007-02-16 05:48:09.000-0600
Resolved in rev 54774 svn trunk.