Skip to content

[Bug, EC] PEES-923: type error in getDataForListfolderGrid#18977

Merged
kingjia90 merged 1 commit into12.3from
listfoldergrid-cast
Feb 9, 2026
Merged

[Bug, EC] PEES-923: type error in getDataForListfolderGrid#18977
kingjia90 merged 1 commit into12.3from
listfoldergrid-cast

Conversation

@kingjia90
Copy link
Copy Markdown
Contributor

Changes in this pull request

Resolves https://github.com/pimcore/service-operations/issues/326

Additional info

@kingjia90 kingjia90 added this to the 12.3.3 milestone Feb 9, 2026
@kingjia90 kingjia90 self-assigned this Feb 9, 2026
Copilot AI review requested due to automatic review settings February 9, 2026 09:01
@kingjia90 kingjia90 added the Bug label Feb 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 9, 2026

Review Checklist

  • Target branch (12.3 for bug fixes, others 12.x)
  • Tests (if it's testable code, there should be a test for it - get help)
  • Docs (every functionality needs to be documented, see here)
  • Migration incl. install.sql (e.g. if the database schema changes, ...)
  • Upgrade notes (deprecations, important information, migration hints, ...)
  • Label
  • Milestone

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes a strict-types TypeError in Pimcore’s asset metadata handling when rendering listfolder grid values by ensuring numeric values are passed to Asset::getById() as an int.

Changes:

  • Cast listfolder-grid numeric metadata values to int before calling \Pimcore\Model\Asset::getById().

@kingjia90 kingjia90 merged commit ab01ae7 into 12.3 Feb 9, 2026
20 checks passed
@kingjia90 kingjia90 deleted the listfoldergrid-cast branch February 9, 2026 09:05
@github-actions github-actions bot locked and limited conversation to collaborators Feb 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants