public Object saveState(FacesContext context)
{
Object values[] = new Object[2];
values[0] = super.saveState(context);
values[1] = getValue() instanceof StateHolder ? saveAttachedState(context, getValue()) : getValue();
return values;
}
public void restoreState(FacesContext context, Object state)
{
Object values[] = (Object[])state;
super.restoreState(context, values[0]);
Object value = restoreAttachedState(context,values[1]);
ValueBinding vb = getValueBinding("value");
if (vb != null)
{
vb.setValue(context, value);
}
}
This works good both with stateholders and lists, also the ugly part is removed since we dont need to tell the component to take special care during restoreState.
Cagatay
On 10/15/06, Martin Marinschek <[EMAIL PROTECTED]> wrote:
The problem is that a list is internally not just serialized, but
specially treated (wrapped, and then on restore you have a different
list than you had before - no good).
And that's something that Cagatay understandably doesn't want
Cagatay, how about adding an additional parameter to the component (on
calling save-state) which tells the component on restore-state if it
should restore the value normally or via restoreAttachedState?
public Object saveState(FacesContext context)
{
Object values[] = new Object[3];
values[0] = super.saveState(context);
boolean stateHolder = getValue() instanceof StateHolder;
values[1] = stateHolder ? saveAttachedState(context,
getValue()) : getValue();
values[2] = Boolean.valueOf(stateHolder);
return values;
}
regards,
Martin
On 10/15/06, Matthias Wessendorf < [EMAIL PROTECTED]> wrote:
> Hi catagay,
>
> javax.faces.component._AttachedStateWrapper is pretty much myfaces_api isn't it?
> so shouldn't be used inside the savastate custom comp.
>
> can you explain why it is failing?
>
> Thanks!
>
>
> On 10/14/06, Cagatay Civici <[EMAIL PROTECTED]> wrote:
> > Hi,
> >
> > I'd like to discuss the latest issues about the savestate component.
> >
> > In order to use the component with a value of type StateHolder,
> > restoreAttachedState-saveAttachedState is used. But using
> > them fails with list implementations other than arraylists.
> >
> > See this one;
> >
> > See this one;
> > http://issues.apache.org/jira/browse/TOMAHAWK-738
> >
> > It seems restoreAttachedState-saveAttachedState should only
> > be used when the value is a stateholder, I've found an ugly solution to the
> > problem as;
> >
> > public Object saveState(FacesContext context)
> > {
> > Object values[] = new Object[2];
> > values[0] = super.saveState(context);
> > values[1] = getValue() instanceof StateHolder ?
> > saveAttachedState(context, getValue()) : getValue();
> > return values;
> > }
> >
> > public void restoreState(FacesContext context, Object state)
> > {
> > Object values[] = (Object[])state;
> > super.restoreState(context, values[0]);
> > Object value =
> > values[1].getClass().getName().equals("
> > javax.faces.component._AttachedStateWrapper") ?
> > restoreAttachedState(context,values[1]) : values[1];
> >
> > ValueBinding vb = getValueBinding("value");
> > if (vb != null)
> > {
> > vb.setValue(context, value);
> > }
> > }
> >
> > Since _AttachedStateWrapper is private, I cant use instanceof, using class
> > name is the ugly part. Other than that it works fine with all cases.
> >
> > Any ideas about this?
> >
> > Cagatay
> >
> >
> >
> >
> >
>
>
> --
> Matthias Wessendorf
> http://tinyurl.com/fmywh
>
> further stuff:
> blog: http://jroller.com/page/mwessendorf
> mail: mwessendorf-at-gmail-dot-com
>
--
http://www.irian.at
Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German
Professional Support for Apache MyFaces
