-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove XML-RPC and move to contrib module #67
Comments
Relevant Drupal.org issues: https://drupal.org/node/1285726 - Remove XML-RPC entirely (postponed) And here's the current stand-alone D8 version of the xmlrpc.module: |
Since this doesn't fall in our 90% use-case, my vote is to remove it entirely. It should live in contrib, not in core. |
I agree to maintain the xmlrpc module as contrib as soon as the most pending technical decisions are taken. |
Thanks! That's an excellent offer @mkalkbrenner! Now all we need is a place to centralize "contrib" ;) There are a number of issues for getting backdropcms.org up and going: https://github.com/backdrop/backdropcms.org-issues/issues One thing we're pretty confident about is that contrib module repositories will be hosted on Github. So when it comes time, step one is just to make a repository on Github. Longer-term, these repositories will be registered with BackdropCMS.org to provide a central place for browsing modules/themes. I agree it's probably best to wait a little longer while the largest technical changes are being made. We're making fast progress already, but we have a lot of work to do. |
Merged in backdrop/backdrop#72. |
In Drupal 8, the XML-RPC system was moved from a stand-alone PHP file (core/xmlrpc.php) to a module (xmlrpc.module). I think that's a logical move, separating out functionality that is rarely used. However, the fact that it's rarely used makes it a candidate for removal. I can go both ways on this one, there's some preliminary conversation at this issue: #60 that was brought up because we were fixing a bug in XML-RPC.
The text was updated successfully, but these errors were encountered: