1

Closed

Bug in frmDNNAzureWizard of DNNAzureWizard project

description

Long story short: regardless of what you specify for the size of the VHD, the wizard always creates it as 1 GB.
 
It seems that there are two textboxes in the form for VHD size: txtVHDSize and txtVHDDriveSize
 
The former is invisible while the latter is not. The latter is (obviously) the one that the user can enter a new value into.
 
However, when text substitution is done on the configuration, the value is being grabbed from the invisible one.
 
This same issue exists for VHD name (txtVHDBlobName and txtVHDName), but it's not as important to me.
 
So, these lines:
 
        // Replace the tokens - VHD settings
        cfgStr = cfgStr.Replace("@@VHDBLOBNAME@@", txtVHDBlobName.Text.Trim().ToLower());
        int driveSize;
        int.TryParse(txtVHDSize.Text, out driveSize);
        cfgStr = cfgStr.Replace("@@VHDBLOBSIZE@@", txtVHDSize.Text.Trim().ToLower());
 
Should be changed to:
 
 
        // Replace the tokens - VHD settings
        cfgStr = cfgStr.Replace("@@VHDBLOBNAME@@", txtVHDName.Text.Trim().ToLower());
        int driveSize;
        int.TryParse(txtVHDDriveSize.Text, out driveSize);
        cfgStr = cfgStr.Replace("@@VHDBLOBSIZE@@", String.Empty + driveSize);
Closed Oct 8, 2012 at 5:10 PM by davidjrh
Fixes packaged on 6.3 release

comments

davidjrh wrote May 16, 2012 at 1:59 PM

Just yesterday I detect that issue and fixed on the check-in 15377 http://dnnazureaccelerator.codeplex.com/SourceControl/changeset/changes/15377

Sorry for the inconvenience and Thanks for the feedback!!

cb55555 wrote May 16, 2012 at 8:14 PM

I see the update. BTW, you're missing the .Trim() calls on the left option of the ternary statements on lines 951 and 952. :)

It seems that left options are the ones that really need it (rather than the right) since the textboxes referenced on the left side of the colon are the ones that allow user input whereas the ones referenced on the right are invisible.

davidjrh wrote May 16, 2012 at 9:48 PM

You're right! Just checked in the fix.

The right part also need the .Trim() -this textboxes are the ones shown if you choose the "not automatic" deployment on the first step.

Thanks!