Move youtubechannelvideo() to a service class [#3577017]

Problem/Motivation

To modernize the youtube data access we should move the youtubechannelvideo() function to a OOP service.
This make the class testable in a future task.

Proposed resolution

Create a new YouTubeChannelFetcher class and swap out youtubechannelvideo() in the .module file.

Issue fork youtubechannel-3577017

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3577017-move-youtubechannelvideo-to changes, plain diff MR !9
Check out this branch for the first time

Check out existing branch, if you already have it locally

About issue forks

Comments

Comment #1

4 March 2026 at 14:04

sunlix created an issue. See original summary.

Comment #2

4 March 2026 at 14:08

sunlix opened merge request !9

Comment #3

sunlix

English

Wesel

commented 4 March 2026 at 14:09

Status:

Active

» Needs review

Comment #4

sunlix

English

Wesel

commented 5 March 2026 at 14:21

In the config method I used the constant YoutubechannelSettingsForm::SETTINGS.
This would be introduced in #3576951: Provide default settings and simplify settings form.
So it would be nice to merge both issues successively.

The YouTube channel data will be cached for 1 hour. So multiple blocks would be possible without any ^n requests to the API.

Would be nice to have this merged. :) Any recommendations are highly welcome.