Hi!
I finally managed to solve the problem and I have finally become a
happy appfuse user + struts 2.1 + fit :)
The problem was (it still puzzles me) that displaytag is not able to
read from the value stack, so I made the wrong assumption and
thought that the value stack was useless.
Adding this line before the display tag table solved the problem
right away:
<s:set name="users" value="users" scope="request"/>
What I still don't understand is why prior to the upgrade display
tag was able to read from the value stack and now this feature is
gone. I didn't see any struts-displaytag plugin so I cannot figure
this one out.
Anyways now it is working. I would be happy to create a wiki entry
on how to integrate struts 2.1 + fit + appfuse if somebody is
interested.
Best regards,
Oscar
On Fri, Apr 11, 2008 at 3:48 AM, Matt Raible
<[EMAIL PROTECTED]> wrote:
On Thu, Apr 10, 2008 at 12:21 PM, Dustin Pearce <[EMAIL PROTECTED]
> wrote:
> *Gasp* How dare you Matt! ;-p Seems like a minor issue to
me. I can't
> remember a time in all the Webwork projects where I didn't find a
miss typed
> property name through front-end unit tests......
>
> But back to 2.1.1. This is still in development, so its kind of
a use at
> your own peril kind of thing. I think Xwork is getting better and
better
> and the 2.1.1 struts stuff is really great!
>
> Can you give me some examples of your struts tags <s:property />
and maybe
> a pojo or two?
If you checkout the rest showcase - or even use your existing app.
1. For the rest showcase, modify orders-edit.jsp:
mate dev/struts2/apps/rest-showcase/src/main/webapp/orders-edit.jsp
2. Change <s:textfield name="amount" label="Amount" /> to <s:textfield
name="amount2" label="Amount" /> and when you load the page, there
won't be any errors.
Matt
>
> -D
>
>
>
>
> On Apr 10, 2008, at 10:52 AM, Matt Raible wrote:
>
>
> >
> >
> >
> > This logging is a result of the following blog post I made and
patch I
> wrote.
> >
> > http://raibledesigns.com/rd/entry/does_struts_2_suck
> >
> > Unfortunately, Struts 2 / OGNL doesn't blow up when a property
is not
> > found - this would be my preference. Folks said it didn't happen
when
> > using FreeMarker but I've noticed it happens there as well.
Because of
> > this, a project I'm currently working on has moved to Spring MVC.
> > Developers need to be told when they fat-finger a property name.
> >
> > You should be able to turn off this warning my modifying your
log4j.xml.
> >
> > Matt
> >
> > On Thu, Apr 10, 2008 at 8:50 AM, oscar perez
> > <[EMAIL PROTECTED]> wrote:
> >
> > > Thank you Matt and Dustin,
> > > I copy-pasted Dustin's code and now the tests run again.
> > > In order to start up the application I had to set also the
parameter
> > > actionPackages. It seems that for the struts 2.1.1 codebehind
plugin
> this is
> > > no longer optional.
> > > However, I am not done yet. Now I am running into the problem
that
> > > basically the OgnlValueStack is not working at all. I am able
to get and
> set
> > > the values as long as I do it via requests/responses but the
value stack
> is
> > > not able to store anything. It keeps showing warnings like this:
> > >
> > > OgnlValueStack.warn(46) | Could not find property [operation]
> > >
> > > I also noticed that in the new struts-default.xml for struts
2.1.1 has
> been
> > > added a new interceptor.
> > > <interceptor-ref name="actionMappingParams"/>
> > > This looks like the cause of my problem but removing it does
not solve
> it.
> > > Has anybody run into the same problem?
> > > Thanks again,
> > > Oscar
> > >
> > >
> > >
> > > On Thu, Apr 10, 2008 at 6:27 PM, Dustin Pearce <[EMAIL PROTECTED]
>
> > > wrote:
> > >
> > >
> > > >
> > > > I started this process, but I have not taken it all the
way. I think
> the
> > > >
> > > main issue is that with Struts 2.1.1 and Xwork 2.1.1 we have an
> opportunity
> > > to change the Struts actions to be less Http dependent. This
is a good
> > > thing. Struts gives you access to the Request and Response
through
> > > interfaces like RequestAware or ServletRequestAware. Using
these in our
> > > actions means we won't have to use ActionContext.getContext()
to get
> what we
> > > need.
> > >
> > > >
> > > >
> > > > So far, I found the package name change in
DateConverter.java to
> import
> > > >
> > > com.opensymphony.xwork2.conversion.TypeConversionException. I
also was
> able
> > > to get the tests to pass with an NPE by adding the following to
> > > BaseActionTestCase to setup the Xwork context:
> > >
> > > >
> > > >
> > > >
> > > > ConfigurationManager configurationManager = new
> > > >
> > > ConfigurationManager();
> > >
> > > > configurationManager.addContainerProvider(new
> > > >
> > > XWorkConfigurationProvider());
> > >
> > > > Configuration config =
configurationManager.getConfiguration();
> > > > Container container = config.getContainer();
> > > >
> > > >
> > > > ValueStack stack =
> > > >
> > >
container.getInstance(ValueStackFactory.class).createValueStack();
> > >
> > > > stack.getContext().put(ActionContext.CONTAINER,
container);
> > > > ActionContext.setContext(new
ActionContext(stack.getContext()));
> > > >
> > > >
> > > > That fixed the NPE, but that is as far as I got. My
ultimate plan was
> to
> > > >
> > > retool the Appfuse Struts actions to use the new Rest plugin
and remove
> all
> > > the xml, and then take it even further with example apps for
Facebook,
> etc!
> > > Let me know if you can start up your webapp once you get the
tests to
> pass.
> > > Also if you have a different NPE from the dojo plugin, put the
> stacktrace up
> > > and we can work it out.
> > >
> > > >
> > > >
> > > > -D
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > On Apr 10, 2008, at 4:48 AM, oscar perez wrote:
> > > >
> > > > Hi all,
> > > >
> > > > I am running into this problem when upgrading appfuse to use
struts
> 2.1
> > > >
> > > and the dojo plugin:
> > >
> > > >
> > > >
> http://www.nabble.com/Getting-ready-for-Struts-2.1.1-td16231688s2369.html
> > > > http://www.mail-archive.com/[EMAIL PROTECTED]/
msg29571.html
> > > >
> > > > Anybody knows of a quick-and-dirty workaround to solve this
issue?
> > > > Thanks
> > > > Oscar
> > > >
> > > >
> > > >
> > > >
> > > >
> > >
> > >
> > >
> >
> >
> >
---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
>
>
>
>
---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]