Problem/Motivation

Upon installing installing the module via /admin/modules/, you immediately get the following error...

Deprecated function: Creation of dynamic property Drupal\aws_cloudwatchlogs\Services\PutLogEvents::$describelogGroupHandler is deprecated in Drupal\aws_cloudwatchlogs\Services\PutLogEvents->__construct() (line 112 of modules/contrib/aws_cloudwatchlogs/src/Services/PutLogEvents.php).
Drupal\aws_cloudwatchlogs\Services\PutLogEvents->__construct(Object, Object, Object, Object, Object, Object, Object, Object) (Line: 261)
Drupal\Component\DependencyInjection\Container->createService(Array, 'aws_cloudwatchlogs.put_log_events') (Line: 179)
Drupal\Component\DependencyInjection\Container->get('aws_cloudwatchlogs.put_log_events', 1) (Line: 438)
Drupal\Component\DependencyInjection\Container->resolveServicesAndParameters(Array) (Line: 239)
Drupal\Component\DependencyInjection\Container->createService(Array, 'aws_cloudwatchlogs.log') (Line: 179)
Drupal\Component\DependencyInjection\Container->get('aws_cloudwatchlogs.log', 1) (Line: 438)
Drupal\Component\DependencyInjection\Container->resolveServicesAndParameters(Array) (Line: 275)
Drupal\Component\DependencyInjection\Container->createService(Array, 'logger.factory') (Line: 179)

Steps to reproduce

Just installing it, I do have the Sodium module also installed which I don't believe is important.

Seeing the error running both PHP 8.2 and PHP 8.3

CommentFileSizeAuthor
#6 6.patch2.65 KBjeremy.oleary
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jeremy.oleary created an issue. See original summary.

jeremy.oleary’s picture

Issue summary: View changes
sarwan_verma’s picture

Assigned: Unassigned » sarwan_verma

sarwan_verma’s picture

Assigned: sarwan_verma » Unassigned
Status: Active » Needs review

Hi,

I have resolved the deprecated function issue and created MR!6.
Could you please review it?

Thanks!

jeremy.oleary’s picture

StatusFileSize
new2.65 KB

Attaching the patch you generated as this is our standard workflow.

greg boggs’s picture

Status: Needs review » Reviewed & tested by the community

The patch works perfectly for us.

ciprian.stavovei’s picture

Status: Reviewed & tested by the community » Fixed

Nice! Thanks for the fix.
Merged and will go out with the next release.

Status: Fixed » Closed (fixed)

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