| @@ -366,16 +366,16 @@ | ||
| 366 | 366 | array( 'enabled' => ! empty( $params['resource_hints'] ) ) |
| 367 | 367 | ); |
| 368 | 368 | } |
| 369 | 369 | |
| 370 | - // Opt-in usage analytics. Only acted on when the key is present in the | |
| 371 | - // payload (legacy onboarding-complete sites are left untouched). The | |
| 372 | - // consent toggle defaults OFF in the wizard, so the common path is | |
| 373 | - // usage_tracking=false → tracker stays dormant, no outbound HTTP. | |
| 374 | - // (True since #437; before that the toggle shipped pre-checked and | |
| 375 | - // this comment described an intent the UI did not implement. The | |
| 376 | - // same consent is now also writable from Settings → Privacy & usage | |
| 377 | - // data, via PrivacyModule, which routes through the same opt_in().) | |
| 370 | + // Usage analytics consent. Only acted on when the key is present in | |
| 371 | + // the payload (legacy onboarding-complete sites are left untouched). | |
| 372 | + // The wizard toggle ships pre-checked, so the common path is | |
| 373 | + // usage_tracking=true → opt_in(true) registers the install. Nothing | |
| 374 | + // is sent before the admin clicks Apply. (#437 shipped it off; later | |
| 375 | + // reverted to pre-checked by product decision. The same consent is | |
| 376 | + // also writable from Settings → Privacy & usage data, via | |
| 377 | + // PrivacyModule, which routes through the same opt_in().) | |
| 378 | 378 | if ( array_key_exists( 'usage_tracking', $params ) ) { |
| 379 | 379 | $tracker = Plugin::instance()->usage_tracker(); |
| 380 | 380 | if ( $tracker ) { |
| 381 | 381 | $tracker->opt_in( ! empty( $params['usage_tracking'] ) ); |
