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.
php drupal-check.phar modules/contrib/rate/
35/35 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
------ --------------------------------------------------------------------------------------
Line src/Plugin/VoteResultFunction/CountUp.php
------ --------------------------------------------------------------------------------------
Class Drupal\votingapi\VoteResultFunctionBase not found and could not be autoloaded.
------ --------------------------------------------------------------------------------------
------ ---------------------------------------------------
Line src/RateBotDetector.php
------ ---------------------------------------------------
170 Call to deprecated function drupal_set_message().
------ ---------------------------------------------------
------ ---------------------------------------------------
Line tests/src/Functional/RateWidgetTest.php
------ ---------------------------------------------------
51 Call to deprecated function entity_get_display().
------ ---------------------------------------------------
[ERROR] Found 3 errors
Comment | File | Size | Author |
---|---|---|---|
#11 | rate-3073450-11.patch | 983 bytes | Krzysztof Domański |
| |||
#5 | interdiff-2-5.txt | 883 bytes | Krzysztof Domański |
#5 | 3073450-5.patch | 2.7 KB | Krzysztof Domański |
| |||
#2 | deprecated-3073450-2.patch | 3.5 KB | Sergiu Stici |
Comments
Comment #2
Sergiu Stici CreditAttribution: Sergiu Stici at FFW commentedHere's the patch, please review.
Comment #4
Krzysztof DomańskiI split this task into two issues.
#3074633: Replace deprecated entity_get_display()
Comment #5
Krzysztof DomańskiComment #6
echo15 CreditAttribution: echo15 at EPAM Systems commentedPatch #5 Looks good. reviewed and tested.
Comment #8
Krzysztof DomańskiThanks!
Comment #10
Krzysztof DomańskiI realized that last changes may break BC. We can use here
MessengerTrait
instead of changing the constructor.Comment #11
Krzysztof DomańskiComment #13
Krzysztof DomańskiComment #15
Krzysztof DomańskiCredited @tomvanetten for working on it in #3091133: Create a drupal 9 port, drupal 9 report.