Festival sign-up without a phone field passes null to setAttribute() (PHP 8.1 deprecation)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 82/100
Línea de trabajo
Lee system/datatypes/festival.php alrededor de la línea 478 en signupAction() y compara cómo el mismo bloque maneja el campo opcional attending. Confirma que dataFor(self::FIELD_NAME_PHONE) puede devolver null cuando el formulario configurado omite [[phone]]; la tarea se considera terminada al proporcionar una cadena vacía como valor predeterminado y confirmar que la deprecación ya no ocurre en ese caso de registro.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem. A festival whose sign-up form has no phone field logs a PHP 8.1 deprecation on every sign-up: Passing null to parameter #2 ($value) of type string is deprecated at system/datatypes/festival.php:478 (main d97ab7b), where signupAction() stores $form->dataFor(self::FIELD_NAME_PHONE) without a default. The phone field is optional in a configured sign-up form, so dataFor() returns null.
This is the same kind of fault as #408 (null to setAttribute()), in a place that #408 and its pull request #508 do not cover.
Steps to reproduce (fictional test site, PHP 8.2 and 8.5). Make a festival page, configure a sign-up form without [[phone]], and sign up as a visitor: the container log shows the deprecation.
Proposed fix. $form->dataFor(self::FIELD_NAME_PHONE, ''), as the other optional fields of the same block do with attending.
Found while writing known-bug tests for #520.
🤖 Generated with Claude Code
- Lenguaje dominante
- JavaScript
- Estrellas
- 5
- Forks
- 9
- Merge medio
- 19 h 10 min
- PR fusionados (30 d)
- 12
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de PlanBCode/hypha
-
Component: User interface Impact: Low Level: Easy Pagetype: Peer reviewed article Type: Bug Value: Visitors
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Impact: Medium Level: Easy Type: Bug Value: Users
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Impact: Medium Level: Easy Pagetype: Mailinglist Type: Bug Value: Users
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
Impact: Low Level: Easy Type: Bug Value: Coders
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
PlanBCode/hypha#586 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Impact: Medium Level: Easy Type: Bug Usecase: De Stadsbron Value: Visitors
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
PlanBCode/hypha#522 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de PlanBCode/hypha
Issues similares
-
follow-redirects updateAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
mailgun/mailgun.js#544 ·
-
beginner bug frontend
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
BistaDinesh03/critique#266 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
solana-foundation/solana-web3.js#4015 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: dependencies auto-generated cwe: 405 dependencies ecosystem: npm help wanted package: minimatch scope: runtime security security-alert severity: medium source: dependabot supply-chain vulnerability
Dificultad 2/5 Medio día Aptitud para principiantes 65/100
tdk-landscape/tdk-cli-core#2811 ·
-
a11y bug confirmed issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
open-webui/open-webui#33442 · 2 comentarios ·
Los mantenedores suelen responder en 1 día