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

[6.x]: Lazy eager loading returns nothing for a multi-instance relation field once its value was read

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#20,235 1 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
68/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ệ
mysql, php
Lĩnh vực
backend, databases

Hướng nghiên cứu

Bắt đầu từ BaseRelationField::getEagerLoadingMap() và so sánh cách nó đọc các điều kiện truy vấn với normalizeValueForInput(), hàm sử dụng getWhereForColumn('elements.id') và khóa values. Kiểm tra cách biểu diễn điều kiện whereIn() của giá trị đã chuẩn hóa, sau đó chạy hoặc bổ sung kiểm thử cho tình huống trường Assets có nhiều phiên bản. Công việc hoàn tất khi lazy eager loading trả về asset cho một trường đã được đọc trên các mục trong tập kết quả có nhiều mục.

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

Mô tả

repo:cms
What happened?

An entry type has one Assets field placed twice in its layout (image, and logoOverHero as a second instance with its own handle). In a site template, {{ element.image.one() }} returns null when element came from a result set of two or more entries (e.g. the nested entries of a Matrix field), although the entry has an image (element.image.ids() returns it, and so does element.image.eagerly(false).one()).

BaseRelationField::getEagerLoadingMap() reads the target IDs from a normalized value's elements.id where with $where['value'], but normalizeValue() adds that constraint with whereIn(), which Laravel stores under values (getQuery()->wheres holds ['type' => 'In', 'column' => 'elements.id', 'values' => [...]]). So the IDs of every source element whose field was already normalized are lost. For single-instance fields fetchRelationsFromDbTable() covers this by querying the relations table, which hides the bug; for multi-instance fields it returns false by design, so the map comes back empty and the element gets an empty eager-loaded result. The element that triggers lazy eager loading always has a normalized value (reading the field is what runs the query), so it always loses its own relations.

The same file reads the same where correctly in normalizeValueForInput() ($value->getWhereForColumn('elements.id') and $where['values']).

Steps to reproduce
  1. Create an Assets field and add it twice to an entry type's field layout (second instance with a different handle).
  2. Add two or more entries of that type to a Matrix field on a page; give the first one an image in the first instance.
  3. In the page's site template: {% for block in entry.myMatrix.all() %}{{ block.image ? block.image.one().id ?? 'null' }}{% endfor %}.
Expected behavior

The asset ID, as with block.image.eagerly(false).one() or with a single-instance field.

Actual behavior

null (block.image.all() is empty). Changing $where['value'] to $where['values'] in BaseRelationField::getEagerLoadingMap() fixes it.

Craft CMS version

6.x-dev (603f0e0, 2026-10-31)

PHP version

8.5

Operating system and version

macOS (Laravel Herd)

Database type and version

MySQL 8

Image driver and version

—

Installed plugins and versions

None needed to reproduce.

Ngôn ngữ chính
PHP
Star
3.6k
Fork
710
Merge trung bình
12 giờ 44 phút
Pull request đã merge (30 ngày)
335

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

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 craftcms/cms

Tất cả issue của craftcms/cms

Issue tương tự

Thêm issue về PHP

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.