Download & Extend
Webform » Issues

Strip tags off confirmation field to ensure wasn't broken by WYSIWYG

Posted by DamienMcKenna on October 1, 2009 at 8:37pm
Project:Webform
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

I've had occasions where I set the "confirmation" node field to e.g. "internal:node/123" but in dealing with a WYSIWYG editor ended up wrapping it in paragraph tags.

Login or register to post comments

Comments

#1

Posted by DamienMcKenna on October 1, 2009 at 8:44pm
Status:active» needs review

First effort at a patch, it only deals with the "internal:" option right now because the other types could legitimately contain HTML.

AttachmentSize
webform-n593496.patch 1.1 KB

#2

Posted by quicksketch on November 6, 2009 at 12:40am
Title:Strip tags off confirmation field to ensure wasn't broken by WYSIWYG?» Strip tags off confirmation field to ensure wasn't broken by WYSIWYG
Status:needs review» fixed

Thanks DamienMcKenna, committed to all branches.

#3

Posted by System Message on November 20, 2009 at 12:40am
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here