diff options
| -rw-r--r-- | engine/schema/upgrades/2009021301.sql | 2 | ||||
| -rw-r--r-- | mod/groups/actions/edit.php | 10 | ||||
| -rw-r--r-- | mod/groups/languages/en.php | 8 | ||||
| -rw-r--r-- | mod/groups/views/default/forms/groups/edit.php | 9 | ||||
| -rw-r--r-- | version.php | 2 |
5 files changed, 16 insertions, 15 deletions
diff --git a/engine/schema/upgrades/2009021301.sql b/engine/schema/upgrades/2009021301.sql new file mode 100644 index 000000000..efec52b93 --- /dev/null +++ b/engine/schema/upgrades/2009021301.sql @@ -0,0 +1,2 @@ +-- Changing the ACLs on existing groups
+UPDATE `prefix_entities` SET access_id=2 WHERE type='group';
\ No newline at end of file diff --git a/mod/groups/actions/edit.php b/mod/groups/actions/edit.php index e30fc7fee..6b66ab9f1 100644 --- a/mod/groups/actions/edit.php +++ b/mod/groups/actions/edit.php @@ -57,14 +57,12 @@ // Group membership - should these be treated with same constants as access permissions?
switch (get_input('membership'))
{
- case 0: $group->membership = ACCESS_PRIVATE;
- case 1 :$group->membership = ACCESS_LOGGED_IN; break;
- case 2:
- default: $group->membership = ACCESS_PUBLIC;
+ case 2: $group->membership = ACCESS_PUBLIC; break;
+ default: $group->membership = ACCESS_PRIVATE;
}
- // Get access
- $group->access_id = get_input('access_id', ACCESS_PRIVATE);
+ // Set access - all groups are public from elgg's point of view.
+ $group->access_id = 2;
// Set group tool options
$group->files_enable = get_input('files_enable', 'yes');
diff --git a/mod/groups/languages/en.php b/mod/groups/languages/en.php index 5f216a9b8..3b678b16a 100644 --- a/mod/groups/languages/en.php +++ b/mod/groups/languages/en.php @@ -32,7 +32,7 @@ 'groups:interests' => 'Interests',
'groups:website' => 'Website',
'groups:members' => 'Group members',
- 'groups:membership' => "Membership",
+ 'groups:membership' => "Group membership permissions",
'groups:access' => "Access permissions",
'groups:owner' => "Owner",
'groups:widget:num_display' => 'Number of groups to display',
@@ -59,6 +59,12 @@ 'groupforumtopic:new' => "New forum post",
/*
+ * Access
+ */
+ 'groups:access:private' => 'Closed - Users must be invited',
+ 'groups:access:public' => 'Open - Any user may join',
+
+ /*
Group tools
*/
'groups:enablepages' => 'Enable group pages',
diff --git a/mod/groups/views/default/forms/groups/edit.php b/mod/groups/views/default/forms/groups/edit.php index 2841c9dfe..5a0976cc7 100644 --- a/mod/groups/views/default/forms/groups/edit.php +++ b/mod/groups/views/default/forms/groups/edit.php @@ -49,16 +49,11 @@ <p>
<label>
<?php echo elgg_echo('groups:membership'); ?><br />
- <?php echo elgg_view('input/access', array('internalname' => 'membership','value' => $vars['entity']->membership, 'options' => array( ACCESS_PRIVATE => elgg_echo('PRIVATE'), ACCESS_PUBLIC => elgg_echo('PUBLIC')))); ?>
+ <?php echo elgg_view('input/access', array('internalname' => 'membership','value' => $vars['entity']->membership, 'options' => array( ACCESS_PRIVATE => elgg_echo('groups:access:private'), ACCESS_PUBLIC => elgg_echo('groups:access:public')))); ?>
</label>
</p>
- <p>
- <label>
- <?php echo elgg_echo('groups:access'); ?><br />
- <?php echo elgg_view('input/access', array('internalname' => 'access_id','value' => $vars['entity']->access_id )); ?>
- </label>
- </p>
+
<p>
<label>
<?php echo elgg_echo('groups:enablepages'); ?><br />
diff --git a/version.php b/version.php index 8ce8f88db..0f3d8bcef 100644 --- a/version.php +++ b/version.php @@ -13,7 +13,7 @@ * @link http://elgg.org/
*/
- $version = 2009021201; // YYYYMMDD = Elgg Date
+ $version = 2009021301; // YYYYMMDD = Elgg Date
// XX = Interim incrementer
$release = '1.4'; // Human-friendly version name
|