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

VERIFIED FIXED

Status

P1
normal
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: BarrettLndstrm, Assigned: shrutiv)

Tracking

Details

Attachments

(4 attachments, 1 obsolete attachment)

(Reporter)

Description

17 years ago
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.
(Reporter)

Comment 1

17 years ago
qa to blee
QA Contact: barrettl → blee
(Assignee)

Comment 2

17 years ago
Display error message if the user adds a custom component, but does not add the
description or the size.

Status: NEW → ASSIGNED
Priority: -- → P2
(Assignee)

Updated

17 years ago
Priority: P2 → P1
(Assignee)

Updated

17 years ago
Keywords: nsbeta1+
(Assignee)

Comment 3

17 years ago
Created attachment 81590 [details] [diff] [review]
Patch to display error message if description or size field is blank

In addition to displaying the error message, this patch also includes some
screen refinement.
(Assignee)

Comment 4

17 years ago
Created attachment 81596 [details] [diff] [review]
Corresponding patch for factory
(Assignee)

Comment 5

17 years ago
Created attachment 81597 [details]
Screen shot of 'Create the installer' screen
(Assignee)

Updated

17 years ago
Whiteboard: need r=

Comment 6

17 years ago
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.
(Assignee)

Comment 7

17 years ago
>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.

(Assignee)

Comment 8

17 years ago
Created attachment 81726 [details] [diff] [review]
Corresponding patch for factory
Attachment #81596 - Attachment is obsolete: true
(Assignee)

Comment 9

17 years ago
Created attachment 81752 [details] [diff] [review]
Patch with more comments
r=smeredith
Whiteboard: need r=
(Assignee)

Comment 11

17 years ago
Marking fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 12

17 years ago
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.