πŸ‡ΊπŸ‡ΈUnited States @mpdonadio

Philadelphia/PA/USA (UTC-5)
Account created on 26 June 2008, almost 16 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)
πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Fixed the new constructs/creates and added a update hook to force rebuild the container.

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Commit credits.

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Commit credits.

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Let the fails fly!

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Let the fails fly!

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Yeah, I know it is postponed...

These don't have create/constructors yet:

core/modules/toolbar/src/Controller/ToolbarController.php
core/modules/history/src/Plugin/views/filter/HistoryUserTimestamp.php
core/modules/aggregator/src/FeedStorage.php

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Smaller starting point.

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Smaller starting point.

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

mpdonadio β†’ created an issue. See original summary β†’ .

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

Postponing on πŸ“Œ Replace REQUEST_TIME in OO code w/o access to the container Needs review so there is better starting point. Also thinking that this may really need to two issues to wrangle impact

- container access b/c inheritance
- container access b/c DI

πŸ‡ΊπŸ‡ΈUnited States mpdonadio Philadelphia/PA/USA (UTC-5)

OK, took the patch from 2902895-43.patch

- Isolated just the OO code (well, I tried); this includes classes w/ and w/o access to the container as a starting point

At this point will start going through `git diff --name-only 8.9.x` to isolate by path/namespace revert changes to classes that def don't have container access

Production build 0.67.2 2024