Default for Campaign Type field causes incorrect wizard

[JimMackin]

Issue

The campaign type field is partially used to decide which template to show for the campaign wizard. A default value for this field can cause the wrong wizard to be shown (asking for an email template for a non email campaign for example.

Expected Behavior

Non email wizard should be displayed.

Actual Behavior

Newsletter wizard is displayed instead.

Possible Fix

Fix the check around line 120 of modules/Campaigns/WizardNewsletter.php to always use the wizardtype from request if it exists.

Steps to Reproduce

1. Set the default for campaign_type to be newsletter
2. Create a new campaign choosing the non email option
3. Observe the campaign wizard - it's showing the newsletter version.

Context

Your Environment

• SuiteCRM Version used:
• Browser name and version (e.g. Chrome Version 51.0.2704.63 (64-bit)):
• Environment name and version (e.g. MySQL, PHP 7):
• Operating System and version (e.g Ubuntu 16.04):

Ubuntu 14.04
PHP 5.5.9
MySQL 5.5.50

[Mac-Rae]

Confirmed in latest 7.10.x

[Kevconry]

Was able to replicate in 7.13.1