← All changes | includes/Frontend/Form/FrontendFormHandler.php +299 -70 3.1.3 → V3.3.2 View file →

		@@ -8,8 +8,9 @@
8	8
9	9	use BitCode\BitForm\Admin\Form\AdminFormHandler;
10	10	use BitCode\BitForm\Admin\Form\FrontEndScriptGenerator;
11	11	use BitCode\BitForm\Admin\Form\Helpers;
	12	+use BitCode\BitForm\Core\Cryptography\Cryptography;
12	13	use BitCode\BitForm\Core\Database\FormEntryMetaModel;
13	14	use BitCode\BitForm\Core\Database\FormModel;
14	15	use BitCode\BitForm\Core\Form\FormManager;
15	16	use BitCode\BitForm\Core\Integration\IntegrationHandler;
		@@ -15,15 +16,21 @@
15	16	use BitCode\BitForm\Core\Integration\IntegrationHandler;
16	17	use BitCode\BitForm\Core\Util\EscapingHelper;
17	18	use BitCode\BitForm\Core\Util\FieldValueHandler;
18	19	use BitCode\BitForm\Core\Util\FileDownloadProvider;
	20	+use BitCode\BitForm\Core\Util\FileHandler;
19	21	use BitCode\BitForm\Core\Util\FrontendHelpers;
	22	+use BitCode\BitForm\Core\Util\Log;
20	23	use BitCode\BitForm\Core\Util\SmartTagRegistry;
21	24	use BitCode\BitForm\Core\Util\SmartTags;
	25	+use BitCode\BitForm\Core\Util\Utilities;
