When l.d.o parsed Drupal 8 RC2, it threw these errors:

15:18:24 Undefined index:                                                        [notice]
15:18:24 drupal-8.0.0-rc2/core/modules/node/src/Controller/NodeController.php::inline-template
15:18:24 l10n_drupal.files.inc:546

15:19:06 Undefined index:                                                        [notice]
15:19:06 /var/www/localize.drupal.org/files-tmp/l10n_drupal_drupalorg_sakAxz_dir/drupal-8.0.0-rc2/core/profiles/testing_config_overrides/config/optional/tour.tour.testing_config_overrides.yml
15:19:06 l10n_drupal.files.inc:546

This happens when we asked back info about a file.

function l10n_drupal_save_file($revision = NULL, $file = NULL) {
  if (!isset($file)) { ...
  }
  elseif (!isset($revision)) {
    // HERE:
    return array($pid, $rid, $files[$file]);
  }

So looks like l10n_drupal_save_file() does not get to know about these files while parsing.

CommentFileSizeAuthor
#3 2601538.patch4.84 KBGábor Hojtsy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy created an issue. See original summary.

Gábor Hojtsy’s picture

Gábor Hojtsy’s picture

Status: Active » Needs review
FileSize
4.84 KB

1. Inline template parsing should just use the file name. No need to make up a special filename for this.
2. Config parsing should use the cleaned-up filename and NOT the full path on the system. That allows us to match with file information pre-recorded.

These I think affected the translatability of config and inline template translatables because the file/line relation could not be recorded.

  • Gábor Hojtsy committed 741f336 on 7.x-3.x
    Issue #2601538 by Gábor Hojtsy: YAML and inline template "files" are...
Gábor Hojtsy’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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