Based on discussions in #1593058: Remove system.info's files[] entry, we are to move the tests in xmlrpc.test over to PSR-0.

CommentFileSizeAuthor
#10 1598622.patch11.79 KBrobloach
#6 1598622-2.patch11.79 KBrobloach
#4 1598622.patch11.37 KBrobloach
#1 1598622.patch11.65 KBrobloach

Comments

robloach’s picture

robloach
he/him
commented
Status: Active » Needs review
StatusFileSize
new 1598622.patch11.65 KB

Not sure about this one.

Tor Arne Thune’s picture

Tor Arne Thune commented

Looks reasonable to me.

aspilicious’s picture

aspilicious commented
Status: Needs review » Needs work 
+++ b/core/modules/system/lib/Drupal/system/Tests/XmlRpc/BasicTest.phpundefined
@@ -0,0 +1,89 @@
+ * Definition of Drupal\system\Tests\XmlRpc\BasicTest.

@file missing

On every file

5 days to next Drupal core point release.

robloach’s picture

robloach
he/him
commented
Status: Needs work » Needs review
StatusFileSize
new 1598622.patch11.37 KB
aspilicious’s picture

aspilicious commented
Status: Needs review » Needs work

Test file needs to be deleted from .info file

robloach’s picture

robloach
he/him
commented
Status: Needs work » Needs review
StatusFileSize
new 1598622-2.patch11.79 KB
aspilicious’s picture

aspilicious commented
Issue tags: -PSR-0

#6: 1598622-2.patch queued for re-testing.

Niklas Fiekas’s picture

Niklas Fiekas commented

#6: 1598622-2.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +PSR-0

The last submitted patch, 1598622-2.patch, failed testing.

robloach’s picture

robloach
he/him
commented
Status: Needs work » Needs review
StatusFileSize
new 1598622.patch11.79 KB
aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

Good

robloach’s picture

robloach
he/him
commented
Status: Reviewed & tested by the community » Postponed

#1033818: Move xmlrpc includes into xmlrpc.module :-) .

robloach’s picture

robloach
he/him
commented
Status: Postponed » Closed (duplicate)

Yay! #1033818: Move xmlrpc includes into xmlrpc.module got in.