https://localize.drupal.org/translate/languages/tr/translate?sid=497193 reads Handler @handler tried to add additional_field @identifier but @table could not be added! which has additional_field in it (not a placeholder), but it really should be additional field in my opinion.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Kartagis created an issue. See original summary.

Balu Ertl’s picture

Title: Underscore in the English string. » Underscore in the English string
Project: localize.drupal.org » Views (for Drupal 7)
Version: 7.x-1.x-dev » 7.x-3.x-dev
Component: Existing language complaint » User interface
Assigned: Unassigned » Balu Ertl
Status: Active » Needs review
FileSize
704 bytes

Based on our HU translation I also share your point that this supposed to be two separate words by regular grammar. However, this issue is not the problem of localize.drupal.org itself. Instead, we need to figure out where this ticket belongs with.

If you check the usage of this string (the little "Show related projects" dropdown link below), you'll see it's used in many other projects as well. The Drupal Core itself is not involved anymore, its 8.5.15 version (released on 17 April 2019) was the last one using this string.
Beside core, regarding contrib projects, I checked through all other projects with any "8.x" version released:

  • Zircon Profile (10 websites)
  • Sportsleague (16 websites)
  • Hunter Shop (There is no usage information for any release of this project)
  • Niobi Research Center (There is no usage information for any release of this project)
  • PanKM (7 websites)
  • deGov (0 websites)
  • Quick Start (0 websites)
  • NewsPublish (There is no usage information for any release of this project)
  • Droopler (0 websites)

Interestingly I found that these all are distributions. So I started to think about like "Which module can be so popular for including it in dozens of distributions?" The Views, of course. After cloning its repository locally I quickly grepped through the codebase and located this string in /handlers/views_handler_field.inc file at line #162.

Patch attached for review, waiting for RTBC.

nevergone’s picture

Status: Needs review » Reviewed & tested by the community

I'm tested and works well!

Kartagis’s picture

You're right Balu, why didn't I think about this before?

DamienMcKenna’s picture

DamienMcKenna’s picture

Status: Reviewed & tested by the community » Fixed

I understand why it was left that way, it was a reference to the underlying variable, but even still in many cases it's used with a space. So yes, thank you for the fix.

DamienMcKenna’s picture

Assigned: Balu Ertl » Unassigned

Status: Fixed » Closed (fixed)

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