diff options
author | ben <ben@36083f99-b078-4883-b0ff-0f9b5a30f544> | 2008-07-31 15:56:43 +0000 |
---|---|---|
committer | ben <ben@36083f99-b078-4883-b0ff-0f9b5a30f544> | 2008-07-31 15:56:43 +0000 |
commit | 1ad49aaf2f1cb23a81909576437997fffa3951d2 (patch) | |
tree | 6156d26c46dea52993708fed082c9c0dbdf37b07 | |
parent | 4ab77c0ef0557d26781cbd3f534aeedd905d8637 (diff) | |
download | elgg-1ad49aaf2f1cb23a81909576437997fffa3951d2.tar.gz elgg-1ad49aaf2f1cb23a81909576437997fffa3951d2.tar.bz2 |
More updates for collections - save now works properly.
git-svn-id: https://code.elgg.org/elgg/trunk@1637 36083f99-b078-4883-b0ff-0f9b5a30f544
-rw-r--r-- | engine/lib/access.php | 21 | ||||
-rw-r--r-- | views/default/friends/picker.php | 4 |
2 files changed, 16 insertions, 9 deletions
diff --git a/engine/lib/access.php b/engine/lib/access.php index a1741f8bf..29f87a1f8 100644 --- a/engine/lib/access.php +++ b/engine/lib/access.php @@ -213,16 +213,21 @@ global $CONFIG;
$collection_id = (int) $collection_id;
- if ($collection = get_access_collection($collection_id)) {
- delete_data("delete from {$CONFIG->dbprefix}access_collection_membership where access_collection_id = {$collection_id}");
- }
+ $collections = get_write_access_array();
+
+ if (array_key_exists($collection_id, $collections)) {
- if (is_array($members) && sizeof($members) > 0) {
- foreach($members as $member) {
- $member = (int) $member;
- add_user_to_access_collection($member, $collection_id);
+ delete_data("delete from {$CONFIG->dbprefix}access_collection_membership where access_collection_id = {$collection_id}");
+
+ if (is_array($members) && sizeof($members) > 0) {
+ foreach($members as $member) {
+ $member = (int) $member;
+ if (get_user($member))
+ insert_data("insert into {$CONFIG->dbprefix}access_collection_membership set access_collection_id = {$collection_id}, user_guid = {$member}");
+ }
+ return true;
}
- return true;
+
}
return false;
diff --git a/views/default/friends/picker.php b/views/default/friends/picker.php index d813e6fe7..2f51a02d7 100644 --- a/views/default/friends/picker.php +++ b/views/default/friends/picker.php @@ -102,7 +102,9 @@ $('#collectionMembersForm<?php echo $friendspicker; ?>').submit(function() {
var inputs = [];
$(':input', this).each(function() {
- inputs.push(this.name + '=' + escape(this.value));
+ if (this.type != 'checkbox' || (this.type == 'checkbox' && this.checked != false)) {
+ inputs.push(this.name + '=' + escape(this.value));
+ }
});
jQuery.ajax({
type: "POST",
|