Skip to content

Conversation

@ictbeheer
Copy link
Contributor

No description provided.

@ictbeheer ictbeheer requested review from a team, Rovasch, SimonvanWijhe, dtakken, hnccox-yard, mvdhoek1 and rivanuff and removed request for a team November 19, 2025 14:58
@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Coverage report for commit: 82f9e47
File: coverage.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ███████████████████████ │ 73.3%
  10% │ ███░░░░░░░░░░░░░░░░░░░░ │  6.7%
  20% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ███░░░░░░░░░░░░░░░░░░░░ │  6.7%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  80% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  90% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
 100% │ █████░░░░░░░░░░░░░░░░░░ │ 13.3%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 11.11% | Methods: 5.88%
FilesLinesMethodsBranches
src/Attributes
   Meta.php--100.00%
   MetaPrefix.php--100.00%
   TaxonomyPrefix.php--100.00%
   Terms.php--100.00%
src/Contracts
   PostDataInterface.php100.00%100.00%100.00%
src/Enums
   PostStatus.php--100.00%
src
   ImageData.php--100.00%
   PostData.php8.93%3.85%100.00%
   TermData.php--100.00%
   UserData.php--100.00%
src/Mappers
   PostPrefixMapper.php--100.00%
   UserPrefixMapper.php--100.00%
src/Normalizers
   WPPostNormalizer.php--100.00%
src/Providers
   DataServiceProvider.php100.00%100.00%100.00%
src/Traits
   HasMeta.php36.00%-100.00%

🤖 comment via lucassabreu/comment-coverage-clover

Copy link

@mvdhoek1 mvdhoek1 left a comment

Choose a reason for hiding this comment

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

Ging fout omdat het WP_Term object als array werd gebruikt waar properties verwacht worden?

@ictbeheer
Copy link
Contributor Author

Ging fout omdat het WP_Term object als array werd gebruikt waar properties verwacht worden?

Nee, de magic from method van Laravel Data gaf problemen (waardoor de termmeta niet gezet werd), vandaar dat ik nu expliciet de constructor aanroep.

@ictbeheer ictbeheer merged commit 3712355 into main Nov 21, 2025
5 checks passed
@ictbeheer ictbeheer deleted the fix/term-data branch November 21, 2025 13:10
@ictbeheer ictbeheer restored the fix/term-data branch January 28, 2026 10:23
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