[forge-dev] Is this correct?

[Doychin Bondzhev]

Hi guys,

While looking at forge code I found this:

/*
        * Make sure we have all the features we need for this to work.
        */
       if (!project.hasAllFacets(Arrays.asList(CDIFacet.class, 
PersistenceFacet.class)))
       {
          request.fire(new InstallFacets(true, EJBFacet.class, 
PersistenceFacet.class));
       }

On first look it is OK but then I noticed that Facets checked in if are 
not the same that are installed  after that. More specifically there is 
check for CDIFacet but then EJBFacet is installed.

Is that supposed to be this way or there is something wrong here?

This is RestPlugin.java

-- 
Doychin Bondzhev
dSoft-Bulgaria Ltd.
PowerPro - billing & provisioning solution for Service providers
PowerStor - Warehouse & POS
http://www.dsoft-bg.com/
Mobile: +359888243116
-------------- next part --------------
A non-text attachment was scrubbed...
Name: doychin.vcf
Type: text/x-vcard
Size: 268 bytes
Desc: not available
Url : http://lists.jboss.org/pipermail/forge-dev/attachments/20111106/6a515649/attachment.vcf