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
CommentFileSizeAuthor
#11 rate-3073450-11.patch983 byteskrzysztof domański
#5 interdiff-2-5.txt883 byteskrzysztof domański
#5 3073450-5.patch2.7 KBkrzysztof domański
#2 deprecated-3073450-2.patch3.5 KBsergiu stici

Comments

Sergiu Stici created an issue. See original summary.

sergiu stici’s picture

sergiu stici commented
Status: Active » Needs review
StatusFileSize
new deprecated-3073450-2.patch3.5 KB

Here's the patch, please review.

Status: Needs review » Needs work

The last submitted patch, 2: deprecated-3073450-2.patch, failed testing. View results

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Title: Deprecated Code Report » Replace deprecated drupal_set_message() with \Drupal::messenger()

I split this task into two issues.

#3074633: Replace deprecated entity_get_display()

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Status: Needs work » Needs review
StatusFileSize
new 3073450-5.patch2.7 KB
new interdiff-2-5.txt883 bytes
echo15’s picture

echo15 commented
Status: Needs review » Reviewed & tested by the community

Patch #5 Looks good. reviewed and tested.

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Status: Closed (fixed) » Needs work

I realized that last changes may break BC. We can use here MessengerTrait instead of changing the constructor.

+   * @param \Drupal\Core\Messenger\MessengerInterface $messenger
+   *   The messenger.
    */
-  public function __construct(ConfigFactoryInterface $config_factory, Connection $database, Client $http_client, RequestStack $request_stack) {
+  public function __construct(ConfigFactoryInterface $config_factory, Connection $database, Client $http_client, RequestStack $request_stack, MessengerInterface $messenger) {
     $this->config = $config_factory->get('rate.settings');
krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Status: Needs work » Needs review
StatusFileSize
new rate-3073450-11.patch983 bytes

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Status: Needs review » Fixed

Krzysztof Domański credited tomvanetten.

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
Parent issue: » #3091133: Create a drupal 9 port, drupal 9 report

Credited @tomvanetten for working on it in #3091133: Create a drupal 9 port, drupal 9 report.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.