Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Add initial support for a fasttoggle block. It's currently usable, but
there's more work to do.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Prior to this commit, spaces caused a redirect on our Ajax POSTs,
turning them into GETs. This confused the callback, making it think
the call was non-ajax. This patch adds support for checking whether a
path will be rewritten by spaces. If so, we seek to rewrite the callback
to the 'correct' version so the POST can continue.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Fix the fallback to the normal view if no fasttoggle access - the
extraction of the value from the default handler's output was broken.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

This patch removes the assumption that field value keys are numeric.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Address an issue which caused fields not to be rendered fasttogglable
any more - properly invoke the field_access function.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Fix the contextual links hook, and remove the huge nesting of if
statements while we're at it.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

According to drumm, Drupal 7 hasn't fully moved away from
using user_save()'s $edit parameter. Switch to the 'old'
way of saving changes to the user object, so that modules
such as Bakery don't break.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Apply patch from issue 2328203. Thanks to Drumm.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

It should depend on fasttoggle_node, not itself.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Use a temporary variable to avoid the dependency on a PHP 5.4
feature.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Update the name of the variable (not denied => undecided) and
remove a couple of words that were wrongly left in a previous edit.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Make comment and modr8 support depend on fasttoggle_node. They
could theoretically be separated, but it would require some
duplication of code. The user can always just not enable node
options they don't want.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

This patch adds support for a link having a set of allowed values
that is a subset of the possible values for the setting. This is
needed for toggling comments between disabled, read only and read
write. If comments for a node are disabled and Fasttoggle is set
up to not allow you to disable comments, an error was displayed
on the page. That's now fixed.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Make the code easier to understand by adding further comments
in the header for the function and renaming a variable from
sitewide to objectwide.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

The views code wasn't checking all the permissions it should. Rely
instead on fasttoggle_get_allowed_links having done the correct checks.
(It won't return the element if it's not allowed).

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Remove some unneeded lines and rely on fasttoggle_get_allowed_links
to do all the access checking and only provide the info we need.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Make the node type settings in fasttoggle forms distinguishable again.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Update the access checks for the modified API.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Add more commenting and ensure that if no function explicitly
allows access to a setting, we deny it. This was the other part
of the problem with the user status issue.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Remove the global variable by passing results back down the call
chain.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Rename fasttoggle_check_access_have_outcome to
fasttoggle_check_access in preparation for getting rid of the
global variable and passing the result back.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Make the meaning of the "We haven't decided yet" value clearer.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Correct the omission of code applying the per-node type settings
for access to links.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Simplify the inclusion of files within the project. The previous version
should be safe, but this is better.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Make the moderate toggle default to being disabled, like other
toggles.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Improve the support for upgrading from 7.x-1.2 by getting the
translation of setting names and values right and removing the need
for the backwards compatibility in the main code.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Rename the variables as necessary.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

The field fasttoggles don't know what kind of view is being built, so
the 'view' parameter isn't set. Default to rendering a full node view
when toggling these settings.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Commit the first version of the API documentation.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

If an access check function does multiple checks (ie multiple calls
to fasttoggle_deny_access_if and fasttoggle_allow_access_if), make
the first one that would set an outcome be the effective one, rather
than the last. This makes it consistent with the overall flow of logic
and what you'd intuitively expect.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Add a comment explaining what we're doing (ie seeking to update the
node content after a setting is toggled).

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

Update documentation and shift the @file tag back up to the top in
fasttoggle.module.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>

The fifth argument isn't always required. Don't add it if it's not
needed and make it optional in the menu config.

Signed-off-by: Nigel Cunningham <nigel@nigelcunningham.com.au>