[jbosstools-issues] [JBoss JIRA] Commented: (JBIDE-1916) Seam find ref/decl assumes root of expression is what to search for, ignores method or properties

[Victor Rubezhny (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBIDE-1916?page=comments#action_12403479 ] 
            
Victor Rubezhny commented on JBIDE-1916:
----------------------------------------

Max Andersen [18/Mar/08 02:28 AM] 

About performance: "I just used the project variables (which are already collected)" ....these variables are (or at least should be done in the background) thus it might not be available since if it blocks the ui for large projects it will be a big showstopper. The initial quickfix popup should not require large calculations and if it requires the seam builder to have completed it will block the creation of that popup - and that is bad plugin behavior. 

About types then I can't recognize that java search search for all calsses with resultList. It looks for references to the resultList variable/method on the same type as testEntityList is. 

So to be explicit here is what I think should happen in case of #{testEntity.resultList.size} declaration search 
(note: not all of these needs to be implemented, we should do them in steps/ that makes sense) 

testEntity -> declarations with the same name (e.g. testEntity) and if we actually know what specific "testEntity" it is pointing to (e.g. it is a local variable used in a loop) then it should only list the specific declarations (e.g. where the variable is declared in the start of the loop) 

testEntity.resultList -> search from declarations of testEntity and find its type (might be multiple types) and then scan those types for possible things that resultList can be declared as (e.g. method or resultList or field resultList) 

testEntity.resultList.size() -> find the type(s) of testEntity, find the things that can be resultList on testEntity (method, field, key in map) and then scan for size() method on those possible types. 

Similar rules can be defined for references. 

What I suggest you/we do is to define a list of possible combiations of xhtml, components.xml and java that illustrates all the possible way of having multiple types and references and express that in unittests that we can use to verify this. 

> Seam find ref/decl assumes root of expression is what to search for, ignores method or properties
> -------------------------------------------------------------------------------------------------
>
>                 Key: JBIDE-1916
>                 URL: http://jira.jboss.com/jira/browse/JBIDE-1916
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: Seam
>    Affects Versions: 2.1.0.beta1
>            Reporter: Max Andersen
>         Assigned To: Victor Rubezhny
>             Fix For: 2.1.0.beta1
>
>
> System.out.println("#{identity.username}");
> If I do Ctrl+1 with the cursor on username it still only search for identity.
> Same happens with java code like:
> identity.getUserName()
> Ctrl+1 on getUserName() only search for identity, not the getUserName()

-- 
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