Skip to content

Make sure repository uses Opencat PHP library#39

Merged
bluetom merged 1 commit intomainfrom
issue-38
May 9, 2025
Merged

Make sure repository uses Opencat PHP library#39
bluetom merged 1 commit intomainfrom
issue-38

Conversation

@ferishili
Copy link
Copy Markdown
Contributor

This PR fixes #38,

Description

After reviewing the usage of the old cURL API call in the tool_opencast plugin, I identified only one instance where the get method is being used.

Solution

This PR contains using the opencastapi property within the api class of the tool_opencast plugin while ensuring that the required behavior remains unchanged.

Testing Instructions

  1. Apply this PR to your setup.
  2. Ensure that the plugin is correctly configured.
  3. Use the plugin alongside the filter_opencast plugin to access the video selection interface from the Opencast repository.
  4. Verify that the displayed list of videos is correct and includes all videos from the selected series.

@ferishili ferishili added the refactoring Something which needs to be refactored in the plugin (which may mean breaking changes) label Mar 1, 2025
@ferishili ferishili requested a review from bluetom March 1, 2025 17:28
@ferishili ferishili self-assigned this Mar 1, 2025
@bluetom bluetom merged commit 0f25c5e into main May 9, 2025
@bluetom bluetom deleted the issue-38 branch May 9, 2025 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Something which needs to be refactored in the plugin (which may mean breaking changes)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactoring: Verify that all Opencast API calls use the Opencast PHP Library

2 participants