Hello guys,
In the last two weeks I've been trying to make the domain/county_country path modules with Drupal 8 language core module works. Our goal is to achieve this particular multi country/language domain system:
https://example.com/US/en/some-user-friendly-url -> United States country, English language
https://example.com/US/es/some-user-friendly-url-in-spain -> United States country, Spanish language
https://example.com/DE/de/some-user-friendly-url-in-german -> Germany country, German language
https://example.com/AT/de/some-user-friendly-url-in-german -> Austria country, German language
For now i developed patch in country module that provides Language Negotiation Plugin who allows to the Routing system to get the correct language prefix. (the second one in the example urls). Now the problem is that my patch doesn't work with Node url-alias :). i will add my patch together with the patch from this issue: https://www.drupal.org/node/2876366 (implemented with my code).
I would be very grateful if you would assist me with the development of this functionality.
Greetings,
Atanas Sotirov
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | url_alias_does_not_work-2888841-8.patch | 4.64 KB | riv16n824 |
| #6 | country_path-multilanguage-path-fix-#2888841-8.x-1.x-dev.patch | 5.19 KB | a.sotirov |
| #5 | 2888841-do-not-worl-with-url-alias-v2.patch | 6.15 KB | a.sotirov |
| #2 | 2888841-doesnt-work-with-url-alias.patch | 49.23 KB | a.sotirov |
Comments
Comment #2
a.sotirov commentedComment #3
itsekhmistro commentedComment #4
itsekhmistro commentedHi,
Important, the patch from https://www.drupal.org/node/2876366 is applied.
Could you update the attached patch to match with the latest 8.x-1.x version code?
Thank you in advance.
Comment #5
a.sotirov commentedHi,
I applied my patch onto last version of 8.x-1.x-dev version of module. The result was the same, so I'm uploading the patch here.
Greetings,
Atanas Sotirov
Comment #6
a.sotirov commentedHi again,
After deep debugging on my custom code I finally find where the bug was. So I tested the code deeply and the changes works fine for my purposes. I'm uploading the FINAL patch so you can test it :)
Greetings,
Atanas Sotirov
Comment #7
a.sotirov commentedComment #8
riv16n824 commentedBased on a.sotirov patch, here is an updated version.
country_path_uninstall()moved to country_path.install file.Fixed php_codesniffer errors.
All thanks to a.sotirov.
Comment #10
itsekhmistro commentedThe patch was applied.
Thanks to a.soritov.