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

Key: ICE-2684
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Judy Guglielmin
Reporter: Judy Guglielmin
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
ICEfaces

fix problems in id generation for list.xhtml.ftl and view.xhtml.ftl

Created: 29/Jan/08 11:54 AM   Updated: 01/May/08 12:00 PM
Component/s: None
Affects Version/s: None
Fix Version/s: None

Environment: Seam

Workaround Exists: Yes
Workaround Description: manually fix id's


 Description  « Hide
look at id's for line 110 on both files. Have to get fixed and back to jboss-seam people before release of 2.0.1.GA

 All   Comments   Change History   Subversion Commits      Sort Order:
Judy Guglielmin [30/Jan/08 11:50 AM]
Also should change the TimerBeanImpl as the ViewListener interface is deprecated (change to DisposableBean interface...but I don't think we really need it since we have @Remove and @Destroy annotations to work with...

Judy Guglielmin [30/Jan/08 01:26 PM]
edit xhtml page has partialSubmit="true" outside of ice:inputText for property id's.

Judy Guglielmin [31/Jan/08 09:56 AM]
changes in s:decorate has changed the way id's are rendered, so must look at id's to ensure all fields have id's for selenium testing. may as well do it at same time.

Judy Guglielmin [31/Jan/08 10:46 AM]
view.xhtml.ftl:- change line 110 to
     <ice:column id="view${parentParentPojo.identifierProperty.name}ColumnName">
edit.xhtml.ftl:- change line 112 to
      <ice:column id="editColumn${componentProperty.name}Id">
editproperty.xhtml.ft:- reverse order of lines 116 & 117 so that partialSubmit="true" comes before value="${....
                                       reverse order of lines 104 and 105 (again so partialSubmit comes before value="...)
                                      

Judy Guglielmin [31/Jan/08 12:21 PM]
get rid of icefaces/src/TimerBean.java (interface not required).

Update TimerBeanImpl:-

1) eliminate implementing ViewListener interface so just have:-
    public class TimerBeanImpl implements Renderable, Serializable{
  also delete line 174 (state.addViewListener(this)) ...it's not required for cleanup of IntervalRenderer since Seam has the @Remove/@Destroy for clean up of IntervalRenderer.

Judy Guglielmin [01/May/08 12:00 PM]
Done in for 2.0 branch (in time for 2.0.2.CR2 release) and Seam trunk (after 2.1.0.A1 release).

Also added in resizable columns for ice:dataTable