Index: openacs-4/packages/project-manager/lib/assignee-portlet.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/project-manager/lib/assignee-portlet.tcl,v diff -u -r1.9 -r1.10 --- openacs-4/packages/project-manager/lib/assignee-portlet.tcl 15 Sep 2005 07:43:14 -0000 1.9 +++ openacs-4/packages/project-manager/lib/assignee-portlet.tcl 15 Sep 2005 22:40:21 -0000 1.10 @@ -49,17 +49,23 @@ set assignee_edit_url [export_vars -base project-assign-edit {project_item_id return_url}] set assign_group_p [parameter::get -parameter "AssignGroupP" -default 0] -if { $assign_group_p } { - set query_name "project_people_groups_query" -} else { - set query_name "project_people_query" -} -db_multirow -extend {contact_url complaint_url name} people $query_name {} { - set name [person::name -person_id $party_id] - if { $assign_group_p && [empty_string_p $name] } { - set name [group::title -group_id $party_id] +db_multirow -extend {contact_url complaint_url name} people project_people_groups_query {} { + if { $assign_group_p } { + # We are going to show all asignees including groups + if { [catch {set name [person::name -person_id $party_id] } err] } { + # person::name give us an error so its probably a group so we get + # the title + set name [group::title -group_id $party_id] + } + } else { + if { [catch {set assignee_name [person::name -person_id $party_id] } err] } { + # person::name give us an error so its probably a group, here we don't want + # to show any group so we just continue the multirow + continue + } } + # If contacts is installed provide a link to the contacts party_id, otherwise don't if {![empty_string_p $contacts_url]} { set contact_url "${contacts_url}$party_id"