Now we have imploded log messages without any space.
So final log message looks like:
Some message1Some message2Some message3

It would be better to explode them with
for example.
Some message1
Some message2
Some meesage3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kalistos created an issue. See original summary.

kalistos’s picture

kalistos’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: ultimate_cron-separate_log_messages-2919000-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

codebymikey’s picture

Rerolled in order to test with the latest version of the module.

As well as ensure that the line break is as memory-efficient as possible.

codebymikey’s picture

Status: Needs work » Needs review
mortona2k’s picture

Status: Needs review » Reviewed & tested by the community

This patch is working great for me, I have a cron hook that runs multiple migrations and the log messages appear on their own lines.

The code change is very simple, it adds a line break to the beginning of each message after the first.

Berdir’s picture

Status: Reviewed & tested by the community » Fixed

Why not I suppose ;)

  • Berdir committed 04da6f84 on 8.x-2.x authored by kalistos
    Issue #2919000 by kalistos, codebymikey: Seprate log messages with...

Status: Fixed » Closed (fixed)

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