#14543 closed defect (bug) (fixed)
Recent Posts Widget post count setting/display discrepancy
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Widgets | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
Recent Posts Widget can only display 1-15 posts.
http://core.trac.wordpress.org/browser/trunk/wp-includes/default-widgets.php#L537
However, you can save any negative/positive integer from the widget setting field.
The number you can use there should be limited to the min (1) and max (15, if this remains so).
Attachments (4)
Change History (23)
#3
@
14 years ago
- Keywords has-patch added
- Status changed from accepted to assigned
Patch submitted.
#4
@
14 years ago
I removed the min/max check in WP_Widget_Recent_Posts::widget and added the same functionality in WP_Widget_Recent_Comments with a filter to allow increasing the default upper limit of the comments widget.
#8
follow-up:
↓ 10
@
14 years ago
Came here to submit this bug. Lets you specify any integer above 15 in the front-end, but the code limits you to 15 in default-widgets.php. Wasted a lot of time figuring this one out.
Does keyword "has-patch" mean this is fixed? Have patience, first post :)
#9
@
14 years ago
Saw the patch details. Looks really cool! Bug tracking is fun!
--
There is no shame whatsoever in double posting, and I do not apologize.
#10
in reply to:
↑ 8
@
14 years ago
Replying to trevmillion:
Does keyword "has-patch" mean this is fixed? Have patience, first post :)
has-patch means that someone has submitted a possible fix, but the fix may not necessarily have made it into core yet. That has to wait until a committer has reviewed it and actually applied the patch.
See the codex for a list of commonly used keywords.
I think the dashboard widgets properly adjust for the limit upon save. We should be doing the same here.