[wp-trac] [WordPress Trac] #11385: a couple of uninstantiated options
WordPress Trac
wp-trac at lists.automattic.com
Fri Dec 11 16:44:22 UTC 2009
#11385: a couple of uninstantiated options
-------------------------------------------+--------------------------------
Reporter: Denis-de-Bernardy | Owner:
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 2.9
Component: Optimization | Version: 2.9
Severity: normal | Resolution:
Keywords: has-patch commit dev-feedback |
-------------------------------------------+--------------------------------
Comment(by nacin):
Replying to [comment:10 ryan]:
> nacin's patch looks good except for the removal of the $posts_per_page
global. That option is defined in the schema. Perhaps it doesn't need to
be made a global, but that can be argued in a different ticket.
Correct me if I'm wrong, but that's the reading option for the number of
posts to display per page in a theme. The global $posts_per_page variable
itself is unused in admin, but in the off-chance a plugin used it, it is
overridden to hold the screen options per_page value in post.php.
I suppose this would make sense if it is being maintained for back compat,
assuming that the admin used to rely on the same setting?
--
Ticket URL: <http://core.trac.wordpress.org/ticket/11385#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list