Some static methods in MailgunHandler are marked as deprecated for ~1 year:

  • validateKey()
  • checkApiSettings()
  • checkLibrary()
  • status()

Let's remove them.

CommentFileSizeAuthor
#3 mailgun-remove_deprecated_functions-3102390-3-D8.patch20.99 KBmatroskeen

Comments

Matroskeen created an issue. See original summary.

matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
commented
Assigned: Unassigned » matroskeen
matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
commented
Status: Active » Needs review
StatusFileSize
new mailgun-remove_deprecated_functions-3102390-3-D8.patch20.99 KB

Also, fixed some coding standards issues and slightly refactored the code.

Let's run the tests before commit.

  • Matroskeen committed f5afb10 on 8.x-1.x 
    Issue #3102390 by Matroskeen: Remove internal deprecated functions
matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
commented
Assigned: matroskeen » Unassigned
Status: Needs review » Fixed

Alright, committed to 8.x-1.x.

Status: Fixed » Closed (fixed)

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