// The temp directory does vary across multiple simpletest instances.
  $file = 'temporary://drupal_debug.txt';

Could become:

  $file = variable_get('drupal_debug_file', 'temporary://drupal_debug.txt');

This way its alterable for example in the settings file.
I had the problem that a debug file was locked and I had to debug some stuff, but I don't have sysadmin powers. This way I could keep on debugging.

CommentFileSizeAuthor
#9 2087321_debug_filename-9.patch921 byteswouters_f
#2 2087321_debug_filename.patch628 byteswouters_f
#2 2087321_d7_debug_filename.patch628 byteswouters_f
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Version: 7.x-1.x-dev » 8.x-1.x-dev

I agree this could be useful, if someone wants to provide a patch, let's use the shiny CMI stuff available in D8 first :)

wouters_f’s picture

wouters_f CreditAttribution: wouters_f commented
FileSize
2087321_d7_debug_filename.patch628 bytes
2087321_debug_filename.patch628 bytes

Patch for d8 and d7 included.

Status: Needs review » Needs work

The last submitted patch, 2087321_d7_debug_filename.patch, failed testing.

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented

A recent core change broke Devel. #2082443: Fields are now bound to an entity type needs to be committed first.

rvilar’s picture

rvilar
Primary language Catalan
Location Montcada i Reixac, Catalonia
CreditAttribution: rvilar commented
Status: Needs work » Needs review

#2: 2087321_debug_filename.patch queued for re-testing.

rvilar’s picture

rvilar
Primary language Catalan
Location Montcada i Reixac, Catalonia
CreditAttribution: rvilar commented

#2: 2087321_d7_debug_filename.patch queued for re-testing.

parthipanramesh’s picture

parthipanramesh CreditAttribution: parthipanramesh commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Latest patch does work fine.

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Reviewed & tested by the community » Needs work

The patch is using variable_get and this stuff needs to be in config for Drupal 8, see #1775842: [meta] Convert all variables to state and/or config systems

wouters_f’s picture

wouters_f CreditAttribution: wouters_f commented
FileSize
2087321_debug_filename-9.patch921 bytes

The d8 patch provided.

willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Needs work » Closed (outdated)

Closing for lack of activity. Feel free to reopen if the issue still exists