Updated: Comment #N

Problem/Motivation

*.config_translation.yml files provide a base_route_name property that the Config Translation module uses to attach child routes to and to generate links with. The problem with link generation is that route parameter values cannot be provided. Because of this the name cannot be of a route that has parameters, as Config Translation does not have any replacement values for the route parameters.

Proposed resolution

Add a base_route_parameters property and pass the values on to the URL generator.

Remaining tasks

User interface changes

None.

API changes

None. The policy enforces non-BC-breaking API additions only.

See #2154741: [regression][policy, no patch] Route parameters must be able to be provided along with route names for the policy issue about this.

CommentFileSizeAuthor
#24 support-route-parameters-config-translation-24.patch3.34 KBhmdnawaz
#6 drupal_2154743_6.patch3.54 KBXano
#3 drupal_2154743_3.patch3.1 KBXano
#1 drupal_2154743_1.patch1.81 KBXano
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Active » Needs review
FileSize
drupal_2154743_1.patch1.81 KB

Lots of unicorns for the lovely people who wrote the existing PHPUnit tests. Making this patch was extremely easy because of those.

Status: Needs review » Needs work

The last submitted patch, 1: drupal_2154743_1.patch, failed testing.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review
FileSize
drupal_2154743_3.patch3.1 KB

Status: Needs review » Needs work

The last submitted patch, 3: drupal_2154743_3.patch, failed testing.

The last submitted patch, 3: drupal_2154743_3.patch, failed testing.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review
FileSize
drupal_2154743_6.patch3.54 KB
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

The other mappers also support passing on route parameters. The names mapper is supposed to be used for single use paths (you can only provide a static set of names), so I'm wondering what is the use case for this?

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Issue tags: +D8MI, +language-config
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Needs work 
[3:09pm] GaborHojtsy: Xano: need more info on the use case
[3:09pm] Xano: GaborHojtsy: Did you happen to see the generic policy issue?
[3:09pm] Xano: GaborHojtsy: Because this problem is not specific to config_translation and having to repeat the use case in every sub-issue is quite a bit of work
[3:10pm] Xano: GaborHojtsy: https://drupal.org/node/2154741
[3:10pm] Xano: That one
[3:10pm] Druplicon: https://drupal.org/node/2154741 => [regression][policy, no patch] Route parameters must be able to be provided along with route names #2154741: [regression][policy, no patch] Route parameters must be able to be provided along with route names => 3 comments, 1 IRC mention
[3:11pm] GaborHojtsy: Xano: you have an info.yml configure use case there
[3:11pm] GaborHojtsy: Xano: not a config translation use case(?)
[3:12pm] Xano: GaborHojtsy: Not to sound like a jerk, but does my particular use case matter? I was hoping the policy issue would describe the fact that this is a generic problem clearly enough
[3:13pm] GaborHojtsy: Xano: it does matter, because your fix looks pretty inadequate based on what your use case may be and/or you may be misunderstanding how config translation works
[3:13pm] Xano: GaborHojtsy: The reason I created the policy issue was to prevent having to explain the use case inj eveyr child issue that fixes one occurence of the problem
[3:13pm] GaborHojtsy: Xano: based on https://drupal.org/node/2147689 you have entities, and config translation has a mapper for them, which does have route params
[3:13pm] Druplicon: https://drupal.org/node/2147689 => [regression] Support route parameters for *info.yml files' "configure" route name #2147689: [regression] Support route parameters for *info.yml files' "configure" route name => 18 comments, 1 IRC mention
[3:14pm] Xano: GaborHojtsy: Well, my base route contains parameters. config_translation doesn't fill them in, and therefore cannot create working links to that route
[3:14pm] Xano: GaborHojtsy: This is not about config entities
[3:14pm] GaborHojtsy: Xano: ok, then the tab / contextual links, etc. should also consider to only provide the tab / contextual link if the proper route params are present
[3:14pm] Xano: GaborHojtsy: What do you mean exactly?
[3:15pm] GaborHojtsy: Xano: your issue is pre in D7 you could says foo/bar/baz and it would work even if this would be a foo/%zap/%zoo link
[3:16pm] GaborHojtsy: Xano: now if you specify the mapper to map to a specific route param only, we cannot attach the tab and show it on the route *regardless* of the route params, because you wanted it to only map to specific route params
[3:16pm] GaborHojtsy: Xano: so use this data in the access checking for tabs / contextual links and so
[3:16pm] GaborHojtsy: on
[3:16pm] GaborHojtsy: Xano: the names mapper is designe dot map to static routes (without route params) because the names are static also
[3:16pm] Xano: GaborHojtsy: *I* only need the translate stuff on the route with a specific value for the parameter
[3:17pm] Xano: GaborHojtsy: Because any other value and the config won't be there anymore
[3:17pm] Xano: So there is no need for a translate tab either
[3:17pm] GaborHojtsy: Xano: the idea is its *highly* unlikely that you would want to attach translate tabs to a dynamic route with infinite route param possibilities but still have the same config names for all of them
[3:17pm] GaborHojtsy: Xano: right, so the access checkers need to be updated too which is what I'm saying 
[3:18pm] GaborHojtsy: Xano: not happening in your patch yet
[3:18pm] GaborHojtsy: Xano: it would put on the tabs / contextual links regardless of the route params
[3:22pm] Xano: GaborHojtsy: Just ConfigTranslationFormAccess?
[3:23pm] Xano: GaborHojtsy: ConfigTranslationOverviewAccess as well, probably
[3:23pm] GaborHojtsy: Xano: anything that deals with access to the routes / tabs / contextual links generated
Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

