Merge Check::evaluate() and Check::evaluatePlain()

Created on 25 November 2015, almost 9 years ago

Updated 16 December 2023, 11 months ago

Per the original issue on GitHub https://github.com/banviktor/security_review/issues/26 :

Reason:
For example take a look at Field check's said functions. It's copy paste for most part, as iterating through the findings can not be done anyway else.

Possible solutions:

Add an argument to evaluate() which could be a string or constants.

Old array based evaluation. The output is keyed by the format (html or plain).

📌 Task

Status

Fixed

Version

3.0

Component

Code

Created by

🇺🇸United States dsnopek USA

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment over 1 year ago →
🇺🇸United States smustgrave
Actually going to close this out. The evaluatePlain is used by Drush for it's output. If we ever convert to plugins we could replace this.
Status changed to Active 11 months ago7:42pm 15 December 2023
Comment 11 months ago →
🇺🇸United States smustgrave
Going to reopen this.
Status changed to Fixed 11 months ago5:05pm 16 December 2023
Comment 11 months ago →
🇺🇸United States smustgrave
Did this in the new 3.0.x branch by doing the suggestion of passing a flag.
Comment 11 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024