Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Attached is a patch to standardize the json output for the Audit Extensions option. This patch converts the text/markup rendered currently with the --json option for both the summary and the detail (--detail) reports.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2496873-generate-json-for-audit-extension.patch | 5.94 KB | lhridley |
Comments
Comment #1
lhridley CreditAttribution: lhridley as a volunteer commentedComment #2
lhridley CreditAttribution: lhridley as a volunteer commentedComment #3
lhridley CreditAttribution: lhridley as a volunteer commentedComment #4
FluxSauce CreditAttribution: FluxSauce commentedThanks for the patch!
Also, I've done some restructuring and cleanup, the patch isn't applying cleanly anymore. Sorry about that, but some of that was to address things you changed, like removing the color text ("a clean site is a happy site").
You can be much more concise.
Same goes for duplicate, development, and count (which is even simpler,
$ret_val = $options;
)Comment #5
lhridley CreditAttribution: lhridley as a volunteer commentedI'm going to resubmit with much more extensive refactoring shortly, which will move the output preparation out of the "checks" class and into the "reports" class, which will make it easier to extend and modify the output with a plugin.
Comment #6
FluxSauce CreditAttribution: FluxSauce commentedIncremental, not monolithic improvements are preferred at this stage. It's a lot easier to review and test. Let's get this smaller work done cleanly first!