22	26	use BitCode\BitForm\Core\WorkFlow\WorkFlow;
23	27
24	28	final class FrontendFormHandler
25	29	{
	30	+ /** Largest stored signature inlined into the page as a data URI. */
	31	+ private const MAX_INLINE_SIGNATURE_BYTES = 2097152;
	32	+
26	33	public function __construct()
27	34	{
28	35	// before markup load - formids [], posts [1,2]
29	36	add_action('wp_enqueue_scripts', [$this, 'loadAssets']);
		@@ -29,29 +36,37 @@
29	36	add_action('wp_enqueue_scripts', [$this, 'loadAssets']);
30	37	// markup loads - formids []
31	38	add_shortcode('bitform', [$this, 'handleFrontendRenderRequest']);
32	39	// after markup load - formids [1,35,3]
33		- add_action('wp_footer', [$this, 'generateJS']);
	40	+ // After popup plugins render at 10 (that is when popup-only forms register
	41	+ // their formID), before wp_print_footer_scripts at 20.
	42	+ add_action('wp_footer', [$this, 'generateJS'], 15);
34	43	}
35	44
36		- private function validPassowordResetToken($token, $userID, $formId)
	45	+ /**
	46	+ * A reset form opened from a used or expired link cannot succeed, so show why instead of
	47	+ * the form. Returns the notice markup, or '' when the link is fine or the form is not a
	48	+ * reset form. Rendered in place of the form (the old exit() cut the page off mid-theme).
	49	+ *
	50	+ * @param string $token key from the reset link
	51	+ * @param string $userID user ID from the reset link
	52	+ * @param int $formId form being rendered
	53	+ *
	54	+ * @return string
	55	+ */
	56	+ private function invalidPasswordResetNotice($token, $userID, $formId)
37	57	{
38	58	$existResetInteg = (new IntegrationHandler($formId))->getAllIntegration('wp_user_auth', 'wp_auth', 1);
39		- if (!is_wp_error($existResetInteg) && count($existResetInteg) > 0) {
40		- if ('reset' === $existResetInteg[0]->integration_name) {
41		- $user = get_userdata($userID);
42		- if ($user) {
43		- $validKey = check_password_reset_key($token, $user->user_login);
44		- if (is_wp_error($validKey)) {
45		- echo "<div id='bf-resp' style='display:grid;justify-content:center;color:#860000;'>This password reset token is invalid.</div>";
46		- exit();
47		- }
48		- } else {
49		- echo "<div id='bf-resp' style='display:grid;justify-content:center;color:#860000;'>Invalid User!!</div>";
50		- exit();
51		- }
52		- }
	59	+ if (is_wp_error($existResetInteg) || empty($existResetInteg) || 'reset' !== $existResetInteg[0]->integration_name) {
	60	+ return '';
53	61	}
	62	+ $user = get_userdata((int) $userID);
	63	+ if ($user && !is_wp_error(check_password_reset_key($token, $user->user_login))) {
	64	+ return '';
	65	+ }
	66	+ return "<div id='bf-resp' role='alert' style='display:grid;justify-content:center;color:#860000;'>"
	67	+ . esc_html__('This password reset link is invalid or has expired. Please request a new one.', 'bit-form')
	68	+ . '</div>';
54	69	}
55	70
56	71	private function getJSFileSrc($postId)
57	72	{
		@@ -60,10 +75,43 @@
60	75
61	76	return $formScriptSrc;
62	77	}
63	78
	79	+ private function getJSFilePath($postId)
	80	+ {
	81	+ return BITFORMS_CONTENT_DIR . "/form-scripts/$postId/bitform-js-$postId.js";
	82	+ }
	83	+
	84	+ /**
	85	+ * Does this page's bundle need (re)generating?
	86	+ *
	87	+ * The DB flag alone is not enough: a page marked generated whose file was never written
	88	+ * (crashed generation, unwritable uploads dir) would enqueue a 404 forever. Conversely a
	89	+ * file that cannot be written must not make every request rebuild it, so a missing file
	90	+ * is retried on a backoff window rather than on every hit.
	91	+ *
	92	+ * @param int $postId
	93	+ * @param bool $regenerateScriptFlag DB-side verdict from regenerateScriptChecker()
	94	+ *
	95	+ * @return bool
	96	+ */
	97	+ private function needsScriptGeneration($postId, $regenerateScriptFlag)
	98	+ {
	99	+ if (file_exists($this->getJSFilePath($postId))) {
	100	+ return (bool) $regenerateScriptFlag;
	101	+ }
	102	+ $retryKey = 'bitforms_js_regen_' . $postId;
	103	+ if (get_transient($retryKey)) {
	104	+ return false;
	105	+ }
	106	+ set_transient($retryKey, 1, 5 * MINUTE_IN_SECONDS);
	107	+ return true;
	108	+ }
	109	+
64	110	public function generateJs($formID = null, $entryID = null, $formType = null)
65	111	{
	112	+ // bitform-js-{postId}.js is disk-cached per post ID with no language key.
	113	+ // Display strings must stay out of it and travel in bf_globals per request.
66	114	// return true;
67	115	$isFormPreview = get_transient('bitform_form_preview');
68	116	if ($isFormPreview && !$formID) {
69	117	delete_transient('bitform_form_preview');
		@@ -123,8 +171,10 @@
123	171	if (!$regenerateScriptFlag) {
124	172	$regenerateScriptFlag = $this->deleteUnusedFormPageIds($postId, $bfUniqFormIds);
125	173	}
126	174	$isJsGenerating = get_option('bitforms_frontend_js_generating');
	175	+ // The fast path also requires the cached bundle to exist on disk, not just be flagged in the DB.
	176	+ $regenerateScriptFlag = $this->needsScriptGeneration($postId, $regenerateScriptFlag);
127	177	if (!$regenerateScriptFlag && !$isJsGenerating && !empty($formIDs)) {
128	178	wp_enqueue_script('bit-form-all-script-test', $this->getJSFileSrc($postId), [], $formUpdateVersion, true);
129	179	return;
130	180	}
		@@ -157,8 +207,13 @@
157	207	$frontendScriptGenObj->generateJsFile($formContents, $allFields, $contentIds, $postId, $formIDs, $previewMode);
158	208	if ('preview' === $previewMode) {
159	209	return;
160	210	}
	211	+ // Only mark the page as generated once the bundle is verifiably on disk; otherwise the
	212	+ // next request must retry generation instead of fast-pathing to a stale/missing file.
	213	+ if (!empty($bfUniqFormIds) && file_exists($this->getJSFilePath($postId))) {
	214	+ $this->markScriptGenerated($bfUniqFormIds, $postId);
	215	+ }
161	216	wp_enqueue_script('bit-form-all-script-test', $this->getJSFileSrc($postId), [], $formUpdateVersion, true);
162	217	}
163	218
164	219	private function deleteUnusedFormPageIds($postId, $formIDs)
		@@ -174,9 +229,9 @@
174	229	);
175	230	$regenerateScriptFlag = false;
176	231	foreach ($forms as $form) {
177	232	$formId = $form->id;
178		- $generatedScriptPageIdsDecoded = json_decode($form->generated_script_page_ids, true);
	233	+ $generatedScriptPageIdsDecoded = json_decode((string) $form->generated_script_page_ids, true);
179	234	$generatedScriptPageIds = is_array($generatedScriptPageIdsDecoded) ? array_keys($generatedScriptPageIdsDecoded) : [];
180	235	if (!empty($generatedScriptPageIds) && !in_array($formId, $formIDs) && in_array($postId, $generatedScriptPageIds)) {
181	236	unset($generatedScriptPageIdsDecoded[$postId]);
182	237	if (empty($generatedScriptPageIdsDecoded)) {
		@@ -204,10 +259,11 @@
204	259	if (!is_a($post, 'WP_Post') && !isset($post->ID)) {
205	260	return;
206	261	}
207	262	$postId = $post->ID;
208		- $regenerateScriptFlag = false;
209		- $formModel = new FormModel();
	263	+ // Read-only check. Marking the page as generated is deferred to markScriptGenerated(),
	264	+ // called only after the bundle file is actually written — marking here left the DB
	265	+ // saying "generated" while the file stayed stale whenever generation failed mid-way.
210	266	foreach ($formsIds as $formId) {
211	267	$formInstance = FormManager::getInstance($formId);
212	268	if (!$formInstance->isExist()) {
213	269	continue;
		@@ -213,18 +269,35 @@
213	269	continue;
214	270	}
215	271	$generatedPages = $formInstance->getFormData('generated_script_page_ids');
216	272	if (empty($generatedPages)) {
217		- $regenerateScriptFlag = true;
218		- } elseif (is_object($generatedPages) && (!isset($generatedPages->{$postId}) || (isset($generatedPages->{$postId}) && false === $generatedPages->{$postId}))) {
219		- $regenerateScriptFlag = true;
	273	+ return true;
220	274	}
221		- if (!$regenerateScriptFlag) {
	275	+ if (is_object($generatedPages) && (!isset($generatedPages->{$postId}) || false === $generatedPages->{$postId})) {
	276	+ return true;
	277	+ }
	278	+ }
	279	+ return false;
	280	+ }
	281	+
	282	+ private function markScriptGenerated($formsIds, $postId)
	283	+ {
	284	+ // Fetched via FormModel rather than FormManager: FormManager keeps its row in a static
	285	+ // property shared across instances, so after the render loop it holds the last form's
	286	+ // data regardless of which instance is asked.
	287	+ $formModel = new FormModel();
	288	+ foreach ($formsIds as $formId) {
	289	+ $form = $formModel->get(['generated_script_page_ids'], ['id' => $formId]);
	290	+ if (is_wp_error($form) || empty($form)) {
222	291	continue;
223	292	}
	293	+ $generatedPages = Utilities::jsonObj($form[0]->generated_script_page_ids ?? '');
224	294	if (!is_object($generatedPages)) {
225	295	$generatedPages = (object) [];
226	296	}
	297	+ if (!empty($generatedPages->{$postId})) {
	298	+ continue;
	299	+ }
227	300	$generatedPages->{$postId} = true;
228	301	$formModel->update(
229	302	[
230	303	'generated_script_page_ids' => \wp_json_encode($generatedPages)
		@@ -233,9 +306,8 @@
233	306	'id' => $formId,
234	307	]
235	308	);
236	309	}
237		- return $regenerateScriptFlag;
238	310	}
239	311
240	312	private function addInlineScript($code, $handle = '', $position = 'after')
241	313	{
		@@ -280,9 +352,9 @@
280	352
281	353	return $fields;
282	354	}
283	355
284		- private function executeOnUserInput($formID, $shortCodeCounter, $fields)
	356	+ private function executeOnUserInput($formID, $shortCodeCounter, $workFlowRunType = 'create')
285	357	{
286	358	$FrontendFormManager = FrontendFormManager::getInstance($formID, $shortCodeCounter);
287	359	$previousValue = $this->getValuesFromQueryParams();
288	360	$formContent = $FrontendFormManager->getFormContentWithValue($previousValue);
		@@ -288,9 +360,9 @@
288	360	$formContent = $FrontendFormManager->getFormContentWithValue($previousValue);
289	361	$customCodesExist = strpos(FrontEndScriptGenerator::getCustomCodes($formID)['JavaScript'], 'bfVars');
290	362	if ($customCodesExist || (!empty($formContent->workFlowExist) && !empty($formContent->workFlowExist->oninput))) {
291	363	$workFlowRunHelper = new WorkFlow($formID);
292		- return $workFlowRunHelper->executeOnUserInput('create', $fields);
	364	+ return $workFlowRunHelper->executeOnUserInput($workFlowRunType);
293	365	}
294	366	}
295	367
296	368	private function getValuesFromQueryParams()
		@@ -345,10 +417,13 @@
345	417	/* translators: %s: form ID */
346	418	return sprintf(__('#%s no. Form doesn\'t exists', 'bit-form'), $formID);
347	419	}
348	420
349		- // Add-ons may detect whether the current visitor is resuming an abandoned entry.
350		- $isAbandoned = (bool) apply_filters('bitform_is_abandoned_entry', false, $formID, $entryId, $atts);
	421	+ // Add-ons return the id of the abandoned entry this visitor can resume. A bare
	422	+ // true (older Pro) carries no id, so nothing is restored rather than entry #1.
	423	+ $abandonedEntryId = apply_filters('bitform_is_abandoned_entry', false, $formID, $entryId, $atts);
	424	+ $abandonedEntryId = (!is_bool($abandonedEntryId) && is_numeric($abandonedEntryId)) ? (int) $abandonedEntryId : 0;
	425	+ $isAbandoned = $abandonedEntryId > 0;
351	426
352	427	FrontendHelpers::setBfFrontendFormIds($formID);
353	428	$bfFrontendFormIds = FrontendHelpers::$bfFrontendFormIds;
354	429	$shortCodeCounter = count($bfFrontendFormIds);
		@@ -368,9 +443,13 @@
368	443	}
369	444
370	445	// Read-only: password reset token from URL for display-time validation. No state written until form is submitted.
371	446	if (!empty($_GET['token']) && !empty($_GET['id'])) {
372		- $this->validPassowordResetToken(sanitize_text_field(wp_unslash($_GET['token'])), sanitize_text_field(wp_unslash($_GET['id'])), $formID);
	447	+ $resetNotice = $this->invalidPasswordResetNotice(sanitize_text_field(wp_unslash($_GET['token'])), sanitize_text_field(wp_unslash($_GET['id'])), $formID);
	448	+ if ('' !== $resetNotice) {
	449	+ ob_end_clean();
	450	+ return $resetNotice;
	451	+ }
373	452	}
374	453
375	454	$previousValue = $this->getValuesFromQueryParams();
376	455	$errorMessages = []; // delete
		@@ -410,9 +489,9 @@
410	489	$fields = apply_filters('bitform_filter_before_workflow_onload_fields', $fields, $formID);
411	490	$fields = $this->triggerWorkflowOnLoad($formID, $shortCodeCounter, $fields, $workFlowRunType);
412	491	$fields = apply_filters('bitform_filter_after_workflow_onload_fields', $fields, $formID);
413	492	do_action('bitform_onload_fields', $fields, $formID);
414		- $workFlowreturnedOnUserInput = $this->executeOnUserInput($formID, $shortCodeCounter, $fields);
	493	+ $workFlowreturnedOnUserInput = $this->executeOnUserInput($formID, $shortCodeCounter, $workFlowRunType);
415	494
416	495	// test for form before remove
417	496	$noLabelFieldTypes = ['decision-box', 'gdpr', 'html', 'shortcode', 'button', 'paypal', 'razorpay', 'recaptcha', 'turnstile', 'hcaptcha', 'stripe', 'spacer'];
418	497	foreach ($fields as $fldKey => $field) {
		@@ -433,12 +512,14 @@
433	512	$FrontendFormManager->getCaptchaSettings()
434	513	&& !is_null($integration->integration_type)
435	514	&& 'gReCaptcha' === $integration->integration_type
436	515	) {
437		- $integrationDetails = json_decode($integration->integration_details);
438		- $integrationDetails->id = $integration->id;
439		- $reCAPTCHA = $integrationDetails;
440		- $reCAPTCHAVersion = 'v2';
	516	+ $integrationDetails = Utilities::jsonObj($integration->integration_details);
	517	+ if ($integrationDetails) {
	518	+ $integrationDetails->id = $integration->id;
	519	+ $reCAPTCHA = $integrationDetails;
	520	+ $reCAPTCHAVersion = 'v2';
	521	+ }
441	522	}
442	523
443	524	if (
444	525	$FrontendFormManager->getTurnstileSettings()
		@@ -444,10 +525,10 @@
444	525	$FrontendFormManager->getTurnstileSettings()
445	526	&& !is_null($integration->integration_type)
446	527	&& 'turnstileCaptcha' === $integration->integration_type
447	528	) {
448		- $integrationDetails = json_decode($integration->integration_details);
449		- $turnstileSiteKey = $integrationDetails->siteKey;
	529	+ $integrationDetails = Utilities::jsonObj($integration->integration_details);
	530	+ $turnstileSiteKey = $integrationDetails->siteKey ?? '';
450	531	}
451	532
452	533	if (
453	534	$FrontendFormManager->isFieldTypeExist('hcaptcha')
		@@ -453,18 +534,20 @@
453	534	$FrontendFormManager->isFieldTypeExist('hcaptcha')
454	535	&& !is_null($integration->integration_type)
455	536	&& 'hcaptcha' === $integration->integration_type
456	537	) {
457		- $integrationDetails = json_decode($integration->integration_details);
458		- $hCaptchaSiteKey = $integrationDetails->siteKey;
	538	+ $integrationDetails = Utilities::jsonObj($integration->integration_details);
	539	+ $hCaptchaSiteKey = $integrationDetails->siteKey ?? '';
459	540	}
460	541
461	542	if ($captchaV3Settings) {
462	543	if (!is_null($integration->integration_type) && 'gReCaptchaV3' === $integration->integration_type) {
463		- $integrationDetails = json_decode($integration->integration_details);
464		- $integrationDetails->id = $integration->id;
465		- $reCAPTCHA = $integrationDetails;
466		- $reCAPTCHAVersion = 'v3';
	544	+ $integrationDetails = Utilities::jsonObj($integration->integration_details);
	545	+ if ($integrationDetails) {
	546	+ $integrationDetails->id = $integration->id;
	547	+ $reCAPTCHA = $integrationDetails;
	548	+ $reCAPTCHAVersion = 'v3';
	549	+ }
467	550	}
468	551	}
469	552	}
470	553	}
		@@ -497,19 +580,20 @@
497	580	$paymentIntegration = $integrationHandler->getAIntegration($fldData->payIntegID);
498	581	if (is_wp_error($paymentIntegration)) {
499	582	continue;
500	583	}
	584	+ $paymentIntegrationRow = Utilities::firstRow($paymentIntegration);
501	585	if ('paypal' === $fldData->typ) {
502		- $integrationDetails = json_decode($paymentIntegration[0]->integration_details);
503		- $clientID = $integrationDetails->clientID;
	586	+ $integrationDetails = Utilities::jsonObj($paymentIntegrationRow->integration_details ?? '');
	587	+ $clientID = $integrationDetails->clientID ?? '';
504	588	$fields->{$fldKey}->clientId = $clientID;
505	589	} elseif ('razorpay' === $fldData->typ) {
506		- $integrationDetails = json_decode($paymentIntegration[0]->integration_details);
507		- $clientID = $integrationDetails->apiKey;
	590	+ $integrationDetails = Utilities::jsonObj($paymentIntegrationRow->integration_details ?? '');
	591	+ $clientID = $integrationDetails->apiKey ?? '';
508	592	$fields->{$fldKey}->clientId = $clientID;
509	593	} elseif ('stripe' === $fldData->typ) {
510		- $integrationDetails = json_decode($paymentIntegration[0]->integration_details);
511		- $publishableKey = $integrationDetails->publishableKey;
	594	+ $integrationDetails = Utilities::jsonObj($paymentIntegrationRow->integration_details ?? '');
	595	+ $publishableKey = $integrationDetails->publishableKey ?? '';
512	596	$fields->{$fldKey}->publishableKey = $publishableKey;
513	597	}
514	598	}
515	599	}
		@@ -539,8 +623,9 @@
