Was looking to integrate, didn't see documentation on the hooks (prob missed it).

CommentFileSizeAuthor
fasttogle_api.patch1012 byteshefox

Comments

nigelcunningham’s picture

Version: 6.x-1.4 » 7.x-1.x-dev
Issue summary: View changes
joachim’s picture

Component: Code » Documentation
Category: Feature request » Bug report
Priority: Minor » Normal

This is a really good start.

From what I can tell by searching the code for 'invoke', there's also a hook_fasttoggle_toggle(). A few other things that need improving:

  1. +++ b/drupal/sites/all/modules/contrib/fasttoggle/fasttoggle.api.php
    @@ -0,0 +1,30 @@
    + * Returns options current user can do for an object.
    

    Should be an imperative verb, so 'Return options...'.

  2. +++ b/drupal/sites/all/modules/contrib/fasttoggle/fasttoggle.api.php
    @@ -0,0 +1,30 @@
    +  return $return;
    

    This needs documenting too.

  3. +++ b/drupal/sites/all/modules/contrib/fasttoggle/fasttoggle.api.php
    @@ -0,0 +1,30 @@
    + *   What style of label, either FASTTOGGLE_LABEL_STATUS or FASTTOGGLE_LABEL_ACTION
    

    Needs wrapping to 80 chars.

  4. +++ b/drupal/sites/all/modules/contrib/fasttoggle/fasttoggle.api.php
    @@ -0,0 +1,30 @@
    +function hook_fasttoggle_labels($style) {
    

    Missing @return here too.

Missing documentation is a bug BTW...

nigelcunningham’s picture

Status: Needs review » Closed (fixed)

Fixed in 7.x-1.3.