Opened 3 years ago
Closed 5 months ago
#1842 closed Defect (worksforme)
group profile view needs more defensive coding
| Reported by: | cash | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Needs Review |
| Component: | Groups | Version: | 1.7 |
| Severity: | minor | Keywords: | |
| Cc: | brettp | Difficulty: |
Description
One line that can cause problems is:
get_user($vars['entity']->owner_guid)->getURL()
This should be broken down into steps to prevent crashes:
$owner = get_user($vars['entity']->owner_guid);
if ($owner) {
do whatever with $owner->getURL();
}
Change History (4)
comment:1 Changed 3 years ago by cash
- Component changed from General to GroupsPlugin
comment:2 Changed 16 months ago by cash
- Milestone set to Needs Review
- Resolution set to fixed
- Severity set to minor
- Status changed from new to closed
- Version set to 1.7
comment:3 Changed 5 months ago by cash
- Resolution fixed deleted
- Status changed from closed to reopened
comment:4 Changed 5 months ago by cash
- Resolution set to worksforme
- Status changed from reopened to closed
Note: See
TracTickets for help on using
tickets.

code rewritten