[izpack-devel] mutually exclusive packs

[Bartz, Klaus]

Hi all,
I have tested the patch given by Vladmir Ralev; thanks
to Vladmir in advance.
The logical behavior seems as described below.

I have changed some positions in the code to support I18N
for this feature. eng.xml and deu.xml contains the additional
strings. For other languages it is needed.
At merging the patch into the sources I have tried to fix the
warnings given by my eclipse predominant beneath the new code. 
After Compiler.java I have forsaken it. 
It is really a stupid work to add the missing or change syntactical
wrong written java docs. Sometimes I do not know for what a param
is. We should only accept patches/new features with correct
I18N, java doc and HTML doc I think.

The HTML docu I have not written. May be Fabrice can do it.
May be it is possible to use the description below.

Cheers

Klaus

>-----Original Message-----
>From: izpack-devel-bounces at lists.berlios.de
>[mailto:izpack-devel-bounces at lists.berlios.de]On Behalf Of Vladimir
>Ralev
>Sent: Thursday, November 16, 2006 7:29 PM
>To: izpack-devel at lists.berlios.de
>Subject: Re: [izpack-devel] mutually exclusive packs
>
>
>I think it's done. The patch should be applied against the current 
>trunk, not over the previous patch. The only deviation from 
>the plan was 
>that I made the packs with a specified excludeGroup to be 
>preselected="no" by default (it's annoying to write it 
>manually for each 
>of the member packs, because most of them need to be preselected="no" 
>anyway). So the feature description becomes:
>
>1. "excludeGroup" is an attribute (ATTLIST) of element "pack"
>
>2. "excludeGroup" can only contain one group.
>
>3. This group is a String which is unique for one group of all
>"excludeGroup"s, but has no relation to other groups like the
>"installGroups" or the pack name or id.
>
>4. A functional  "excludeGroup" can contain at least two members. A 
>member is
>a pack which has the attribute "excludeGroup" with the given name.
>If only one member is declared, nothing will be done.
>
>5. A member in the "excludeGroup" cannot have the attribute 
>"required="yes"".
>
>6. Only one member in the "excludeGroup" can have the attribute 
>"preselected="yes"".
>Packs, which have an excludeGroup attribute are not preselected by 
>default (Unlike
>the packs with no specified excludeGroup).
>
>7. From all members of an "excludeGroup" only one member can be
>selected. One member of the group is mutually exclusive to each other 
>member.
>If a second pack is selected, the previous will be deselected. If no 
>previous pack
>from the same excludeGroup is selected, the pack will be 
>selected without
>excluding any other packs.
>
>Let me know.
>
>Bartz, Klaus wrote:
>> Hi Vladimir,
>> of course "pack"...
>> Using the require text area for both is OK, I think. Code
>> have to be addapted.
>> Will be good if you can expand the docu with points related
>> to the added rules.
>> In the moment I am very busy, therefore staccato style :-)
>>
>> Cheers
>>
>> Klaus
>>   
>-- 
>Vladimir Ralev, JBoss/RedHat
>
>