Fixed a bug where the message log could try to add duplicate entries for the same user.
This commit is contained in:
parent
e91e424787
commit
03175dcbcb
1 changed files with 3 additions and 2 deletions
|
|
@ -19,6 +19,7 @@ use WebGUI::Session;
|
||||||
use WebGUI::SQL;
|
use WebGUI::SQL;
|
||||||
use WebGUI::URL;
|
use WebGUI::URL;
|
||||||
use WebGUI::User;
|
use WebGUI::User;
|
||||||
|
use WebGUI::Utility;
|
||||||
|
|
||||||
#-------------------------------------------------------------------
|
#-------------------------------------------------------------------
|
||||||
sub _notify {
|
sub _notify {
|
||||||
|
|
@ -54,7 +55,7 @@ sub addEntry {
|
||||||
if ($groupId ne "") {
|
if ($groupId ne "") {
|
||||||
@users = WebGUI::SQL->buildArray("select userId from groupings where groupId=$groupId");
|
@users = WebGUI::SQL->buildArray("select userId from groupings where groupId=$groupId");
|
||||||
}
|
}
|
||||||
@users = ($userId,@users) if ($userId ne "");
|
@users = ($userId,@users) if ($userId ne "" && !isIn($userId, @users));
|
||||||
foreach $user (@users) {
|
foreach $user (@users) {
|
||||||
$u = WebGUI::User->new($user);
|
$u = WebGUI::User->new($user);
|
||||||
if ($u->userId ne "") {
|
if ($u->userId ne "") {
|
||||||
|
|
@ -83,7 +84,7 @@ sub addInternationalizedEntry {
|
||||||
if ($groupId ne "") {
|
if ($groupId ne "") {
|
||||||
@users = WebGUI::SQL->buildArray("select userId from groupings where groupId=$groupId");
|
@users = WebGUI::SQL->buildArray("select userId from groupings where groupId=$groupId");
|
||||||
}
|
}
|
||||||
@users = ($userId,@users) if ($userId ne "");
|
@users = ($userId,@users) if ($userId ne "" && !isIn($userId, @users));
|
||||||
foreach $user (@users) {
|
foreach $user (@users) {
|
||||||
$u = WebGUI::User->new($user);
|
$u = WebGUI::User->new($user);
|
||||||
if ($u->userId ne "") {
|
if ($u->userId ne "") {
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue