Problem/Motivation

\Drupal\babel\EventSubscriber\ConfigSubscriber::removeTranslationsForChangedSource accounts for more than 7% of the install time. It checks for $event->getConfig()->isNew() which does not work and always returns false, see \Drupal\Core\Config\Config::save.

As a result, most of the method runs without effect. This case is more complex, less rewarding, and many configurations are uninstalled later during the final stages of Drupal installation.

Proposed resolution

Remaining tasks

User interface changes

Nothing.

API changes

No API changes expected.

Data model changes

No data model changes expected.

Issue fork babel-3568967

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

huzooka created an issue. See original summary.

huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Assigned: Unassigned » huzooka

huzooka opened merge request !73

huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Assigned: huzooka » Unassigned
Status: Active » Needs review
huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Title: Resolve slowness from Babel during site install » Resolve site install slowness caused by Babel
herved’s picture

herved commented
Status: Needs review » Reviewed & tested by the community

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented
Status: Reviewed & tested by the community » Fixed

Thank you!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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