Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Festival sign-up without a phone field passes null to setAttribute() (PHP 8.1 deprecation)

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#588 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
82/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
php
Lĩnh vực
backend

Hướng nghiên cứu

Đọc system/datatypes/festival.php quanh dòng 478 trong signupAction() và so sánh cách khối tương tự xử lý trường tùy chọn attending. Xác nhận rằng dataFor(self::FIELD_NAME_PHONE) có thể trả về null khi biểu mẫu được cấu hình bỏ qua [[phone]]; công việc được xem là hoàn tất khi cung cấp chuỗi rỗng làm giá trị mặc định và xác nhận rằng cảnh báo deprecated không còn xảy ra trong trường hợp đăng ký đó.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Impact: Low Level: Easy Pagetype: Festival Type: Bug Value: Coders

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

Ngôn ngữ chính
JavaScript
Star
5
Fork
9
Merge trung bình
18 giờ 7 phút
Pull request đã merge (30 ngày)
11

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của PlanBCode/hypha

Tất cả issue của PlanBCode/hypha

Issue tương tự

Thêm issue về JavaScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.