Problem/Motivation

Some of the code in Encrypt takes an EncryptionProfile object as parameter, where it should be an EncryptionProfileInterface, to conform to proper OO principles.

Proposed resolution

Fix the parameters. This won't break anything, because EncryptionProfile parameters passed in our implementations implement the EncryptionProfileInterface anyway.

CommentFileSizeAuthor
#2 2677128-2.patch5.27 KBsvendecabooter
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

svendecabooter created an issue. See original summary.

svendecabooter’s picture

svendecabooter
Primary language Dutch
Location Gent
CreditAttribution: svendecabooter for Acquia commented
Status: Active » Needs review
FileSize
2677128-2.patch5.27 KB
rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Status: Needs review » Reviewed & tested by the community

Looks good.

rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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