Support wp_mail()

[mikachan]

This issue is very specific to using the Create Block Theme plugin on a WordPress Playground instance, so it could be that there needs to be a fix in that plugin instead/as well!

Create Block Theme allows you to add font files to your WordPress install. When I add a Google font in a Playground instance, I'm seeing the following error: wasm_popen: mode 'w' not supported (cmd: /usr/sbin/sendmail -t -i)!

I think this is because the 'w' mode is not supported by Playground WebAssembly implementation, or perhaps it's a permissions issue. I wondered if there was a possible fix for this or a way around this from the Create Block Theme side.

To reproduce:

1. Install Create Block Theme on a WordPress Playground instance (example)
2. Go to Appearance -> Manage Theme Fonts -> Add Google font button
3. Select a Google font from the dropdown menu
4. Select a font variant from the list of fonts
5. Click the 'Add Google fonts to your theme' button
6. You should see the above error

[adamziel]

How weird is that rename() somehow leads to a sendmail issue?!

What I think is happening is this:

1. class-manage-fonts.php calls rename() and fails because $temp_filename is not a string (as seen at the top of the Fatal Error message)
2. Some error handler in WordPress notices that and tries to send an email to notify the admin
3. Sending the email fails because of the popen() thing

Let's keep this issue open to track the w mode, but it won't fix the problem you're having. For that, you'll need to figure out why $temp_file is not a string here:

https://github.com/WordPress/create-block-theme/blob/0e9b25247975eb2d5e47b65ebfcea077ccf5210f/admin/class-manage-fonts.php#L221-L227

[adamziel]

My guess: Installing Google fonts involves network calls. Playground in the browser doesn't support that at the moment because of CORS and SSL problems – see the related issue. Can the Google fonts feature be disabled for now?

[mikachan]

That makes sense, thanks for explaining! Yes, we could disable adding Google fonts, as there is also the ability to add local fonts which works well with Playground. Maybe we could disable Google fonts if the plugin is on a Playground instance for now.

Thanks for your help. Happy for this to be closed as a duplicate of #85.

[mikachan]

Oh I've just seen your first comment! I'm sure that wasn't there before, I'm going to blame GitHub 😅

Let's keep this issue open to track the w mode, but it won't fix the problem you're having. For that, you'll need to figure out why $temp_file is not a string here:

Ok! We'll take a look at the $temp_file string issue. I've opened an issue here: WordPress/create-block-theme#360

[adamziel]

Thinking about Playground - it would be good to turn that fatal error into a warning somewhere here:

wordpress-playground/packages/php-wasm/compile/build-assets/phpwasm-emscripten-library.js

Line 386 in 388e805

return _W_EXITCODE(0, 2); // 2 is SIGINT

[carolinan]

I mentioned this issue to Adam during WCEU - and came here to open a new issue; good to know it is documented and is being discussed.

[adamziel]

cc @dmsnell – you might be interested this issue as it touches the major PHP/JavaScript integration points