539	623	];
540	624
541	625	if ($entryId) {
542	626	$bitFormFrontArr['entryId'] = $entryId;
	627	+ self::markResponseUncacheable();
543	628	}
544	629
545	630	if (isset($additional->enabled->validateFocusLost)) {
546	631	$bitFormFrontArr['validateFocusLost'] = true;
		@@ -545,12 +630,19 @@
545	630	if (isset($additional->enabled->validateFocusLost)) {
546	631	$bitFormFrontArr['validateFocusLost'] = true;
547	632	}
548	633
549		- if (!empty($isAbandoned)) {
550		- $bitFormFrontArr['oldValues'] = $this->getFieldsValue($formID, $isAbandoned);
	634	+ if ($isAbandoned) {
	635	+ // One visitor's typed values, so this response must not be page-cached.
	636	+ $bitFormFrontArr['oldValues'] = $this->getFieldsValue($formID, $abandonedEntryId);
	637	+ self::markResponseUncacheable();
551	638	if (empty($entryId)) {
552		- $bitFormFrontArr['entryId'] = $entryId;
	639	+ // Later draft saves must update this entry, not start a new one each time.
	640	+ $bitFormFrontArr['entryId'] = $abandonedEntryId;
	641	+ $entryToken = Cryptography::encrypt($abandonedEntryId, Helpers::getBitformSalt());
	642	+ if (is_string($entryToken)) {
	643	+ $bitFormFrontArr['entryToken'] = $entryToken;
	644	+ }
553	645	}
554	646	}
555	647
556	648	$formInfo = $FrontendFormManager->getFormInfo();
		@@ -585,27 +677,31 @@
585	677	$layout = wp_json_encode($layout);
586	678	$buttons = wp_json_encode($buttons);
587	679	$frontArr = wp_json_encode($bitFormFrontArr);
588	680
589		- $bfGlobals = sprintf('
590		- if(!window.bf_globals) {
591		- window.bf_globals = {}
592		- } if(!window.bf_globals.%1$s) {
593		- window.bf_globals.%1$s = {}
	681	+ $bfGlobals = sprintf('
	682	+ if(!window.bf_globals) {
	683	+ window.bf_globals = {}
	684	+ } if(!window.bf_globals.%1$s) {
	685	+ window.bf_globals.%1$s = {}
594	686	}
595		- if(document.getElementById("%1$s")) {
596		- window.bf_globals.%1$s = {
597		- ...window.bf_globals.%1$s,
598		- ...%2$s
599		- };
600		- }', $FormIdentifier, $frontArr);
	687	+ window.bf_globals.%1$s = {
	688	+ ...window.bf_globals.%1$s,
	689	+ ...%2$s
	690	+ };
	691	+ if (typeof window.bitformInit === "function") { window.bitformInit("%1$s"); }', $FormIdentifier, $frontArr);
601	692
	693	+ // Inert copy of the config. Optimizers only rewrite executable scripts, so
	694	+ // this survives and travels with the markup; the runtime hydrates from it
	695	+ // whenever bf_globals is missing.
	696	+ $configTag = self::buildFormConfigTag($FormIdentifier, $bitFormFrontArr);
	697	+
602	698	if ('conversational' === $formType
603	699	&& isset($formContent->formInfo->conversationalSettings->enable)
604	700	&& $formContent->formInfo->conversationalSettings->enable) {
605		- $html = $FrontendFormManager->conversationalFormView($fields, $file, $errorMessages);
	701	+ $html = $FrontendFormManager->conversationalFormView($fields, $file, $errorMessages, null, !empty($entryId));
606	702	} else {
607		- $html = $FrontendFormManager->formView($fields, $file, $errorMessages);
	703	+ $html = $FrontendFormManager->formView($fields, $file, $errorMessages, null, !empty($entryId));
608	704	}
609	705
610	706	// if form preview then return html otherwise echo with output buffer
611	707	if ($formPreview) {
		@@ -613,8 +709,9 @@
613	709	$formViewObject = new \stdClass();
614	710	$formViewObject->html = $html;
615	711	$formViewObject->font = $font;
616	712	$formViewObject->bfGlobals = $bfGlobals;
	713	+ $formViewObject->configTag = $configTag;
617	714	$formViewObject->formContent = $formContent;
618	715	return $formViewObject;
619	716	}
620	717
		@@ -621,13 +718,60 @@
621	718	$bfGlobalsHandle = 'bitform-bf-globals-' . sanitize_key($FormIdentifier);
622	719	$this->addInlineScript($bfGlobals, $bfGlobalsHandle, 'after');
623	720	$this->emitShowPickerBridge();
624	721
	722	+ // Printed outside wp_kses rather than allowing <script> in form markup.
	723	+ // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- built by buildFormConfigTag(), JSON_HEX_* escaped.
	724	+ echo $configTag;
625	725	echo wp_kses(trim($html), EscapingHelper::getFormAllowedHtml($formContent));
626	726	return ob_get_clean();
627	727	}
628	728
629	729	/**
	730	+ * Keep per-visitor config (oldValues, entryId) out of full-page caches.
	731	+ *
	732	+ * @return void
	733	+ */
	734	+ public static function markResponseUncacheable()
	735	+ {
	736	+ // DONOTCACHEPAGE does the work; caches read it at shutdown. Rendering
	737	+ // usually runs after headers are sent, so nocache_headers() is a bonus.
	738	+ if (!defined('DONOTCACHEPAGE')) {
	739	+ define('DONOTCACHEPAGE', true);
	740	+ }
	741	+ if (!headers_sent() && function_exists('nocache_headers')) {
	742	+ nocache_headers();
	743	+ }
	744	+ }
	745	+
	746	+ /**
	747	+ * Build the inert JSON config block for a rendered form.
	748	+ *
	749	+ * JSON_HEX_* escapes < > & as \u00XX so no field value can close the script
	750	+ * element or inject markup.
	751	+ *
	752	+ * @param string $formIdentifier
	753	+ * @param array $config
	754	+ *
	755	+ * @return string
	756	+ */
	757	+ public static function buildFormConfigTag($formIdentifier, $config)
	758	+ {
	759	+ $json = wp_json_encode($config, JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT);
	760	+ if (false === $json) {
	761	+ return '';
	762	+ }
	763	+ // Some optimizers wrap any inline <script>, including application/json,
	764	+ // in DOMContentLoaded boilerplate that corrupts the JSON. These attributes
	765	+ // make the common ones skip it; the JS side also salvage-parses.
	766	+ return sprintf(
	767	+ '<script type="application/json" class="bf-form-config" id="bf-config-%1$s" data-bf-form="%1$s" data-no-optimize="1" data-no-defer="1" data-no-minify="1" data-cfasync="false" nowprocket>%2$s</script>',
	768	+ esc_attr($formIdentifier),
	769	+ $json
	770	+ );
	771	+ }
	772	+
	773	+ /**
630	774	* Build the smart-tag map exposed to the browser in window.bf_globals[formId].smartTags.
631	775	*
632	776	* Security: the legacy code shipped the ENTIRE ~43-tag map to every visitor, leaking
633	777	* PII (admin/user/author email) and freezing per-visitor request data (IP, time,
		@@ -692,8 +836,15 @@
692	836	$code = 'if(!window.__bfShowPickerBound){window.__bfShowPickerBound=true;document.addEventListener("click",function(e){var t=e.target;if(t&&t.matches&&t.matches("input[data-bf-show-picker=\"1\"]")&&typeof t.showPicker==="function"){try{t.showPicker();}catch(_){}}});}';
693	837	$this->addInlineScript($code, 'bitform-show-picker-bridge', 'after');
694	838	}
695	839
	840	+ /**
	841	+ * Does this form row exist?
	842	+ *
	843	+ * @param int $formID
	844	+ *
	845	+ * @return bool
	846	+ */
696	847	private function isExist($formID)
697	848	{
698	849	$formModel = new FormModel();
699	850	$form = $formModel->get(
		@@ -703,12 +854,32 @@
703	854	[
704	855	'id' => $formID,
705	856	]
706	857	);
707		- if (!is_wp_error($form)) {
708		- return true;
	858	+
	859	+ if (is_wp_error($form)) {
	860	+ if ('result_empty' !== $form->get_error_code()) {
	861	+ Log::debug_log([
	862	+ 'message' => 'Form lookup failed — reported to the visitor as a missing form',
	863	+ 'formID' => $formID,
	864	+ 'code' => $form->get_error_code(),
	865	+ 'error' => $form->get_error_message(),
	866	+ ]);
	867	+ }
	868	+
	869	+ return false;
709	870	}
710		- return false;
	871	+
	872	+ if (empty($form)) {
	873	+ Log::debug_log([
	874	+ 'message' => 'Form lookup returned no rows without an error (is the form table present?)',
	875	+ 'formID' => $formID,
	876	+ ]);
	877	+
	878	+ return false;
	879	+ }
	880	+
	881	+ return true;
711	882	}
712	883
713	884	private function getFieldsValue($formID, $entryID)
714	885	{
		@@ -738,8 +909,12 @@
738	909	} else {
739	910	$metaVal = implode(BITFORMS_BF_SEPARATOR, $metaVal);
740	911	}
741	912	}
	913	+ // A restored draft never gets the password back; the visitor types it again.
	914	+ if (isset($formFields[$metaKey]['type']) && 'password' === $formFields[$metaKey]['type']) {
	915	+ continue;
	916	+ }
742	917	if (!isset($fldsData->{$metaKey})) {
743	918	$fldsData->{$metaKey} = '';
744	919	}
745	920	$fldsData->{$metaKey} = $metaVal;
		@@ -792,8 +967,16 @@
792	967	} else {
793	968	$metaVal = implode(BITFORMS_BF_SEPARATOR, $metaVal);
794	969	}
795	970	}
	971	+ if (property_exists($fields, $metaKey) && 'password' === $fields->{$metaKey}->typ) {
	972	+ // Never hand a stored password back to the browser; blank keeps it on update.
	973	+ $fields->{$metaKey}->val = '';
	974	+ if ('' !== (string) $metaValue->meta_value && isset($fields->{$metaKey}->valid) && is_object($fields->{$metaKey}->valid)) {
	975	+ $fields->{$metaKey}->valid->req = false;
	976	+ }
	977	+ continue;
	978	+ }
796	979	if (property_exists($fields, $metaKey)) {
797	980	$fields->{$metaKey}->val = $metaVal;
798	981	if ('file-up' === $fields->{$metaKey}->typ || 'advanced-file-up' === $fields->{$metaKey}->typ) {
799	982	$fields->{$metaKey}->val = $metaValue->meta_value;
		@@ -798,8 +981,11 @@
798	981	if ('file-up' === $fields->{$metaKey}->typ || 'advanced-file-up' === $fields->{$metaKey}->typ) {
799	982	$fields->{$metaKey}->val = $metaValue->meta_value;
800	983	$fields->{$metaKey}->config->oldFiles = $metaValue->meta_value;
801	984	}
	985	+ if ('signature' === $fields->{$metaKey}->typ) {
	986	+ $this->setOldSignature($fields->{$metaKey}, $formID, $entryID, $metaValue->meta_value);
	987	+ }
802	988	}
803	989	}
804	990	}
805	991	return $fields;
		@@ -804,8 +990,51 @@
804	990	}
805	991	return $fields;
806	992	}
807	993
	994	+ /** Give the signature field its stored signature: a data URI to redraw, and the name it posts back as `_old`. */
	995	+ private function setOldSignature($field, $formID, $entryID, $storedValue)
	996	+ {
	997	+ $fileName = is_string($storedValue) ? trim($storedValue) : '';
	998	+ $decoded = json_decode($fileName, true);
	999	+ if (is_array($decoded)) {
	1000	+ $fileName = empty($decoded) ? '' : trim((string) reset($decoded));
	1001	+ }
	1002	+ // signature-failed.png means the stored signature was never usable.
	1003	+ if ('' === $fileName || 'signature-failed.png' === $fileName) {
	1004	+ return;
	1005	+ }
	1006	+ $fileName = sanitize_file_name($fileName);
	1007	+ if (!isset($field->config)) {
	1008	+ $field->config = (object) [];
	1009	+ } elseif (is_array($field->config)) {
	1010	+ $field->config = (object) $field->config;
	1011	+ }
	1012	+ $field->config->oldSignatureFile = $fileName;
	1013	+
	1014	+ $filePath = FileHandler::getEntriesFileUploadDir($formID, $entryID) . DIRECTORY_SEPARATOR . $fileName;
	1015	+ if (!is_file($filePath) || !is_readable($filePath)) {
	1016	+ return;
	1017	+ }
	1018	+ // The types getSignatureFilePath() writes; wp_check_filetype() reports none for SVG.
	1019	+ $signatureMimeTypes = ['png' => 'image/png', 'jpg' => 'image/jpeg', 'svg' => 'image/svg+xml'];
	1020	+ $extension = strtolower((string) pathinfo($fileName, PATHINFO_EXTENSION));
	1021	+ if (!isset($signatureMimeTypes[$extension])) {
	1022	+ return;
	1023	+ }
	1024	+ $mimeType = $signatureMimeTypes[$extension];
	1025	+ // A hand-drawn signature is a few KB; a larger file is not worth inlining.
	1026	+ if (filesize($filePath) > self::MAX_INLINE_SIGNATURE_BYTES) {
	1027	+ return;
	1028	+ }
	1029	+ // phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_operations_file_get_contents -- local upload dir read, inlined as a data URI for the signature pad.
	1030	+ $contents = file_get_contents($filePath);
	1031	+ if (false === $contents || '' === $contents) {
	1032	+ return;
	1033	+ }
	1034	+ $field->config->oldSignature = 'data:' . $mimeType . ';base64,' . base64_encode($contents);
	1035	+ }
	1036	+
