[Rejected] [Issue] Default theme template output in xmlhttp.php?action=get_buddyselect

[Johnny S]

After revamping private_send template i have noticed that buddy select popup content remains the same even if xmlhttp_buddy template contains completely different code (divs instead of tables). Once i have looked at source code i have noticed that that template (xmlhttp_buddy) still outputs default template content. After looking at  xmlhttp.php?action=get_buddyselect default theme template content was still there (regardless of what content was in xmlhttp_buddyselect in my custom theme). Basically: regardless what theme user has selected in UserCP  xmlhttp.php?action=get_buddyselect will output xmlhttp_buddy template content from the theme that is set as default.

[Johnny S]

After digging around in xmlhttp.php i have made following edit (same code is used in global.php - beta 3)

xmlhttp.php line 79 - 89

from

// Load basic theme information that we could be needing.
if($loadstyle == "def='1'")
{
if(!$cache->read('default_theme'))
{
 $cache->update_default_theme();
 }
$theme = $cache->read('default_theme');
}
$theme = @array_merge($theme, my_unserialize($theme['properties']));

to

// Load basic theme information that we could be needing.
if($loadstyle == "def='1'")
{
	if(!$cache->read('default_theme'))
	{
		$cache->update_default_theme();
	}
	$theme = $cache->read('default_theme');
}
else 
{
	$query = $db->simple_select('themes', 'name, tid, properties, stylesheets', $loadstyle, array('limit' => 1));
	$theme = $db->fetch_array($query);
}

$theme = @array_merge($theme, unserialize($theme['properties']));

[Eldenroot]

Johnny - dont you wanna fix and make a pull request for all your issues? You made about 5 or more thread with working solution... it needs to be just pulled to mybb git. It needs some discussion with MyBB dev team - coordinate work and make a decision - 1.8.0 or 1.8.1?

[Jones H]

I reported this already and made a PR for it

[Johnny S]

Johnny - dont you wanna fix and make a pull request for all your issues? You made about 5 or more thread with working solution... it needs to be just pulled to mybb git. It needs some discussion with MyBB dev team - coordinate work and make a decision - 1.8.0 or 1.8.1?

It's not up to me to decide if a issue is going to be pushed on github or it will be labeled for 1.8 or 1.8.1. If a PR for opened issue is opened and that issue is labeled with 1.8.1 that request won't be included in 1.8.

@Jones H just saw your issue on github https://github.com/mybb/mybb/issues/1208

This thread can be closed (or junked).