Many hooks are currently undocumented and some just don't exist (mailing_list hooks).

Additionally, add a few initial api doc groups like API, mail to group similar functions together.

Comments

berdir’s picture

Status: Active » Needs review
StatusFileSize
new19.74 KB

Start of a better api documentation.

The patch contains mostly non-functional changes so it's easy to do partial updates here.

The patch adds:

- @mainpage docblock with a textual overview with links to more specific topics/groups
- @defgroup docblocks various topics like subscribers, newsletters, issues, mail spool, source and so on. All of them are actually still missing detailed documentation
- @ingroup for all function in simplenews.module for which it makes sense (pretty much all except hook implementations + a few irrelevant helper functions).

Functional changes (might want to commit them separately):
- While going through the functions, I noticed that our hook_locale() implementations hasn't been removed... done that
- Also renamed simplenews_update_sent_status() to simplenews_newsletter_update_sent_status()

The result of this patch can be seen live on http://api.worldempire.ch/api/simplenews

berdir’s picture

StatusFileSize
new27.66 KB

Added a few more ingroup statements.

berdir’s picture

Status: Needs review » Active

Commited, setting back to active for more documentation for the doc groups.

miro_dietiker’s picture

Pushing...

  • Berdir committed edbc137 on 8.x-1.x
    Issue #1392314 by Berdir: Cleaned up simplenews.api.php and created...