Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
There are following 3x Issues at current AMP Version:
1. Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException: Circular reference detected for service "asset.css.collection_renderer", path: "ajax_response.subscriber
ArgumentCountError: Too few arguments to function Drupal\Core\Asset\CssCollectionRenderer::__construct(), 1 passed in /modules/contrib/amp/src/Asset/AmpCssCollectionRenderer
2. TypeError: Drupal\amp\EventSubscriber\AmpEventSubscriber::onView(): Argument #1 ($event) must be of type Symfony\Component\HttpKernel\Event\GetResponseForControllerResultEvent
3. Error: Call to undefined function drupal_get_path() in amp_theme_registry_alter() (line 327 of docroot/modules/contrib/amp/amp.module).
Provided Patch solves all of these 3x Issues. The Patch should be compatible with current dev version.
Please integrate it into the latest version
Comment | File | Size | Author |
---|---|---|---|
#3 | amp--d10-compatibility--3353999-3.patch | 2.25 KB | flefle |
#2 | amp--d10-compatibility--3353999-1.patch | 4.53 KB | flefle |
Comments
Comment #2
flefle CreditAttribution: flefle as a volunteer commentedPatch solving the 3x Issues.
Comment #3
flefle CreditAttribution: flefle as a volunteer commentedThis patch focuses only at:
Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException: Circular reference detected for service "asset.css.collection_renderer", path: "ajax_response.subscriber
ArgumentCountError: Too few arguments to function Drupal\Core\Asset\CssCollectionRenderer::__construct(), 1 passed in /modules/contrib/amp/src/Asset/AmpCssCollectionRenderer