| 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 |
| Priority: | Trivial | Regression? | No |
| Status: | Closed/Complete | Components: | Resources/res_agi |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| 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. | ||