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

Created on 25 November 2015, about 9 years ago
Updated 16 December 2023, about 1 year 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.
Sign in to follow issues

Comments & Activities

Not all content is available!

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

Production build 0.71.5 2024