Skip to content

Conversation

@jclausen
Copy link
Contributor

@jclausen jclausen commented Oct 2, 2025

Resolves incorrect metadata lookup when resolving a class that has a different entity name than the file name

Description

Resolves new errors seen in ContentBox builds using coldbox@be

Please delete options that are not relevant.

  • Bug Fix

Checklist

  • My code follows the style guidelines of this project cfformat
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@jclausen jclausen requested review from Copilot and lmajano October 2, 2025 23:13
Copy link
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

This PR fixes a metadata lookup issue in the ObjectPopulator class where entity names differ from file names. The bug was causing ContentBox build failures when resolving relationships between entities.

Key changes:

  • Modified how class metadata is extracted from relational properties to preserve full path information
  • Updated the entity name lookup logic to properly handle cases where entity names don't match file names
  • Added clarifying comments for the class lookup fallback mechanism

@github-actions
Copy link

github-actions bot commented Oct 2, 2025

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit ae82453. ± Comparison against base commit b31e191.

♻️ This comment has been updated with latest results.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@lmajano lmajano merged commit b4bc8a4 into development Oct 3, 2025
28 checks passed
@lmajano lmajano deleted the patch/Object-Populator-Fix branch October 3, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants