Problem/Motivation

Site Audit fails to run from local environment with error message:

$ ddev composer require --dev 'drupal/site_audit:^4.0@RC' 
$ ddev drush @site1.ddev en site_audit
$ ddev drush @site1.ddev audit --format=html --bootstrap --detail > site1.html

The "report" argument does not exist.

Failed to run drush @site1.ddev audit --format=html --bootstrap --detail: exit status 1

There is no "report" argument passed, so I'm not certain what this error message actually means.

Steps to reproduce

Ddev 1.23.0
Composer 2
Php 8.2
Drupal 10.2 multisite.
Site Audit 4.0.0-rc5.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#8 site_audit-report-argument-3443345-8.patch1.05 KBcraigra

Issue fork site_audit-3443345

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jwilson3 created an issue. See original summary.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Issue summary: View changes
jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Issue summary: View changes
jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Priority: Normal » Critical

I'm bumping this to critical because the bug here renders the module useless.

jrochate’s picture

jrochate commented

Same here, but not multi-site.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

I hit this snag using site_audit:4.0.0-rc5

Site Audit module now has a 4.1.1 version, which may have resolved this issue, but needs additional testing.

I have an internal ticket to pick this up during maintenance cycle, but it may be some time before we can come back around to this.

redbrickone’s picture

redbrickone commented

FWIW I am seeing this report flag error on 4.1.1 after installing the module, submodules and trying to run some audit drush commands as outlined in the readme.

craigra’s picture

craigra commented
Version: 4.0.0-rc5 » 4.1.1
Status: Active » Needs review
StatusFileSize
new site_audit-report-argument-3443345-8.patch1.05 KB

Changed "report" argument to "checklist" for site_audit:audit command interaction hook.

the_g_bomb’s picture

the_g_bomb commented
Status: Needs review » Reviewed & tested by the community

I have applied this patch as a part of a patch cocktail to get site_audit working locally.

I can confirm this patch resolves the issue we were experiencing.

### Environment:
- Drupal 10.4.8
- Site Audit: 4.0.x-dev

### Error Before Patch:

The "report" argument does not exist.

### After Applying Patch:
The error is resolved. The patch correctly updates the argument name from 'report' to 'checklist' to match the current command signature. Site_audit drush commands now execute successfully.

froboy made their first commit to this issue’s fork.

  • froboy committed 99b2ef1b on 4.x 
    [#3443345] Changed "report" argument to "checklist" for site_audit:audit...

froboy opened merge request !22

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented

Thanks folks. Patch applied and merged to 4.x. Release pending.

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented
Version: 4.1.1 » 4.1.2
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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