anonymous node [#1171720]

This module allows nodes that are created by an anonymous user to be handled by the authenticated user who logged in from a same machine. The connection is identifed and built automatically.

When an anonymous user who has created some nodes logs in, the module will identify the anonymous user with the authenticated user, and automatically created duplicated nodes for the latter. Every change on those nodes made by either the anonymous or authenticated user will be applied to the corresponding nodes.

The functionality provides the convenience that users can perform orperations without logging in, and be free from the risk of exposing all nodes by *remember me* machanism.

http://drupal.org/sandbox/henry/1150502

Comments

Comment #1

alexreinhart commented 6 July 2011 at 20:03

Status:

Needs review

» Needs work

I've taken a brief look through your module to look for obvious issues. Here's a few comments:

You should use Drupal's $cookie_domain variable when setting cookies so your module obeys global Drupal settings.
In anonymous_node_nodeapi, you use the variable $node_copy. It is never defined, only used. This must be a bug.
In several places there are drupal_set_message calls without helpful messages. I presume they are for debugging; they shouldn't be in your final module version without more helpful messages.
Why do you need to call cache_clear_all()?
You should run your module through the Coder module automatic code review to fix some minor formatting errors.
Does the process of giving a new node ID break all links to that node? If so, it seems your module has a major caveat. Why is this necessary?

Once you've resolved these issues, please set the status here back to "needs review" and someone will take another look at your module.