Include mu-plugins in Studio site import - #852
Conversation
TODO: - support mu-plugins in other importers
2b87a69 to
21f86e3
Compare
| /** | ||
| * If the file does not exist, skip it. | ||
| * This can happen if a empty directory is included in the backup | ||
| * because the empty directory won't be included in the extraction. | ||
| */ |
There was a problem hiding this comment.
I discovered this issue in a Local export.
The uploads/2025/01 folder was empty on the site and the 01 folder is missing from the extracted ZIP.
This caused lstat to fail because the directory didn't exist.
| uploads: string[]; | ||
| plugins: string[]; | ||
| themes: string[]; | ||
| muPlugins?: string[]; |
There was a problem hiding this comment.
Should I make muPlugins mandatory?
There was a problem hiding this comment.
Let's keep it optional, as none of the supported backup formats includes it by default.
|
@bgrgicak the code change looks good and works as expected for the packages you provided. However, it doesn't work for my Playground import file, which works fine in Studio 1.3.2. Also, we should include |
It seems it happens in the trunk, too. I've added https://github.com/Automattic/dotcom-forge/issues/10369 |
| uploads: string[]; | ||
| plugins: string[]; | ||
| themes: string[]; | ||
| muPlugins?: string[]; |
There was a problem hiding this comment.
Let's keep it optional, as none of the supported backup formats includes it by default.
I've opened https://github.com/Automattic/dotcom-forge/issues/10373 for that. |
Related issues
Proposed Changes
mu-plugins/sqlite-integration-pluginfolder during importTesting Instructions
Jetpack Import
Playground Import
Wpress Import
Local Import
Pre-merge Checklist