Admin Module Displaying 'Random' Characters

[erica26]

Running sugar version 5.2.0.

In modules/Administration/index.php the following if block (starting on line 84) results in a new row in the Administration Network table.

if(!empty($license->settings['license_latest_versions'])){
    	$encodedVersions = $license->settings['license_latest_versions'];
    	$versions = unserialize(base64_decode( $encodedVersions));
    	include('sugar_version.php');
    	if(!empty($versions)){
    		foreach($versions as $version){
    			if($version['version'] > $sugar_version )
    			{
    				$admin_option_defs['update'][] ='red';
    				if(!isset($admin_option_defs['update']['additional_label']))$admin_option_defs['update']['additional_label']= '('.$version['version'].')';
    			}
    		}
    	}
    }

Displayed on the page is the 'r' of the word red in one table cell, and a '(' in a second table cell. The '(' is colored red and from what it looks like in the code should have a version number following it. But it doesn't. Has anyone ever seen this behavior before?

[ddbtek]

We're seeing this too, exactly as you have described. Have not seen this anywhere else. Is this something 5.2.0a patch would fix? Have not seen any info on how to apply the 5.2.0a patch, it's ignored on all of the online literature.

I believe this problem is happening because the variable $license->settings['license_latest_versions'] is uninitialized and so it has garbage and the interpeter thinks it's nonzero and so it executes this code block. Of course, this code block then renders additional links with also uninitialized variables so you see the "base" url + the uninitialized variable which generates the onscreen garbage and incomplete character.

[roblaus]

I've seen it on the German forum. Looks like a leftover from Pro/Ent. I'm running one 5.2a but I haven't seen it there.

Upgrade: download the patch and simply apply it in the admin section under upgrades. No unzip. That's it.

[ddbtek]

I completely agree, remnant. The 5.2.0a patch *did* fix it.

[erica26]

Thanks! Installing the patch does remove those characters.

However, I'm weary of thinking it actually fixed the issue because the code in index.php is still the same -- so my fear is that the same behavior will occur the next time there's a newer version available. Thoughts?

[roblaus]

I think this is a minor issue compared to others.

One should never change or upgrade a running software just because there is a new version available. Just wait and see, do a test installation and if everything is ok go ahead with your production box.

Our customers are still on 5 and 5.1, one even on 4.5. As long as you do not need new functionality there is no need to upgrade...