[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-1190) Disabling install of core components no longer works

[Pete Muir (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBSEAM-1190?page=comments#action_12361754 ] 
            
Pete Muir commented on JBSEAM-1190:
-----------------------------------

Also, I don't see why this is consistent:

<my:component installed="true" /> on a @Install(false) causes the component to be installed

AND

<my:component installed="false" /> on a @Install(true) causes the component to be installed

Huh?

> Disabling install of core components no longer works
> ----------------------------------------------------
>
>                 Key: JBSEAM-1190
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1190
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.2.1.GA
>         Environment: Any
>            Reporter: Mike Quilleash
>         Assigned To: Gavin King
>            Priority: Blocker
>
> Upgrading from Seam 1.1.6 to 1.2.1 seems to have broken the ability to do this...
>   <component class="org.jboss.seam.web.ExceptionFilter" installed="false"/>
> In my WEB-INF/components.xml.  The logic in the dependency manager scans both component descriptors, ignores the first as it is set to not install and then installs the second (default) one as normal.  I believe the precedence of two components is not being honoured with regard to the installed flag.
> Possible in
>     private boolean tryToInstall(String key) {       
> it should stop looping immediatly if it finds a non-installed component rather than skipping that descriptor and continuing to the next loop.
> This is a rather major issue for me as the exception filter is causing problems elsewhere (see bug 1188) and I can't seem to find out how to turn it off!

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira