| @@ -27,8 +27,9 @@ | ||
| 27 | 27 | private function mxchat_init_ajax_hooks() { |
| 28 | 28 | // Settings AJAX |
| 29 | 29 | add_action('wp_ajax_mxchat_save_setting', array($this, 'mxchat_save_setting_callback')); |
| 30 | 30 | add_action('wp_ajax_mxchat_save_prompts_setting', array($this, 'mxchat_save_prompts_setting_callback')); |
| 31 | + add_action('wp_ajax_mxchat_acf_toggle_group', array($this, 'mxchat_acf_toggle_group_callback')); | |
| 31 | 32 | add_action('wp_ajax_migrate_pinecone_settings', array($this, 'ajax_migrate_pinecone_settings')); |
| 32 | 33 | |
| 33 | 34 | // License AJAX |
| 34 | 35 | add_action('wp_ajax_mxchat_handle_activate_license', array($this, 'mxchat_handle_activate_license')); |
| @@ -53,11 +54,119 @@ | ||
| 53 | 54 | add_action('wp_ajax_mxchat_reset_global_rate_limit', array($this, 'mxchat_reset_global_rate_limit_callback')); |
| 54 | 55 | |
| 55 | 56 | // Custom (OpenAI-compatible) Provider connection test |
| 56 | 57 | add_action('wp_ajax_mxchat_test_custom_provider', array($this, 'mxchat_test_custom_provider_callback')); |
| 58 | + | |
| 59 | + // Built-in provider key validation — cheap per-provider auth check (plan-mxchat-20260623-c41f74) | |
| 60 | + add_action('wp_ajax_mxchat_test_provider_key', array($this, 'mxchat_test_provider_key_callback')); | |
| 61 | + | |
| 62 | + // Custom Post Meta discovery scan for the KB whitelist picker (plan-mxchat-20260709-fe8e4e) | |
| 63 | + add_action('wp_ajax_mxchat_scan_custom_meta_keys', array($this, 'mxchat_scan_custom_meta_keys_callback')); | |
| 57 | 64 | } |
| 58 | 65 | |
| 59 | 66 | /** |
| 67 | + * Discover non-ACF custom post-meta keys present on published public content, so the | |
| 68 | + * KB → Custom Post Meta section can offer a click-to-add picker instead of a blind | |
| 69 | + * "type the exact key you already know" textarea. plan-mxchat-20260709-fe8e4e. | |
| 70 | + * | |
| 71 | + * Bounded + button-triggered only (never on page load). Returns up to 50 keys by | |
| 72 | + * frequency, each with a short sample value, so the owner can judge relevance before | |
| 73 | + * whitelisting. Underscore-prefixed (protected/internal) keys are hidden unless the | |
| 74 | + * caller opts in; ACF-managed keys are excluded so this picker never double-lists the | |
| 75 | + * sibling ACF discovery picker on the same page. | |
| 76 | + */ | |
| 77 | +public function mxchat_scan_custom_meta_keys_callback() { | |
| 78 | + check_ajax_referer('mxchat_prompts_setting_nonce'); | |
| 79 | + | |
| 80 | + if (!current_user_can('manage_options')) { | |
| 81 | + wp_send_json_error(['message' => esc_html__('Unauthorized', 'mxchat')]); | |
| 82 | + } | |
| 83 | + | |
| 84 | + global $wpdb; | |
| 85 | + | |
| 86 | + $include_internal = isset($_POST['include_internal']) && $_POST['include_internal'] === '1'; | |
| 87 | + | |
| 88 | + // Restrict discovery to public post types (the content the KB actually embeds). | |
| 89 | + $post_types = get_post_types(array('public' => true), 'names'); | |
| 90 | + if (empty($post_types)) { | |
| 91 | + wp_send_json_success(array('keys' => array(), 'scanned' => 0)); | |
| 92 | + } | |
| 93 | + $pt_placeholders = implode(',', array_fill(0, count($post_types), '%s')); | |
| 94 | + | |
| 95 | + // Build the set of ACF-managed meta keys to exclude. ACF stores, alongside each | |
| 96 | + // value key `foo`, a reference key `_foo` whose value is the ACF field key | |
| 97 | + // (`field_xxxxx`). Strip the leading underscore from every such reference key to | |
| 98 | + // get the real meta key, and exclude those — the ACF picker on this page owns them. | |
| 99 | + $acf_managed = array(); | |
| 100 | + $acf_refs = $wpdb->get_col( | |
| 101 | + $wpdb->prepare( | |
| 102 | + "SELECT DISTINCT meta_key FROM {$wpdb->postmeta} WHERE meta_key LIKE %s AND meta_value LIKE %s", | |
| 103 | + $wpdb->esc_like('_') . '%', | |
| 104 | + $wpdb->esc_like('field_') . '%' | |
| 105 | + ) | |
| 106 | + ); | |
| 107 | + foreach ((array) $acf_refs as $ref_key) { | |
| 108 | + if (strlen($ref_key) > 1 && $ref_key[0] === '_') { | |
| 109 | + $acf_managed[substr($ref_key, 1)] = true; | |
| 110 | + } | |
| 111 | + } | |
| 112 | + | |
| 113 | + // Discover keys + counts + a sample value in one bounded aggregate query. | |
| 114 | + // SUBSTRING(MIN(...)) keeps the sample selection ONLY_FULL_GROUP_BY-safe. | |
| 115 | + $params = $post_types; | |
| 116 | + $sql = "SELECT pm.meta_key AS mk, COUNT(*) AS n, SUBSTRING(MIN(pm.meta_value), 1, 200) AS sample | |
| 117 | + FROM {$wpdb->postmeta} pm | |
| 118 | + INNER JOIN {$wpdb->posts} p ON p.ID = pm.post_id | |
| 119 | + WHERE p.post_status = 'publish' | |
| 120 | + AND p.post_type IN ($pt_placeholders) | |
| 121 | + AND pm.meta_key <> ''"; | |
| 122 | + if (!$include_internal) { | |
| 123 | + $sql .= " AND pm.meta_key NOT LIKE %s"; | |
| 124 | + $params[] = $wpdb->esc_like('_') . '%'; | |
| 125 | + } | |
| 126 | + $sql .= " GROUP BY pm.meta_key ORDER BY n DESC, pm.meta_key ASC LIMIT 200"; | |
| 127 | + | |
| 128 | + // phpcs:ignore WordPress.DB.PreparedSQL — placeholders assembled above, values in $params. | |
| 129 | + $rows = $wpdb->get_results($wpdb->prepare($sql, $params)); | |
| 130 | + | |
| 131 | + $keys = array(); | |
| 132 | + foreach ((array) $rows as $row) { | |
| 133 | + $mk = $row->mk; | |
| 134 | + if (isset($acf_managed[$mk])) { | |
| 135 | + continue; // already offered by the ACF picker | |
| 136 | + } | |
| 137 | + | |
| 138 | + $raw = (string) $row->sample; | |
| 139 | + if ($raw !== '' && (is_serialized($raw) || preg_match('/^(a:\d+:\{|O:\d+:"|s:\d+:")/', $raw))) { | |
| 140 | + $sample = esc_html__('[structured value]', 'mxchat'); | |
| 141 | + } else { | |
| 142 | + $sample = trim(preg_replace('/\s+/', ' ', $raw)); | |
| 143 | + if (function_exists('mb_strlen') ? mb_strlen($sample) > 60 : strlen($sample) > 60) { | |
| 144 | + $sample = (function_exists('mb_substr') ? mb_substr($sample, 0, 60) : substr($sample, 0, 60)) . '…'; | |
| 145 | + } | |
| 146 | + if ($sample === '') { | |
| 147 | + $sample = esc_html__('(empty value)', 'mxchat'); | |
| 148 | + } | |
| 149 | + } | |
| 150 | + | |
| 151 | + $keys[] = array( | |
| 152 | + 'key' => $mk, | |
| 153 | + 'count' => (int) $row->n, | |
| 154 | + 'sample' => $sample, | |
| 155 | + ); | |
| 156 | + | |
| 157 | + if (count($keys) >= 50) { | |
| 158 | + break; | |
| 159 | + } | |
| 160 | + } | |
| 161 | + | |
| 162 | + wp_send_json_success(array( | |
| 163 | + 'keys' => $keys, | |
| 164 | + 'scanned' => is_array($rows) ? count($rows) : 0, | |
| 165 | + )); | |
| 166 | +} | |
| 167 | + | |
| 168 | +/** | |
| 60 | 169 | * Test connection to a Custom (OpenAI-compatible) provider by hitting its /models endpoint |
| 61 | 170 | * with whichever auth scheme the user configured. Reports model count or a clean error. |
| 62 | 171 | */ |
| 63 | 172 | public function mxchat_test_custom_provider_callback() { |
| @@ -125,8 +234,122 @@ | ||
| 125 | 234 | 'count' => $count, |
| 126 | 235 | )); |
| 127 | 236 | } |
| 128 | 237 | |
| 238 | +/** | |
| 239 | + * Validate a BUILT-IN provider key with the lightest authenticated call per | |
| 240 | + * provider (a /models or key-info GET — never a generation). Reads the posted | |
| 241 | + * key value so the owner can test BEFORE saving; falls back to the saved option | |
| 242 | + * when the field is empty. Mirrors mxchat_test_custom_provider_callback and the | |
| 243 | + * add-on test buttons (cf5bd5 veo / 8d16f1 perplexity). The key is never logged. | |
| 244 | + * plan-mxchat-20260623-c41f74. | |
| 245 | + */ | |
| 246 | +public function mxchat_test_provider_key_callback() { | |
| 247 | + check_ajax_referer('mxchat_test_provider_key'); | |
| 248 | + if (!current_user_can('manage_options')) { | |
| 249 | + wp_send_json_error(array('message' => esc_html__('Unauthorized', 'mxchat'))); | |
| 250 | + } | |
| 251 | + | |
| 252 | + $provider = isset($_POST['provider']) ? sanitize_key(wp_unslash($_POST['provider'])) : ''; | |
| 253 | + $posted_key = isset($_POST['key']) ? trim((string) wp_unslash($_POST['key'])) : ''; | |
| 254 | + | |
| 255 | + $option_map = array( | |
| 256 | + 'openai' => 'api_key', | |
| 257 | + 'xai' => 'xai_api_key', | |
| 258 | + 'claude' => 'claude_api_key', | |
| 259 | + 'deepseek' => 'deepseek_api_key', | |
| 260 | + 'gemini' => 'gemini_api_key', | |
| 261 | + 'openrouter' => 'openrouter_api_key', | |
| 262 | + ); | |
| 263 | + if (!isset($option_map[$provider])) { | |
| 264 | + wp_send_json_error(array('message' => esc_html__('Unknown provider.', 'mxchat'))); | |
| 265 | + } | |
| 266 | + | |
| 267 | + // Prefer the just-typed value (test-before-save); fall back to the saved key. | |
| 268 | + $key = $posted_key; | |
| 269 | + if ($key === '') { | |
| 270 | + $options = get_option('mxchat_options', array()); | |
| 271 | + $key = isset($options[$option_map[$provider]]) ? trim((string) $options[$option_map[$provider]]) : ''; | |
| 272 | + } | |
| 273 | + if ($key === '') { | |
| 274 | + wp_send_json_error(array('message' => esc_html__('No API key entered or saved for this provider.', 'mxchat'))); | |
| 275 | + } | |
| 276 | + | |
| 277 | + // Lightest authenticated metadata call per provider — model-agnostic, no generation. | |
| 278 | + $headers = array(); | |
| 279 | + switch ($provider) { | |
| 280 | + case 'openai': | |
| 281 | + $url = 'https://api.openai.com/v1/models'; | |
| 282 | + $headers = array('Authorization' => 'Bearer ' . $key); | |
| 283 | + break; | |
| 284 | + case 'xai': | |
| 285 | + $url = 'https://api.x.ai/v1/models'; | |
| 286 | + $headers = array('Authorization' => 'Bearer ' . $key); | |
| 287 | + break; | |
| 288 | + case 'deepseek': | |
| 289 | + $url = 'https://api.deepseek.com/models'; | |
| 290 | + $headers = array('Authorization' => 'Bearer ' . $key); | |
| 291 | + break; | |
| 292 | + case 'openrouter': | |
| 293 | + // /auth/key validates the key itself (the public /models list does not). | |
| 294 | + $url = 'https://openrouter.ai/api/v1/auth/key'; | |
| 295 | + $headers = array('Authorization' => 'Bearer ' . $key); | |
| 296 | + break; | |
| 297 | + case 'gemini': | |
| 298 | + $url = add_query_arg(array('pageSize' => 1, 'key' => $key), 'https://generativelanguage.googleapis.com/v1beta/models'); | |
| 299 | + break; | |
| 300 | + case 'claude': | |
| 301 | + $url = 'https://api.anthropic.com/v1/models'; | |
| 302 | + $headers = array('x-api-key' => $key, 'anthropic-version' => '2023-06-01'); | |
| 303 | + break; | |
| 304 | + default: | |
| 305 | + wp_send_json_error(array('message' => esc_html__('Unknown provider.', 'mxchat'))); | |
| 306 | + } | |
| 307 | + | |
| 308 | + $response = wp_remote_get($url, array( | |
| 309 | + 'headers' => $headers, | |
| 310 | + 'timeout' => 10, | |
| 311 | + )); | |
| 312 | + | |
| 313 | + if (is_wp_error($response)) { | |
| 314 | + wp_send_json_error(array('message' => sprintf(esc_html__('Network error: %s', 'mxchat'), esc_html($response->get_error_message())))); | |
| 315 | + } | |
| 316 | + | |
| 317 | + $code = (int) wp_remote_retrieve_response_code($response); | |
| 318 | + if ($code >= 200 && $code < 300) { | |
| 319 | + wp_send_json_success(array('message' => esc_html__('Key is valid.', 'mxchat'))); | |
| 320 | + } | |
| 321 | + | |
| 322 | + // Surface the provider's own error text when present (trimmed; key never echoed). | |
| 323 | + $detail = ''; | |
| 324 | + $body = json_decode(wp_remote_retrieve_body($response), true); | |
| 325 | + if (is_array($body)) { | |
| 326 | + if (isset($body['error']['message'])) { | |
| 327 | + $detail = $body['error']['message']; | |
| 328 | + } elseif (isset($body['error']) && is_string($body['error'])) { | |
| 329 | + $detail = $body['error']; | |
| 330 | + } elseif (isset($body['message'])) { | |
| 331 | + $detail = $body['message']; | |
| 332 | + } | |
| 333 | + } | |
| 334 | + $detail = trim((string) $detail); | |
| 335 | + if (strlen($detail) > 200) { | |
| 336 | + $detail = substr($detail, 0, 200) . '…'; | |
| 337 | + } | |
| 338 | + | |
| 339 | + if ($code === 401 || $code === 403) { | |
| 340 | + $msg = ($detail !== '') | |
| 341 | + ? sprintf(esc_html__('Key rejected (HTTP %1$d): %2$s', 'mxchat'), $code, esc_html($detail)) | |
| 342 | + : sprintf(esc_html__('Key rejected (HTTP %d). Check the API key.', 'mxchat'), $code); | |
| 343 | + wp_send_json_error(array('message' => $msg)); | |
| 344 | + } | |
| 345 | + | |
| 346 | + $msg = ($detail !== '') | |
| 347 | + ? sprintf(esc_html__('Provider returned HTTP %1$d: %2$s', 'mxchat'), $code, esc_html($detail)) | |
| 348 | + : sprintf(esc_html__('Provider returned HTTP %d.', 'mxchat'), $code); | |
| 349 | + wp_send_json_error(array('message' => $msg)); | |
| 350 | +} | |
| 351 | + | |
| 129 | 352 | // ======================================== |
| 130 | 353 | // SETTINGS AJAX HANDLERS |
| 131 | 354 | // ======================================== |
| 132 | 355 | |
| @@ -140,10 +363,10 @@ | ||
| 140 | 363 | wp_send_json_error(['message' => esc_html__('Unauthorized', 'mxchat')]); |
| 141 | 364 | } |
| 142 | 365 | |
| 143 | 366 | $name = isset($_POST['name']) ? $_POST['name'] : ''; |
| 144 | - // Strip slashes from the value before saving | |
| 145 | - $value = isset($_POST['value']) ? stripslashes($_POST['value']) : ''; | |
| 367 | + // Remove WP's added slashes before saving (wp_unslash is the canonical form; plan-3f8158). | |
| 368 | + $value = isset($_POST['value']) ? wp_unslash($_POST['value']) : ''; | |
| 146 | 369 | |
| 147 | 370 | //error_log('MXChat Save: Processing field name: ' . $name); |
| 148 | 371 | //error_log('MXChat Save: Field value: ' . $value); |
| 149 | 372 | |
| @@ -164,8 +387,113 @@ | ||
| 164 | 387 | } |
| 165 | 388 | |
| 166 | 389 | // Handle special cases |
| 167 | 390 | switch ($field_name) { |
| 391 | + // Editor Assistant enable toggle (plan-8cb0cb). STANDALONE option — NOT part | |
| 392 | + // of mxchat_options, so it skips the mxchat_sanitize strip-trap entirely. Save | |
| 393 | + // it directly and short-circuit (mirrors the mxchat_transcripts_options pattern | |
| 394 | + // below); never falls through to the generic mxchat_options save. Default OFF. | |
| 395 | + case 'mxchat_editor_assistant_enabled': | |
| 396 | + $ea_value = ($value === 'on' || $value === '1') ? 'on' : 'off'; | |
| 397 | + update_option('mxchat_editor_assistant_enabled', $ea_value); | |
| 398 | + wp_send_json_success(['message' => esc_html__('Setting saved', 'mxchat')]); | |
| 399 | + return; | |
| 400 | + | |
| 401 | + // Smart asset loading toggle (plan-915355). STANDALONE option, same | |
| 402 | + // reasoning as the Editor Assistant case above — saved directly and | |
| 403 | + // short-circuited so it never touches mxchat_options / mxchat_sanitize. | |
| 404 | + // Default OFF (opt-in performance optimization). | |
| 405 | + case 'mxchat_smart_asset_loading': | |
| 406 | + $sal_value = ($value === 'on' || $value === '1') ? 'on' : 'off'; | |
| 407 | + update_option('mxchat_smart_asset_loading', $sal_value); | |
| 408 | + wp_send_json_success(['message' => esc_html__('Setting saved', 'mxchat')]); | |
| 409 | + return; | |
| 410 | + | |
| 411 | + // Hybrid keyword boost toggle (plan-38ffa1). STANDALONE option, same | |
| 412 | + // pattern. Enabling runs capability detection HERE, at admin-save time — | |
| 413 | + // building the FULLTEXT index during a visitor's chat request is not | |
| 414 | + // acceptable, and detection is a one-time cost the admin can wait on. | |
| 415 | + case 'mxchat_hybrid_keyword_toggle': | |
| 416 | + $hkb_value = ($value === 'on' || $value === '1') ? 'on' : 'off'; | |
| 417 | + update_option('mxchat_hybrid_keyword_toggle', $hkb_value); | |
| 418 | + if ($hkb_value === 'on') { | |
| 419 | + MxChat_Utils::mxchat_hybrid_detect_capability(true); | |
| 420 | + } | |
| 421 | + wp_send_json_success(['message' => esc_html__('Setting saved', 'mxchat')]); | |
| 422 | + return; | |
| 423 | + | |
| 424 | + // ACF→PDF import-time extraction (plan 11720c). STANDALONE option, same | |
| 425 | + // pattern. Moved from a per-import modal checkbox to an install-level | |
| 426 | + // setting on Knowledge → ACF Fields. Stored '1'/'0' to match the | |
| 427 | + // knowledge page's sibling toggles. Default OFF. | |
| 428 | + case 'mxchat_acf_pdf_extraction': | |
| 429 | + $apx_value = ($value === 'on' || $value === '1') ? '1' : '0'; | |
| 430 | + update_option('mxchat_acf_pdf_extraction', $apx_value); | |
| 431 | + wp_send_json_success(['message' => esc_html__('Setting saved', 'mxchat')]); | |
| 432 | + return; | |
| 433 | + | |
| 434 | + // In-chat YouTube card: master switch + its own confidence floor | |
| 435 | + // (plan f52492). STANDALONE options, same pattern as the cases above. | |
| 436 | + // Default ON — the card already ships, so this is an opt-OUT. | |
| 437 | + case 'mxchat_video_embed_enabled': | |
| 438 | + $vce_value = ($value === 'on' || $value === '1') ? 'on' : 'off'; | |
| 439 | + update_option('mxchat_video_embed_enabled', $vce_value); | |
| 440 | + wp_send_json_success(['message' => esc_html__('Setting saved', 'mxchat')]); | |
| 441 | + return; | |
| 442 | + | |
| 443 | + // Clamped to the same 20-95 the field advertises. An out-of-range or | |
| 444 | + // non-numeric POST is corrected rather than refused, and the corrected | |
| 445 | + // value is echoed back so the field can reconcile — a silently stored | |
| 446 | + // 0 here would put a video on every answer, which is the bug. | |
| 447 | + case 'mxchat_video_embed_threshold': | |
| 448 | + $vct_value = is_numeric($value) | |
| 449 | + ? (int) $value | |
| 450 | + : MXCHAT_VIDEO_EMBED_THRESHOLD_DEFAULT; | |
| 451 | + if ($vct_value < 20) { $vct_value = 20; } | |
| 452 | + if ($vct_value > 95) { $vct_value = 95; } | |
| 453 | + update_option('mxchat_video_embed_threshold', $vct_value); | |
| 454 | + wp_send_json_success([ | |
| 455 | + 'message' => esc_html__('Setting saved', 'mxchat'), | |
| 456 | + 'value' => $vct_value, | |
| 457 | + ]); | |
| 458 | + return; | |
| 459 | + | |
| 460 | + // Live-agent availability schedules (plans 8ccaa2 + 99d7a4). STANDALONE | |
| 461 | + // options, same reasoning as the Editor Assistant case above — nested | |
| 462 | + // structures that mxchat_sanitize() would strip on the next autosave of any | |
| 463 | + // other field. Each channel's value arrives as JSON from its own hidden | |
| 464 | + // input, which that channel's schedule editor keeps in sync; the class owns | |
| 465 | + // all validation. The bare legacy name is kept as a defense against a | |
| 466 | + // browser still running pre-split cached admin JS: that UI edited "both | |
| 467 | + // channels" as one, so its save writes both. | |
| 468 | + case 'live_agent_schedule_slack': | |
| 469 | + case 'live_agent_schedule_telegram': | |
| 470 | + case 'live_agent_schedule_webhook': | |
| 471 | + case 'live_agent_schedule': | |
| 472 | + if (!class_exists('MxChat_Live_Agent_Schedule')) { | |
| 473 | + wp_send_json_error(['message' => esc_html__('Schedule unavailable', 'mxchat')]); | |
| 474 | + return; | |
| 475 | + } | |
| 476 | + $decoded = json_decode($value, true); | |
| 477 | + if (!is_array($decoded)) { | |
| 478 | + wp_send_json_error(['message' => esc_html__('Invalid schedule', 'mxchat')]); | |
| 479 | + return; | |
| 480 | + } | |
| 481 | + $channels = ($field_name === 'live_agent_schedule') | |
| 482 | + ? array('slack', 'telegram') | |
| 483 | + : array(substr($field_name, strlen('live_agent_schedule_'))); | |
| 484 | + $saved_schedule = null; | |
| 485 | + foreach ($channels as $schedule_channel) { | |
| 486 | + $saved_schedule = MxChat_Live_Agent_Schedule::save($schedule_channel, $decoded); | |
| 487 | + } | |
| 488 | + // Echo the normalized result so the editor can reconcile if it ever | |
| 489 | + // disagrees with the server (e.g. a time the class rejected). | |
| 490 | + wp_send_json_success([ | |
| 491 | + 'message' => esc_html__('Setting saved', 'mxchat'), | |
| 492 | + 'schedule' => $saved_schedule, | |
| 493 | + ]); | |
| 494 | + return; | |
| 495 | + | |
| 168 | 496 | case 'model': |
| 169 | 497 | //error_log('MXChat Save: Processing model selection'); |
| 170 | 498 | //error_log('MXChat Save: Model value received: ' . $value); |
| 171 | 499 | //error_log('MXChat Save: Value type: ' . gettype($value)); |
| @@ -237,8 +565,15 @@ | ||
| 237 | 565 | //error_log('MXChat Save: Processing email_blocker_header_content'); |
| 238 | 566 | // Allow HTML content but sanitize it safely |
| 239 | 567 | $options[$field_name] = wp_kses_post($value); |
| 240 | 568 | break; |
| 569 | + case 'intro_message': | |
| 570 | + // Stored-XSS hardening (Wordfence CWE-79, plan-3f8158): sanitize on save as | |
| 571 | + // defense in depth. wp_kses_post mirrors mxchat_sanitize() (the options.php | |
| 572 | + // save path) so both save routes treat intro_message identically and strip | |
| 573 | + // <script>/</textarea> breakout while keeping basic formatting + {visitor_name}. | |
| 574 | + $options[$field_name] = wp_kses_post($value); | |
| 575 | + break; | |
| 241 | 576 | case 'email_blocker_button_text': |
| 242 | 577 | //error_log('MXChat Save: Processing email_blocker_button_text'); |
| 243 | 578 | $options[$field_name] = sanitize_text_field($value); |
| 244 | 579 | break; |
| @@ -245,8 +580,13 @@ | ||
| 245 | 580 | case 'name_field_placeholder': |
| 246 | 581 | //error_log('MXChat Save: Processing name_field_placeholder'); |
| 247 | 582 | $options[$field_name] = sanitize_text_field($value); |
| 248 | 583 | break; |
| 584 | + case 'consent_checkbox_label': | |
| 585 | + // b062c4 — same allowlist as the options.php save path and the | |
| 586 | + // widget render, so the stored label always equals the shown label. | |
| 587 | + $options[$field_name] = MxChat_Utils::sanitize_consent_label($value); | |
| 588 | + break; | |
| 249 | 589 | case 'similarity_threshold': |
| 250 | 590 | //error_log('MXChat Save: Processing similarity_threshold'); |
| 251 | 591 | // Validate and save - enforce min 20, max 85 |
| 252 | 592 | $threshold = intval($value); |
| @@ -273,8 +613,49 @@ | ||
| 273 | 613 | //error_log('MXChat Save: Processing live_agent_status'); |
| 274 | 614 | // Set the new value |
| 275 | 615 | $options[$field_name] = ($value === 'on') ? 'on' : 'off'; |
| 276 | 616 | break; |
| 617 | + // Shared handoff channel (plan 1a2666): re-probe the channel's privacy | |
| 618 | + // at configuration time so the settings screen can warn about private | |
| 619 | + // channels (their inbound events arrive as message.groups, which the | |
| 620 | + // documented app setup never subscribes to). Only id-shaped values can | |
| 621 | + // be checked before the first handoff resolves a #name — the handoff | |
| 622 | + // path probes those when it caches the resolved id. | |
| 623 | + case 'live_agent_shared_channel': | |
| 624 | + $options[$field_name] = sanitize_text_field($value); | |
| 625 | + delete_option('mxchat_slack_shared_channel_privacy'); | |
| 626 | + $shared_channel_target = ltrim(trim((string) $options[$field_name]), '#'); | |
| 627 | + if ($shared_channel_target !== '' && preg_match('/^[CG][A-Z0-9]{6,}$/', $shared_channel_target)) { | |
| 628 | + if (!class_exists('MxChat_Integrator')) { | |
| 629 | + require_once plugin_dir_path(dirname(__FILE__)) . 'includes/class-mxchat-integrator.php'; | |
| 630 | + } | |
| 631 | + MxChat_Integrator::mxchat_probe_slack_channel_privacy( | |
| 632 | + $options['live_agent_bot_token'] ?? '', | |
| 633 | + $shared_channel_target, | |
| 634 | + trim((string) $options[$field_name]) | |
| 635 | + ); | |
| 636 | + } | |
| 637 | + break; | |
| 638 | + // Webhook handoff destination (plan d88e22). Status normalized like the | |
| 639 | + // other channel toggles; the URL is refused outright when it isn't | |
| 640 | + // https so the admin hears about it at save time instead of the | |
| 641 | + // handoff silently never firing. | |
| 642 | + case 'webhook_handoff_status': | |
| 643 | + $options[$field_name] = ($value === 'on') ? 'on' : 'off'; | |
| 644 | + break; | |
| 645 | + case 'webhook_handoff_url': | |
| 646 | + $wh_url = trim((string) $value); | |
| 647 | + if ($wh_url === '') { | |
| 648 | + $options[$field_name] = ''; | |
| 649 | + break; | |
| 650 | + } | |
| 651 | + $wh_clean = esc_url_raw($wh_url, array('https')); | |
| 652 | + if ($wh_clean === '' || stripos($wh_clean, 'https://') !== 0) { | |
| 653 | + wp_send_json_error(['message' => esc_html__('Webhook URL must start with https://', 'mxchat')]); | |
| 654 | + return; | |
| 655 | + } | |
| 656 | + $options[$field_name] = $wh_clean; | |
| 657 | + break; | |
| 277 | 658 | case 'enable_web_search': |
| 278 | 659 | //error_log('MXChat Save: Processing enable_web_search'); |
| 279 | 660 | $options[$field_name] = ($value === 'on') ? 'on' : 'off'; |
| 280 | 661 | break; |
| @@ -322,9 +703,16 @@ | ||
| 322 | 703 | $transcripts_options = array(); |
| 323 | 704 | } |
| 324 | 705 | |
| 325 | 706 | // Handle checkbox values (convert 'on'/'off' to 1/0) |
| 326 | - if ($value === 'on' || $value === '1') { | |
| 707 | + if ($field_name === 'mxchat_retention_days') { | |
| 708 | + // Number field, NOT a checkbox — without this branch a | |
| 709 | + // value of '1' would hit the 'on'/'1' coercion below | |
| 710 | + // (harmlessly) but nothing would clamp: this direct-DB | |
| 711 | + // path bypasses the registered sanitiser and its | |
| 712 | + // 0-3650 clamp entirely (plan-3c3338). | |
| 713 | + $transcripts_options[$field_name] = max(0, min(3650, (int) $value)); | |
| 714 | + } else if ($value === 'on' || $value === '1') { | |
| 327 | 715 | $transcripts_options[$field_name] = 1; |
| 328 | 716 | } else if ($value === 'off' || $value === '0' || $value === '') { |
| 329 | 717 | $transcripts_options[$field_name] = 0; |
| 330 | 718 | } else { |
| @@ -329,9 +717,20 @@ | ||
| 329 | 717 | $transcripts_options[$field_name] = 0; |
| 330 | 718 | } else { |
| 331 | 719 | // For text/select fields, sanitize appropriately |
| 332 | 720 | if ($field_name === 'mxchat_notification_email') { |
| 333 | - $transcripts_options[$field_name] = sanitize_email($value); | |
| 721 | + // sanitize_email() alone CANNOT validate this field: given | |
| 722 | + // "[email protected], [email protected]" it returns the single concatenated | |
| 723 | + // address "[email protected]", which is_email() then accepts. | |
| 724 | + // That is how two addresses used to be stored as one dead | |
| 725 | + // one, silently. MxChat_Utils validates the raw parts first | |
| 726 | + // and refuses the whole list if any of them is bad. | |
| 727 | + $parsed = MxChat_Utils::parse_notification_emails($value); | |
| 728 | + if ($parsed['error'] !== '') { | |
| 729 | + // Reject: the previously stored value stays untouched. | |
| 730 | + wp_send_json_error(array('message' => $parsed['error'])); | |
| 731 | + } | |
| 732 | + $transcripts_options[$field_name] = implode(', ', $parsed['emails']); | |
| 334 | 733 | } else { |
| 335 | 734 | $transcripts_options[$field_name] = sanitize_text_field($value); |
| 336 | 735 | } |
| 337 | 736 | } |
| @@ -508,10 +907,14 @@ | ||
| 508 | 907 | 'contextual_awareness_toggle', |
| 509 | 908 | 'citation_links_toggle', |
| 510 | 909 | 'enable_email_block', |
| 511 | 910 | 'enable_name_field', |
| 911 | + 'enable_consent_checkbox', | |
| 912 | + 'consent_checkbox_required', | |
| 512 | 913 | 'custom_provider_for_embeddings', |
| 513 | - 'custom_provider_for_images' | |
| 914 | + 'custom_provider_for_images', | |
| 915 | + 'print_button_enabled', | |
| 916 | + 'reset_chat_enabled' | |
| 514 | 917 | ])) { |
| 515 | 918 | //error_log('MXChat Save: Processing toggle: ' . $field_name); |
| 516 | 919 | $options[$field_name] = ($value === 'on') ? 'on' : 'off'; |
| 517 | 920 | } else { |
| @@ -654,8 +1057,42 @@ | ||
| 654 | 1057 | if ($field_name === 'mxchat_pinecone_host') { |
| 655 | 1058 | $new_value = str_replace(['https://', 'http://'], '', $new_value); |
| 656 | 1059 | } |
| 657 | 1060 | break; |
| 1061 | + case 'mxchat_pinecone_index_type': | |
| 1062 | + // 362c31: document only once the host has been checked (see sanitize_pinecone_settings). | |
| 1063 | + $new_value = ($value === 'document') ? 'document' : 'vector'; | |
| 1064 | + if ($new_value === 'document' && strtolower(trim((string) ($current_options['mxchat_pinecone_host'] ?? ''), '/')) !== strtolower((string) ($current_options['mxchat_pinecone_docs_verified_host'] ?? ''))) { | |
| 1065 | + wp_send_json_error(['message' => esc_html__('Check the document index host first (Check index or Create index for me).', 'mxchat')]); | |
| 1066 | + } | |
| 1067 | + // 3e83e4: same gate as the form save — no switch onto a half-copied index. | |
| 1068 | + if ($new_value === 'document' && (($current_options['mxchat_pinecone_index_type'] ?? 'vector') !== 'document') && class_exists('MxChat_Pinecone_Documents')) { | |
| 1069 | + $running = MxChat_Pinecone_Documents::migration_blocks_switch((string) ($current_options['mxchat_pinecone_host'] ?? '')); | |
| 1070 | + if ($running !== null) { | |
| 1071 | + wp_send_json_error(['message' => MxChat_Pinecone_Documents::switch_blocked_message($running)]); | |
| 1072 | + } | |
| 1073 | + } | |
| 1074 | + break; | |
| 1075 | + case 'mxchat_pinecone_vector_host': | |
| 1076 | + $new_value = trim(str_replace(['https://', 'http://'], '', sanitize_text_field($value)), '/'); | |
| 1077 | + break; | |
| 1078 | + case 'mxchat_pinecone_vector_index': | |
| 1079 | + case 'mxchat_pinecone_docs_region': | |
| 1080 | + $new_value = sanitize_text_field($value); | |
| 1081 | + break; | |
| 1082 | + case 'mxchat_pinecone_docs_cloud': | |
| 1083 | + $new_value = in_array($value, array('aws', 'gcp', 'azure'), true) ? $value : 'aws'; | |
| 1084 | + break; | |
| 1085 | + case 'mxchat_pinecone_docs_language': | |
| 1086 | + $lang_value = sanitize_key($value); | |
| 1087 | + $new_value = (class_exists('MxChat_Pinecone_Documents') && array_key_exists($lang_value, MxChat_Pinecone_Documents::languages())) ? $lang_value : 'en'; | |
| 1088 | + break; | |
| 1089 | + case 'mxchat_pinecone_top_k': | |
| 1090 | + // d0cae1: out-of-range and junk normalize to the default 50 — | |
| 1091 | + // same clamp the read site applies. | |
| 1092 | + $top_k = absint($value); | |
| 1093 | + $new_value = (string) (($top_k >= 1 && $top_k <= 1000) ? $top_k : 50); | |
| 1094 | + break; | |
| 658 | 1095 | default: |
| 659 | 1096 | wp_send_json_error(['message' => esc_html__('Unknown Pinecone field', 'mxchat')]); |
| 660 | 1097 | } |
| 661 | 1098 | |
| @@ -685,14 +1122,16 @@ | ||
| 685 | 1122 | ); |
| 686 | 1123 | //error_log('[MXCHAT-PROMPTS] Updated existing option, result: ' . ($save_result !== false ? 'SUCCESS' : 'FAILED')); |
| 687 | 1124 | } else { |
| 688 | 1125 | // Insert new option |
| 1126 | + // Credential option — must NOT autoload (holds the Pinecone secret; | |
| 1127 | + // autoloaded rows are read into memory on every request). | |
| 689 | 1128 | $save_result = $wpdb->insert( |
| 690 | 1129 | $wpdb->options, |
| 691 | 1130 | array( |
| 692 | 1131 | 'option_name' => 'mxchat_pinecone_addon_options', |
| 693 | 1132 | 'option_value' => $serialized_options, |
| 694 | - 'autoload' => 'yes' | |
| 1133 | + 'autoload' => 'off' | |
| 695 | 1134 | ), |
| 696 | 1135 | array('%s', '%s', '%s') |
| 697 | 1136 | ); |
| 698 | 1137 | //error_log('[MXCHAT-PROMPTS] Inserted new option, result: ' . ($save_result !== false ? 'SUCCESS' : 'FAILED')); |
| @@ -810,10 +1249,18 @@ | ||
| 810 | 1249 | } |
| 811 | 1250 | |
| 812 | 1251 | // Handle ACF field exclusion toggles |
| 813 | 1252 | if (strpos($name, 'mxchat_acf_field_') === 0) { |
| 814 | - // Extract field name from the input name (e.g., mxchat_acf_field_private_notes -> private_notes) | |
| 815 | - $field_name = str_replace('mxchat_acf_field_', '', $name); | |
| 1253 | + // The identifier after the prefix is the ACF field KEY (unique per | |
| 1254 | + // field), not the field name — names are shared across groups and | |
| 1255 | + // collide (plan 30e81f). Reject anything that isn't key-shaped so a | |
| 1256 | + // stale pre-3.2.20 page (or its exit beacon) posting a bare name | |
| 1257 | + // can't write junk into the key-based list. | |
| 1258 | + $field_key = str_replace('mxchat_acf_field_', '', $name); | |
| 1259 | + if (!preg_match('/^field_[A-Za-z0-9_\-]+$/', $field_key)) { | |
| 1260 | + wp_send_json_error(['message' => esc_html__('Invalid ACF field identifier', 'mxchat')]); | |
| 1261 | + return; | |
| 1262 | + } | |
| 816 | 1263 | $is_enabled = ($value === 'on' || $value === '1'); |
| 817 | 1264 | |
| 818 | 1265 | // Get current excluded fields |
| 819 | 1266 | $excluded_fields = get_option('mxchat_acf_excluded_fields', array()); |
| @@ -822,13 +1269,19 @@ | ||
| 822 | 1269 | } |
| 823 | 1270 | |
| 824 | 1271 | if ($is_enabled) { |
| 825 | 1272 | // Remove from exclusion list (field should be included) |
| 826 | - $excluded_fields = array_values(array_diff($excluded_fields, array($field_name))); | |
| 1273 | + $excluded_fields = array_values(array_diff($excluded_fields, array($field_key))); | |
| 1274 | + // Lazy legacy-name conversion: if this field's NAME is still | |
| 1275 | + // stored (its group was inactive when the 30e81f migration | |
| 1276 | + // ran), including this one field must not silently include | |
| 1277 | + // its same-named twins — swap the name entry for the keys of | |
| 1278 | + // every OTHER field currently wearing that name. | |
| 1279 | + $excluded_fields = $this->mxchat_expand_legacy_acf_name_entry($excluded_fields, $field_key); | |
| 827 | 1280 | } else { |
| 828 | 1281 | // Add to exclusion list (field should be excluded) |
| 829 | - if (!in_array($field_name, $excluded_fields)) { | |
| 830 | - $excluded_fields[] = $field_name; | |
| 1282 | + if (!in_array($field_key, $excluded_fields, true)) { | |
| 1283 | + $excluded_fields[] = $field_key; | |
| 831 | 1284 | } |
| 832 | 1285 | } |
| 833 | 1286 | |
| 834 | 1287 | $updated = update_option('mxchat_acf_excluded_fields', $excluded_fields); |
| @@ -835,10 +1288,10 @@ | ||
| 835 | 1288 | |
| 836 | 1289 | if ($updated || true) { // Always report success since the state may already be correct |
| 837 | 1290 | wp_send_json_success([ |
| 838 | 1291 | 'message' => $is_enabled |
| 839 | - ? sprintf(esc_html__('Field "%s" will be included in imports', 'mxchat'), $field_name) | |
| 840 | - : sprintf(esc_html__('Field "%s" will be excluded from imports', 'mxchat'), $field_name) | |
| 1292 | + ? esc_html__('Field will be included in imports', 'mxchat') | |
| 1293 | + : esc_html__('Field will be excluded from imports', 'mxchat') | |
| 841 | 1294 | ]); |
| 842 | 1295 | } else { |
| 843 | 1296 | wp_send_json_error(['message' => esc_html__('Failed to save ACF field setting', 'mxchat')]); |
| 844 | 1297 | } |
| @@ -858,12 +1311,12 @@ | ||
| 858 | 1311 | } |
| 859 | 1312 | return; |
| 860 | 1313 | } |
| 861 | 1314 | |
| 862 | - // Handle other prompts options | |
| 1315 | + // Handle other prompts options (autoload false — can hold the Pinecone secret) | |
| 863 | 1316 | $options = get_option('mxchat_prompts_options', []); |
| 864 | 1317 | $options[$name] = $value; |
| 865 | - $updated = update_option('mxchat_prompts_options', $options); | |
| 1318 | + $updated = update_option('mxchat_prompts_options', $options, false); | |
| 866 | 1319 | |
| 867 | 1320 | if ($updated) { |
| 868 | 1321 | wp_send_json_success(['message' => esc_html__('Setting saved', 'mxchat')]); |
| 869 | 1322 | } else { |
| @@ -870,10 +1323,130 @@ | ||
| 870 | 1323 | wp_send_json_error(['message' => esc_html__('No changes detected', 'mxchat')]); |
| 871 | 1324 | } |
| 872 | 1325 | } |
| 873 | 1326 | |
| 1327 | + /** | |
| 1328 | + * If the field behind $included_key still has its NAME stored in the | |
| 1329 | + * exclusion list (a legacy entry the 30e81f migration could not resolve | |
| 1330 | + * because the group was inactive), replace that name with the keys of | |
| 1331 | + * every OTHER current field wearing it. Including one field must never | |
| 1332 | + * silently include its same-named twins — that would be the original | |
| 1333 | + * collision bug in reverse, in the unsafe (privacy-losing) direction. | |
| 1334 | + */ | |
| 1335 | + private function mxchat_expand_legacy_acf_name_entry($excluded_fields, $included_key) { | |
| 1336 | + if (!function_exists('acf_get_field')) { | |
| 1337 | + return $excluded_fields; | |
| 1338 | + } | |
| 1339 | + $field = acf_get_field($included_key); | |
| 1340 | + if (!$field || empty($field['name'])) { | |
| 1341 | + return $excluded_fields; | |
| 1342 | + } | |
| 1343 | + $field_name = $field['name']; | |
| 1344 | + if (!in_array($field_name, $excluded_fields, true)) { | |
| 1345 | + return $excluded_fields; | |
| 1346 | + } | |
| 1347 | + $excluded_fields = array_values(array_diff($excluded_fields, array($field_name))); | |
| 1348 | + foreach ($this->mxchat_acf_keys_for_name($field_name) as $twin_key) { | |
| 1349 | + if ($twin_key !== $included_key && !in_array($twin_key, $excluded_fields, true)) { | |
| 1350 | + $excluded_fields[] = $twin_key; | |
| 1351 | + } | |
| 1352 | + } | |
| 1353 | + return $excluded_fields; | |
| 1354 | + } | |
| 874 | 1355 | |
| 875 | 1356 | /** |
| 1357 | + * Keys of every currently-registered top-level ACF field with this name. | |
| 1358 | + */ | |
| 1359 | + private function mxchat_acf_keys_for_name($field_name) { | |
| 1360 | + $keys = array(); | |
| 1361 | + if (!function_exists('acf_get_field_groups') || !function_exists('acf_get_fields')) { | |
| 1362 | + return $keys; | |
| 1363 | + } | |
| 1364 | + foreach (acf_get_field_groups() as $group) { | |
| 1365 | + $group_fields = acf_get_fields($group['key']); | |
| 1366 | + if (empty($group_fields)) { | |
| 1367 | + continue; | |
| 1368 | + } | |
| 1369 | + foreach ($group_fields as $field) { | |
| 1370 | + if (isset($field['name'], $field['key']) && $field['name'] === $field_name) { | |
| 1371 | + $keys[] = $field['key']; | |
| 1372 | + } | |
| 1373 | + } | |
| 1374 | + } | |
| 1375 | + return $keys; | |
| 1376 | + } | |
| 1377 | + | |
| 1378 | + /** | |
| 1379 | + * Group-level ACF toggle (plan bf57e0): sets every field in one ACF field | |
| 1380 | + * group included or excluded in a SINGLE option write. The client must | |
| 1381 | + * never loop the per-field endpoint for this — get_option → modify → | |
| 1382 | + * update_option once per field from twenty concurrent requests is a | |
| 1383 | + * lost-update race that silently drops most of the group. | |
| 1384 | + */ | |
| 1385 | + public function mxchat_acf_toggle_group_callback() { | |
| 1386 | + check_ajax_referer('mxchat_prompts_setting_nonce'); | |
| 1387 | + | |
| 1388 | + if (!current_user_can('manage_options')) { | |
| 1389 | + wp_send_json_error(['message' => esc_html__('Insufficient permissions', 'mxchat')], 403); | |
| 1390 | + return; | |
| 1391 | + } | |
| 1392 | + | |
| 1393 | + if (!function_exists('acf_get_fields')) { | |
| 1394 | + wp_send_json_error(['message' => esc_html__('ACF is not active', 'mxchat')]); | |
| 1395 | + return; | |
| 1396 | + } | |
| 1397 | + | |
| 1398 | + $group_key = isset($_POST['group_key']) ? sanitize_text_field(wp_unslash($_POST['group_key'])) : ''; | |
| 1399 | + if (!preg_match('/^group_[A-Za-z0-9_\-]+$/', $group_key)) { | |
| 1400 | + wp_send_json_error(['message' => esc_html__('Invalid ACF group identifier', 'mxchat')]); | |
| 1401 | + return; | |
| 1402 | + } | |
| 1403 | + $state = isset($_POST['state']) ? sanitize_text_field(wp_unslash($_POST['state'])) : ''; | |
| 1404 | + $include = ($state === 'on' || $state === '1'); | |
| 1405 | + | |
| 1406 | + // Resolve the group's fields SERVER-side — a client-supplied key list | |
| 1407 | + // is not trusted. This is the same call the settings UI lists from, | |
| 1408 | + // so the toggle covers exactly the rendered set (top-level fields). | |
| 1409 | + $group_fields = acf_get_fields($group_key); | |
| 1410 | + if (empty($group_fields)) { | |
| 1411 | + wp_send_json_error(['message' => esc_html__('No fields found for this group', 'mxchat')]); | |
| 1412 | + return; | |
| 1413 | + } | |
| 1414 | + | |
| 1415 | + $excluded_fields = get_option('mxchat_acf_excluded_fields', array()); | |
| 1416 | + if (!is_array($excluded_fields)) { | |
| 1417 | + $excluded_fields = array(); | |
| 1418 | + } | |
| 1419 | + | |
| 1420 | + $touched = array(); | |
| 1421 | + foreach ($group_fields as $field) { | |
| 1422 | + if (empty($field['key'])) { | |
| 1423 | + continue; | |
| 1424 | + } | |
| 1425 | + if ($include) { | |
| 1426 | + $excluded_fields = array_values(array_diff($excluded_fields, array($field['key']))); | |
| 1427 | + $excluded_fields = $this->mxchat_expand_legacy_acf_name_entry($excluded_fields, $field['key']); | |
| 1428 | + } elseif (!in_array($field['key'], $excluded_fields, true)) { | |
| 1429 | + $excluded_fields[] = $field['key']; | |
| 1430 | + } | |
| 1431 | + $touched[] = array( | |
| 1432 | + 'name' => 'mxchat_acf_field_' . $field['key'], | |
| 1433 | + 'value' => $include ? 'on' : 'off', | |
| 1434 | + ); | |
| 1435 | + } | |
| 1436 | + | |
| 1437 | + // The one write — the whole point of this endpoint. | |
| 1438 | + update_option('mxchat_acf_excluded_fields', array_values($excluded_fields)); | |
| 1439 | + | |
| 1440 | + wp_send_json_success([ | |
| 1441 | + 'message' => $include | |
| 1442 | + ? esc_html__('All fields in this group will be included in imports', 'mxchat') | |
| 1443 | + : esc_html__('All fields in this group will be excluded from imports', 'mxchat'), | |
| 1444 | + 'fields' => $touched, | |
| 1445 | + ]); | |
| 1446 | + } | |
| 1447 | + | |
| 1448 | + /** | |
| 876 | 1449 | * Handles AJAX request for Pinecone settings migration |
| 877 | 1450 | */ |
| 878 | 1451 | public function ajax_migrate_pinecone_settings() { |
| 879 | 1452 | // Verify nonce |
| @@ -909,9 +1482,9 @@ | ||
| 909 | 1482 | 'mxchat_pinecone_index' => sanitize_text_field($old_options['mxchat_pinecone_index'] ?? ''), |
| 910 | 1483 | 'mxchat_pinecone_environment' => sanitize_text_field($old_options['mxchat_pinecone_environment'] ?? '') |
| 911 | 1484 | ); |
| 912 | 1485 | |
| 913 | - update_option('mxchat_pinecone_addon_options', $migrated_options); | |
| 1486 | + update_option('mxchat_pinecone_addon_options', $migrated_options, false); | |
| 914 | 1487 | |
| 915 | 1488 | wp_send_json_success(array( |
| 916 | 1489 | 'migrated' => true, |
| 917 | 1490 | 'message' => 'Settings migrated successfully from Pinecone add-on' |
| @@ -1070,9 +1643,17 @@ | ||
| 1070 | 1643 | //error_log('MxChat deactivate: Nonce check failed'); |
| 1071 | 1644 | wp_send_json_error('Security check failed.'); |
| 1072 | 1645 | return; |
| 1073 | 1646 | } |
| 1074 | - | |
| 1647 | + | |
| 1648 | + // plan-mxchat-20260731-c63fb6 — nonce is not authorization. Without this, | |
| 1649 | + // any authenticated user holding the nonce could revoke the site's PRO | |
| 1650 | + // licence. Every sibling handler in this file already checks. | |
| 1651 | + if (!current_user_can('manage_options')) { | |
| 1652 | + wp_send_json_error(esc_html__('Unauthorized', 'mxchat'), 403); | |
| 1653 | + return; | |
| 1654 | + } | |
| 1655 | + | |
| 1075 | 1656 | //error_log('MxChat deactivate: Nonce check passed'); |
| 1076 | 1657 | |
| 1077 | 1658 | $license_key = get_option('mxchat_activation_key'); |
| 1078 | 1659 | $email = get_option('mxchat_pro_email'); |
