id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc
19465	'Show on screen' (per_page) options don't save for certain custom taxonomies	jolley_small		"I've identified 2 situations where the screen options don't save (per_page) when administrating custom taxonomies:

1) When a taxonomy's name contains an integer
2) When a taxonomy's name contains a hyphen (-)

The first issue is due to a line in misc.php set_screen_options():

{{{
if ( !preg_match( '/^[a-z_-]+$/', $option ) )
			return;
}}}

This returns (not saving the option) if the taxonomy contains numbers. This preg_match should be expanded to also allow integers.

The second issue is due to this line in the same function:

{{{
$option = str_replace('-', '_', $option);
}}}

That replaces hyphens with underscores, which would be fine, however, the function in screen.php (render_per_page_options) which loads the options contains this:

{{{
$option = $this->get_option( 'per_page', 'option' );
		
		if ( ! $option )
			$option = str_replace( '-', '_', ""{$this->id}_per_page"" );
}}}

In my test cases $option was loaded by the first part, meaning the hyphen to underscore replacement never occurred, thus the saved per_page option was not loaded and the default value of 20 was used instead."	defect (bug)	closed	normal		Administration	3.3	normal	duplicate