There are quite a lot of deprecated functions being used in 8.x-1.x, mainly because it was written during beta.
However I think it would be nice to fix those now, as D8 is stable.

CommentFileSizeAuthor
#4 security_review-deprecated-4.patch25.39 KBbanviktor
#3 security_review-deprecated-3.patch19.22 KBbanviktor
#2 security_review-deprecated.patch5.76 KBstefan.r
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

banviktor created an issue. See original summary.

stefan.r’s picture

stefan.r CreditAttribution: stefan.r commented
Status: Active » Needs review
FileSize
security_review-deprecated.patch5.76 KB
banviktor’s picture

banviktor CreditAttribution: banviktor at CARD.com commented
FileSize
security_review-deprecated-3.patch19.22 KB

Extended the patch.

banviktor’s picture

banviktor CreditAttribution: banviktor at CARD.com commented
FileSize
security_review-deprecated-4.patch25.39 KB

There should be no deprecated functions/classes in this one.

  • banviktor committed 3c13158 on 8.x-1.x 
    Issue #2656372 by banviktor, stefan.r: Avoid usage of deprecated...
banviktor’s picture

banviktor CreditAttribution: banviktor at CARD.com commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

smustgrave credited Nitesh Pawar.

smustgrave’s picture

smustgrave CreditAttribution: smustgrave commented

Closed https://www.drupal.org/project/security_review/issues/2785401 as duplicate of this and moved over credit.