Part of meta-issue #1310084: [meta] API documentation cleanup sprint

This is for the system module, subdirectory tests, files and sub-directories in there starting with J through Z, including the themes and upgrade subdirectories.

Files: 
CommentFileSizeAuthor
#10 system_tests_j_z-1431664-10.patch11.89 KBAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 56,254 pass(es). View
#10 interdiff.txt2.75 KBAlbert Volkman
#8 system_tests_j_z-1431664-8.patch11.4 KBAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 58,014 pass(es). View
#8 interdiff.txt643 bytesAlbert Volkman
#7 system_tests_j_z-1431664-7.patch11.39 KBAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 57,971 pass(es). View
#6 system_tests_j_z-1431664-6.patch17.16 KBAlbert Volkman
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch system_tests_j_z-1431664-6.patch. Unable to apply patch. See the log in the details link for more information. View
#6 interdiff.txt12.15 KBAlbert Volkman
#4 system_tests_j_z-1431664-4.patch17.37 KBAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 49,053 pass(es). View

Comments

xjm’s picture

Title: Clean up API docs for simpletest/tests, J-Z » Clean up API docs for simpletest/tests, J-Z, including subdirectories
xjm’s picture

Title: Clean up API docs for simpletest/tests, J-Z, including subdirectories » Clean up API docs for system/tests, J-Z, including subdirectories
xjm’s picture

Issue summary: View changes

Added explicit mention of subdirs

mmartinov’s picture

Assigned: Unassigned » mmartinov

I'm taking this, I should be able to post a patch soon.

Albert Volkman’s picture

Status: Active » Needs review
FileSize
17.37 KB
PASSED: [[SimpleTest]]: [MySQL] 49,053 pass(es). View

Here's a first rough patch.

jhodgdon’s picture

Status: Needs review » Needs work

Thanks!

The standard for menu callbacks is:
http://drupal.org/node/1354#menu-callback
- You need : instead of ;
- The next thing should be a verb like "Records the active trail for 403/404 pages" rather than "Used for ..."

So, most of this patch needs an update.

Albert Volkman’s picture

FileSize
12.15 KB
17.16 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch system_tests_j_z-1431664-6.patch. Unable to apply patch. See the log in the details link for more information. View

This patch resolves the semicolon versus colon mixup. Next up is fixing the verbiage.

Albert Volkman’s picture

Assigned: mmartinov » Albert Volkman
FileSize
11.39 KB
PASSED: [[SimpleTest]]: [MySQL] 57,971 pass(es). View

Re-roll of #6.

Albert Volkman’s picture

Status: Needs work » Needs review
FileSize
643 bytes
11.4 KB
PASSED: [[SimpleTest]]: [MySQL] 58,014 pass(es). View

Fixed incorrect docblock for system_test_basic_auth_page().

jhodgdon’s picture

Status: Needs review » Needs work

Thanks, looking better!

+++ b/core/modules/system/tests/modules/module_test/module_test.module
@@ -93,7 +93,7 @@ function module_test_menu() {
 }
 
 /**
- * Page callback for 'hook dynamic loading' test.
+ * Page callback: For 'hook dynamic loading' test.

These do not follow our standards for hook_menu callbacks:
https://drupal.org/node/1354#menu-callback
They need a verb. There are 4 in this file that need updating, one in session_test.module, and a couple in theme_test.module.

Other than that, the patch looks pretty good... Oh, here's another problem, in session_test.module:

+ * Page callback: Attempts to save a value with after disabling session.
  */
 function _session_test_no_set($value) {

"with after"? This doesn't quite make sense. :)

And can we fix the typo here:

+++ b/core/modules/system/tests/modules/url_alter_test/url_alter_test.install
...
+/**
  * Impelement hook_install().
Albert Volkman’s picture

FileSize
2.75 KB
11.89 KB
PASSED: [[SimpleTest]]: [MySQL] 56,254 pass(es). View

Give this a try.

Albert Volkman’s picture

Status: Needs work » Needs review
Albert Volkman’s picture

Issue summary: View changes

.

jhodgdon’s picture

Issue summary: View changes
Status: Needs review » Closed (won't fix)

These 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!