Skip to content

use previous version of symfony cache (WP-996)#612

Merged
vsolovei-smartling merged 1 commit intomasterfrom
WP-996-composer-install
Apr 17, 2026
Merged

use previous version of symfony cache (WP-996)#612
vsolovei-smartling merged 1 commit intomasterfrom
WP-996-composer-install

Conversation

@vsolovei-smartling
Copy link
Copy Markdown
Contributor

@vsolovei-smartling vsolovei-smartling commented Apr 16, 2026

I've ran into a similar issue that is likely related to symfony/symfony#46477. Two fixes: fix-double-namespace now includes an attempt to copy the unprocessed file to their namespaced location. The real fix is downgrading the symfony cache package to the previous major version to avoid an unprocessable file in docker environments.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Comment thread fix-double-namespace.php
$filesCopied = 0;
$filesMissing = 0;

foreach (new RecursiveIteratorIterator(
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is iterator that iterates over iterators? looks like 'pimp my ride' :)

Comment thread fix-double-namespace.php
Comment on lines +46 to +51
// Copy classmap files that are present in inc/third-party but absent from inc/lib.
// The namespacer skips namespace-less files (e.g. ValueWrapper.php), leaving them missing
// in inc/lib even though the generated autoloader still expects them.
$libDir = __DIR__ . '/inc/lib';
$thirdPartyDir = __DIR__ . '/inc/third-party';
$prefix = 'smartling-connector-';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this file work? is it some kind of "migration"? does it run only when user updates the plugin?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this runs before release

"php fix-double-namespace.php"

@vsolovei-smartling vsolovei-smartling merged commit ee9729b into master Apr 17, 2026
3 checks passed
@vsolovei-smartling vsolovei-smartling deleted the WP-996-composer-install branch April 17, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants