Automated Drupal 11 compatibility fixes for symfony_mailer

Thanks, I made some comments

Thanks for reviewing!!
Done with required changes, let me know if we can merge and do a D11 release.

I think the tests are failing after the changes , will look into this and fix the pipeline.

Fixed the small copy/paste mistakes.

It can be improved by using the deprecation helper but so far it looks good to me to release a D11 compatible version.

Thank you everyone for your work! The changes seem working so it's good to be RTBC by me. The only unresolved thread is regarding the .gitlab-ci.yml variables.

I just want to add some really kind and gentle pressure on this in order to say that websites using this module (32500+ currently) will not be able to upgrade to Drupal 11 without a stable version containing these changes. Drupal Lenient composer plugin won't solve the problem since we still have the dependency issue coming from the "symfony/mailer": "^5.3 || ^6.0" requirement and Lenient only fixes "drupal/core" requirements.

A possible workaround I have found is to change my composer.json's repositories property to the following, while also adding the diff from the MR as patch of course:

"repositories": [
    {
        "type": "package",
        "package": {
            "version": "1.4.1",
            "name": "drupal/symfony_mailer",
            "description": "Symfony Mailer",
            "type": "drupal-module",
            "license": "GPL-2.0-or-later",
            "dist": {
                "url": "https://ftp.drupal.org/files/projects/symfony_mailer-1.4.1.zip",
                "type": "zip"
            },
            "source": {
                "url": "https://git.drupalcode.org/project/symfony_mailer.git",
                "type": "git",
                "reference": "1.4.1"
            },
            "require": {
                "html2text/html2text": "^4.0.1",
                "symfony/mailer": "^5.3 || ^6.0 || ^7.1@beta",
                "tijsverkoyen/css-to-inline-styles": "^2.2"
            },
            "extra": {
                "drush": {
                    "services": {
                        "drush.services.yml": "^11"
                    }
                }
            }
        }
    },
    {
        "type": "composer",
        "url": "https://packages.drupal.org/8"
    }
],

I feel that the key is to limit the range of supported core versions. We should not need to call version_compare! Instead we can remove the old code and increase the minimum core version.

1. Using the Random class makes D11 work but breaks D9.
2. The change from renderPlain() to renderInIsolation() makes D12 work, but breaks D10.2.

So core_version_requirement should be 10.3 || ^11. I'll bump the minor version to 1.5, and anyone still on D10.2 will stay on 1.4. Easy😃.

Done with all the feedback changes, hence moving into review.

Great thanks it looks good just one thread is unresolved still

I was able to install & enable this in Drupal 11. Great work!

Yes thanks but please see #15.

I see a potential problem with the "Broaden test coverage" part. It seems to me that in future the tests are likely to break when code is added to 11.x-dev e.g. new deprecations. Or CORE_PHP_MAX will eventually jump to a new version which we don't necessarily yet support. Also it just slows down every test run.

What's the downside if we remove these lines? Well someone could accidentally write code that uses something that has been removed from 11.x, and we wouldn't notice. But then anyway it was likely deprecated in 10.3? To me this feels much less of a problem than the above.

Hello AdamPS, I have updated MR with the requested changes and also created a separate issue for broaden test coverage. Hence requesting review.

renderPlain is deprecated in D10.2 so I would suggest to keep minimum support of ^10.2 in info.yml and I'm not seeing any other deprecation fixes other then renderPlain.
What are your thoughts?

Great thanks

I believe the minimum support is because renderInIsolation was only added in 10.3