Improve error messages to be clearer about the path causing the problem

Created on 6 December 2023, 7 months ago

Problem/Motivation

If drush tome:static gets an error on a page, the errors are collected up and output. But the output is pretty hard to understand:

 [ERROR] Error when running "'/var/www/html/vendor/bin/drush' 'tome:static-export-path'
         '/terminate,/authenticated,/mlr/get,/qualtrics-test-form,/layout_builder_iframe_modal/redirect'
         '--return-json' '--process-count=5' '--uri=https://bi-com-giotrif-static-site.ddev.site'":

You need to be fairly familiar with the internals of Tome to understand what's going on here -- that the processing is done in batches, that tome:static-export-path is a subcommand, and so on.

Also, because of the way --process-count is used, you don't know which path caused the problem -- there are 5 possibilities with the default setting. So you then need to run it all over again with '--process-count=1' to get a finer-grained error output.

This really needs to say something like 'Got this error when exporting path /PATH'.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

✨ Feature request
Status

Active

Version

1.0

Component

Code

Created by

πŸ‡¬πŸ‡§United Kingdom joachim

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.69.0 2024