Propose https prefix, not http for external links

[casiepa]

In ​https://build.trac.wordpress.org/browser/trunk/wp-includes/class-wp-editor.php?marks=1269#L1269
The text says: The URL you entered seems to be an external link. Do you want to add the required ​http:// prefix?

Might be better to propose to prefix '​https://' as default.

[noisysocks]

Sure, good suggestion.

Update the default HTTP prefix to HTTPS as most sites now a days has SSL enabled and it's good for everything.

Trac ticket: https://core.trac.wordpress.org/ticket/50191

Hi @iamazik! 👋

Thank you for your contribution to WordPress! 💖

It looks like this is your first pull request to wordpress-develop. Here are a few things to be aware of that may help you out!

No one monitors this repository for new pull requests. Pull requests must be attached to a Trac ticket to be considered for inclusion in WordPress Core. To attach a pull request to a Trac ticket, please include the ticket's full URL in your pull request description.

Pull requests are never merged on GitHub. The WordPress codebase continues to be managed through the SVN repository that this GitHub repository mirrors. Please feel free to open pull requests to work on any contribution you are making.

More information about how GitHub pull requests can be used to contribute to WordPress can be found in ​this blog post.

Please include automated tests. Including tests in your pull request is one way to help your patch be considered faster. To learn about WordPress' test suites, visit the ​Automated Testing page in the handbook.

If you have not had a chance, please review the ​Contribute with Code page in the ​WordPress Core Handbook.

The ​Developer Hub also documents the various ​coding standards that are followed:

• ​PHP Coding Standards
• ​CSS Coding Standards
• ​HTML Coding Standards
• ​JavaScript Coding Standards
• ​Accessibility Coding Standards
• ​Inline Documentation Standards

Thank you,
The WordPress Project

Patch looks good to me :+1:

[desrosj]

This one seems to be a bit more involved than just simply changing these two strings. Those strings are copied directly from the TinyMCE library and included here for translation purposes. I believe that changing them in class-wp-editor.php without changing them in TinyMCE would result in those strings no longer being translated properly.

TinyMCE is currently on version 6.2, and WordPress includes 4.9.11 (last updated in [49557]). And far as I can tell, the 4.x branch is no longer maintained upstream, and we have not made any changes to the version of the library included in Core.

We could make changes to the bundled version on a case by case basis. But it would make it much more difficult to merge in an update from the maintainers should one become available (though unlikely). While this change does make sense (HTTPS is the ​protocol used on 80%+ of websites now), I'm not sure that it's worth the effort. Especially where WordPress no longer uses this library by default.

@azaozz Do you have any thoughts here?

[azaozz]

Replying to desrosj:

Yes, the strings can be changed easily, but the function that adds the http:// prefix has to be updated too. That seems to be part of the link plugin (if I'm not mistaken, it's been years...).

I'm also not sure it's worth the effort, unless it is a very easy/straightforward change.

[davidbaumwald]

This came up at #WCUS Contributor Day. Given the maintenance issues with fixing this and needing some sort of decision on whether it should even be handled, removing the good-first-bug keyword.