- First commit to issue fork.
- πΊπΈUnited States jrglasgow Idaho
It looks to me like the thing that is need most here to get this done is telling ableplayer.info.yml that the module is compatible with Drupal 10
Based on these deprecated functions/methods I suggest it be changed to
core_version_requirement: ^9.3 || ^10
drupal_get_path(): ^9.3
render: ^9.3
UiHelperTrait::drupalPostForm: ^9.1 - πΊπΈUnited States jrglasgow Idaho
here is a patch that will do just that
- @jrglasgow opened merge request.
- Status changed to Active
almost 2 years ago 6:16pm 3 February 2023 - πΊπΈUnited States theMusician
Thanks jrglasgow. The patch has been applied to 3.x-dev.
- πΊπΈUnited States theMusician
Also on 3.x-dev are updated templates using apply spaceless and endapply in place of spaceless and endspaceless which are not valid in D10 twig.
file_get_url has also been replaced.
- πΊπΈUnited States jrglasgow Idaho
@theMusician - I don't see the patch having been applies to the 1.x branch in the repository.
Can you please check again, or just merge the pull request.
- πΊπΈUnited States theMusician
It is merged into 3.x-dev. https://git.drupalcode.org/project/ableplayer/-/blob/3.x-dev/ableplayer....
- πΊπΈUnited States jrglasgow Idaho
When you said it was merged into 3.x-dev I took a look at the source code link in the right sidebar
which links to https://git.drupalcode.org/project/ableplayer which shows the default branch "3.x" not "3.x-dev"I suppose the question would be... why is there a 3.x branch along with a 3.x-dev branch? Seems kind of redundant to me, not to mention it leads to a bit of confusion as Drupal.org will automatically create a release called 3.x-dev whenever the 3.x branch is updated, if you opt to have development releases for that branch. Take a look at the Release naming conventions - Release branches and tags - Release branches β
- πΊπΈUnited States theMusician
Thank you for the tip. I'll try to clean that up later this week. I can see how the dev branch would be causing confusion. Certainly not my intent.
Max
- πΊπΈUnited States jrglasgow Idaho
any chance we can get a release where the info file says it is compatible with Drupal 10?
-
jrglasgow β
authored 0f7b1b14 on 3.x
Issue #3296514 by jrglasgow, Project Update Bot: Automated Drupal 10...
-
jrglasgow β
authored 0f7b1b14 on 3.x
- πΊπΈUnited States theMusician
Alpha6 is out now with the D10 info file. Thanks for the support on the project jrglasgow.
- Status changed to Closed: duplicate
11 months ago 7:21pm 15 December 2023 - πΊπΈUnited States theMusician
The 3.x branch is now getting the Drupalbot attention