Opened 3 years ago
Closed 9 months ago
#1495 closed Defect (fixed)
group urls are invalid
| Reported by: | cash | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Elgg 1.8.0 |
| Component: | Core | Version: | Github Master |
| Severity: | minor | Keywords: | |
| Cc: | brettp | Difficulty: | moderate |
Description (last modified by ewinslow)
The ElggGroup get method returns group:<guid> when the object's username is requested. This is then used in urls. It is invalid to include a ':' in a url like this.
One effect of this is that it breaks get_submenu(). get_submenu() depends on php's parse_url which chokes on invalid urls. This means the selected menu item doesn't show up in group sidebar menus.
Change History (4)
comment:1 Changed 3 years ago by brettp
comment:2 Changed 3 years ago by brettp
- Difficulty set to moderate
- Type changed from unconfirmed defect to confirmed defect
Would
comment:3 Changed 3 years ago by cash
- Milestone changed from Elgg 1.8 to Elgg 1.9
comment:4 Changed 9 months ago by ewinslow
- Description modified (diff)
- Milestone changed from Near Term Future Release to Elgg 1.8.0
- Resolution set to fixed
- Status changed from new to closed
Looks like this is no longer an issue.
Note: See
TracTickets for help on using
tickets.
As a note, including group:<guid> in a URL does some rather annoying magic and sets the pageowner to the group. This is not documented well (anywhere?) and can make debugging complicated when trying to get the page owner.