Fix REST on WordPress

[MegaphoneJon]

Overview

Civi 5.13.0 broke the REST endpoint on WordPress.

Before

A call like: http://wpmaster.localhost/wp-content/plugins/civicrm/civicrm/extern/rest.php?entity=system&action=check&key=cTJjAAadwbEB2n5w&api_key=54321

results in a 500 error with stack trace:

#0 /var/www/9to5.org/htdocs/wp-content/plugins/civicrm/civicrm/CRM/Utils/System.php(1474): CRM_Utils_System_WordPress->loadBootStrap(Array, true, false, NULL)
#1 /var/www/9to5.org/htdocs/wp-content/plugins/civicrm/civicrm/CRM/Utils/REST.php(640): CRM_Utils_System::loadBootStrap(Array, true, false)
#2 /var/www/9to5.org/htdocs/wp-content/plugins/civicrm/civicrm/CRM/Utils/REST.php(114): CRM_Utils_REST->loadCMSBootstrap()
#3 /var/www/9to5.org/htdocs/wp-content/plugins/civicrm/civicrm/extern/rest.php(42): CRM_Utils_REST->bootAndRun()
#4 {main}
  thrown in /var/www/9to5.org/htdocs/wp-content/plugins/civicrm/civicrm/CRM/Utils/System/WordPress.php on line 476

After

REST endpoint works as expected.

Technical Details

This commit declared an exception when loading the CMS bootstrap by uid as "not implemented". So I implemented it and removed the exception :)

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[JoeMurray]

@eileenmcnaughton @totten @kcristiano this seems important enough to make an announcement and quick fix release.

[seamuslee001]

from @kcristiano on mattermost "The PR makes sense to me, but I am not in a position to run a test. Looks like I need to add REST Calls to the testing checklist"

[seamuslee001]

have also tested this using the E2E Matrix in test.civicrm.org and it fixes some of the tests there and seems to be sensible change to me

[seamuslee001]

Jenkins re test this please

[eileenmcnaughton]

@seamuslee001 has this been merged to the rc?

[seamuslee001]

yes #14189

[totten]

have also tested this using the E2E Matrix in test.civicrm.org and it fixes some of the tests there and seems to be sensible change to me

Oooh, nice. So we're now only one test away from a green E2E matrix. :)

[totten]

Reported failure is a common false-negative.

[seamuslee001]

Test fail unrelated merging