DevelMailLog in the 8.x branch is quite a messy.. it is basically a copy/paste of 7.x version. I think we can improve it by:

- use a more OO approach
- use the adeguate access modifiers for methods
- add some documentations
- migrate tests from WebTestBase to KernelTestBase
- improve test coverage

CommentFileSizeAuthor
#2 improve_develmaillog-2832798-2.patch15.44 KBwillzyx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

willzyx created an issue. See original summary.

willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Active » Needs review
FileSize
improve_develmaillog-2832798-2.patch15.44 KB
lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
CreditAttribution: lussoluca at H2H Creative Productions commented
Status: Needs review » Reviewed & tested by the community

  • willzyx committed e693a59 on 8.x-1.x 
    Issue #2832798 by willzyx: Improve DevelMailLog
willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x

Status: Fixed » Closed (fixed)

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