Skip to content

Add all episodes page#31

Merged
leftouterjoins merged 4 commits intodevelopmentfrom
codex/add-all-episodes-page
Jun 17, 2025
Merged

Add all episodes page#31
leftouterjoins merged 4 commits intodevelopmentfrom
codex/add-all-episodes-page

Conversation

@leftouterjoins
Copy link
Owner

Summary

  • add global episodes route
  • show link to episodes in base template
  • list all episodes ordered by published date
  • expose getAllItems() in State
  • test State::getAllItems

Testing

  • composer test

https://chatgpt.com/codex/tasks/task_e_685173737bbc83228cd57bcf831728c2

cursor[bot]

This comment was marked as outdated.

leftouterjoins and others added 3 commits June 17, 2025 08:19
@leftouterjoins leftouterjoins merged commit e028f2c into development Jun 17, 2025
4 checks passed
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: Image Source Logic Error

The episodes.html.php template incorrectly checks for $item['item_image'] to determine the image source. The State::getAllItems() query only returns COALESCE(items.image, feeds.image) AS image, so the field item_image does not exist in the result set. This causes the condition !empty($item['item_image']) to always be false, forcing image URLs to always use feed_id instead of item_id even when an item has its own image.

templates/episodes.html.php#L9-L10

<a href="/item?item_id=<?= $item['id'] ?>">
<img src="/image?<?= !empty($item['item_image']) ? 'item_id=' . $item['id'] : 'feed_id=' . $item['feed_id'] ?>" class="w-32 border-solid border-neutral-800 border inline float-left mr-4">

Fix in Cursor


Was this report helpful? Give feedback by reacting with 👍 or 👎

@leftouterjoins leftouterjoins deleted the codex/add-all-episodes-page branch June 17, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant