Opened 9 years ago
Closed 9 years ago
#2742 closed defect (bug) (fixed)
Plugin Directory: Admin tool Author Card doesn't show plugins that weren't approved
| Reported by: | Ipstenu | Owned by: | coffee2code |
|---|---|---|---|
| Priority: | high | Milestone: | Plugin Directory v3.0 |
| Component: | Plugin Directory | Keywords: | has-patch |
| Cc: |
Description
As the subject says.
If a dev has a plugin in "pending initial review" or "pending" it doesn't show up as a plugin on their list. This can cause issues as if you don't KNOW someone submitted the same plugin, twice, you may accidentally approve it twice.
cc @coffee2code
Attachments (1)
Change History (6)
#1
@
9 years ago
- Summary Plugin Directory: Admin tool Author Card doesn't show pending plugins → Plugin Directory: Admin tool Author Card doesn't show plugins that weren't approved
#4
@
9 years ago
- Keywords has-patch added; needs-patch removed
- Resolution fixed
- Status closed → reopened
The query for user IPs a few lines below just uses 'post_status' => 'any'. Can't we use the same here instead of a hardcoded list? Seems more consistent and future-proof.
#5
@
9 years ago
- Resolution → fixed
- Status reopened → closed
The check for user IPs can take into account any post status because all it cares about is getting all known IP addresses associated with the user. However, the check for author plugins is for a section that isn't future-proof for unknown post statuses. Of course that section could be reworked, but I don't feel the flexibility for supporting unknown post statuses was warranted since it would lead to them being displayed in the author's plugins list with no attributable meaning (is it in some sort of open state? some sort of closed state?).
If new statuses get added in the future (as talked about), I think it's reasonable to explicitly account for them (which includes adding a custom explanatory tooltip, class, styling, and perhaps handling for each).
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Correction.
It doesn't show anything that wasn't approved. No pending, no rejections :(