Checklist API

Note: PAReview gives two warnings:

FILE: ...ew_temp/test_candidate/checklistapi_example/checklistapi_example.module
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 2 WARNING(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
  8 | WARNING | Line exceeds 80 characters; contains 98 characters
 10 | WARNING | Line exceeds 80 characters; contains 98 characters
--------------------------------------------------------------------------------

They refer to @link ... @endlink code, which as I understand should be all on the same line and is permitted to exceed 80 characters. I can't find this fact documented anywhere at present. I'll have it confirmed and then see if I can't get it noted in the standards. Then I'll put in an issue to update the Code Sniffer accordingly. Update: This has been confirmed to be a false positive. See #1559536: False positive/Support @link...@endlink standards.

Updated issue summary.

Adding tag.

Thank you, @scarer.

The PAReview errors are false positives. See #1.

Vertical tabs always display that way in Bartik; it's not a function of my implementation.

Thank you, @targoo. I thought about that, but the variables won't belong to the API—they'll belong to the implementing modules. If I take responsibility for removing them in checklistapi_uninstall() they may not always be properly removed. For example, suppose foochecklist.module and barchecklist.module both implement the API. If a user installs and uses both and then only uninstalls foochecklist.module, the API won't be uninstalled, so checklistapi_uninstall() won't be executed and foochecklist.module's variables won't be deleted. If you have a good solution for this, I would most gladly hear it. Otherwise, maybe I just need to add a note to the documentation that it's the responsibility of implementing modules to delete their own variables on hook_uninstall().