Following #1033818: Move xmlrpc includes into xmlrpc.module, should the xmlrpc.php menu callback move to just xmlrpc? On one hand, backwards compatibility to Drupal 7, and keeping web services working is important, but on the other hand, index.php/xmlrpc.php does look funny.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cweagans’s picture

+1 for renaming the menu callback to just xmlrpc

catch’s picture

What about changing the callback, but also adding a path alias during the upgrade path?

RobLoach’s picture

Status: Active » Postponed

Could do that too! #1033818: Move xmlrpc includes into xmlrpc.module first though ;-) .

RobLoach’s picture

Status: Postponed » Active

Oh! It's committed! Thanks catch.

greggles’s picture

Project: Drupal core » xmlrpc
Version: 8.0.x-dev »
Component: xml-rpc system » Code
Issue summary: View changes

After #1285726-48: Remove XML-RPC moving this to the contributed module home for xmlrpc issues.

fgm’s picture

Version: » 8.x-1.x-dev

Show of hands : does anyone have opinions on this change ? I would be in favor : compatibility with D7 does not seem to be really relevant here.

fgm’s picture

Status: Active » Needs review
FileSize
1.16 KB

Suggested patch.

fgm’s picture

Updated to add change in xmlrpc_example.

  • fgm committed fd7f9a4 on 8.x-1.x
    Issue #1627496 by fgm: Switch XML-RPC callback from xmlrpc.php to xmlrpc...
fgm’s picture

Status: Needs review » Fixed

Committed in today's HEAD.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.