WordPress自定义PHP联系表单发信提示与重定向故障排查

一、重定向死循环（too many redirects）修复 #

这个问题是几个低级错误叠加导致的，按顺序改：

• 立刻删掉代码最顶部、REQUEST_METHOD判断之外的两行wp_redirect赋值代码。wp_redirect本身是直接发送HTTP跳转头的函数，你在全局无判断直接调用，等于访问任何加载这段代码的页面都会触发跳转，跳转到thank-you/error页面后又会重复执行这段跳转逻辑，直接死循环。
• 修正跳转地址拼接错误：你之前写的'"'.home_url().'/thank-you"'会拼出带多余双引号的无效地址，直接用home_url('/thank-you')、home_url('/error')生成地址即可。
• WordPress环境下跳转优先用wp_safe_redirect()，比原生header()和wp_redirect安全，跳转语句后必须加exit;终止后续代码执行，否则程序会继续跑后面的逻辑，引发不可预期的问题。
• 不要把表单处理逻辑放在模板文件全局位置，最好挂到WordPress的template_redirect钩子下执行，确保只有访问表单所在页面时才会跑提交处理逻辑，从根源避免其他页面触发跳转。

正确跳转代码示例：

// 所有校验通过、邮件发送成功后执行
wp_safe_redirect( home_url('/thank-you') );
exit;

// 校验失败、验证码无效、发信失败时执行
wp_safe_redirect( home_url('/error') );
exit;

二、后端PHP校验不生效修复 #

你写的校验规则完全没起作用，对应问题点：

• 手机号校验逻辑写错变量：判断条件里写的是!is_numeric($number)，但你根本没有定义$number变量，实际要校验的是$phone，这段规则永远不会触发。另外注意你占位符写的是带+、空格的手机号格式，is_numeric()会把这类合法格式判定为非法，建议换成正则校验：!preg_match('/^[0-9+\-\s]*$/', $phone)。
• 缺少校验结果拦截逻辑：你现在字段校验规则写完就放那了，没有判断是否存在校验错误，只要请求带了POST参数、验证码过了就直接发信，等于校验规则白写。需要加一个全局错误标记，只要任意字段不符合规则就标记为校验失败，直接终止后续发信逻辑。
• 表单存在嵌套错误：HTML不支持<form>标签嵌套，你代码里套了三层form，最内层带提交按钮的表单action指向了varify_captcha.php，提交的数据根本不会发到你写了校验逻辑的当前页面，这也是校验不生效、JS注入无反应的核心原因。直接删掉所有多余的form标签，整个联系区域只保留一个form，action留空提交到当前页，把reCAPTCHA区块、提交按钮全部放到这个表单内部。

三、前端JS校验说明 #

JS前端校验是合理的优化方向，但绝对不能替代后端PHP校验——前端校验可以被用户轻易绕过（关JS、改浏览器调试代码就能跳过），只能用来做即时反馈提升用户体验，安全校验必须以后端PHP为准。
你之前JS逻辑调不通，大概率是嵌套表单导致提交事件绑定对象错误，等你把表单结构修正后，直接用Bootstrap自带的needs-validation校验逻辑即可，不需要额外写复杂的校验代码。

四、其他代码隐患修复 #

• 发信不要用原生mail()函数，换成WordPress原生的wp_mail()，兼容性更好，邮件进收件箱的概率更高，能自动适配站点的SMTP配置。
• reCAPTCHA接口请求不要用file_get_contents()，很多服务器环境会禁用远程文件读取，换成WordPress原生的wp_remote_post()发起验证请求，稳定性更高。
• 补全所有if/else块的大括号，你现在验证码判断的分支因为缺大括号逻辑完全错位，验证码验证失败的提示和JS注入代码不管验证成功与否都会执行，很容易引发逻辑混乱。
• 删掉表单里重复闭合的label标签（Email、Phone字段的label多写了一个</label>），属于HTML语法错误，虽然不影响核心功能但建议修正。

五、JS注入无反应原因 #

之前echo输出JS不执行有两个原因：一是表单嵌套导致数据根本没提交到当前脚本，那段JS代码压根没输出到页面；二是如果提交后立刻触发跳转，浏览器还没解析执行JS就收到跳转头，自然看不到效果。做提交状态反馈最稳定的方式是直接在thank-you、error页面写固定提示内容，不要靠注入JS实现。

内容的提问来源于stack exchange，提问作者ScorpVisual