History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: IDEADEV-25887
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Dmitry Avdeev
Reporter: Stephen Friedrich
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
IDEA: Development

Idea thinks setter methods of EL ValueBindings/ValueExpressions are unused

Created: 13 Oct 07 02:03   Updated: 17 Nov 08 11:51
Component/s: J2EE.JSF, J2EE.JSP, Refactoring
Fix Version/s: Diana 8.0.1

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
Issue Links:
Duplicate
 
This issue is duplicated by:
IDEA-17033 ValueExpression setXXX method is not ... Resolved
IDEADEV-26617 Seam: Good code is red: Property defi... Normal Resolved

Build: 7,348
Fixed in build: 9,028
Severity: High


 Description  « Hide
I actually deleted some methods because Idea did not find any usages.
After that my app broke at runtime because in fact the methods were used form value bindings in EL.

For example Idea thinks this is unused

public void setValuation(@Nullable Integer valuation)

but my web page needs it:

<tr:inputText id="valuation" value="#{scoreEntry.valuation}" autoComplete="false"/>


 All   Comments   Work Log   Change History      Sort Order:
Maxim Mossienko - 30 Nov 07 00:43
facelet method refs

Stephen Friedrich - 11 Apr 08 02:29
IMHO this should be included in Diana as facelets and seam support is one of the major new features.

Stephen Friedrich - 10 Sep 08 01:16
Excuse me for being persistent, but is there any chance to see this in Diana?

Refactoring my facelets file feels like in the beginning of the 90's: Just do a global search/replace and carefully review each occurence.


Stephen Friedrich - 30 Oct 08 17:31
Still broken in 8972: Placing the caret on the setter and searching for usages only gives the "No usages found in Project Files" popup.