ZF-11652: addEventListener should also take 3rd argument in ReCaptcha decorator

Issue Type:
Bug
Created:
2011-08-08T19:22:06.000+0000
Last Updated:
2011-08-13T19:42:45.000+0000
Status:
Resolved
Fix version(s):
    Reporter:
    Michiel Thalen (chielsen)
    Assignee:
    Pádraic Brady (padraic)
    Tags:
    • Zend_Form
    • Zend_Service_ReCaptcha
    • zf-crteam-padraic
    • zf-crteam-priority
    Related issues:
    Attachments:

    Description

    The javascript in the Decorator for ReCaptcha throws JS errors in Firefox (even 5.01) as well as IE.

    Fix is to add a 3rd argument:

    windowOnLoad(function(){ document.getElementById("$challengeId").form.addEventListener("submit", function(e) { document.getElementById("$challengeId").value = document.getElementById("recaptcha_challenge_field").value; document.getElementById("$responseId").value = document.getElementById("recaptcha_response_field").value; }, false); });

    Comments

    Posted by Pádraic Brady (padraic) on 2011-08-13T19:42:45.000+0000

    Fixed in r24370. The third param is a Firefox requirement but it's optional under MSIE. Cross testing on Gecko, Webkit is needed for any Javascript.

    Thanks to the reporter!