Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
drupal-check results on commit hash:
source : [git] https://git.drupal.org/project/flexslider cfe2d30ce9477fea0ec21cfdeda92b17deeddb56
source : http://cgit.drupalcode.org/flexslider
------ ---------------------------------------------------
Line flexslider.module
------ ---------------------------------------------------
83 Call to deprecated function libraries_get_path().
------ ---------------------------------------------------
------ --------------------------------------------------
Line src/Controller/FlexsliderOptionsetController.php
------ --------------------------------------------------
25 Call to deprecated method url() of class
Drupal\Core\Config\Entity\ConfigEntityBase.
39 Call to deprecated method url() of class
Drupal\Core\Config\Entity\ConfigEntityBase.
------ --------------------------------------------------
------ ---------------------------------------------------
Line src/Form/FlexsliderDeleteForm.php
------ ---------------------------------------------------
41 Call to deprecated function drupal_set_message().
------ ---------------------------------------------------
------ ---------------------------------------------------
Line src/Form/FlexsliderForm.php
------ ---------------------------------------------------
421 Call to deprecated function drupal_set_message().
427 Call to deprecated function drupal_set_message().
485 Call to deprecated function libraries_detect().
487 Call to deprecated function drupal_set_message().
------ ---------------------------------------------------
------ -------------------------------------------------------------------
Line src/Tests/Update/FlexsliderSettingsUpdateTest.php
------ -------------------------------------------------------------------
12 Class Drupallexslider\Tests\Update\FlexsliderSettingsUpdateTest
extends deprecated class
Drupal\system\Tests\Update\UpdatePathTestBase.
------ -------------------------------------------------------------------
[ERROR] Found 9 errors
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#12 | interdiff_10-11.txt | 17.35 KB | Sudeepthi Peteti |
#11 | flexslider-3042778-11.patch | 8.83 KB | Sudeepthi Peteti |
#10 | flexslider-3042778-10.patch | 7.52 KB | Tolyan4ik |
#8 | interdiff-5-8.txt | 3.05 KB | andralex |
#8 | flexslider-3042778-8.patch | 9.4 KB | andralex |
Comments
Comment #2
daiwik.addweb CreditAttribution: daiwik.addweb at AddWeb Solution Pvt. Ltd. commented@mcdwayne, Kindly review the attached patch on the same hope this will helps you.
Thanks!
Comment #3
daiwik.addweb CreditAttribution: daiwik.addweb at AddWeb Solution Pvt. Ltd. commentedComment #5
Tolyan4ik CreditAttribution: Tolyan4ik at EPAM Systems commentedComment #6
jenlamptonI believe that if you add drupal 9 as an option to the
composer.json
file in this project you can get a nice little Compatible with Drupal 9 badge in the Project information section on the module page.Below is a code sample from a module that has the badge.
It also looks like it may possible to get the badge by adding the 'core_version_requirement' key in the modules info.yml file, which, in turn, will add the version to the require section of
composer.json
. Example follows.Do you want to include the change in this Patch or open a separate issue?
Comment #7
Tolyan4ik CreditAttribution: Tolyan4ik at EPAM Systems commentedComment #8
andralex CreditAttribution: andralex at EPAM Systems for EPAM Systems commentedFigured out that the last commit is now changed to 3daf2116a523a7a6816bb0f22ce495dae0e4fb61 and there are some extra deprecations exists now.
Here is a new patch with fixes of extra deprecations. Skipped libraries_detect(), libraries_get_path() and tests. Going to move them into separate issues to not overweight here.
Comment #9
Steven Brown CreditAttribution: Steven Brown as a volunteer commentedRan drupal-check against
3daf211
. Here are some new errors that are not in the list above.Comment #10
Tolyan4ik CreditAttribution: Tolyan4ik at EPAM Systems commentedComment #11
Sudeepthi Peteti CreditAttribution: Sudeepthi Peteti as a volunteer and at TA Digital commentedHi,
After applying patch#10, there is still some deprecated code. Changed them.
Also added core_version_requirement as well in info.yml file.
Comment #12
Sudeepthi Peteti CreditAttribution: Sudeepthi Peteti as a volunteer and at TA Digital commentedComment #13
amaria CreditAttribution: amaria as a volunteer commentedI will review these patches and change accordingly since some of the changes are taken care of in other tickets.
Comment #15
amaria CreditAttribution: amaria as a volunteer commentedThis has been committed. I made a few minor changes to fix the messenger calls and removed the library api and test code changes that was done in other tickets. Thanks to all who participated. Note that it will still complain about a couple of library api calls that I left in for backward compatibility until 8.9 becomes the minimum supported version of Drupal.