| 1 | Index: web_ui.py |
|---|
| 2 | =================================================================== |
|---|
| 3 | --- web_ui.py (revision 7499) |
|---|
| 4 | +++ web_ui.py (working copy) |
|---|
| 5 | @@ -1270,6 +1270,14 @@ |
|---|
| 6 | ticket.values.update(values) |
|---|
| 7 | |
|---|
| 8 | context = Context.from_request(req, ticket.resource) |
|---|
| 9 | + |
|---|
| 10 | + # display the owner and reporter links only if the user is known |
|---|
| 11 | + known_users = [user[0] for user in self.env.get_known_users()] |
|---|
| 12 | + for user in 'reporter', 'owner': |
|---|
| 13 | + link = '%s_link' % user |
|---|
| 14 | + if ticket[user] in known_users: |
|---|
| 15 | + data[link] = self._query_link(req, user, ticket[user]) |
|---|
| 16 | + |
|---|
| 17 | data.update({ |
|---|
| 18 | 'context': context, |
|---|
| 19 | 'fields': fields, 'changes': changes, |
|---|
| 20 | @@ -1278,8 +1286,6 @@ |
|---|
| 21 | 'action_controls': action_controls, |
|---|
| 22 | 'action': selected_action, |
|---|
| 23 | 'change_preview': change_preview, |
|---|
| 24 | - 'reporter_link': self._query_link(req, 'reporter', ticket['reporter']), |
|---|
| 25 | - 'owner_link': self._query_link(req, 'owner', ticket['owner']) |
|---|
| 26 | }) |
|---|
| 27 | |
|---|
| 28 | def rendered_changelog_entries(self, req, ticket, when=None): |
|---|
| 29 | Index: templates/ticket.html |
|---|
| 30 | =================================================================== |
|---|
| 31 | --- templates/ticket.html (revision 7499) |
|---|
| 32 | +++ templates/ticket.html (working copy) |
|---|
| 33 | @@ -137,9 +137,18 @@ |
|---|
| 34 | not in ('type', 'owner')]"> |
|---|
| 35 | <tr> |
|---|
| 36 | <th id="h_reporter">Reported by:</th> |
|---|
| 37 | - <td headers="h_reporter" class="searchable">${reporter_link}</td> |
|---|
| 38 | + <td headers="h_reporter" class="searchable"> |
|---|
| 39 | + <py:choose test="defined('reporter_link')"> |
|---|
| 40 | + <py:when test="True">${reporter_link}</py:when> |
|---|
| 41 | + <py:otherwise>${authorinfo(ticket.reporter)}</py:otherwise> |
|---|
| 42 | + </py:choose> |
|---|
| 43 | + </td> |
|---|
| 44 | <th id="h_owner">Owned by:</th> |
|---|
| 45 | - <td headers="h_owner">${owner_link} |
|---|
| 46 | + <td headers="h_owner"> |
|---|
| 47 | + <py:choose test="defined('owner_link')"> |
|---|
| 48 | + <py:when test="True">${owner_link}</py:when> |
|---|
| 49 | + <py:otherwise>${ticket.owner and authorinfo(ticket.owner) or ''}</py:otherwise> |
|---|
| 50 | + </py:choose> |
|---|
| 51 | </td> |
|---|
| 52 | </tr> |
|---|
| 53 | <tr py:for="row in group(fields, 2, lambda f: f.type != 'textarea')" |
|---|
