Disabled local store ('local-empty' dummy) silently accepts registrations

Aperta
  • Qualcosa non va?
#459 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

  • #523 di @faisalahammad — chiusa senza merge

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
68/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
php
Ambito
backend

Direzione di ricerca

Inizia da includes/local-fields.php:87-93 e leggi come acf_get_local_store() seleziona il local-empty store condiviso. Esegui tests/php/includes/test-local-fields.php::test_local_setting_filter_disables_local e analizza il suo reset nel teardown. Il lavoro è completato quando le registrazioni effettuate mentre local è disabilitato vengono scartate oppure il dummy store viene reimpostato in modo sicuro, senza perdite di stato tra i test.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

[Type] Code Quality

Description

When local fields are disabled (via acf_disable_local() or the acf/settings/local filter), acf_get_local_store() (includes/local-fields.php:87-93) returns the shared local-empty "dummy" store — but it is a normal writable ACF_Data store. Registrations made while local is disabled silently accumulate there instead of being rejected, and the same dummy instance is shared between the fields and groups paths.

Low severity (the data is never read while disabled), but it is a foot-gun: re-enabling local does not surface those registrations, and the accumulated state can confuse debugging.

Reproduction

Characterized in #450: tests/php/includes/test-local-fields.php::test_local_setting_filter_disables_local (resets the dummy store in teardown to avoid cross-test pollution).

Suggested fix

Return a write-discarding store (no-op set/append), or reset the dummy store when toggling.

Found during the 2026-06 test campaign (see PR #450).

Lingua principale
PHP
Stelle
136
Fork
73
Merge medio
18h 21m
PR unite (30g)
21

Preparare l'ambiente

  • Nessun Dockerfile né file Docker Compose
  • Ha un modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di WordPress/secure-custom-fields

Tutte le issue di WordPress/secure-custom-fields

Issue simili

Altre issue su PHP

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.