Problem/Motivation
The procedural watchdog_exception() function has been deprecated in drupal 10.1.0 and is removed from drupal:11.0.0.
See: https://www.drupal.org/node/2932520
Proposed resolution
Use Error::logException instead
use \Drupal\Core\Utility\Error;
catch (\Exception $e) {
Error::logException($this->logger, $e);
}
Note that this change requires Drupal 10.1, so it can't be merged into older branches.
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 3422683-4.patch | 2.51 KB | chandreshgiri gauswami |
Issue fork linkit-3422683
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
Comment #2
sarwan_verma commentedHi @solideogloria
created patch please review,Thanks
Comment #4
chandreshgiri gauswami commentedCreated patch with DI to fix coding standard test failure.
Comment #5
solideogloria commentedComment #6
solideogloria commentedComment #9
solideogloria commentedPlease ignore MR !40. MR 41 is the correct one.
Comment #11
mark_fullmerComment #13
mark_fullmerThe merge request didn't apply against the latest changes in 7.x, but I've credited everyone in the commit message. Thanks for the work, folks!
Comment #14
solideogloria commentedCan you merge it into 6.x as well? There's no release for 7.x yet. Or are you planning to just leave the deprecated code in 6.x?
Comment #15
mark_fullmerThe 7.x branch is the target branch for Drupal 11 compatibility; I don't plan to make the 6.x branch Drupal 11 compatible. Therefore, I don't think there's a need to change this deprecated code in the 6.x branch.
I do expect an alpha release of the 7.x branch in the coming weeks, FYI.