The user can then unfollow them if she wants to.

CommentFileSizeAuthor
#7 commons_follow-automatically_follow_nodes-1816536-7.patch1.44 KBgnuget
#5 commons_follow-automatically_follow_nodes-1816536-5.patch1.33 KBgnuget

Comments

ezra-g’s picture

ezra-g commented
Priority: Normal » Major
ezra-g’s picture

ezra-g commented
Priority: Major » Critical
gnuget’s picture

gnuget
Primary language Spanish
commented
Assigned: Unassigned » gnuget
gnuget’s picture

gnuget
Primary language Spanish
commented

I almost finish this issue but i found two bugs who needs be fixed before to fix this one:

#1845520: $node undefined in commons_radioactivity_flag().
#1845510: Fatal error when a comment is created

I will working in these two issues, and after i will back to this.

gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new commons_follow-automatically_follow_nodes-1816536-5.patch1.33 KB

Done.

I attach a patch for the commons_follow module.

ezra-g’s picture

ezra-g commented
Status: Needs review » Needs work

Thanks, gnuget!

This looks good overall. One improvement:

   $flag->flag('flag', $nid);

Let's use an $account parameter there, taken from the $node or $comment uid. That way, programmatic insertions of nodes and comments will be consistent with ones done through the UI.

gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Needs work » Needs review
StatusFileSize
new commons_follow-automatically_follow_nodes-1816536-7.patch1.44 KB

I added the change on this new patch.

In the version of today seems the follow_node flag is not longer associated with posts, i don't sure if this is a bug or is a new default behaviour

ezra-g’s picture

ezra-g commented
Status: Needs review » Fixed

Looks good - Thanks, gnuget!

http://drupalcode.org/project/commons_follow.git/commit/ee018d0

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.