Index: site-manage/site-manage-tool/tool/src/java/org/sakaiproject/site/tool/SiteInfoToolServlet.java
===================================================================
--- site-manage/site-manage-tool/tool/src/java/org/sakaiproject/site/tool/SiteInfoToolServlet.java (revision 2150)
+++ site-manage/site-manage-tool/tool/src/java/org/sakaiproject/site/tool/SiteInfoToolServlet.java (revision 2151)
@@ -329,7 +329,7 @@
Element participantNode = doc.createElement(PARTICIPANT_NODE_NAME);
// participant name
- String participantName= participant.getName();
+ String participantName= participant.getDisplayName();
if (participant.getDisplayId() != null)
{
participantName +="( " + participant.getDisplayId() + " )";
Index: site-manage/site-manage-tool/tool/src/webapp/vm/sitesetup/chef_site-siteInfo-list.vm
===================================================================
--- site-manage/site-manage-tool/tool/src/webapp/vm/sitesetup/chef_site-siteInfo-list.vm (revision 2150)
+++ site-manage/site-manage-tool/tool/src/webapp/vm/sitesetup/chef_site-siteInfo-list.vm (revision 2151)
@@ -400,7 +400,7 @@
- #set ($participantName=$participant.getName())
+ #set ($participantName=$participant.getDisplayName())
#if ($participantName ==", ")
#else
$validator.escapeHtml($participantName)
Index: site-manage/site-manage-tool/tool/src/java/org/sakaiproject/site/tool/SiteAction.java
===================================================================
--- site-manage/site-manage-tool/tool/src/java/org/sakaiproject/site/tool/SiteAction.java (revision 2158)
+++ site-manage/site-manage-tool/tool/src/java/org/sakaiproject/site/tool/SiteAction.java (revision 2159)
@@ -6593,6 +6593,9 @@
}
}
} catch (UserNotDefinedException e) {
+ // add non-registered participant
+ realmEdit.removeMember(rId);
+
M_log.warn(this + ".doUpdate_participant: IdUnusedException " + rId + ". ", e);
}
}
@@ -11861,5 +11864,20 @@
}
}
+ public static String makeUserDisplayName(String userId) {
+ String userDisplayName = NULL_STRING;
+ String at = "@";
+
+ if (userId.indexOf(at) == -1) {
+ // non-email
+ userDisplayName = userId + " [deregistered]";
+ } else {
+ // email
+ userDisplayName = userId + " [pending]";
+ }
+
+ return userDisplayName;
+ }
+
} |