oqapi/webgui
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

change group INSERT to REPLACE to prevent merge problems in multi-master DB clusters

Browse source
This commit is contained in:
Doug Bell 2011-03-21 18:49:45 -05:00
parent 5e6483e97f
commit 2a519d5ae9
2 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
docs/changelog/7.x.x.txt
View file

@ -8,6 +8,7 @@
- fixed #12050: Thing query caching #1 - fixed #12050: Thing query caching #1
- added: Let CHI cache request $dbh via an "args" : [ "dbh" ] cache conf - added: Let CHI cache request $dbh via an "args" : [ "dbh" ] cache conf
- added: lineage increment_step and increment_offset for multi-master DB clusters - added: lineage increment_step and increment_offset for multi-master DB clusters
- Change Group INSERT to REPLACE to prevent merge problems in multi-master DB clusters
7.10.11 7.10.11
- fixed #12057: WebGUI::Search, assetIds search clause - fixed #12057: WebGUI::Search, assetIds search clause

4
lib/WebGUI/Group.pm
View file

@ -137,7 +137,7 @@ sub addGroups {
my $group = WebGUI::Group->new($self->session, $gid); my $group = WebGUI::Group->new($self->session, $gid);
my $recursive = isIn($self->getId, @{$group->getGroupsIn(1)}); my $recursive = isIn($self->getId, @{$group->getGroupsIn(1)});
next GROUP if $recursive; next GROUP if $recursive;
$self->session->db->write("insert into groupGroupings (groupId,inGroup) values (?,?)",[$gid, $self->getId]); $self->session->db->write("REPLACE into groupGroupings (groupId,inGroup) values (?,?)",[$gid, $self->getId]);
} }
$self->clearCaches(); $self->clearCaches();
return 1; return 1;
@ -169,7 +169,7 @@ sub addUsers {
foreach my $uid (@{$users}) { foreach my $uid (@{$users}) {
my ($isIn) = $self->session->db->quickArray("select count(*) from groupings where groupId=? and userId=?", [$self->getId, $uid]); my ($isIn) = $self->session->db->quickArray("select count(*) from groupings where groupId=? and userId=?", [$self->getId, $uid]);
unless ($isIn) { unless ($isIn) {
$self->session->db->write("insert into groupings (groupId,userId,expireDate) values (?,?,?)", [$self->getId, $uid, (time()+$expireOffset)]); $self->session->db->write("REPLACE into groupings (groupId,userId,expireDate) values (?,?,?)", [$self->getId, $uid, (time()+$expireOffset)]);
$self->session->stow->delete("gotGroupsForUser"); $self->session->stow->delete("gotGroupsForUser");
} else { } else {
$self->userGroupExpireDate($uid,(time()+$expireOffset)); $self->userGroupExpireDate($uid,(time()+$expireOffset));