The problem here is that the names mapper makes assumptions about the setup of the route the config is on, while it absolutely should not do that at all.

As discussed on IRC, \Drupal\config_translation\Access\ConfigNameCheck and \Drupal\config_translation\Access\ConfigTranslationOverviewAccess will need to be updated as well.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review

6: drupal_2154743_6.patch queued for re-testing.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Needs work

Still needs work for #10.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

I'd love to have some extra input on this, as I'm a bit lost in how config translation does access control

juanramonperez’s picture

juanramonperez CreditAttribution: juanramonperez commented

Any status of this?

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.6 was released on February 1, 2017 and is the final full bugfix release for the Drupal 8.2.x series. Drupal 8.2.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.3.0 on April 5, 2017. (Drupal 8.3.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.3.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.6 was released on August 2, 2017 and is the final full bugfix release for the Drupal 8.3.x series. Drupal 8.3.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.4.0 on October 4, 2017. (Drupal 8.4.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.4.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.4 was released on January 3, 2018 and is the final full bugfix release for the Drupal 8.4.x series. Drupal 8.4.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.5.0 on March 7, 2018. (Drupal 8.5.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.5.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.6 was released on August 1, 2018 and is the final bugfix release for the Drupal 8.5.x series. Drupal 8.5.x will not receive any further development aside from security fixes. Sites should prepare to update to 8.6.0 on September 5, 2018. (Drupal 8.6.0-rc1 is available for testing.)

Bug reports should be targeted against the 8.6.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Manuel Garcia’s picture

Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer and at Appnovation for Pfizer, Inc. commented
Related issues: +#2780293: Add GUI to configure the site's logo alt attribute

Arriving here after a long dark road... trying to get the update path for #2780293: Add GUI to configure the site's logo alt attribute right. In there we're trying to account for theme settings with translate configuration overrides to update, however I'm finding out that core theme settings are actually not translatable as far as I can see.

Getting this issue fixed would make that possible since that's what theme settings pages are, a path with a parameter.

Version: 8.6.x-dev » 8.8.x-dev

Drupal 8.6.x will not receive any further development aside from security fixes. Bug reports should be targeted against the 8.8.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.9.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.7 was released on June 3, 2020 and is the final full bugfix release for the Drupal 8.8.x series. Drupal 8.8.x will not receive any further development aside from security fixes. Sites should prepare to update to Drupal 8.9.0 or Drupal 9.0.0 for ongoing support.

Bug reports should be targeted against the 8.9.x-dev branch from now on, and new development or disruptive changes should be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

hmdnawaz’s picture

hmdnawaz CreditAttribution: hmdnawaz commented
FileSize
support-route-parameters-config-translation-24.patch3.34 KB

Rerolled patch # 10 for 9.1.x

mossy2100’s picture

mossy2100 CreditAttribution: mossy2100 commented

I've just been working on this and after some excruciating digging through deep Drupal code, found my way to this issue.
Just wanted to report the patch by hmdnawaz (#24) works but the one by Xeno (#6) doesn't, despite the green "PASSED" text. I thought that was a bit misleading. Drupal version 9.2.10.

Version: 8.9.x-dev » 9.2.x-dev

Drupal 8 is end-of-life as of November 17, 2021. There will not be further changes made to Drupal 8. Bugfixes are now made to the 9.3.x and higher branches only. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.15 was released on June 1st, 2022 and is the final full bugfix release for the Drupal 9.3.x series. Drupal 9.3.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.4.x-dev branch from now on, and new development or disruptive changes should be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.9 was released on December 7, 2022 and is the final full bugfix release for the Drupal 9.4.x series. Drupal 9.4.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.5.x-dev branch from now on, and new development or disruptive changes should be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.5.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.