#4481 closed Defect (fixed)
can_edit_extender does not pass entity in $params
| Reported by: | mrclay | Owned by: | mrclay |
|---|---|---|---|
| Priority: | normal | Milestone: | Elgg 1.8.6 |
| Component: | Core | Version: | 1.8.3 |
| Severity: | minor | Keywords: | |
| Cc: | brett@… | Difficulty: |
Description
It passes $entity, which is an undefined variable. It should pass $extender.
Change History (6)
comment:1 Changed 13 months ago by mrclay
- Owner set to mrclay
- Status changed from new to assigned
comment:2 Changed 13 months ago by ewinslow
- Milestone changed from Needs Review to Elgg 1.8.5
comment:3 Changed 12 months ago by mrclay
Fixed (thanks Evan) PR: https://github.com/Elgg/Elgg/pull/237
comment:4 Changed 12 months ago by Steve Clay
- Resolution set to fixed
- Status changed from assigned to closed
Fixes #4481: In can_edit_extender, correctly pass entity to permissions hook. Also fixes missing attribute name in ProblemUpdatingMeta import exception.
Changeset: 07b6df3d3d5a4b4724d1c573543b67b98a80fd65
comment:5 Changed 11 months ago by Steve Clay
Fixes #4481: In can_edit_extender, correctly pass entity to permissions hook. Also fixes missing attribute name in ProblemUpdatingMeta import exception.
Changeset: 07b6df3d3d5a4b4724d1c573543b67b98a80fd65
comment:6 Changed 11 months ago by Cash Costello
Merge pull request #237 from mrclay/4481_18
Fixes #4481: In can_edit_extender, correctly pass entity to permissions hook.
Changeset: 912bcd65be8fb4b12aef4b78e8e9fa5ad68fc412
Note: See
TracTickets for help on using
tickets.
https://github.com/Elgg/Elgg/pull/205