Problem/Motivation
This is a child issue of #2144861: [meta] Replace Drupal in UI text with the name of the distribution for the following modules/components:
config_translation, content_translation, locale, language, and translation server parts of installer.
Drupal has a deal with distributions, that they can replace Drupal with their own Distribution name. Also we have a UI text standard
https://www.drupal.org/docs/develop/user-interface-standards/interface-t...
that says:
Use "Site"—not "Drupal". Referring to Drupal by name complicates distributions, and users may not know the site is running on Drupal.
You can also use terms like "the core software".
But Drupal still appears in the UI.
Proposed resolution
Replace Drupal in UI text with the distribution name, or phrases like "the site" and "the core software".
Remaining tasks
- Novice task -- Locate the word "Drupal" appearing in UI text in this group of modules/themes. Here's a grep command that yields some false positives, but should find most/all of what we're looking for:
egrep -i -e "'[^']*[^/\]drupal[^/\]" -e '"[^"]*[^/\]drupal[^/\]' `find . \( -type d -name tests \) -prune -o -type f -print` | grep -i --color drupal
- Novice task -- Replace each occurence either with "the core software", "the site", or the distribution name. There is a function that will return the distribution name: drupal_install_profile_distribution_name().
- Novice task -- Make and upload a patch file, or use an issue fork / merge request workflow.
User interface changes
We will not use the word Drupal in the UI, but instead either show the distribution name or the phrases "the site" or "the core software".
API changes
None.
Data model changes
None.
Release notes snippet
Not necessary.
Comment | File | Size | Author |
---|---|---|---|
#23 | replaced-drupal-site-2143615-23.patch | 7.45 KB | Anjali Rathod |
#6 | distro_name_translation_website-2143615-6-do-not-test.patch | 5.53 KB | YesCT |
Comments
Comment #1
Gábor HojtsyI don't think that is a problem. The translation website of the distro in that case is hte same as the Drupal translation website.
Comment #2
YesCT CreditAttribution: YesCT commented#1861930-53: Use "Drupal translations website" instead of Drupal translation server or Community translation server proposes to take out the Drupal to @Drupal change.
Here we can replace (or remove) all the "Drupal"s in locale. there are more than just in the phrase Drupal translation website.
Also, lets leave the very first screen in the language installer for #1386394: Add back the ability for install profiles (or at least distros) to pre-select a language or modify the language-selection screen,
or
just call it translation website and leave off the Drupal/Distribution name.
===
patch coming to move the code over from the other server -> website issue.
Comment #3
YesCT CreditAttribution: YesCT commentedComment #4
YesCT CreditAttribution: YesCT commentedComment #5
YesCT CreditAttribution: YesCT commentedin server/website issue I was wondering why there was a new function, maybe so we dont have to:
Comment #6
YesCT CreditAttribution: YesCT commentedpatch is do-not-test because it wont apply until #1861930: Use "Drupal translations website" instead of Drupal translation server or Community translation server is committed.
patch has just those Drupal @Drupal replacements I took out of #1861930-41: Use "Drupal translations website" instead of Drupal translation server or Community translation server which was by @mr.york
but attempts to just reuse the install function: drupal_install_profile_distribution_name()
does not put a new function in common.inc yet (I'm still not clear on why we need that.)
Do we want to replace the others in locale? (or reword them to take out "Drupal")
I think the remaining ones are (mostly in the module help):
Comment #15
jhodgdonUpdating issue summary/title as per standards of parent issue.
Comment #16
jhodgdonComment #17
jhodgdonStreamline title
Comment #18
dabley CreditAttribution: dabley commentedFixed missing open < in issue summary.
Comment #19
dabley CreditAttribution: dabley commentedMinor edit to description punctuation.
Comment #20
dabley CreditAttribution: dabley commentedBetter fix for description punctuation.
Comment #21
jhodgdonAdding regular expression for finding 'drupal'.
Comment #22
Anjali RathodComment #23
Anjali RathodComment #24
guilhermevp CreditAttribution: guilhermevp at CI&T commentedLooks like everything is ok based in my experience with similar issue.
Comment #25
catchThe parent issue was opened in 2013 and it feels a bit like things have moved on since then. IMO we need to rethink the basis for the overall change.
- In 2013, if you installed a distribution, Drupal core would change very little while the distribution might change a lot - until you got to a new major Drupal version which could be years away. Drupal core now has minor releases every six months, and the string changes here obfuscate this a bit. We're really talking about Drupal version and that is meaningful information, not 'site version' which is not really a concept.
- If we replaced Drupal with $distribution_name instead of 'site' that might help, but the version of a distribution is not going to match the Drupal core version at all, so it would still be misleading.
- Also I wonder how many distributions actually try to suppress the fact they're built on Drupal?
On top of that, this link goes to localize.drupal.org, so changing the string name doesn't actually help to obfuscate the fact we're running Drupal at all.
Comment #30
xjmDoing cleanups like this on a per-module basis is explicitly prohibited by the core issue scope guidelines. See #2144861-31: [meta] Replace Drupal in UI text with the name of the distribution for more information. Thanks!