diff --git a/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserToGroupUI.inc b/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserToGroupUI.inc index 847b8b2..37b8296 100644 --- a/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserToGroupUI.inc +++ b/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserToGroupUI.inc @@ -16,9 +16,9 @@ function getPage($iUserID, $iGroupID) { if (isset($iUserID)) { $oUser = User::get($iUserID); - $heading = "Are you Sure you wish to assign the User to the Group? \n"; + $heading = "Are you Sure you wish to assign the User to the Group?\n"; }else{ - $heading = "Please Assign a User to a Group: \n"; + $heading = "Please Assign a User to a Group:\n"; } @@ -26,13 +26,10 @@ function getPage($iUserID, $iGroupID) { $oGroup = Group::get($iGroupID); } - - $sToRender .= "

\n"; - $sToRender .= "Assign User to Group:\n"; + + $sToRender .= "" . renderHeading("Add User to Group") . "
"; $sToRender .= "\n"; - $sToRender .= "

\n"; $sToRender .= $heading; - $sToRender .= "
\n"; $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; @@ -57,12 +54,10 @@ function getPage($iUserID, $iGroupID) { // page that comes up when there is nothing selected function getPageNotSelected() { global $default; - - $sToRender .= "

\n"; - $sToRender .= "Sorry, Selection Failed\n"; + + $sToRender .= "
User Name: " . getUserDisplay($oUser) . "
" . renderHeading("Add User to Group") . "
"; $sToRender .= "\n"; - $sToRender .= "

\n"; - $sToRender .= "\n"; + $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; @@ -75,15 +70,13 @@ function getPageNotSelected() { return $sToRender; } -//if edition successful -function getPageSuccess() { +//if addition successful +function getPageSuccess() { global $default; - - $sToRender .= "

\n"; - $sToRender .= "Success!\n"; + + $sToRender .= "
Sorry, Selection Failed
Please select BOTH a User AND a Group
" . renderHeading("Add User to Group") . "
"; $sToRender .= "\n"; - $sToRender .= "

\n"; - $sToRender .= "\n"; + $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; @@ -97,14 +90,12 @@ function getPageSuccess() { } //if edition successful -function getPageFail() { +function getPageFail() { global $default; - - $sToRender .= "

\n"; - $sToRender .= "Failed!\n"; - $sToRender .= "
Success!
The User was successfully assigned to the specified Group
\n"; - $sToRender .= "

\n"; - $sToRender .= "\n"; + + $sToRender .= "
" . renderHeading("Add User to Group") . "
"; + $sToRender .= "\n"; + $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; $sToRender .= "\n"; @@ -148,7 +139,9 @@ function getGroupDisplay($oGroup) { if (Permission::userIsSystemAdministrator()) { // if this is the system administrator, prepend group names with unit name $oPatternListBox = & new PatternListBox($default->owl_groups_table, "name", "id", "fGroupID"); - //$oPatternListBox->setFromClause("INNER JOIN $default->"); + $oPatternListBox->setFromClause("LEFT OUTER JOIN groups_units_link GUL on ST.id=GUL.group_id " . + "INNER JOIN units_lookup UL on GUL.unit_id=UL.id"); + $oPatternListBox->setCompositeDisplayName("DISTINCT CONCAT(CONCAT(UL.name, '-'),ST.name)"); } else if (Permission::userIsUnitAdministrator()) { // else if this is a unit administrator, only display the groups in your unit $oPatternListBox = & new PatternListBox($default->owl_groups_table, "name", "id", "fGroupID"); @@ -167,9 +160,9 @@ function getGroupDisplay($oGroup) { function getAssignButton($oUser) { global $default; if (!isset($oUser)) { - return "\n"; + return "\n"; } else { - return "\n"; + return "\n"; } } // show button after User selected
Failed!
The User already belongs to the specified Group
graphicsUrl/widgets/select.gif\" value=\"submit\" />graphicsUrl/widgets/select.gif\" value=\"submit\" border=\"0\"/>graphicsUrl/widgets/assign.gif\" value=\"submit\" />graphicsUrl/widgets/assign.gif\" value=\"submit\" border=\"0\"/>