Hi. Message in watchdog isn't link (see screenshot). After reload page message becomes the link.

CommentFileSizeAuthor
#5 message_in_watchdog_is_not_link-2624736-5.patch1.62 KBcherebedov.s
#2 message_in_watchdog_is_not_link-2624736-2.patch1.04 KBcherebedov.s
Selection_237.png60.67 KBcherebedov.s
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cherebedov.s created an issue. See original summary.

cherebedov.s’s picture

cherebedov.s CreditAttribution: cherebedov.s commented
Assigned: cherebedov.s » Unassigned
Status: Active » Needs review
FileSize
message_in_watchdog_is_not_link-2624736-2.patch1.04 KB

Hi. I fixed it. Can you check my patch, please? Thanks.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Needs work

thanks for the patch.

i think we could do more than check for more than just the existence of dblog. how do you feel about adding a hook_module_implements_alter() implementation that makes sure to push put the nodejs watchdog implementation last?

cherebedov.s’s picture

cherebedov.s CreditAttribution: cherebedov.s commented

@beejeebus, Hi, thank You for a review. About hook_module_implements_alter, I understood, I can make it anytime soon.

cherebedov.s’s picture

cherebedov.s CreditAttribution: cherebedov.s at DrupalJedi commented
FileSize
message_in_watchdog_is_not_link-2624736-5.patch1.62 KB

Hi. I added implementation hook_module_implements_alter(). Please, check my new patch. Thank's.

cherebedov.s’s picture

cherebedov.s CreditAttribution: cherebedov.s commented
Status: Needs work » Needs review
Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Thanks, on my phone now, will review later.

  • beejeebus committed 0950e44 on 7.x-1.x authored by cherebedov.s 
    Issue #2624736 by cherebedov.s: Fix message in watchdog report page isn'...
Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Fixed

thanks! committed and pushed to 7.x-1.x.

Status: Fixed » Closed (fixed)

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