#2900 closed enhancement (fixed)
Separator argument not normalized
Reported by: | casiepa | Owned by: | johnjamesjacoby |
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | normal | Version: | 2.0 |
Component: | API - Actions/Filters | Keywords: | good-first-bug has-patch |
Cc: |
Description (last modified by )
Most functions use $args['sep'
] to define the separator. However (at least) bbp_list_forums()
and bbp_get_admin_repair_tool_overhead_filters()
seem to use $args['separator'
]
Normalization including backward compatibility would be needed.
Attachments (1)
Change History (7)
#1
@
8 years ago
- Description modified (diff)
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Under Consideration
#2
@
7 years ago
- Keywords good-first-bug added
- Milestone changed from Under Consideration to Future Release
- Version changed from 2.5.8 to 2.0
#3
@
7 years ago
- Keywords has-patch added; needs-patch removed
As this is the first time I am contributing to bbpress please let me know if my patch/method is correct.
many thanks
#4
@
7 years ago
- Milestone changed from Future Release to 2.6
Sorry for the delay in getting to this @milindmore22 let's take a look and try to sneak this into 2.6
#5
@
7 years ago
- Owner set to johnjamesjacoby
- Resolution set to fixed
- Status changed from new to closed
In 6628:
This ticket was mentioned in Slack in #bbpress by casiepa. View the logs.
7 years ago
Note: See
TracTickets for help on using
tickets.
Moving to future release to clear up Under Consideration milestone, we can milestone this once it has a patch