To avoid long lists and clutter both the screen and logs. It will also improve performance as we do not make less Drupal API calls while processing by just adding project-specific info to an array to be used in the aggregated message as an item list.

Comments

lolandese created an issue. See original summary.

lolandese’s picture

To note that with log messages the latest appears on top while with screen messages that is opposite.

lolandese’s picture

Status: Active » Needs review
StatusFileSize
new48.23 KB

Saving some work. (WIP)

lolandese’s picture

Status: Needs review » Needs work

WIP, not finished.

lolandese’s picture

Borrowed code from the patch in #1325736: Create a consistent method to link to watchdog log reports to get the link to the latest log message. To compress the screen messages to the max, we just provide a link to the associated log message for details.

lolandese’s picture

Status: Needs work » Needs review
StatusFileSize
new58.08 KB
lolandese’s picture

Status: Needs review » Needs work

To restore the links to the instructions (private repos and 'sudo git' for user www-data).

Furthermore, to see if we can identify and group the highlighted issues.

lolandese’s picture

Status: Needs work » Needs review
StatusFileSize
new59.65 KB

We prefer to not extend the screen and log messages to the report items itself but limit it to HEAD fix, fetch, and pull.

Links to instructions are restored and even included in the log messages besides the screen messages.

  • lolandese committed a812eca on 8.x-1.x
    Issue #3144338 by lolandese: Aggregate similar log and screen messages...
lolandese’s picture

Status: Needs review » Fixed
lolandese’s picture

Status: Fixed » Needs review
StatusFileSize
new10.72 KB

After many recent changes, the messages were screwed up. The patch fixes that.

  • lolandese committed 535a205 on 8.x-1.x
    Issue #3144338 by lolandese: Aggregate similar log and screen messages...
lolandese’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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