🇫🇷France @andypost

Routes probably should be deprecated as there's 📌 Support route aliasing (Symfony 5.4) and allow deprecating the route name Needs work

it's merged but needs to fix pipeline to have a green release

As the issue 📌 Use LRU Cache for static entity cache Postponed is postponed it makes sense to file follow-up to clean-up and add TODO there, for #113

and needs rebase/reroll too

Thanks Alex! I did rebase on top of 11.x and reverted all unrelated changes

Removed cache as this kind of checks should be cached statically, also removed useless logging of error

I see no reason to use cache except of static variable because this check should happen only once while PHP is running

btw it could use follow-up to add UI to configure quality and speed for images in terms of 🌱 [Meta] High-performance images (nearly) out of the box Active

ref https://www.php.net/manual/function.imageavif.php

Thank you, applied clean-up!

Thank you, looks ready

it needs summary update about API change at least, or change record

Maybe start with real goal and minimize amount of assets? ✨ Support inlining critical CSS for faster core web vitals Active

Then it will be more clear how "to decouple" into placeholders

Looks the issue could be closed as there's contrib module https://www.drupal.org/project/masquerade_log →

Bump rebuilds using fresh base image https://git.drupalcode.org/project/drupalci_environments/-/jobs/4116513

Core tests using it https://git.drupalcode.org/issue/drupal-3501859/-/pipelines/404220

$ docker run --rm mysql:9
2025-01-23 22:30:59+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 9.2.0-1.el9 started.

Production image pushed using

$ docker run --rm mysql:9
2025-01-23 22:30:59+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 9.2.0-1.el9 started.

Thank you! looks ready

better title

Testing dev-image https://git.drupalcode.org/project/drupalci_environments/-/jobs/4116280

Testing dev image https://git.drupalcode.org/project/drupalci_environments/-/jobs/4116280 via https://git.drupalcode.org/issue/drupal-3501859/-/pipelines/404210

better title

andypost → created an issue.

andypost → created an issue.

the blocker is in

The blocker is in 📌 Support route aliasing (Symfony 5.4) and allow deprecating the route name Needs work

@claudiu.cristea Thank you! Since D8 we just adding new formats like ✨ Let GDToolkit support AVIF image format Needs work

sometimes it's easier to chunk commits by 400 loc but having overall changes as 10k

Deprecation suppose removal of the usage, 240LOC surely needs chunking

git grep REQUIREMENT_ |wc -l
241

@jcnventura it sounds like option for contrib module a-la Schema →

Needs rebase and deprecation for 12.x

It needs summary update, see #3499524-4: Mysql Schema::addField() incorectly processing indexes specification →

Mysql is using specific schema validation but does not support extra settings for for indexes

It should be done as upload progress needs revival but not clear about " the modern approach"

removal is targeted to D12 📌 Remove jQuery Form dependency from misc/ajax.js Needs work

but this issue needs new approach

ini_set('zend.assertions', 1);ini_set('zend.assertions', 1);

This settings can be changed only starting PHP https://www.php.net/manual/en/ini.core.php#ini.zend.assertions

Any reason to use jQuery.form which is deprecated for a long time?

tests pass

I think it's ready for commiters

Then at least doc-block needs rewording to point out that fields are required

without the 'fields' element

from \Drupal\Core\Database\Schema::addField()

   * @param array|null $keys_new
   *   (optional) Keys and indexes specification to be created on the
   *   table along with adding the field. The format is the same as a
   *   table specification but without the 'fields' element. If you are
   *   adding a type 'serial' field, you MUST specify at least one key
   *   or index including it in this array. See ::changeField() for more
   *   explanation why.

It was added in #1314214: MySQL driver does not support full UTF-8 (emojis, asian symbols, mathematical symbols) →

The mysql/Schema::getNormalizedIndexes() require fields to be passed

the commit which breaks https://git.drupalcode.org/project/drupal/-/commit/eea37702137e1dce6036f...

it means mysql require to pass "fields" but other drivers are not (as doc block said)

@daffie please check if it works for mongodb

Fixed pgsql and filed issue for mysql schema 🐛 Mysql Schema::addField() incorectly processing indexes specification Active

andypost → created an issue.

on pgsql able to reproduce, the failing value is

php -r 'var_dump(unserialize(hex2bin("4f3a33313a2253796d666f6e795c436f6d706f6e656e745c526f7574696e675c416c696173223a323a7b733a34343a22")));'
PHP Warning:  unserialize(): Error at offset 44 of 48 bytes in Command line code on line 1
PHP Stack trace:
PHP   1. {main}() Command line code:0
PHP   2. unserialize($data = 'O:31:"Symfony\\Component\\Routing\\Alias":2:{s:44:"') Command line code:1
Command line code:1:
bool(false)

The only failure is pgsql with strange warning

    1) /builds/issue/drupal-3159210/core/lib/Drupal/Core/Routing/RouteProvider.php:267
    unserialize(): Error at offset 44 of 48 bytes

@amateescu Thank you fixing update hook! Probably it needs follow-up and todo added to allow add field with index to mysql table in one transaction

upgrade fails on mysql (every upgrade test) https://git.drupalcode.org/issue/drupal-3159210/-/jobs/4004297

pgsql fails only 1 test RouteProviderTest https://git.drupalcode.org/issue/drupal-3159210/-/jobs/4004507

