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.
Now that watchdog() is deprecated we should remove all usage, and then in a followup issue, remove the function itself
Comment | File | Size | Author |
---|---|---|---|
#36 | drupal-watchdog-2271251-36.patch | 41 KB | ParisLiakos |
#27 | drupal-watchdog-2271251-27.patch | 41 KB | undertext |
#25 | drupal-watchdog-2271251-25.patch | 40.98 KB | undertext |
#23 | drupal-watchdog-2271251-23.patch | 40.78 KB | ParisLiakos |
#22 | drupal-watchdog-2271251-22.patch | 46.75 KB | JeroenT |
Comments
Comment #1
tim.plunkettHere's FormValidator. I just *really* wanted those damn watchdog hacks gone :)
Comment #2
ParisLiakos CreditAttribution: ParisLiakos commentedthis is probably too big to be in a single issue...maybe we split it to 2/3 parts..but anyway, lets have it green first
Comment #4
ParisLiakos CreditAttribution: ParisLiakos commentedreroll
Comment #6
ParisLiakos CreditAttribution: ParisLiakos commentedlets try again
Comment #8
ParisLiakos CreditAttribution: ParisLiakos commentedone more
Comment #10
ParisLiakos CreditAttribution: ParisLiakos commentedComment #11
ParisLiakos CreditAttribution: ParisLiakos commented#1289536: Switch Watchdog to a PSR-3 logging framework was reverted
Comment #12
ParisLiakos CreditAttribution: ParisLiakos commentedOk, i splitted this to 3 issues.
This one
One for procedural code: #2272467: Remove usages of watchdog() from procedural code
One for forms, controllers and plugins: #2272481: Remove usages of watchdog() from forms, plugins and controllers
Comment #14
ParisLiakos CreditAttribution: ParisLiakos commentedComment #16
ParisLiakos CreditAttribution: ParisLiakos commentedwe cant really inject a logger in the
module_handler
because during install/uninstall container gets rebuilt and module handler ends up to have an outdated instance injectedComment #17
ParisLiakos CreditAttribution: ParisLiakos commentedSome fixes..i missed what MigrateMessage actually did
Comment #18
StevenPatzComment #20
ParisLiakos CreditAttribution: ParisLiakos commentedComment #21
ParisLiakos CreditAttribution: ParisLiakos commentedreroll
Comment #22
JeroenTPatch no longer applied so I created a reroll.
Comment #23
ParisLiakos CreditAttribution: ParisLiakos commentedreroll.
After this patch, no usage remains:
RTBC anyone?
Comment #24
penyaskitoArgh
Comment #25
undertext CreditAttribution: undertext commentedComment #27
undertext CreditAttribution: undertext commentedOh. randomName() is now randomMachineName().One more reroll.
Comment #28
m1r1k CreditAttribution: m1r1k commentedComment #32
penyaskitoI didn't know we could do this :)
Should it be injected?
Should it be injected?
Should it be injected?
Same.
Comment #33
ParisLiakos CreditAttribution: ParisLiakos commented1. See #16
2. Those are being instantiated from all over the place, they are not in the container or controllers/plugins
3. It already uses \Drupal for the state service, so i didnt bother injecting it
4. Its a test so i didnt bother
Comment #34
penyaskitoThen let's RTBC it.
Comment #35
penyaskitoNo, it does not apply again :(
Comment #36
ParisLiakos CreditAttribution: ParisLiakos commentedthanks! i edited the patch manually to make it apply again.
conflicted with #2322889: Various setUp() and tearDown() methods are not protected
Comment #37
alexpottCommitted 9be30ef and pushed to 8.0.x. Thanks!