Closed Bug 106748 Opened 23 years ago Closed 22 years ago

When adding custom component, the description and size may be left blank

Categories

(CCK Graveyard :: CCK-Wizard, defect, P1)

x86
Windows 2000
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: BarrettLndstrm, Assigned: shrutiv)

Details

Attachments

(4 files, 1 obsolete file)

I don't consider this a blocker by any means, but it's something to look at for 
the future.  

Currently, if you add a custom component, but do not add the description or the 
size, the component is added anyways, and the installer builds with no problems. 
 But the reported disk space needed by the installer is incorrect (offset by the 
actual size of the custom component).  I think there should be some sort of 
catch that forces users to add the name and size if one has not been given for 
the component.
qa to blee
QA Contact: barrettl → blee
Display error message if the user adds a custom component, but does not add the
description or the size.

Status: NEW → ASSIGNED
Priority: -- → P2
Priority: P2 → P1
Keywords: nsbeta1+
In addition to displaying the error message, this patch also includes some
screen refinement.
Attached patch Corresponding patch for factory (obsolete) — Splinter Review
Whiteboard: need r=
I think you could be a little more descriptive in the code describing what 
IsFieldEmpty() does. Seems that the other parameters listed are only checked if 
the first parameter has a value. Is that right? 

This line in cckwiz/iniFiles/build_page1.ini looks suspect because IsFieldEmpty 
is in there twice: 

+onNext=IsFieldEmpty(%CustomComponentPath1%,%CustomComponentDesc1%,%
CustomComponentSize1%,Enter the description and size for the additional 
component);IsFieldEmpty(%CustomComponentPath2%,%CustomComponentDesc2%,%
CustomComponentSize2%,Enter the description and size for the additional 
component);Message(Are you ready to build your customized installer?);RunIB
();Msg(Installer creation is complete. The build is in %Root%Configs\%
CustomizationList%\Output)

This line in Factory's build_page1.ini looks suspect for the same reason as 
above and because the functions are in a different order than CCK's: 

+onNext=Message(Are you ready to build your customized installer?);IsFieldEmpty
(%CustomComponentPath1%,%CustomComponentDesc1%,%CustomComponentSize1%,Enter the 
description and size for the additional component);IsFieldEmpty(%
CustomComponentPath2%,%CustomComponentDesc2%,%CustomComponentSize2%,Enter the 
description and size for the additional component);RunIB();Msg(Installer 
creation is complete. The build is in %Root%Configs\%CustomizationList%\Output)
 
And I think you missed MyEnterprise/deploykit.che in Factory.
>I think you could be a little more descriptive in the code describing what 
>IsFieldEmpty() does. 
I'll add a few more comments 

>Seems that the other parameters listed are only checked if 
>the first parameter has a value. Is that right? 
Yes.

>This line in cckwiz/iniFiles/build_page1.ini looks suspect because IsFieldEmpty 
>is in there twice
There are 2 additional components. So, I'm performing the check twice... once
for each component. 

>This line in Factory's build_page1.ini looks suspect for the same reason as 
>above and because the functions are in a different order than CCK's: 
good catch. I have accidentally put the functions in wrong order  for factory
and I have missed MyEnterprise/deploykit.che.

Attachment #81596 - Attachment is obsolete: true
r=smeredith
Whiteboard: need r=
Marking fixed.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Verified in 5/8 CCK and NCADM blds.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: