Problem/Motivation

The pdfa11y_report Views configuration (config/optional/views.view.pdfa11y_report.yml) was added in 1.0.5. Drupal only imports config/optional files during a fresh module install, not during a module update. Sites upgrading from 1.0.4 or earlier will not have the report view and will get a 404 at /admin/reports/pdf-accessibility/media and /admin/reports/pdf-accessibility/files.

Proposed resolution

Add an update hook (pdfa11y_update_10004()) that imports views.view.pdfa11y_report from the module's config/optional directory if it does not already exist in active configuration.

Remaining tasks

  • Add pdfa11y_update_10004() to pdfa11y.install
  • Test on a simulated upgrade path (install 1.0.4, run update, confirm view is present)
  • Review
  • Merge

User interface changes

The report view will appear at /admin/reports/pdf-accessibility/media and /admin/reports/pdf-accessibility/files after running drush updb.

API changes

None.

Data model changes

None.

Issue fork pdfa11y-3588852

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

joshuami created an issue. See original summary.

joshuami opened merge request !15

  • joshuami committed eefa85c2 on 1.0.x 
    Issue #3588852 by joshuami: Import pdfa11y_report Views config on...
joshuami’s picture

joshuami
he/him
Primary language English
Location Portland, OR
commented
Status: Active » 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.