task: #3624733 Adopt the single media audio field from Varbase Podcasts Base

Issue: #3624733 Adopt the single media audio field from Varbase Podcasts Base

What and why

Varbase Podcasts Base #3624729 (MR !5 (merged)) replaces the podcast episode's two audio fields — field_audio (local file upload) and field_audio_url (link) — with one entity reference to a media entity targeting the audio and remote_audio bundles.

#3624658 (MR !40 (merged)) had already moved the podcast content model out of RightUp into varbase_podcasts_base, but five pieces of the old model were left behind in RightUp and now contradict that change. This removes them and adopts the single media audio field.

The change

  1. Deleted the duplicate field storages. config/field.storage.node.field_audio.yml and config/field.storage.node.field_audio_url.yml both duplicate storages the base recipe owns. The first declared type: file where the base now declares type: entity_reference with target_type: media, and under strict: false whichever imports first wins silently.
  2. Dropped the removed link field from the podcast view displays. Removed the field.field.node.podcast.field_audio_url dependency and the field_audio_url: true hidden entry from the live_feed, search_index and search_result displays. field_audio stays hidden in all three, deliberately.
  3. Added the audio media entities. Seven audio media entities under content/media/, one per podcast audio file, with each demo episode's field_audio repointed at its media UUID and the depends: entry moved from file to media.
  4. Updated the Canvas node.podcast.full override (the simpleConfigUpdate action in recipe.yml). The local audio player now walks field_audio > entity > media:audio > field_media_audio_file > entity > file > uri, matching the base recipe. The remote player component bound to the removed link field is dropped, along with the link module dependency and the remote-player config dependency; media.type.audio is added.

Ordering

This must land together with or immediately after Varbase Podcasts Base #3624729. Neither is safe alone:

  • !5 (merged) alone leaves RightUp contradicting its dependency.
  • This alone removes storages that nothing else provides yet.

Verification status

Done:

  • Every changed YAML file was parsed to confirm it is valid.
  • No tracked file still references field_audio_url.

Not done yet, stated plainly:

  • Not yet verified on a fresh Drupal CMS build with the RightUp site template picked in the installer. That is still open.
  • gitlab-ci-local is not installed on this machine, so the pipeline was not run locally. The remote pipeline is the gate.

Known limitation

Carried over from #3624729 and not solved here: vartheme_bs5 has no audio SDC, so inside a Canvas template local audio renders through the video component and remote audio not at all.

AI-Generated: Yes

Checkpoints:

  • File an issue
  • Addition/Change/Update/Fix
  • Testing to ensure no regression
  • Automated unit testing coverage
  • Automated functional testing coverage
  • UX/UI designer responsibilities
  • Readability
  • Accessibility
  • Performance
  • Security
  • Developer Documentation
  • User Guide Documentation
  • Reviewed by human
  • Code review by maintainers
  • Full testing and approval
  • Credit contributors
  • Review with the product owner
  • Release notes snippet
  • Release
Edited by Rajab Natshah

Merge request reports

Loading
Loading