Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Part of meta-issue #1310084: [meta] API documentation cleanup sprint
Comment | File | Size | Author |
---|---|---|---|
#7 | xmlrpc_docs-1649140-6.patch | 9.03 KB | Albert Volkman |
Comments
Comment #1
mjonesdinero CreditAttribution: mjonesdinero commentedwill work on this.. thanks
Comment #2
mjonesdinero CreditAttribution: mjonesdinero commented@jhodgdon
it seems the file on XMLrpc are already clean, i have the updated clone of the drupal 8 on my local
or are there things need to be done their because the indention even the 80 characters limit per line is also okie
Comment #3
jhodgdonI took a look at the files in the xmlrpc directory and its subdirectories, and found a few things that need cleaning up:
a) xmlrpc.api.php - hook_xmlrpc_alter() doc starts with wrong verb tense, see
http://drupal.org/node/1354#hooks
b) xmlrpc.server.inc - first function starts with wrong verb tense, see
http://drupal.org/node/1354#functions
c) xmlrpc.inc - several functions do not have parameter docs
d) tests subdirectory - xmlrpc_test.module has no documentation in it at all.
e) lib subdirectory - the test files in there need some work. XmlRpcBasicTest.php for instance has wrong verb tenses in its documentation headers for class and methods. See:
http://drupal.org/node/1354#classes
One note: the getInfo() and setUp() functions in tests currently are required *not* to have documentation headers, so don't add those. Long story.
Comment #4
mjonesdinero CreditAttribution: mjonesdinero commentedComment #5
fotuzlab CreditAttribution: fotuzlab commentedfollowing up with #1650894: Make xmlrpc module pass Coder Review
assigning this to myself as both the issues are related.
Comment #7
Albert Volkman CreditAttribution: Albert Volkman commentedFirst pass at this.
Comment #8
jhodgdonThese issues are a lot of work with very little tangible payoff, so I'm closing the rest of them as "won't fix". Your efforts on working on this issue were appreciated... it was just my fault for starting a task that was very difficult to get right.
Let's instead put our effort into fixing and reviewing documentation that is really unclear and/or wrong, and I hope that the people who worked on these issues are not afraid to jump into a more reasonable issue!