808	1037	public function loadAssets($formID = 0, $fromType = 'classic')
809	1038	{
810	1039	$bfUniqFormIds = FrontendHelpers::getAllFormIdsInPage();
811	1040	$isPageBuilder = FrontendHelpers::$isPageBuilder;
		@@ -829,9 +1058,9 @@
829	1058	$formUpdateVersion = get_option('bitform_form_update_version');
830	1059	if (!wp_style_is('bitform-style-' . $newFormId) && is_readable(BITFORMS_CONTENT_DIR . '/form-styles/bitform-' . $newFormId . '.css')) {
831	1060	wp_enqueue_style(
832	1061	'bitform-style-' . $newFormId,
833		- BITFORMS_UPLOAD_BASE_URL . "/form-styles/bitform-{$newFormId}.css",
	1062	+ FrontendHelpers::styleSrc("form-styles/bitform-{$newFormId}.css"),
834	1063	[],
835	1064	$formUpdateVersion
836	1065	);
837	1066	if ($isPageBuilder) {
		@@ -841,9 +1070,9 @@
841	1070	}
842	1071	if (!wp_style_is('bitform-style-custom-' . $formID) && is_readable(BITFORMS_CONTENT_DIR . '/form-styles/bitform-custom-' . $formID . '.css')) {
843	1072	wp_enqueue_style(
844	1073	'bitform-style-custom-' . $formID,
845		- BITFORMS_UPLOAD_BASE_URL . "/form-styles/bitform-custom-{$formID}.css",
	1074	+ FrontendHelpers::styleSrc("form-styles/bitform-custom-{$formID}.css"),
846	1075	[],
847	1076	$formUpdateVersion
848	1077	);
849	1078	if ($isPageBuilder) {
		@@ -856,9 +1085,9 @@
856	1085	if (!wp_style_is('bitform-conversational-style-' . $formID) &&
857	1086	is_readable(BITFORMS_CONTENT_DIR . "/form-styles/bitform-conversational-{$formID}.css")) {
858	1087	wp_enqueue_style(
859	1088	'bitform-conversational-style',
860		- BITFORMS_UPLOAD_BASE_URL . "/form-styles/bitform-conversational-{$formID}.css",
	1089	+ FrontendHelpers::styleSrc("form-styles/bitform-conversational-{$formID}.css"),
861	1090	[],
862	1091	$formUpdateVersion
863	1092	);
864	1093	}