By default access is set to public.
This would allow users to set the default to friends only in their account settings.

Comments

socketwench’s picture

Assigned: Unassigned » socketwench
StatusFileSize
new3.66 KB

Okay, I like the idea behind this issue. The problem is that the patch is completely broken. So, based on the code contained therein, I rewrote it with a few changes of my own.

socketwench’s picture

StatusFileSize
new3.66 KB
+++ b/flag_friend_access/flag_friend_access.moduleundefined
@@ -50,13 +52,48 @@ function flag_friend_access_form_alter(&$form, &$form_state, $form_id) {
-    $access_value = isset($form['#node']->nid) ? flag_friend_access_value($form['#node']->nid) : FALSE;
+    ¶
+    // Get the node's friend visibility.
+    if (isset($form['#node']->nid)) {

Whitespace issue.

imyour’s picture

The last submitted patch, 1: 1168250.1b.perUserAccessSetting.patch, failed testing.

imyour’s picture

Yup ... i doubt that!

socketwench’s picture

Issue summary: View changes
StatusFileSize
new3.66 KB

@imyour Yeah. There's a Testbot bug that affected the entire 1.x branch of Flag Friend. I've created a new 2.x branch that supports Flag 3.x. This should work around the testbot bug.

socketwench’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev

Needed to fix the branches.

socketwench’s picture

Issue tags: +Needs tests
StatusFileSize
new3.21 KB

Reroll.

socketwench’s picture

StatusFileSize
new5.24 KB

  • socketwench committed 43a7f8c on 7.x-2.x
    #1168250 by socketwench Add per-user access setting default
    
socketwench’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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