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
- Deleted the duplicate field storages.
config/field.storage.node.field_audio.ymlandconfig/field.storage.node.field_audio_url.ymlboth duplicate storages the base recipe owns. The first declaredtype: filewhere the base now declarestype: entity_referencewithtarget_type: media, and understrict: falsewhichever imports first wins silently. - Dropped the removed link field from the podcast view displays. Removed the
field.field.node.podcast.field_audio_urldependency and thefield_audio_url: truehidden entry from thelive_feed,search_indexandsearch_resultdisplays.field_audiostays hidden in all three, deliberately. - Added the audio media entities. Seven
audiomedia entities undercontent/media/, one per podcast audio file, with each demo episode'sfield_audiorepointed at its media UUID and thedepends:entry moved fromfiletomedia. - Updated the Canvas
node.podcast.fulloverride (thesimpleConfigUpdateaction inrecipe.yml). The local audio player now walksfield_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 thelinkmodule dependency and the remote-player config dependency;media.type.audiois 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-localis 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