__group__	ticket	summary	owner	component	_version	priority	severity	milestone	type	_status	workflow	_created	modified	_description	_reporter
3.6	20839	"""Visit plugin site"" should open in a new window"	helen	Administration	3.4	normal	minor	3.6	enhancement	reopened	close	2012-06-04T20:13:05Z	2013-05-05T05:56:44Z	It's weird to take users off the site when they click on this link in the plugins section. Just a little thing to make the experience that much better :)	empireoflight
3.6	22476	Standardize single and double quotes in CSS url()s		Administration		low	trivial	3.6	enhancement	new	has-patch	2012-11-16T17:26:05Z	2013-04-10T19:48:14Z	"As per http://www.w3.org/TR/CSS2/syndata.html#value-def-uri (and other locations), single quotes {{{'}}} and double quotes {{{""}}} in {{{url()}}} values in CSS like
{{{
body { background: url(""../img/image,gif"");
}}}
are optional. Most of core's CSS is already written without such quotes. The patch 22476-remove-quotes-in-url.diff removes the remaining ones, for consistency and to save us some bytes again, even in the minified CSS."	TobiasBg
3.6	21241	Default value for background_image_thumb fails when background_image URL includes a percent sign		Appearance	3.4.1	normal	minor	3.6	defect (bug)	reopened	has-patch	2012-07-12T16:49:14Z	2013-01-17T04:14:38Z	"If the background_image theme mod is set to a URL that includes a percent sign, and if background_image_thumb is not set, the code that uses background_image as the default setting for background_image_thumb fails due to get_theme_mod interpreting the percent signs in the background_image URL as sprintf conversion specifications.

I don't think that this situation can occur in a vanilla WordPress install (but that might be host-specific), but it's entirely possible that a plugin would set background_image to a URL including a percent sign."	cfinke
3.6	7392	Don't create new autosave revision if nothing has changed yet	westi	Autosave	2.6	normal	minor	3.6	defect (bug)	reopened	has-patch	2008-07-23T22:03:39Z	2013-05-09T15:22:06Z	"I noticed that simply loading and saving a post creates a new revision. This seems rather dumb to me. Could we not load the post up, compare it with the revision, and not create a whole new revision if nothing changed (ignoring post save time, of course)?

Discuss."	Otto42
3.6	21272	reporting typo in dbDelta		Database	3.4.1	normal	minor	3.6	defect (bug)	new	has-patch	2012-07-14T14:45:25Z	2013-01-08T07:05:05Z	"In dbDelta, there is the section:


{{{
                // For every remaining index specified for the table
                foreach ( (array) $indices as $index ) {
                        // Push a query line into $cqueries that adds the index to that table
                        $cqueries[] = ""ALTER TABLE {$table} ADD $index"";
                        $for_update[$table.'.'.$fieldname] = 'Added index '.$table.' '.$index;
                }
}}}

Note that $for_update is updated with the wrong array index.  This causes some updates not to be returned in $for_update.  Better would be:

{{{
                        $for_update[$table.'.'.$index] = 'Added index '.$table.' '.$index;
}}}

Even this is not really right, since indexes and columns may have the same names.  Given that the users of dbDelta don't even seem to care about the keys, it might be simpler just to change all the $for_update lines to

{{{
                        $for_update[] = ...
}}}

except for the first one that must be keyed on table name."	apimlott
3.6	23132	wp_timezone_override_offset() has noticeable performance hit		Date/Time	3.5	normal	minor	3.6	enhancement	new	has-patch	2013-01-06T22:05:19Z	2013-03-03T05:01:15Z	"While wandering through XHProf generated callgraph of my test stack I came upon `wp_timezone_override_offset()` eating 3.4% of runtime (73 calls). I do not expect noticeable resource consumption from such minor configuration-related function.

The obvious performance aspect is that it creates and discards two objects (`DateTime` and `DateTimeZone` - latter seemingly more heavy) per each call, while calculating same result each run.

The less obvious aspect is that the only use of this function is to pre-filter `gmt_offset` option, and it essentially hijacks option cache for it.

Possible solutions:

 - calculate result once per run and cache in static variable inside function (straightforward, but retains multiple function call)
 - filter `gmt_offset` on set rather than get (more involved)
 - scrap the extra function and/or option altogether? is there still practical need for having two options for timezone information?"	Rarst
3.6	23299	Definition List in Editor		Editor	2.2	low	minor	3.6	defect (bug)	new	has-patch	2013-01-26T18:38:12Z	2013-01-27T19:13:36Z	After switch from Text to Visual and then from Visual to Text the formatting of the definition list is lost.	gputignano
3.6	16276	Default value for search field - (#11420 Easy within-site linking from new content)		Editor	3.1	normal	minor	3.6	enhancement	new	has-patch	2011-01-17T20:14:42Z	2013-05-05T00:33:59Z	"The within-site linking available in the current release candidate looks like a great idea, but I'm wondering if it might make sense to pre-popular the search box area with the selected text that is highlighted.

Currently the search box (which is actually more of a filter box) is empty by default, but it seems to me that most of the time if you are linking to internal content, you will be linking to the material you are highlighting which will have a similar title."	jamie.richard
3.6	23433	Custom background select needs a contextualized label		I18N	3.5	normal	minor	3.6	defect (bug)	new	has-patch	2013-02-09T16:55:23Z	2013-02-20T01:11:46Z	"In /themes.php?page=custom-background, the fixed/scroll selector uses the 'Attachment' label.

Problem is, that label name is also used for another type of attachment: files.

Ergo, translators might find themselves with an incorrect wording. In effect, the French translation displays ""Fichier attaché"" (attached file) instead of a more appropriate term.

I suggest turning the string into a contextualized one.

Another solution would be to change the wording altogether, for instance: ""Behaviour when scrolling"", with ""Scroll along"" and ""Keep in place"" as its options.

Hence, I submit both (hopefully correct) patches -- built against trunk for now, but the issue obviously exists in 3.5."	xibe
3.6	23907	Scandinavian ligatures transcribed wrong in remove_accents()		I18N	trunk	normal	minor	3.6	defect (bug)	new	has-patch	2013-03-30T08:52:56Z	2013-03-30T16:32:08Z	"remove_accents() is transcribing old scandinavian ligatures wrong: ÆØÅøå should be transcribed as 'Ae', 'Oe', 'Aa', 'oe' and 'aa' respectively.

Transcribation rules are the same in swedish, danish and norwegian. In trunk they are transcribed as 'AE', 'O', 'A', 'o', 'a'.

See #4739, #9591 for history."	dnusim
3.6	23642	wp_mail() returns true if sending failed		Mail	3.2	normal	minor	3.6	defect (bug)	reopened	commit	2013-02-28T15:54:38Z	2013-05-22T17:41:30Z	"Here: source:/trunk/wp-includes/pluggable.php@23507#L449

The code is:
{{{
	try {
		$phpmailer->Send();
	} catch ( phpmailerException $e ) {
		return false;
	}
	
		return true;
}}}
However, $phpmailer->Send() may return false without throwing an exception. A simple change fixes this like so:
{{{
	try {
		return $phpmailer->Send();
	} catch ( phpmailerException $e ) {
		return false;
	}
}}}"	chmac
3.6	23380	3gp mime support		Media	3.5	normal	minor	3.6	enhancement	new	has-patch	2013-02-04T13:15:09Z	2013-05-23T05:16:29Z	Some devices using 3gp format for wp mobile app.	m_uysl
3.6	18584	Nav menu UI needs more hooks		Menus	3.3	low	minor	3.6	enhancement	new	has-patch	2011-09-04T02:38:16Z	2013-04-16T15:29:35Z	"I'm trying to add some additional fields to the nav menu blocks but sadly there are no hooks for doing so. Zero.

We should add some and get in a better habit of just doing so when the code is written.

My specific suggestions are before the Remove/Cancel links (i.e. for adding standard stuff), one above the ""original"" field, and one after those action links (to add more actions)."	Viper007Bond
3.6	15765	wp-admin/ms-sites.php doesn't indicate required fields		Network Admin	3.0.2	normal	minor	3.6	defect (bug)	new	commit	2010-12-10T16:14:49Z	2013-05-05T00:12:24Z	In '''wp-admin/ms-sites.php''', apparently all fields under '''Add Site''' are required, but there is no visual indication.	novasource
3.6	16843	wp_unique_post_slug() doesn't check pagination base when CPT has archive		Post Types	3.1	normal	minor	3.6	defect (bug)	new	has-patch	2011-03-13T06:22:22Z	2013-01-14T00:12:14Z	Title says it all.	scribu
3.6	23984	Thickbox top margin		Themes	2.8	normal	minor	3.6	defect (bug)	new	has-patch	2013-04-08T07:54:55Z	2013-05-08T00:45:01Z	"The ""theme update details"" popup looks funky, the thickbox titlebar is hidden behind WP's adminbar: http://cl.ly/image/2m27131t3508"	tillkruess
3.6	23362	Fields with HTML5 form validation constraints break post save UI		Validation	3.5	normal	minor	3.6	enhancement	new	has-patch	2013-02-01T22:12:41Z	2013-02-01T22:26:14Z	Consider a metabox added to the edit post screen which includes fields that utilize HTML5 form validation constraints (by supplying a specific `type` attribute or the `required` or `pattern` attributes). Normally upon hitting the Publish or Save Draft button, the buttons go disabled and a spinner appears until the page reloads. However, if any of the metabox input fields violate the HTML5 form validation constraints, then upon hitting a submit button the browser will display the native validation error messages, but the Save Draft and Publish buttons will still go into their disable state and the spinner will appear, even though the `submit` event never fired. This unfortunately discourages the use of HTML5 form validation and forces plugin authors to include a `formnovalidate` attribute on such input fields.  Attached is a patch that allows input fields to use HTML5 form validation constraints and which prevents the UI from going into an an indeterminate/broken state. See attached screenshot for a depiction of the issue.	westonruter
3.6	24356	Admin Nav Menu Walker - Strict Standards compatibility		Warnings/Notices	3.5.1	normal	minor	3.6	enhancement	new	has-patch	2013-05-17T16:34:31Z	2013-05-20T16:52:13Z	"Admin -> Appearance -> Menus produces some warning for me for some methods overriding abstract definitions do not follow strict standards.

Patch is here:
http://pastebin.com/6a2ugms7"	dvarga