sqlite pass all

any somehow update hooks fails on mysql

Run updates
       ┐
       ├ The update failed with the following message: "Failed: Drupal\Core\Database\SchemaException: MySQL needs the 'alias' field specification in order to normalize the 'alias' index in Drupal\mysql\Driver\Database\mysql\Schema->getNormalizedIndexes() (line 332 of /builds/issue/drupal-3159210/core/modules/mysql/src/Driver/Database/mysql/Schema.php)."
       │
       │ /builds/issue/drupal-3159210/core/tests/Drupal/Tests/UpdatePathTestTrait.php:68
       │ /builds/issue/drupal-3159210/core/tests/Drupal/FunctionalTests/Update/UpdatePathTestBase.php:196
       │ /builds/issue/drupal-3159210/core/modules/workspaces/tests/src/Functional/Update/WorkspaceAssociationStringIdsUpdatePathTest.php:42

the only question is array vs iterable

kind of it

Measuring queries executed in big-pipe and while rendering also can give wrong results on timing for db-queries

Probably makes sense to patch core so it will react on fiber suspend,
also maybe to add attribute to spans to point out that span generated inside of fiber and time could be inaccurate

needs to add filtering and inject configuration cleaner, also needs tests

Filed follow-up to add extra attributes ✨ Add optianal logging and filtering query arguments Active

andypost → created an issue.

Looks like it's not a blocker anymore, not clear why but that's great!

Congrats!

Would be great to add a help topic(s) https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →

penyaskito → credited andypost → .

Added MR for 10.5.x

added 2 suggestions for PHP 8.4 compatibility

This files are not from core, please file new issue

Looks like it needs follow-up as help is not shown after install

I bet it needs to check CI template as very probably it needs changes but it doable in follow-up if any maintainer can be found

Updated CI and conditionally call block_rebuild() for 11.1+ core (looks like it has no effect)

Few more deprecations in test files and usage of block_rebuild() must be converted to new hooks for 11.x https://www.drupal.org/node/3486534 →

As 10.4/10.5 are compatible the commit needs backport to 10.x

Used to run pipelines for the current 10.4 and 10.5 and both are green!

- https://git.drupalcode.org/issue/drupal-3427903/-/pipelines/378581
- https://git.drupalcode.org/issue/drupal-3427903/-/pipelines/378594

So closing this one and 10.4.0 release post can mention about compatibility like https://www.drupal.org/blog/drupal-11-1-0 →

andypost → changed the visibility of the branch 10.4.x to hidden.

andypost → changed the visibility of the branch 10.5.x to hidden.

It's very helpful for PHPStorm so maybe makes sense to add rector rule)

Looking at CI jobs I see no viable perf regression, so except fixing the last test it looks ready to go (good to get it in in early days on 11.2)

Looks great and ready to go!

Just not sure about return type which I think which be iterable (traversable) instead of array

Ludwig already mentioning that it should be replaced, so asked how to backport route in #3377094-17: Retire Ludwig? →

Module needs new issue to deal with 11.2 upgrade which gonna remove authorize.php route #3491731-35: Remove the ability to update modules and themes via authorize.php →
Guess moving route should not be a blocker

Probably CI images needs update #3468905: Update default system encoding setting for Mysql images → before merging it as all Mysql 8.0+ images complain in logs about default encoding

Probably core needs own issue to discus default encoding additionally to ✨ Make ROW_FORMAT configurable Needs work

• InnoDB tables have a max index length of 767 bytes (in MySQL 5.6 and older) or 3072 bytes (MySQL 5.7+ with innodb_large_prefix enabled).
• UTF-8 MB4 encoding uses up to 4 bytes per character, meaning strings longer than 191 characters can exceed the max index size.

Undrafted MR

Looks it still blocked on 📌 Replace \PDO::FETCH_* constants to indicate fetch mode with an enumeration Active

in 10.3 there was added handling of exceptions → in queries

Docs should be added

Module could provide hybrid context and raise required OT version to 1.1

Kinda

use OpenTelemetry\Context\ContextStorage;
use OpenTelemetry\Context\FiberBoundContextStorage;

class HybridContextStorage extends ContextStorage {
    private $fiberStorage;
    private $defaultStorage;

    public function __construct() {
        $this->fiberStorage = new FiberBoundContextStorage();
        $this->defaultStorage = new ContextStorage();
    }

    public function current(): Context {
        if (\Fiber::getCurrent()) {
            return $this->fiberStorage->current();
        }
        return $this->defaultStorage->current();
    }

    public function attach(Context $context): Scope {
        if (\Fiber::getCurrent()) {
            return $this->fiberStorage->attach($context);
        }
        return $this->defaultStorage->attach($context);
    }
}

Yes, there's server-setup.sh script which can be improved, so only a backend question remains - fpm or what?!

I did it intentionally because Mysql 9.x is Innovation releases (9.0 is outdated) and latest PHP is 8.4 so oldest and newest combinations are covered

Moreover in related we can use better matrix 📌 [CI] Introduce a separate stage for unit tests Active

Curious how we can create image containing both nginx and apache

looks failures are random

meantime mysql 9.1 is out so tested it

So filed 📌 Upgrade Mysql 9.0 CI job to use 9.1 image Active