Add token for "Street address lines 1 and 2 combined"

Reworked to apply to latest 8.x-1.x, fixed few coding standard issues.
Renamed CountryNameTokenTest to AddressTokenTest and extended it to contain test case for new token

Good idea!

Needs a reroll for 2.0.x.

We also need to add address_line3 the same way we added address_line2.

I have created a patch for the 2.0.x.

I have adapted this patch for the 2.0.x branch, which includes support for an additional third address line.

Changes Made:
1. Renamed the token from 'address_lines_both' to 'address_lines_combined' to better reflect its functionality, as it now combines up to three address lines.
2. Updated the logic in address.tokens.inc to handle the third address line. The token now correctly concatenates the available address lines, separating them with commas as appropriate.
3. Manually tested the new token with the Meta Tag module. I used the [node:field_address:address_lines_combined] token in the default meta description for content. The token correctly outputs:

• Just the first line when only the first address line is present.
• The first and second lines, separated by a comma, when both are present.
• All three lines, each separated by a comma, when all are present.
• No additional text in the meta description when the address field is empty.

Thanks!

This is now 2.1.0 material, so bumping the version.

ericgsmith → changed the visibility of the branch 3350959-add-token-for to hidden.

Thanks all - we've been using this patch for a couple of months and it works perfectly for us.

Code changes look good and so does the test 👍

Moving patch from #9 to a MR against 2.1.x

Looks like tests and a few jobs are failing but they may not be related to this change as it looks like the pipelines aren't running on a schedule and I can't see the last run - I'd RTBC it but will investigate these failures first.

Test passed locally:

docker compose exec php-cli phpunit -c /data/app/core/phpunit.xml /data/app/modules/contrib/address/tests/src/Kernel/AddressTokenTest.php
PHPUnit 10.5.39 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.3.14
Configuration: /data/app/core/phpunit.xml

...                                                                 3 / 3 (100%)

Time: 00:16.928, Memory: 12.00 MB

OK, but there were issues!
Tests: 3, Assertions: 58, PHPUnit Deprecations: 1.

Needs 📌 [Gitlab] PHPUnit tests fail after --verbose removed, and no scheduled pipelines exist Active to pass in CI as there is a bit of work to get the pipeline green - good enough for RTBC for me :)

Merged, thanks.

Let's get the CI green in the other issue.

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