Problems with struts-faces

2012-07-10 Thread Ramiro González Gómez
Hi Craig,

I’m a profesional developer from Spain. We are decided to used strtus-faces
jar to migrate an application from Struts to JSF. We have followed all
steps described in PDF  Migrating from Struts to
JavaServer Faces from Kito. We can see the JSF page and everything is OK
excepts the action form. When we see the generated source code for the page
is like that

form id=j_id_jsp_517671304_8 name=mantenimientoControlStockForm
action=/Diana/WEB-INF/ram/almacen/pages/faces/MantenimientoControlStock.do
method=post


And the proper code action would be like that

form id=j_id_jsp_517671304_8 name=mantenimientoControlStockForm
action=/Diana/MantenimientoControlStock.do method=post

So we can’t submit the form properly. Looking for Internet some people had
the same error but there are no answers for them. Could you help us? Thanks
a lot.

Regards.


struts-faces problem not invoking Action class displaying blank page

2007-07-03 Thread sadasivarao

Hi,

 

I am using struts and JSF using struts-faces 1.3.8.jar file and
using WSAD 5.1.2

 

My jsf page is like this

s:form action=/logon

..

h:commandbutton type=submit  value=ok

/s:form

As soon as I click on submit button blank page displaying. No
error it is showing it is populating ActionForm but it not invoking my
Action class

 

My struts-config.xml file is

 

!-- Form Beans --

  form-beans

form-bean name=logonForm type=example.LogonForm/

  /form-beans

 

  !-- Global Exceptions --

  global-exceptions

  /global-exceptions

 

  !-- Global Forwards --

  global-forwards

  /global-forwards

 

  !-- Action Mappings --

  action-mappings

  action path=/logon

  type=example.LogonAction

  name=logonForm

  input=login.faces 

  scope=request

forward name=success path=/Success.faces/forward

forward name=failure path=/error.faces/forward

  /action

  /action-mappings

  

controller locale=true

  set-property property=processorClass 

value=org.apache.struts.faces.application.FacesRequestProcessor/

  /controller

 

  Please help me it is urgent..



 

Thanks  Regards,
Sadasivarao Alokam,
Wipro Technologies,
Madhapur, Hyderabad, India
Tel : +91-40-30795443
Mobile: +91-9989228936

 




The information contained in this electronic message and any attachments to 
this message are intended for the exclusive use of the addressee(s) and may 
contain proprietary, confidential or privileged information. If you are not the 
intended recipient, you should not disseminate, distribute or copy this e-mail. 
Please notify the sender immediately and destroy all copies of this message and 
any attachments. 

WARNING: Computer viruses can be transmitted via email. The recipient should 
check this email and any attachments for the presence of viruses. The company 
accepts no liability for any damage caused by any virus transmitted by this 
email.
 
www.wipro.com

Re: struts-faces problem not invoking Action class displaying blank page

2007-07-03 Thread Niall Pemberton

On 7/3/07, sadasivarao [EMAIL PROTECTED] wrote:


Hi,
I am using struts and JSF using struts-faces 1.3.8.jar file and
using WSAD 5.1.2

My jsf page is like this

s:form action=/logon
h:commandbutton type=submit  value=ok
/s:form

As soon as I click on submit button blank page displaying. No
error it is showing it is populating ActionForm but it not invoking my
Action class

My struts-config.xml file is

!-- Form Beans --

  form-beans
form-bean name=logonForm type=example.LogonForm/
  /form-beans

  !-- Global Exceptions --
  global-exceptions
  /global-exceptions

  !-- Global Forwards --
  global-forwards
  /global-forwards

  !-- Action Mappings --
  action-mappings

  action path=/logon
  type=example.LogonAction
  name=logonForm
  input=login.faces
  scope=request
forward name=success path=/Success.faces/forward
forward name=failure path=/error.faces/forward
  /action

  /action-mappings

controller locale=true
  set-property property=processorClass
value=org.apache.struts.faces.application.FacesRequestProcessor/
  /controller

  Please help me it is urgent..


Please ask questions like this on the user mailing list - this list is
to discuss the development of Struts:

http://struts.apache.org/mail.html

Niall

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: struts-faces integration problem with existing struts based application?

2007-05-24 Thread Laurie Harper
Please post questions about using Struts to the Struts Users list only. 
The Struts Dev list is for discussing the development and enhancement of 
Struts itself.


http://struts.apache.org/mail.html

vijendra wrote:

I am trying to integrate struts-faces in my previous struts based
application.
i am unable to get required results.

the libraries i am using are:-(mainly)
commons-beanutils-1.7.0,
commons-chain-1.1,
commons-digester-1.8,
commons-lang-2.3,
struts-faces-1.3.8,
tomahawk-1.1.5,
log4j-1.2.8,
myfaces-api,
myfaces-impl,
struts,
struts-el-1.3.8,
struts-taglib-1.3.8,
struts-extras-1.3.8,

and many more.
i feel these are the responsible files which are really required to
be there.

Now what i did is configured web.xml with faces settings.
configured faces-config.xml.

when i modified web.xml for faces integration and calling a page from
some other page giving extention as faces to my new page either it
gives me exception.or sometimes it comes as many templets shown in
loop(one in other and so on:- useless) but original content is not
shown.
Exception is :
javax.faces.FacesException: javax.servlet.jsp.JspException:
org.apache.myfaces.context.servlet.ServletExternalContextImpl.dispatch(ServletExternalContextImpl.java:
422)

and if i try using /faces/* combination it dosen't comes out at all.
(In this case my doubt is its unable to find out the page itself)

But when i am calling a particular jsp page(reporthome.faces) which
was earlier normal jsp from index page.i modified this page with
struts-faces libraries with what i require.
but it first of all loads very slowly , sometime shows the same page
in looped condition so upper tags are getting displayed many times and
other data is gone.
and finally it comes up with exception
javax.faces.FacesException:
org.apache.myfaces.context.servlet.ServletExternalContextImpl.dispatch(ServletExternalContextImpl.java:
422)

this exception comes in many different situation but comes mostly.
so what is the reason behind.

please guide me. thanks

vijendra
-
Posted via Jive Forums
http://forums.opensymphony.com/thread.jspa?threadID=84385messageID=151283#151283



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: struts-faces integration problem with existing struts based application?

2007-05-21 Thread vijendra
I am trying to integrate struts-faces in my previous struts based
application.
i am unable to get required results.

the libraries i am using are:-(mainly)
commons-beanutils-1.7.0,
commons-chain-1.1,
commons-digester-1.8,
commons-lang-2.3,
struts-faces-1.3.8,
tomahawk-1.1.5,
log4j-1.2.8,
myfaces-api,
myfaces-impl,
struts,
struts-el-1.3.8,
struts-taglib-1.3.8,
struts-extras-1.3.8,

and many more.
i feel these are the responsible files which are really required to
be there.

Now what i did is configured web.xml with faces settings.
configured faces-config.xml.

when i modified web.xml for faces integration and calling a page from
some other page giving extention as faces to my new page either it
gives me exception.or sometimes it comes as many templets shown in
loop(one in other and so on:- useless) but original content is not
shown.
Exception is :
javax.faces.FacesException: javax.servlet.jsp.JspException:
org.apache.myfaces.context.servlet.ServletExternalContextImpl.dispatch(ServletExternalContextImpl.java:
422)

and if i try using /faces/* combination it dosen't comes out at all.
(In this case my doubt is its unable to find out the page itself)

But when i am calling a particular jsp page(reporthome.faces) which
was earlier normal jsp from index page.i modified this page with
struts-faces libraries with what i require.
but it first of all loads very slowly , sometime shows the same page
in looped condition so upper tags are getting displayed many times and
other data is gone.
and finally it comes up with exception
javax.faces.FacesException:
org.apache.myfaces.context.servlet.ServletExternalContextImpl.dispatch(ServletExternalContextImpl.java:
422)

this exception comes in many different situation but comes mostly.
so what is the reason behind.

please guide me. thanks

vijendra
-
Posted via Jive Forums
http://forums.opensymphony.com/thread.jspa?threadID=84385messageID=151283#151283


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



struts-faces integration problem with existing struts based application?

2007-05-18 Thread vijendra
i don't know what is wrong but i tried all he things  i can do to get the 
results.
http://www-128.ibm.com/developerworks/library/j-integrate/ 
is the first page i found useful in order to start.
the first sample application itself is having many flaws. not designed properly.
i tried many modification finally able to run upto some extent.
but even to completely run the application supporting libraries ViewHandler and 
tree, i am unable to find them anywhere.

now coming to my application integration part i am trying to modify web.xml for 
faces navigation where i used /faces/* and *.faces both options separately but 
nothing has given be desired results till now.
Eevn i tried posting my problem in some diff sites but till now unable to get 
the solution.
if either some can give me the real path with libraries to be used i may retry 
doing it again.
till now i am unable to dispaly the fist jsf(faces) page in my application.
i don't know how to prooceed now.
is this whole problem because of version mismatch.
 what i tried is here:-
http://forum.java.sun.com/thread.jspa?threadID=5173587

please guide me which way to go now.
-
Posted via Jive Forums
http://forums.opensymphony.com/thread.jspa?threadID=84385messageID=150592#150592


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[s1.3.6] Where to download struts-faces?

2006-11-27 Thread Paul Benedict

We have this site issue
https://issues.apache.org/struts/browse/STR-2926

The complaint is we don't have any download links for Struts-Faces. Now 
with the 1.3.5 release, struts-faces is part of the library. However, 
that wasn't the case with earlier releases.


Do we want to provide a link on the website anywhere to previous 
releases? Or consider this defect solved by 1.3.5?


Paul

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [s1.3.6] Where to download struts-faces?

2006-11-27 Thread Niall Pemberton

On 11/28/06, Paul Benedict [EMAIL PROTECTED] wrote:

We have this site issue
https://issues.apache.org/struts/browse/STR-2926

The complaint is we don't have any download links for Struts-Faces. Now
with the 1.3.5 release, struts-faces is part of the library. However,
that wasn't the case with earlier releases.

Do we want to provide a link on the website anywhere to previous
releases? Or consider this defect solved by 1.3.5?


I don't think we ever had a release of faces on its own. In the 1.2.x
series it was in the Sandbox (and distrubuted in the contrib folder
of the release). For 1.3.5 it had been merged back in under Struts1.
So I don't think we need a download link for it.

Niall


Paul


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [Struts-Faces] commandLink obsolete ?

2006-10-28 Thread Matthias Wessendorf

wow, it is still there :)



On 10/27/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:

commandLink was create in 2004, b/c of a RI bug.
I guess (or assume) that it has been fixed there.
works fine w/ MyFaces.

however inside trinidad/adf the commandLink will not work ...

-M

--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com




--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [Struts-Faces] commandLink obsolete ?

2006-10-28 Thread Matthias Wessendorf

the bug,

so can sb apply the patch ?


On 10/28/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:

wow, it is still there :)



On 10/27/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:
 commandLink was create in 2004, b/c of a RI bug.
 I guess (or assume) that it has been fixed there.
 works fine w/ MyFaces.

 however inside trinidad/adf the commandLink will not work ...

 -M

 --
 Matthias Wessendorf
 http://tinyurl.com/fmywh

 further stuff:
 blog: http://jroller.com/page/mwessendorf
 mail: mwessendorf-at-gmail-dot-com



--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com




--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [Struts-Faces] possible Bug in FormRenderer

2006-10-28 Thread Craig McClanahan

On 10/26/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:


Hey,

two things.

a) is this the right place to ask questions on Struts-Faces, or where ?



It's the right place for dev type questions ... the Struts User list for
user type questions :-).

if so b)


the demos of struts faces show a portal which uses links (done w/
JSF-components) to navigate to a tiles pages, which finally contains a
s:form based formular. that is the key... ActionForm/Action on the
java side and JSF comp.s on the page side.

Every thing works. A form-submit sends a req. to the action

So far so good...

When I am converting one of my own apps w/ struts-faces I'd like to
stay with html:link for linking to form pages (that are using
s:form). But ...



In other words, you want to use the Struts html:link tag in a form
surrounded by the JSF s:form component?  That's very likely not going to
work reliably.  You should stick to one technology or the other for all of
your view tier tags on any particular page -- either the Struts HTML tags or
the appropriate JSF component tags.  You are only scratching the surface of
a lot of problems that will show up trying to mix them.

Craig

since that is a GET request the form renders html like


form name=fooForm action=/blub/tilesMasterLayout.do ...


(only when using GET / html:link)

so ... a submit causes a 404. Not wondering :)

When I use firebug to change the action attr value to
/blub/tilesMasterLayout.faces every thing works. Action is called.


The action() method in FormRenderer returns in the wrong szenario a .do
URL
I think it is not to bad to check agains a struts pattern in that URL
and replace that .do
(if needed) with the jsf pattern (like .faces or add /faces/)

I think this is valid, because the form renders already fine, which
ensures you are inside the FacesContext ...

If you all agree, I'd like to provide the patch for that.

Greetz,
Matt


--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com



Re: [Struts-Faces] commandLink obsolete ?

2006-10-28 Thread Wendy Smoak

On 10/28/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:

the bug,
so can sb apply the patch ?

On 10/28/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:
 wow, it is still there :)


What patch?  If it's something in the Struts JIRA, add a comment to the issue.

--
Wendy

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [Struts-Faces] commandLink obsolete ?

2006-10-28 Thread Matthias Wessendorf

The bug is still in the RI
(1.1._02)

so s:commandLink is not obsolate.

the bug in s:commandLink is, that it is not working w/ Trinidad
http://issues.apache.org/struts/browse/STR-2966

-M

On 10/28/06, Wendy Smoak [EMAIL PROTECTED] wrote:

On 10/28/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:
 the bug,
 so can sb apply the patch ?

 On 10/28/06, Matthias Wessendorf [EMAIL PROTECTED] wrote:
  wow, it is still there :)

What patch?  If it's something in the Struts JIRA, add a comment to the issue.

--
Wendy

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[Struts-Faces] possible Bug in FormRenderer

2006-10-27 Thread Matthias Wessendorf

Hey,

two things.

a) is this the right place to ask questions on Struts-Faces, or where ?

if so b)

the demos of struts faces show a portal which uses links (done w/
JSF-components) to navigate to a tiles pages, which finally contains a
s:form based formular. that is the key... ActionForm/Action on the
java side and JSF comp.s on the page side.

Every thing works. A form-submit sends a req. to the action

So far so good...

When I am converting one of my own apps w/ struts-faces I'd like to
stay with html:link for linking to form pages (that are using
s:form). But ...

since that is a GET request the form renders html like

form name=fooForm action=/blub/tilesMasterLayout.do ...


(only when using GET / html:link)

so ... a submit causes a 404. Not wondering :)

When I use firebug to change the action attr value to
/blub/tilesMasterLayout.faces every thing works. Action is called.


The action() method in FormRenderer returns in the wrong szenario a .do URL
I think it is not to bad to check agains a struts pattern in that URL
and replace that .do
(if needed) with the jsf pattern (like .faces or add /faces/)

I think this is valid, because the form renders already fine, which
ensures you are inside the FacesContext ...

If you all agree, I'd like to provide the patch for that.

Greetz,
Matt


--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[Struts-Faces] commandLink obsolete ?

2006-10-27 Thread Matthias Wessendorf

commandLink was create in 2004, b/c of a RI bug.
I guess (or assume) that it has been fixed there.
works fine w/ MyFaces.

however inside trinidad/adf the commandLink will not work ...

-M

--
Matthias Wessendorf
http://tinyurl.com/fmywh

further stuff:
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [SAF1 - Struts-Faces] s:loadMessages / can't load the correct MessageResources in a multimodule webapp

2006-05-13 Thread A. Alonso Dominguez

Hi Craig,

I found also a similiar problem with the s:message / tag. I will open a
now issue about them two.

Alonso

2006/5/12, Craig McClanahan [EMAIL PROTECTED]:


On 5/12/06, A. Alonso Dominguez [EMAIL PROTECTED] wrote:

 Hi there,

 I think that there is a bug in the implementation of the 
 org.apache.struts.faces.taglib.LoadMessagesTag. When working in a
 multimodule webapp with several messages-resources / per module, the
 s:loadMessages / tag can't find the appropiate MessageResources
instance
 via messages attribute.

 The reason for this is the way that the tag uses to locate the
appropiate
 MessageResources instance. This code explains it (line 76):

 public int doStartTag() {

 // Acquire the Locale to be wrapped
 Locale locale =
 FacesContext.getCurrentInstance().getViewRoot().getLocale();

 // Acquire the MessageResources to be wrapped
 MessageResources messages = null;
 if (this.messages == null) {
 messages = (MessageResources)
 pageContext.getAttribute(Globals.MESSAGES_KEY,
  PageContext.REQUEST_SCOPE);
 if (messages == null) {
 messages = (MessageResources)
 pageContext.getAttribute(Globals.MESSAGES_KEY,

PageContext.APPLICATION_SCOPE
 );
 }
 } else {
 messages = (MessageResources)
 pageContext.getAttribute(this.messages,
  PageContext.REQUEST_SCOPE);
 }

 // Expose a Map instance under the specified request attribute
key
 pageContext.setAttribute(var,
  new MessagesMap(messages, locale),
  PageContext.APPLICATION_SCOPE);

 // Skip the body of this tag (if any)
 return (SKIP_BODY);

 }

 As you can see, this code doesn't supports modules.

 If we want to load a different bundle than the one configured as the
 default, we must use the messages attribute but this code tries to
load
 that MessageResources instance from the APPLICATION_SCOPE rather that
 search
 for it inside the REQUEST_SCOPE.

 In a multimodule webapp the MessageResources instances are loaded on
 startup
 inside the APPLICATION_SCOPE with the key:
 org.apache.struts.action/MESSAGE + [modulePrefix]

 If we have more than one MessageResources instance per module the will
be
 loaded with the key:
 [bundleKey] + [modulePrefix]


 So, the above code can't load any of those kind of messages resources.
 This
 can be solved easily:

 public int doStartTag() throws JspException {

 // Acquire the Locale to be wrapped
 Locale locale =
 FacesContext.getCurrentInstance().getViewRoot().getLocale();

 ModuleConfig modConfig = ModuleUtils.getInstance
 ().getModuleConfig(
 (HttpServletRequest) pageContext.getRequest(),
 pageContext.getServletContext());

 // Acquire the MessageResources to be wrapped
 MessageResources messages = null;
 if (this.messages == null) {
 messages = (MessageResources)
 pageContext.getAttribute(Globals.MESSAGES_KEY,
  PageContext.REQUEST_SCOPE);
 if (messages == null) {
 messages = (MessageResources)
 pageContext.getAttribute(Globals.MESSAGES_KEY +
 modConfig.getPrefix(),

PageContext.APPLICATION_SCOPE
 );
 }
 } else {
 messages = (MessageResources)
 pageContext.getAttribute(this.messages,
  PageContext.REQUEST_SCOPE);

 if(messages == null) {
 messages = (MessageResources)
 pageContext.getAttribute(this.messages +
 modConfig.getPrefix(),

PageContext.APPLICATION_SCOPE
 );
 }
 }

 if(messages == null)
 throw new JspException(MessageResources bundle  +
this.messages +  not
 found);

 // Expose a Map instance under the specified request attribute
key
 pageContext.setAttribute(var,
  new MessagesMap(messages, locale),
  PageContext.REQUEST_SCOPE);

 // Skip the body of this tag (if any)
 return (SKIP_BODY);

 }

 Should I open a new issue about this?


Yes please ... this is not an issue (pardon the pun :-) that I was aware
of.

http://issues.apache.org/struts/

Alonso

 Craig




[SAF1 - Struts-Faces] s:loadMessages / can't load the correct MessageResources in a multimodule webapp

2006-05-12 Thread A. Alonso Dominguez

Hi there,

I think that there is a bug in the implementation of the 
org.apache.struts.faces.taglib.LoadMessagesTag. When working in a
multimodule webapp with several messages-resources / per module, the
s:loadMessages / tag can't find the appropiate MessageResources instance
via messages attribute.

The reason for this is the way that the tag uses to locate the appropiate
MessageResources instance. This code explains it (line 76):

public int doStartTag() {

   // Acquire the Locale to be wrapped
   Locale locale =
   FacesContext.getCurrentInstance().getViewRoot().getLocale();

   // Acquire the MessageResources to be wrapped
   MessageResources messages = null;
   if (this.messages == null) {
   messages = (MessageResources)
   pageContext.getAttribute(Globals.MESSAGES_KEY,
PageContext.REQUEST_SCOPE);
   if (messages == null) {
   messages = (MessageResources)
   pageContext.getAttribute(Globals.MESSAGES_KEY,
PageContext.APPLICATION_SCOPE);
   }
   } else {
   messages = (MessageResources)
   pageContext.getAttribute(this.messages,
PageContext.REQUEST_SCOPE);
   }

   // Expose a Map instance under the specified request attribute key
   pageContext.setAttribute(var,
new MessagesMap(messages, locale),
PageContext.APPLICATION_SCOPE);

   // Skip the body of this tag (if any)
   return (SKIP_BODY);

   }

As you can see, this code doesn't supports modules.

If we want to load a different bundle than the one configured as the
default, we must use the messages attribute but this code tries to load
that MessageResources instance from the APPLICATION_SCOPE rather that search
for it inside the REQUEST_SCOPE.

In a multimodule webapp the MessageResources instances are loaded on startup
inside the APPLICATION_SCOPE with the key:
org.apache.struts.action/MESSAGE + [modulePrefix]

If we have more than one MessageResources instance per module the will be
loaded with the key:
[bundleKey] + [modulePrefix]


So, the above code can't load any of those kind of messages resources. This
can be solved easily:

public int doStartTag() throws JspException {

   // Acquire the Locale to be wrapped
   Locale locale =
   FacesContext.getCurrentInstance().getViewRoot().getLocale();

   ModuleConfig modConfig = ModuleUtils.getInstance().getModuleConfig(
   (HttpServletRequest) pageContext.getRequest(),
pageContext.getServletContext());

   // Acquire the MessageResources to be wrapped
   MessageResources messages = null;
   if (this.messages == null) {
   messages = (MessageResources)
   pageContext.getAttribute(Globals.MESSAGES_KEY,
PageContext.REQUEST_SCOPE);
   if (messages == null) {
   messages = (MessageResources)
   pageContext.getAttribute(Globals.MESSAGES_KEY +
modConfig.getPrefix(),
PageContext.APPLICATION_SCOPE);
   }
   } else {
   messages = (MessageResources)
   pageContext.getAttribute(this.messages,
PageContext.REQUEST_SCOPE);

   if(messages == null) {
   messages = (MessageResources)
   pageContext.getAttribute(this.messages +
modConfig.getPrefix(),
   PageContext.APPLICATION_SCOPE);
   }
   }

   if(messages == null)
   throw new JspException(MessageResources bundle  +
  this.messages +  not
found);

   // Expose a Map instance under the specified request attribute key
   pageContext.setAttribute(var,
new MessagesMap(messages, locale),
PageContext.REQUEST_SCOPE);

   // Skip the body of this tag (if any)
   return (SKIP_BODY);

   }

Should I open a new issue about this?

Alonso


Re: [SAF1 - Struts-Faces] s:loadMessages / can't load the correct MessageResources in a multimodule webapp

2006-05-12 Thread Craig McClanahan

On 5/12/06, A. Alonso Dominguez [EMAIL PROTECTED] wrote:


Hi there,

I think that there is a bug in the implementation of the 
org.apache.struts.faces.taglib.LoadMessagesTag. When working in a
multimodule webapp with several messages-resources / per module, the
s:loadMessages / tag can't find the appropiate MessageResources instance
via messages attribute.

The reason for this is the way that the tag uses to locate the appropiate
MessageResources instance. This code explains it (line 76):

public int doStartTag() {

// Acquire the Locale to be wrapped
Locale locale =
FacesContext.getCurrentInstance().getViewRoot().getLocale();

// Acquire the MessageResources to be wrapped
MessageResources messages = null;
if (this.messages == null) {
messages = (MessageResources)
pageContext.getAttribute(Globals.MESSAGES_KEY,
 PageContext.REQUEST_SCOPE);
if (messages == null) {
messages = (MessageResources)
pageContext.getAttribute(Globals.MESSAGES_KEY,
 PageContext.APPLICATION_SCOPE
);
}
} else {
messages = (MessageResources)
pageContext.getAttribute(this.messages,
 PageContext.REQUEST_SCOPE);
}

// Expose a Map instance under the specified request attribute key
pageContext.setAttribute(var,
 new MessagesMap(messages, locale),
 PageContext.APPLICATION_SCOPE);

// Skip the body of this tag (if any)
return (SKIP_BODY);

}

As you can see, this code doesn't supports modules.

If we want to load a different bundle than the one configured as the
default, we must use the messages attribute but this code tries to load
that MessageResources instance from the APPLICATION_SCOPE rather that
search
for it inside the REQUEST_SCOPE.

In a multimodule webapp the MessageResources instances are loaded on
startup
inside the APPLICATION_SCOPE with the key:
org.apache.struts.action/MESSAGE + [modulePrefix]

If we have more than one MessageResources instance per module the will be
loaded with the key:
[bundleKey] + [modulePrefix]


So, the above code can't load any of those kind of messages resources.
This
can be solved easily:

public int doStartTag() throws JspException {

// Acquire the Locale to be wrapped
Locale locale =
FacesContext.getCurrentInstance().getViewRoot().getLocale();

ModuleConfig modConfig = ModuleUtils.getInstance
().getModuleConfig(
(HttpServletRequest) pageContext.getRequest(),
pageContext.getServletContext());

// Acquire the MessageResources to be wrapped
MessageResources messages = null;
if (this.messages == null) {
messages = (MessageResources)
pageContext.getAttribute(Globals.MESSAGES_KEY,
 PageContext.REQUEST_SCOPE);
if (messages == null) {
messages = (MessageResources)
pageContext.getAttribute(Globals.MESSAGES_KEY +
modConfig.getPrefix(),
 PageContext.APPLICATION_SCOPE
);
}
} else {
messages = (MessageResources)
pageContext.getAttribute(this.messages,
 PageContext.REQUEST_SCOPE);

if(messages == null) {
messages = (MessageResources)
pageContext.getAttribute(this.messages +
modConfig.getPrefix(),
PageContext.APPLICATION_SCOPE
);
}
}

if(messages == null)
throw new JspException(MessageResources bundle  +
   this.messages +  not
found);

// Expose a Map instance under the specified request attribute key
pageContext.setAttribute(var,
 new MessagesMap(messages, locale),
 PageContext.REQUEST_SCOPE);

// Skip the body of this tag (if any)
return (SKIP_BODY);

}

Should I open a new issue about this?



Yes please ... this is not an issue (pardon the pun :-) that I was aware of.

http://issues.apache.org/struts/

Alonso


Craig


Re: Struts-Faces: Is this a bug?

2006-05-11 Thread A. Alonso Dominguez

A created a new issue about this bug:

http://issues.apache.org/struts/browse/STR-2873

2006/5/11, Craig McClanahan [EMAIL PROTECTED]:


On 5/9/06, A. Alonso Dominguez [EMAIL PROTECTED] wrote:

 Hi there!

 I'm trying to use the new struts-action framework, version 1.3.3 with
 tiles
 and faces. I'm following the example applications but I found possibly a
 bug
 when rendering a struts-based FormBean.

 The jsp code that I was using is as follows:

 s:errors /
 s:form action=/login id=login styleClass=loginForm
 focus=username
 h:panelGrid columns=2
 f:facet name=header
 h:panelGroup
 h:graphicImage
 url=/common/skin/social-labs/img/circle1.png
 /
 s:message key=page.logon.form
 f:param name=arg0 value=Social Labs /
 /s:message
 /h:panelGroup
 /f:facet
 h:outputLabel for=username
 s:message key=form.logon.username /
 /h:outputLabel
 h:inputText id=username size=25 value=#{logonForm.username
}
 /

 h:outputLabel for=password
 s:message filter=false key=form.logon.password /
 /h:outputLabel
 h:inputSecret id=password size=25 value=#{
logonForm.password}
 /

 f:facet name=footer
 h:commandButton id=submit type=SUBMIT styleClass=img
 image=/common/skin/social-labs/img/login.png /
 /f:facet
 /h:panelGrid
 /s:form

 but when it's rendered to the view, the HTML form's action attribute
 references /common/skin/default/layout/base.do. There is not any
action
 in
 my application with that url. Instead there is a file
 /common/skin/default/layout/base.jsp which is the base file for the
 tiles
 definitions:

 definition name=base.layout
 page=/common/skin/default/layout/base.jsp
 put name=title value= /
 put name=navigation value= /
 put name=body value= /
 putList name=scripts
 add value= /
 /putList
 /definition

 I dived inside the org.apache.struts.faces.renderer.FormRenderer code
 and
 I found this code (line 280):

 protected String action(FacesContext context, UIComponent component) {
 String actionURL =
 context.getApplication().getViewHandler().
 getActionURL(context, context.getViewRoot().getViewId());
 if (log.isTraceEnabled()) {
 log.trace(getActionURL( + context.getViewRoot
().getViewId()
 +
   ) --  + actionURL);
 }
 return (context.getExternalContext
().encodeActionURL(actionURL));

 }


 which really does nothing with the actionURL, It just takes de
ViewRootId
 and returns it to the caller.
 Playing with this code I have now the following:

 protected String action(FacesContext context, UIComponent component,
 ModuleConfig moduleConfig) {
 //String actionURL =
 //context.getApplication ().getViewHandler().
 //getActionURL(context, context.getViewRoot().getViewId());
 FormComponent form = (FormComponent) component;
 String actionURL = context.getApplication().getViewHandler().
   getActionURL(context, moduleConfig.getPrefix() +
 form.getAction());
 if (log.isTraceEnabled()) {
 log.trace(getActionURL( + context.getViewRoot
().getViewId()
 +
   ) --  + actionURL);
 }
 return (context.getExternalContext
().encodeActionURL(actionURL));

 }

 I don't know if this is the correct way for retrieving the real
actionURL
 but It works now and I'm using it.

 Should this be posted as a bug??

 Yes, this does sound like a bug.  Could you please file an issue on it,
using project name Struts Action 1 and component Faces?

  http://issues.apache.org/struts/

Craig




Re: Struts-Faces: Is this a bug?

2006-05-10 Thread Craig McClanahan

On 5/9/06, A. Alonso Dominguez [EMAIL PROTECTED] wrote:


Hi there!

I'm trying to use the new struts-action framework, version 1.3.3 with
tiles
and faces. I'm following the example applications but I found possibly a
bug
when rendering a struts-based FormBean.

The jsp code that I was using is as follows:

s:errors /
s:form action=/login id=login styleClass=loginForm
focus=username
h:panelGrid columns=2
f:facet name=header
h:panelGroup
h:graphicImage
url=/common/skin/social-labs/img/circle1.png
/
s:message key=page.logon.form
f:param name=arg0 value=Social Labs /
/s:message
/h:panelGroup
/f:facet
h:outputLabel for=username
s:message key=form.logon.username /
/h:outputLabel
h:inputText id=username size=25 value=#{logonForm.username}
/

h:outputLabel for=password
s:message filter=false key=form.logon.password /
/h:outputLabel
h:inputSecret id=password size=25 value=#{logonForm.password}
/

f:facet name=footer
h:commandButton id=submit type=SUBMIT styleClass=img
image=/common/skin/social-labs/img/login.png /
/f:facet
/h:panelGrid
/s:form

but when it's rendered to the view, the HTML form's action attribute
references /common/skin/default/layout/base.do. There is not any action
in
my application with that url. Instead there is a file
/common/skin/default/layout/base.jsp which is the base file for the
tiles
definitions:

definition name=base.layout
page=/common/skin/default/layout/base.jsp
put name=title value= /
put name=navigation value= /
put name=body value= /
putList name=scripts
add value= /
/putList
/definition

I dived inside the org.apache.struts.faces.renderer.FormRenderer code
and
I found this code (line 280):

protected String action(FacesContext context, UIComponent component) {
String actionURL =
context.getApplication().getViewHandler().
getActionURL(context, context.getViewRoot().getViewId());
if (log.isTraceEnabled()) {
log.trace(getActionURL( + context.getViewRoot().getViewId()
+
  ) --  + actionURL);
}
return (context.getExternalContext().encodeActionURL(actionURL));

}


which really does nothing with the actionURL, It just takes de ViewRootId
and returns it to the caller.
Playing with this code I have now the following:

protected String action(FacesContext context, UIComponent component,
ModuleConfig moduleConfig) {
//String actionURL =
//context.getApplication ().getViewHandler().
//getActionURL(context, context.getViewRoot().getViewId());
FormComponent form = (FormComponent) component;
String actionURL = context.getApplication().getViewHandler().
  getActionURL(context, moduleConfig.getPrefix() +
form.getAction());
if (log.isTraceEnabled()) {
log.trace(getActionURL( + context.getViewRoot().getViewId()
+
  ) --  + actionURL);
}
return (context.getExternalContext().encodeActionURL(actionURL));

}

I don't know if this is the correct way for retrieving the real actionURL
but It works now and I'm using it.

Should this be posted as a bug??

Yes, this does sound like a bug.  Could you please file an issue on it,

using project name Struts Action 1 and component Faces?

 http://issues.apache.org/struts/

Craig


Struts-Faces: Is this a bug?

2006-05-09 Thread A. Alonso Dominguez

Hi there!

I'm trying to use the new struts-action framework, version 1.3.3 with tiles
and faces. I'm following the example applications but I found possibly a bug
when rendering a struts-based FormBean.

The jsp code that I was using is as follows:

s:errors /
s:form action=/login id=login styleClass=loginForm focus=username
   h:panelGrid columns=2
   f:facet name=header
   h:panelGroup
   h:graphicImage url=/common/skin/social-labs/img/circle1.png
/
   s:message key=page.logon.form
   f:param name=arg0 value=Social Labs /
   /s:message
   /h:panelGroup
   /f:facet
   h:outputLabel for=username
   s:message key=form.logon.username /
   /h:outputLabel
   h:inputText id=username size=25 value=#{logonForm.username}
/

   h:outputLabel for=password
   s:message filter=false key=form.logon.password /
   /h:outputLabel
   h:inputSecret id=password size=25 value=#{logonForm.password} /

   f:facet name=footer
   h:commandButton id=submit type=SUBMIT styleClass=img
image=/common/skin/social-labs/img/login.png /
   /f:facet
   /h:panelGrid
/s:form

but when it's rendered to the view, the HTML form's action attribute
references /common/skin/default/layout/base.do. There is not any action in
my application with that url. Instead there is a file
/common/skin/default/layout/base.jsp which is the base file for the tiles
definitions:

definition name=base.layout page=/common/skin/default/layout/base.jsp
   put name=title value= /
   put name=navigation value= /
   put name=body value= /
   putList name=scripts
   add value= /
   /putList
   /definition

I dived inside the org.apache.struts.faces.renderer.FormRenderer code and
I found this code (line 280):

protected String action(FacesContext context, UIComponent component) {
   String actionURL =
   context.getApplication().getViewHandler().
   getActionURL(context, context.getViewRoot().getViewId());
   if (log.isTraceEnabled()) {
   log.trace(getActionURL( + context.getViewRoot().getViewId() +
 ) --  + actionURL);
   }
   return (context.getExternalContext().encodeActionURL(actionURL));

   }


which really does nothing with the actionURL, It just takes de ViewRootId
and returns it to the caller.
Playing with this code I have now the following:

protected String action(FacesContext context, UIComponent component,
ModuleConfig moduleConfig) {
   //String actionURL =
   //context.getApplication ().getViewHandler().
   //getActionURL(context, context.getViewRoot().getViewId());
   FormComponent form = (FormComponent) component;
   String actionURL = context.getApplication().getViewHandler().
 getActionURL(context, moduleConfig.getPrefix() +
form.getAction());
   if (log.isTraceEnabled()) {
   log.trace(getActionURL( + context.getViewRoot().getViewId() +
 ) --  + actionURL);
   }
   return (context.getExternalContext().encodeActionURL(actionURL));

   }

I don't know if this is the correct way for retrieving the real actionURL
but It works now and I'm using it.

Should this be posted as a bug??


[jira] Moved: (SHALE-91) [struts-faces] Example application doesn't deploy

2006-04-26 Thread Craig McClanahan (JIRA)
 [ http://issues.apache.org/struts/browse/SHALE-91?page=all ]

Craig McClanahan moved STR-2792 to SHALE-91:


  Project: Shale  (was: Struts Action 1)
  Key: SHALE-91  (was: STR-2792)
Component: (was: Faces)
  Version: (was: Unknown)
Assign To: (was: Struts Developer Mailing List)

 [struts-faces] Example application doesn't deploy
 -

  Key: SHALE-91
  URL: http://issues.apache.org/struts/browse/SHALE-91
  Project: Shale
 Type: Bug

  Environment: Operating System: other
 Platform: Other
 Reporter: Roland
 Priority: Critical


 Hello,
 I'm running Apache Tomcat/4.1.31
 And java version 1.4.2_06 on Linux.
 I'm trying to deploy the example applications which come with
 struts-faces-20060313 nightly but I get the following errors:
 2006-03-13 14:45:44 WebappLoader[/struts-faces-example2]: Deploy JAR
 /WEB-INF/lib/struts-faces.jar to
 /var/roland/CATALINA_BASE/webapps/struts-faces-example2/WEB-INF/lib/struts-faces.jar
 2006-03-13 14:45:44 WebappLoader[/struts-faces-example2]: Deploy JAR
 /WEB-INF/lib/struts.jar to
 /var/roland/CATALINA_BASE/webapps/struts-faces-example2/WEB-INF/lib/struts.jar
 2006-03-13 14:45:44 StandardManager[/struts-faces-example2]: Seeding random
 number generator class java.security.SecureRandom
 2006-03-13 14:45:44 StandardManager[/struts-faces-example2]: Seeding of random
 number generator has been completed
 2006-03-13 14:45:46 StandardContext[/struts-faces-example2]: Exception sending
 context initialized event to listener instance of class
 com.sun.faces.config.ConfigureListener
 java.lang.UnsupportedClassVersionError:
 org/apache/struts/faces/application/ActionListenerImpl (Unsupported 
 major.minor
 version 49.0)
   at java.lang.ClassLoader.defineClass0(Native Method)
   at java.lang.ClassLoader.defineClass(ClassLoader.java:539)
   at 
 java.security.SecureClassLoader.defineClass(SecureClassLoader.java:123)
   at
 org.apache.catalina.loader.WebappClassLoader.findClassInternal(WebappClassLoader.java:1649)
   at
 org.apache.catalina.loader.WebappClassLoader.findClass(WebappClassLoader.java:931)
   at
 org.apache.catalina.loader.WebappClassLoader.loadClass(WebappClassLoader.java:1373)
   at
 org.apache.catalina.loader.WebappClassLoader.loadClass(WebappClassLoader.java:1252)
   at com.sun.faces.util.Util.loadClass(Util.java:403)
   at com.sun.faces.util.Util.createInstance(Util.java:1150)
   at 
 com.sun.faces.config.ConfigureListener.configure(ConfigureListener.java:460)
   at 
 com.sun.faces.config.ConfigureListener.configure(ConfigureListener.java:402)
   at
 com.sun.faces.config.ConfigureListener.contextInitialized(ConfigureListener.java:332)
   at
 org.apache.catalina.core.StandardContext.listenerStart(StandardContext.java:3212)
   at 
 org.apache.catalina.core.StandardContext.start(StandardContext.java:3554)
   at 
 org.apache.catalina.core.ContainerBase.addChildInternal(ContainerBase.java:774)
   at 
 org.apache.catalina.core.ContainerBase.addChild(ContainerBase.java:760)
   at org.apache.catalina.core.StandardHost.addChild(StandardHost.java:548)
   at
 org.apache.catalina.core.StandardHostDeployer.install(StandardHostDeployer.java:260)
   at org.apache.catalina.core.StandardHost.install(StandardHost.java:741)
   at 
 org.apache.catalina.startup.HostConfig.deployWARs(HostConfig.java:445)
   at 
 org.apache.catalina.startup.HostConfig.deployApps(HostConfig.java:353)
   at org.apache.catalina.startup.HostConfig.start(HostConfig.java:671)
   at 
 org.apache.catalina.startup.HostConfig.lifecycleEvent(HostConfig.java:311)
   at
 org.apache.catalina.util.LifecycleSupport.fireLifecycleEvent(LifecycleSupport.java:119)
   at org.apache.catalina.core.ContainerBase.start(ContainerBase.java:1149)
   at org.apache.catalina.core.StandardHost.start(StandardHost.java:707)
   at org.apache.catalina.core.ContainerBase.start(ContainerBase.java:1141)
   at 
 org.apache.catalina.core.StandardEngine.start(StandardEngine.java:316)
   at 
 org.apache.catalina.core.StandardService.start(StandardService.java:450)
   at 
 org.apache.catalina.core.StandardServer.start(StandardServer.java:2143)
   at org.apache.catalina.startup.Catalina.start(Catalina.java:463)
   at org.apache.catalina.startup.Catalina.execute(Catalina.java:350)
   at org.apache.catalina.startup.Catalina.process(Catalina.java:129)
   at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
   at 
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
   at
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
   at java.lang.reflect.Method.invoke(Method.java:324)
   at org.apache.catalina.startup.Bootstrap.main(Bootstrap.java:156)
 2006-03-13 14:45:46

Re: Struts Faces - what to do?

2006-04-14 Thread Dakota Jack
Isn't the rule that this should have [faces] or something like that in the
subject matter?

On 4/13/06, James Mitchell [EMAIL PROTECTED] wrote:

 Perhaps Craig can shed some light on this, but are we going to
 continue to support the struts-faces project?  If so, can someone
 please fix the faces example (1 and 2) apps?

 --
 James Mitchell





 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




--
You can lead a horse to water but you cannot make it float on its back.
~Dakota Jack~


Re: Struts Faces - what to do?

2006-04-14 Thread Craig McClanahan
On 4/14/06, Dakota Jack [EMAIL PROTECTED] wrote:

 Isn't the rule that this should have [faces] or something like that in the
 subject matter?


It is a convention, not a rule.  And it is more for the user list than the
dev list -- people subscribed here are presumed to have a clue :-).

Craig


Re: Struts Faces - what to do?

2006-04-13 Thread Craig McClanahan
On 4/13/06, James Mitchell [EMAIL PROTECTED] wrote:

 Perhaps Craig can shed some light on this, but are we going to
 continue to support the struts-faces project?


We should.  There will be people who want to have a few JSF components on a
few pages (or they want to migrate) -- and using this library is the right
strategy for them.  Don was also talking about porting this to action2,
which I also support.

  If so, can someone
 please fix the faces example (1 and 2) apps?


Has the source organization and the build scripts settled down now?  If so,
I can devote some time to this ... but not until the end of next week.  One
more long trip first (St. Petersburg, Russia).

--
 James Mitchell


Craig


Re: Struts Faces - what to do?

2006-04-13 Thread James Mitchell
Yes, I believe things have settled down.  Although I'm still working  
changes in the struts-scripting webapp.


The error I was getting when deploying the faces-example1 app was:
javax.faces.FacesException: Undefined component type  
org.apache.struts.faces.Html


I was under the assumption that the struts-faces library only buys  
someone partial compatibility.  And what they get was only jsf-1.0  
compatible anyway, but I will be happy to be corrected.



--
James Mitchell




On Apr 14, 2006, at 12:14 AM, Craig McClanahan wrote:


On 4/13/06, James Mitchell [EMAIL PROTECTED] wrote:


Perhaps Craig can shed some light on this, but are we going to
continue to support the struts-faces project?



We should.  There will be people who want to have a few JSF  
components on a
few pages (or they want to migrate) -- and using this library is  
the right
strategy for them.  Don was also talking about porting this to  
action2,

which I also support.

  If so, can someone

please fix the faces example (1 and 2) apps?



Has the source organization and the build scripts settled down  
now?  If so,
I can devote some time to this ... but not until the end of next  
week.  One

more long trip first (St. Petersburg, Russia).

--

James Mitchell



Craig



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Struts Faces - what to do?

2006-04-13 Thread Craig McClanahan
On 4/13/06, James Mitchell [EMAIL PROTECTED] wrote:

 Yes, I believe things have settled down.  Although I'm still working
 changes in the struts-scripting webapp.

 The error I was getting when deploying the faces-example1 app was:
 javax.faces.FacesException: Undefined component type
 org.apache.struts.faces.Html


Sounds like a jar file is missing from the webapp ... not all that
surprising giving all the changes going on.  I'll take a look.

I was under the assumption that the struts-faces library only buys
 someone partial compatibility.  And what they get was only jsf-1.0
 compatible anyway, but I will be happy to be corrected.


It buys you either JSF 1.0 or 1.1 or 1.2 ... but, more importantly, it buys
you the ability to use the UI components part of JSF even if you don't want
to use the controller part.  For example, it has full integration with
Commons Validator and Tiles, but uses JSF component tags instead of Struts
HTML tags.

--
 James Mitchell


Craig


svn commit: r393833 - in /struts/action/trunk/faces: ./ core-library/ core-library/src/ src/ systest1-webapp/src/java/org/apache/struts/faces/systest1/

2006-04-13 Thread jmitchell
Author: jmitchell
Date: Thu Apr 13 08:31:04 2006
New Revision: 393833

URL: http://svn.apache.org/viewcvs?rev=393833view=rev
Log:
Fixing Faces library and sample apps (Part 1)
 * moving src back under core-library
 * moving pom.xml back under core-library, the pom at the root of faces should 
be similar to the one at the root of apps, which simply delegates to it's 
children (modules)
 * change ActionError usage to ActionMessage

Added:
struts/action/trunk/faces/core-library/pom.xml
  - copied unchanged from r393817, struts/action/trunk/faces/pom.xml
struts/action/trunk/faces/core-library/src/
  - copied from r393785, struts/action/trunk/faces/src/
Removed:
struts/action/trunk/faces/pom.xml
struts/action/trunk/faces/src/
Modified:

struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/ContextAction.java

struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/LogonAction.java

Modified: 
struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/ContextAction.java
URL: 
http://svn.apache.org/viewcvs/struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/ContextAction.java?rev=393833r1=393832r2=393833view=diff
==
--- 
struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/ContextAction.java
 (original)
+++ 
struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/ContextAction.java
 Thu Apr 13 08:31:04 2006
@@ -29,7 +29,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.action.ActionError;
 import org.apache.struts.action.ActionErrors;
 
 

Modified: 
struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/LogonAction.java
URL: 
http://svn.apache.org/viewcvs/struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/LogonAction.java?rev=393833r1=393832r2=393833view=diff
==
--- 
struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/LogonAction.java
 (original)
+++ 
struts/action/trunk/faces/systest1-webapp/src/java/org/apache/struts/faces/systest1/LogonAction.java
 Thu Apr 13 08:31:04 2006
@@ -29,8 +29,9 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.action.ActionError;
 import org.apache.struts.action.ActionErrors;
+import org.apache.struts.action.ActionMessage;
+import org.apache.struts.action.ActionMessages;
 
 
 /**
@@ -60,21 +61,21 @@
 PropertyUtils.getSimpleProperty(form, username);
 if ((username == null) || (.equals(username))) {
 errors.add(username,
- new ActionError(logon.username));
+ new ActionMessage(logon.username));
 }
 String password = (String)
 PropertyUtils.getSimpleProperty(form, password);
 if ((password == null) || (.equals(password))) {
 errors.add(password,
- new ActionError(logon.password));
+ new ActionMessage(logon.password));
 }
 if (log.isTraceEnabled()) {
 log.trace(username= + username + ,password= + password);
 }
 if (errors.isEmpty() 
 (!gooduser.equals(username) || !goodpass.equals(password))) {
-errors.add(ActionErrors.GLOBAL_ERROR,
- new ActionError(logon.mismatch));
+errors.add(ActionMessages.GLOBAL_MESSAGE,
+ new ActionMessage(logon.mismatch));
 }
 if (errors.isEmpty()) {
 if (log.isDebugEnabled()) {



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Struts Faces - what to do?

2006-04-13 Thread James Mitchell
Perhaps Craig can shed some light on this, but are we going to  
continue to support the struts-faces project?  If so, can someone  
please fix the faces example (1 and 2) apps?


--
James Mitchell





-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r392070 - in /struts/action/trunk: ./ faces/core-library/src/ faces/src/ faces/src/conf/ faces/src/java/ faces/src/java/org/apache/struts/faces/application/ faces/src/test/

2006-04-06 Thread mrdon
Author: mrdon
Date: Thu Apr  6 13:54:30 2006
New Revision: 392070

URL: http://svn.apache.org/viewcvs?rev=392070view=rev
Log:
Adding faces and scripting to the module list, starting to restructure
faces, adding a maven 2 build for faces which works and all tests pass,
made faces work with latest action although backwards compatibility
might be compromised due the use of ModuleUtils and
InvalidCancelException.  This should be addressed if we plan to keep
support for old versions of Struts Action 1.

Added:
struts/action/trunk/faces/src/
  - copied from r392045, struts/action/trunk/faces/core-library/src/
struts/action/trunk/faces/src/conf/
  - copied from r392064, struts/action/trunk/faces/core-library/src/conf/
struts/action/trunk/faces/src/java/
  - copied from r392064, struts/action/trunk/faces/core-library/src/java/
struts/action/trunk/faces/src/test/
  - copied from r392064, struts/action/trunk/faces/core-library/src/test/
Removed:
struts/action/trunk/faces/core-library/src/
Modified:

struts/action/trunk/faces/src/java/org/apache/struts/faces/application/ActionListenerImpl.java

struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
struts/action/trunk/pom.xml

Modified: 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/ActionListenerImpl.java
URL: 
http://svn.apache.org/viewcvs/struts/action/trunk/faces/src/java/org/apache/struts/faces/application/ActionListenerImpl.java?rev=392070r1=392064r2=392070view=diff
==
--- 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/ActionListenerImpl.java
 (original)
+++ 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/ActionListenerImpl.java
 Thu Apr  6 13:54:30 2006
@@ -36,6 +36,7 @@
 import org.apache.struts.faces.Constants;
 import org.apache.struts.faces.component.FormComponent;
 import org.apache.struts.util.RequestUtils;
+import org.apache.struts.util.ModuleUtils;
 
 
 /**
@@ -152,7 +153,7 @@
 // Invoke the appropriate request processor for this request
 try {
 request.setAttribute(Constants.ACTION_EVENT_KEY, event);
-RequestUtils.selectModule(request, servletContext);
+ModuleUtils.getInstance().selectModule(request, servletContext);
 ModuleConfig moduleConfig = (ModuleConfig)
 request.getAttribute(Globals.MODULE_KEY);
 if (log.isTraceEnabled()) {

Modified: 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
URL: 
http://svn.apache.org/viewcvs/struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java?rev=392070r1=392064r2=392070view=diff
==
--- 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
 (original)
+++ 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
 Thu Apr  6 13:54:30 2006
@@ -38,6 +38,7 @@
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
 import org.apache.struts.action.RequestProcessor;
+import org.apache.struts.action.InvalidCancelException;
 import org.apache.struts.config.FormBeanConfig;
 import org.apache.struts.config.ForwardConfig;
 import org.apache.struts.faces.Constants;
@@ -403,7 +404,7 @@
   HttpServletResponse response,
   ActionForm form,
   ActionMapping mapping)
-throws IOException, ServletException {
+throws IOException, ServletException, InvalidCancelException {
 
 if (log.isTraceEnabled()) {
 log.trace(Performing standard validation);

Modified: struts/action/trunk/pom.xml
URL: 
http://svn.apache.org/viewcvs/struts/action/trunk/pom.xml?rev=392070r1=392069r2=392070view=diff
==
--- struts/action/trunk/pom.xml (original)
+++ struts/action/trunk/pom.xml Thu Apr  6 13:54:30 2006
@@ -273,6 +273,8 @@
 moduleel/module
 moduleextras/module
 moduletaglib/module
+modulefaces/module
+modulescripting/module
 !--moduleapps/module--
 /modules
 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r392072 - in /struts/action/trunk/faces/src/java/org/apache/struts/faces/application: FacesTilesRequestProcessor.java FacesTilesRequestProcessor.java.bak

2006-04-06 Thread mrdon
Author: mrdon
Date: Thu Apr  6 14:00:43 2006
New Revision: 392072

URL: http://svn.apache.org/viewcvs?rev=392072view=rev
Log:
Moving the tiles version of the Faces request processor out of the
build, until we determine what to do with it

Added:

struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java.bak
  - copied unchanged from r392071, 
struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
Removed:

struts/action/trunk/faces/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r389368 - in /struts/faces/trunk: core-library/build.xml default.properties example1-webapp/build.xml example2-webapp/build.xml sysclient-app/build.xml systest1-webapp/build.xml

2006-03-27 Thread wsmoak
Author: wsmoak
Date: Mon Mar 27 18:43:25 2006
New Revision: 389368

URL: http://svn.apache.org/viewcvs?rev=389368view=rev
Log:
Add source and target attributes to all javac tasks to target JDK 1.3.
Both example webapps now deploy properly in Tomcat 4.1.31 using JDK 1.4.2.
Bug: 38955
Reported By: Roland roland at gisplan.com.br

Added:
struts/faces/trunk/default.properties   (with props)
Modified:
struts/faces/trunk/core-library/build.xml
struts/faces/trunk/example1-webapp/build.xml
struts/faces/trunk/example2-webapp/build.xml
struts/faces/trunk/sysclient-app/build.xml
struts/faces/trunk/systest1-webapp/build.xml

Modified: struts/faces/trunk/core-library/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/build.xml?rev=389368r1=389367r2=389368view=diff
==
--- struts/faces/trunk/core-library/build.xml (original)
+++ struts/faces/trunk/core-library/build.xml Mon Mar 27 18:43:25 2006
@@ -29,6 +29,7 @@
   property file=build.properties/
   property file=../build.properties/
   property file=../../build.properties/
+  property file=../default.properties/
   property file=${user.home}/build.properties/
 
 
@@ -186,7 +187,9 @@
destdir=${build.home}/classes
  debug=${compile.debug}
deprecation=${compile.deprecation}
-  optimize=${compile.optimize}
+  optimize=${compile.optimize}
+source=${platform.source}
+target=${platform.target}
   classpath refid=compile.classpath /
 /javac
 
@@ -296,7 +299,9 @@
destdir=${build.home}/test-classes
  debug=${compile.debug}
deprecation=${compile.deprecation}
-  optimize=${compile.optimize}
+  optimize=${compile.optimize}
+source=${platform.source}
+target=${platform.target}
   classpath refid=test.classpath /
 /javac
 

Added: struts/faces/trunk/default.properties
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/default.properties?rev=389368view=auto
==
--- struts/faces/trunk/default.properties (added)
+++ struts/faces/trunk/default.properties Mon Mar 27 18:43:25 2006
@@ -0,0 +1,25 @@
+# default.properties
+# ---
+#
+# 
==
+# Copyright 2006 The Apache Software Foundation.
+# 
+# Licensed under the Apache License, Version 2.0 (the License);
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+# 
+#  http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an AS IS BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+# 
==
+
+# Java platform version which source code must conform to
+platform.source=1.3
+
+# Java platform version which generated class files must conform to
+platform.target=1.3
+

Propchange: struts/faces/trunk/default.properties
--
svn:eol-style = native

Modified: struts/faces/trunk/example1-webapp/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example1-webapp/build.xml?rev=389368r1=389367r2=389368view=diff
==
--- struts/faces/trunk/example1-webapp/build.xml (original)
+++ struts/faces/trunk/example1-webapp/build.xml Mon Mar 27 18:43:25 2006
@@ -29,6 +29,7 @@
   property file=build.properties/
   property file=../build.properties/
   property file=../../build.properties/
+  property file=../default.properties/
   property file=${user.home}/build.properties/
 
 
@@ -272,7 +273,9 @@
destdir=${build.home}/${context.path}/WEB-INF/classes
  debug=${compile.debug}
deprecation=${compile.deprecation}
-  optimize=${compile.optimize}
+  optimize=${compile.optimize}
+source=${platform.source}
+target=${platform.target}
   classpath refid=compile.classpath /
 /javac
 

Modified: struts/faces/trunk/example2-webapp/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/build.xml?rev=389368r1=389367r2=389368view=diff
==
--- struts/faces/trunk/example2-webapp/build.xml (original)
+++ struts/faces/trunk/example2-webapp/build.xml Mon Mar 27 18:43:25 2006
@@ -29,6 +29,7 @@
   property file=build.properties

DO NOT REPLY [Bug 38955] - [struts-faces] Example application doesn't deploy

2006-03-27 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38955.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38955


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2006-03-28 03:48 ---

Fixed in r389368 for the 20060328 nightly build.

   http://svn.apache.org/viewcvs?rev=389368view=rev

I added source and target attributes to all the javac tasks, so we should now be
targetting 1.3 for all the binaries.  Both examples deploy and work in Tomcat
4.1.31 w/ JDK 1.4.2 even though I built the project with 1.5.0.  

Thanks for your patience. :)

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 36054] - [struts-faces] Example 2 application does not get deployed properly.

2006-03-27 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=36054.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36054


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||WORKSFORME




--- Additional Comments From [EMAIL PROTECTED]  2006-03-28 04:02 ---
While resolving Bug 38955 I noted that both example apps start up and work
correctly.  I see the following in the Tomcat 4.1.31 console:

Mar 27, 2006 7:49:10 PM org.apache.struts.tiles.TilesPlugin 
initDefinitionsFactory
INFO: Tiles definition factory loaded for module ''.

Feel free to reopen this if the example apps are still not working for you.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38955] - [struts-faces] Example application doesn't deploy

2006-03-15 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38955.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38955





--- Additional Comments From [EMAIL PROTECTED]  2006-03-16 00:29 ---
(In reply to comment #1)
 The problem is that the struts-faces.jar was compiled with a higher java
 version(probably java 5.0). Please fix this.

Adding source and target attributes to the javac task should fix it.  

default.properties
# Java platform version which source code must conform to
platform.source=1.4
# Java platform version which generated class files must conform to
platform.target=1.4

build.xml
property file=../default.properties/
...
!-- Compile Java Sources --
javac  srcdir=src/java
   destdir=${build.home}/classes
 debug=${compile.debug}
   deprecation=${compile.deprecation}
  optimize=${compile.optimize}
source=${platform.source}
target=${platform.target}
  classpath refid=compile.classpath /
/javac


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38955] - [struts-faces] Example application doesn't deploy

2006-03-15 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38955.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38955





--- Additional Comments From [EMAIL PROTECTED]  2006-03-16 00:37 ---
(In reply to comment #2)
 default.properties
 # Java platform version which source code must conform to
 platform.source=1.4
 # Java platform version which generated class files must conform to
 platform.target=1.4

I copied that from Shale, but I believe Struts Faces should target 1.3 instead.
 Craig, is that correct?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38955] - [struts-faces] Example application doesn't deploy

2006-03-13 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38955.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38955


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Example application doesn't |[struts-faces] Example
   |deploy  |application doesn't deploy




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r384525 - /struts/faces/tags/STRUTS_1_2_9/

2006-03-09 Thread niallp
Author: niallp
Date: Thu Mar  9 07:07:53 2006
New Revision: 384525

URL: http://svn.apache.org/viewcvs?rev=384525view=rev
Log:
Tag Struts 1.2.9

Added:
struts/faces/tags/STRUTS_1_2_9/
  - copied from r384524, struts/faces/branches/STRUTS_1_2_BRANCH/


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 36054] - [struts-faces] Example 2 application does not get deployed properly.

2006-01-18 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=36054.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36054





--- Additional Comments From [EMAIL PROTECTED]  2006-01-18 17:38 ---
The tiles-def.xml dtd cannot be resolved. tiles-config.dtd is not bundled in the
struts.jar. Just change the DOCTYPE by this one.

!DOCTYPE tiles-definitions PUBLIC
  -//Apache Software Foundation//DTD Tiles Configuration 1.1//EN
  http://jakarta.apache.org/struts/dtds/tiles-config_1_1.dtd;

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31829] - [struts-faces] Exception in second example

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=31829.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31829


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 30342] - Struts-faces JSF commandLink bug in RI 1.1

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=30342.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30342


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 26317] - struts-faces example2 - 2004-01-13 build

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=26317.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=26317


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31351] - JSF 1.1_01 and Struts-faces Problem

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=31351.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31351


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31809] - [struts-faces] Unable to deploy struts-faces-example2.war

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=31809.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31809


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31097] - Struts-faces renders blank pages when refreshing page.

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=31097.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31097


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31803] - Struts-Faces and Validator framework (of Struts 1.2.4)

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=31803.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31803


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31842] - [struts-faces] Simple JSF integration with Tiles

2005-12-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=31842.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31842


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|--- |1.2 Family




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 36054] - [struts-faces] Example 2 application does not get deployed properly.

2005-11-13 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=36054.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36054


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Example 2 application does  |[struts-faces] Example 2
   |not get deployed properly.  |application does not get
   ||deployed properly.




--- Additional Comments From [EMAIL PROTECTED]  2005-11-13 22:36 ---
Possibly duplicates Bug 35265

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r331730 - /struts/faces/tags/STRUTS_1_2_8/

2005-11-08 Thread niallp
Author: niallp
Date: Tue Nov  8 00:17:40 2005
New Revision: 331730

URL: http://svn.apache.org/viewcvs?rev=331730view=rev
Log:
Tag Struts 1.2.8

Added:
struts/faces/tags/STRUTS_1_2_8/
  - copied from r331729, struts/faces/branches/STRUTS_1_2_BRANCH/


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r278868 - /struts/faces/trunk/xdocs/

2005-09-05 Thread wsmoak
Author: wsmoak
Date: Mon Sep  5 15:25:57 2005
New Revision: 278868

URL: http://svn.apache.org/viewcvs?rev=278868view=rev
Log:
Deleting xdocs to allow moving site/xdocs/faces into its place.

Removed:
struts/faces/trunk/xdocs/


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Common build file issues (was Re: struts-faces won't compile)

2005-08-14 Thread Wendy Smoak

Wendy wrote:

But there are things in the common build file that are *not*
common to all subprojects.


We needs a way to make sure that core/taglib/el/apps/plugins are all using
the same versions of  their dependencies, without imposing those
dependencies on 'standalone' subprojects.

How about:

/current/build/project.xml
... has the list of committers, reports, etc.  Only things that are common
to *all* sub-projects

/current/build/project-core.xml
... extendproject.xml/extend
... has the list of dependencies for struts-core

Then each subproject chooses which build file to extend.

- core, taglib, el, struts-tiles, apps, plugins can all extend
build/project-core.xml in order to keep their dependencies on [mostly]
commons libraries in sync.  Even if the subproject doesn't necessarily need
one of the dependencies that struts-core does, you can't use these
subprojects without core, so you end up needing that dependency anyway.
Subprojects can add additional dependencies, but can't remove the ones they
inherit from core.

- Faces, Standalone Tiles, etc., can extend from the generic 'project.xml'
to pick up the list of committers, reports, etc., but are responsible for
their own dependencies.

--
Wendy



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Common build file issues (was Re: struts-faces won't compile)

2005-08-14 Thread James Mitchell
I also had the same thoughts during the initial timeframe when I was setting 
up the Maven config.


I think we should stay away from getting too complex with config 
inheritance.


I was thinking about this the other day, and what we might want do is move 
to a completely independent maven configuration for all subprojects.  We 
will still share the build/ dir via svn:externals for common stuff like 
license and CheckStyle config, but each subproject will be a complete 
configuration itself (no extends).


By doing it this way, we can create a new project.xml and maven.xml in 
current/ which is actually the normal way of doing it.  This also puts the 
subprojects in a position to go ahead and do releases independent of other 
subprojects.  We need to further discuss how we handle this with respect to 
how Maven deals with dependencies.  Specifically, someone will need to 
maintain a chart of compatibility issues for all projects under Struts. 
This will be the only way to keep what version of core works with what 
version of taglibs or el or foo type questions from making this split even 
worth the effort.  As it is now, the same issue (sort of) applies across the 
commons dependencies.  Things can get very complicated, very fast.


In addition to the above, it will be easier for me to fix the problems with 
the nightly build that come up every time anything is changed or released. 
Currently the release targets are hardcoded in my cron script (because of 
the way the files have to be named for the cleanup cron to work properly) 
and I'd like to remove that inefficiency.


I'll be making some changes a little later today and more this week (as free 
time allows).


With all hope, we can simplify this whole process and get it moved over to 
our zone and stop relying on our personal machines and (my) sometimes flaky 
network connection.


(Wendy, just got your other email, I'll look into doing that when things 
settle down a bit)



--
James Mitchell
Software Engineer / Open Source Evangelist
Consulting / Mentoring / Freelance
EdgeTech, Inc.
http://www.edgetechservices.net/
678.910.8017
AIM:   jmitchtx
MSN:   [EMAIL PROTECTED]
Skype: jmitchtx

- Original Message - 
From: Wendy Smoak [EMAIL PROTECTED]

To: Struts Developers List dev@struts.apache.org
Sent: Sunday, August 14, 2005 8:27 AM
Subject: Common build file issues (was Re: struts-faces won't compile)



Wendy wrote:

But there are things in the common build file that are *not*
common to all subprojects.


We needs a way to make sure that core/taglib/el/apps/plugins are all using
the same versions of  their dependencies, without imposing those
dependencies on 'standalone' subprojects.

How about:

/current/build/project.xml
... has the list of committers, reports, etc.  Only things that are common
to *all* sub-projects

/current/build/project-core.xml
... extendproject.xml/extend
... has the list of dependencies for struts-core

Then each subproject chooses which build file to extend.

- core, taglib, el, struts-tiles, apps, plugins can all extend
build/project-core.xml in order to keep their dependencies on [mostly]
commons libraries in sync.  Even if the subproject doesn't necessarily 
need

one of the dependencies that struts-core does, you can't use these
subprojects without core, so you end up needing that dependency anyway.
Subprojects can add additional dependencies, but can't remove the ones 
they

inherit from core.

- Faces, Standalone Tiles, etc., can extend from the generic 'project.xml'
to pick up the list of committers, reports, etc., but are responsible for
their own dependencies.

--
Wendy



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



Re: Common build file issues (was Re: struts-faces won't compile)

2005-08-14 Thread James Mitchell
I also had the same thoughts during the initial timeframe when I was setting 
up the Maven config.


I think we should stay away from getting too complex with config 
inheritance.


I was thinking about this the other day, and what we might want do is move 
to a completely independent maven configuration for all subprojects.  We 
will still share the build/ dir via svn:externals for common stuff like 
license and CheckStyle config, but each subproject will be a complete 
configuration itself (no extends).


By doing it this way, we can create a new project.xml and maven.xml in 
current/ which is actually the normal way of doing it.  This also puts the 
subprojects in a position to go ahead and do releases independent of other 
subprojects.  We need to further discuss how we handle this with respect to 
how Maven deals with dependencies.  Specifically, someone will need to 
maintain a chart of compatibility issues for all projects under Struts. 
This will be the only way to keep what version of core works with what 
version of taglibs or el or foo type questions from making this split even 
worth the effort.  As it is now, the same issue (sort of) applies across the 
commons dependencies.  Things can get very complicated, very fast.


In addition to the above, it will be easier for me to fix the problems with 
the nightly build that come up every time anything is changed or released. 
Currently the release targets are hardcoded in my cron script (because of 
the way the files have to be named for the cleanup cron to work properly) 
and I'd like to remove that inefficiency.


I'll be making some changes a little later today and more this week (as free 
time allows).


With all hope, we can simplify this whole process and get it moved over to 
our zone and stop relying on our personal machines and (my) sometimes flaky 
network connection.


(Wendy, just got your other email, I'll look into doing that when things 
settle down a bit)



--
James Mitchell
Software Engineer / Open Source Evangelist
Consulting / Mentoring / Freelance
EdgeTech, Inc.
http://www.edgetechservices.net/
678.910.8017
AIM:   jmitchtx
MSN:   [EMAIL PROTECTED]
Skype: jmitchtx

- Original Message - 
From: Wendy Smoak [EMAIL PROTECTED]

To: Struts Developers List dev@struts.apache.org
Sent: Sunday, August 14, 2005 8:27 AM
Subject: Common build file issues (was Re: struts-faces won't compile)



Wendy wrote:

But there are things in the common build file that are *not*
common to all subprojects.


We needs a way to make sure that core/taglib/el/apps/plugins are all using
the same versions of  their dependencies, without imposing those
dependencies on 'standalone' subprojects.

How about:

/current/build/project.xml
... has the list of committers, reports, etc.  Only things that are common
to *all* sub-projects

/current/build/project-core.xml
... extendproject.xml/extend
... has the list of dependencies for struts-core

Then each subproject chooses which build file to extend.

- core, taglib, el, struts-tiles, apps, plugins can all extend
build/project-core.xml in order to keep their dependencies on [mostly]
commons libraries in sync.  Even if the subproject doesn't necessarily 
need

one of the dependencies that struts-core does, you can't use these
subprojects without core, so you end up needing that dependency anyway.
Subprojects can add additional dependencies, but can't remove the ones 
they

inherit from core.

- Faces, Standalone Tiles, etc., can extend from the generic 'project.xml'
to pick up the list of committers, reports, etc., but are responsible for
their own dependencies.

--
Wendy



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



Re: struts-faces won't compile

2005-08-11 Thread Craig McClanahan
On 8/11/05, Michael Rasmussen [EMAIL PROTECTED] wrote:
 Craig,
   I know this issue came up about a year ago that struts-faces
 wouldn't compile against the latest version of Struts (I think it was
 a validator issue).  It makes sense to me that it should always work
 with the latest version of Struts.  I think it would serve the project
 well to cut a *.0 release of struts-faces and give it a 1.1-1.2.x
 compatability...after that put it into maintenence mode.  Then cut a
 new *.0 release that brings struts-faces up to date with struts
 core...
 
 I haven't been following faces too closely lately...has it gone to 1.x
 yet?  If so, maybe this Struts dependency change to 1.2.x should
 denote a v 2.0?
 
 I totally understand that the target audience for struts-faces is the
 developer trying to migrate a struts app off of struts to jsf.  I have
 a hard time (and sort of balked at struts-faces because of it)
 commiting to a path that may force me to run my app on struts 1.2.x
 and 1.3 in paralell if I decide later that JSF just won't get it done
 for me.  Basically I just mean that I am forced to limit my options if
 I use struts-faces, and I thought the spirit of the library was to
 increase my options.
 

The Maven-generated build script has been getting tweaked lately, and
does indeed impose a Struts 1.2 dependency.  I'll have to take a look
at that -- but there seem to have been some incompatible changes in
the underlying utility classes that is going to make supporting both
1.1 and 1.2 interesting -- let alone supporting 1.3 as well.

NB - the nightly builds are compiled against 1.2.6 at the moment.

  http://cvs.apache.org/builds/struts/nightly/struts-faces/

 My $0.02
 
 Michael
 

Craig


 On 8/8/05, Craig McClanahan [EMAIL PROTECTED] wrote:
  On 8/8/05, Wendy Smoak [EMAIL PROTECTED] wrote:
   From: Craig McClanahan [EMAIL PROTECTED]
  
Maybe the Maven mavens can figure out a way to share the build
infrastructure without sharing the dependency information?
  
   Not a problem... just change the dependency in project.xml.  Looks like it
   needs at least 1.2.2 to compile.  (It won't compile against Struts 1.1.
   Should it?)
  
 
  Given that Struts changed incompatibly, I'm ok with 1.2.x as a
  restriction.  But doesn't that mean we still need an independent
  project.xml file instead of a shared one?
 
   If it makes sense, we can remove the extendbuild/project.xml/extend 
   from
   project.xml, and that will make the build stand on its own.  That
   seems more appropriate for Tiles than Struts-Faces, though.
 
  Yep ... but without disrupting all the subprojects that *do* want to
  share dependencies.  Maybe another opportunity to use SVN externals
  creatively.
 
  
   Thanks,
   Wendy
 
  Craig
 
  -
  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]



Re: struts-faces won't compile

2005-08-11 Thread Wendy Smoak

From: Craig McClanahan [EMAIL PROTECTED]


+import org.apache.struts.util.ModuleUtils;
-RequestUtils.selectModule(request, servletContext);
+ModuleUtils.getInstance().selectModule(request,servletContext);



Conceptually that fix makes sense in that it solves the compilation
problem ... but I don't believe that struts-faces should inherit a
dependency on 1.3.x.


I don't see how switching from a deprecated method call to the replacement
causes a dependency on 1.3.x.  That change still compiles under 1.2.2.


Today it works (runtime) on 1.1 and 1.2,  and it
would be pretty pointless to throw away the vast majority of the
potential target market.


Does it really still work on 1.1?  It won't compile against 1.1 due to the
use of org.apache.struts.taglib.TagUtils and
org.apache.struts.util.ModuleUtils, which must not have existed then.


I think Struts-Faces (if it's going to be built with Maven :-) needs
its own set of dependencies, independent of whatever Struts Core is
using.


It was inheriting from the Struts Common Build, not from Struts Core.  The
incorrect dependency on struts-core-1.3.0-dev was actually *in* the
struts-faces build file and has been corrected to struts-1.2.2.  (You said
you're using 1.2.6 for the Ant build, but that one isn't on ibiblio.  Can we
agree on 1.2.7 by any chance?)

James, can you comment on how you'd like to see this handled?  The website 
builds just fine whether or not the extend tag is there (so I'm happy
either way).  But there are things in the common build file that are *not* 
common to all subprojects.  Maven is smart enough to allow sub-projects to
override dependencies by specifying a different version number, 
(servletapi-2.2 instead of servletapi-2.3, for example.)  But by extending 
the common build, Struts Faces picks up dependencies on antlr, 
commons-chain, commons-digester, commons-fileupload and oro, none of which 
it really needs.  I don't see a way to override a dependency coming from the 
common build with nothing.


Thanks,
--
Wendy Smoak



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Struts-Faces build (was Re: Vote: ANT or Maven ... )

2005-08-09 Thread Wendy Smoak

From: Craig McClanahan [EMAIL PROTECTED]

Craig wrote:

There was talk on some thread or another ... hmm, I seem to
remember a certain advocate for a compromise position :-) ... that you
could have it both ways by simply generating a build.xml file from
Maven.


I advocated for both... but not for Maven-generated Ant build files.  You 
brought those up wrt Commons projects, I think.


I was talking about Tiles (now we're back to the original topic) where I 
would like Maven to build it and install the .jar in the local repository so 
I can easily have it as a dependency in my own projects.  I just want Maven 
to work for Tiles, completely separate from whatever is happening with the 
Ant build.


--
Wendy







-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



struts-faces won't compile

2005-08-08 Thread Wendy Smoak
Faces is using a formerly deprecated, and now removed, method in 
RequestUtils.  This breaks the Maven build, which works against 
struts-core-1.3.0-dev.


Is this the correct fix?

Thanks,
Wendy

Index: 
core-library/src/java/org/apache/struts/faces/application/ActionListenerImpl.java

===
---  
core-library/src/java/org/apache/struts/faces/application/ActionListenerImpl.java 
(revision 230657)
+++ 
core-library/src/java/org/apache/struts/faces/application/ActionListenerImpl.java 
(working copy)

@@ -35,6 +35,7 @@
import org.apache.struts.config.ModuleConfig;
import org.apache.struts.faces.Constants;
import org.apache.struts.faces.component.FormComponent;
+import org.apache.struts.util.ModuleUtils;
import org.apache.struts.util.RequestUtils;


@@ -152,7 +153,7 @@
// Invoke the appropriate request processor for this request
try {
request.setAttribute(Constants.ACTION_EVENT_KEY, event);
-RequestUtils.selectModule(request, servletContext);
+ModuleUtils.getInstance().selectModule(request, 
servletContext);

ModuleConfig moduleConfig = (ModuleConfig)
request.getAttribute(Globals.MODULE_KEY);
if (log.isTraceEnabled()) {



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: struts-faces won't compile

2005-08-08 Thread Craig McClanahan
On 8/8/05, Wendy Smoak [EMAIL PROTECTED] wrote:
 Faces is using a formerly deprecated, and now removed, method in
 RequestUtils.  This breaks the Maven build, which works against
 struts-core-1.3.0-dev.
 
 Is this the correct fix?
 

Conceptually that fix makes sense in that it solves the compilation
problem ... but I don't believe that struts-faces should inherit a
dependency on 1.3.x.  Today it works (runtime) on 1.1 and 1.2,  and it
would be pretty pointless to throw away the vast majority of the
potential target market.

I think Struts-Faces (if it's going to be built with Maven :-) needs
its own set of dependencies, independent of whatever Struts Core is
using.  That may mean decoupling from the shared build
infrastructure that is so handy across Struts Core ... but this
problem is going to get faced by other subcomponents as soon as they
contemplate independent release cycles.

Maybe the Maven mavens can figure out a way to share the build
infrastructure without sharing the dependency information?

Craig

 Thanks,
 Wendy
 
 Index:
 core-library/src/java/org/apache/struts/faces/application/ActionListenerImpl.java
 ===
 ---
 core-library/src/java/org/apache/struts/faces/application/ActionListenerImpl.java
 (revision 230657)
 +++
 core-library/src/java/org/apache/struts/faces/application/ActionListenerImpl.java
 (working copy)
 @@ -35,6 +35,7 @@
  import org.apache.struts.config.ModuleConfig;
  import org.apache.struts.faces.Constants;
  import org.apache.struts.faces.component.FormComponent;
 +import org.apache.struts.util.ModuleUtils;
  import org.apache.struts.util.RequestUtils;
 
 
 @@ -152,7 +153,7 @@
  // Invoke the appropriate request processor for this request
  try {
  request.setAttribute(Constants.ACTION_EVENT_KEY, event);
 -RequestUtils.selectModule(request, servletContext);
 +ModuleUtils.getInstance().selectModule(request,
 servletContext);
  ModuleConfig moduleConfig = (ModuleConfig)
  request.getAttribute(Globals.MODULE_KEY);
  if (log.isTraceEnabled()) {
 
 
 
 -
 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]



Re: struts-faces won't compile

2005-08-08 Thread Wendy Smoak

From: Craig McClanahan [EMAIL PROTECTED]


Maybe the Maven mavens can figure out a way to share the build
infrastructure without sharing the dependency information?


Not a problem... just change the dependency in project.xml.  Looks like it
needs at least 1.2.2 to compile.  (It won't compile against Struts 1.1.
Should it?)

If it makes sense, we can remove the extendbuild/project.xml/extend from
project.xml, and that will make the build stand on its own.  That
seems more appropriate for Tiles than Struts-Faces, though.

Thanks,
Wendy



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: struts-faces won't compile

2005-08-08 Thread Craig McClanahan
On 8/8/05, Wendy Smoak [EMAIL PROTECTED] wrote:
 From: Craig McClanahan [EMAIL PROTECTED]
 
  Maybe the Maven mavens can figure out a way to share the build
  infrastructure without sharing the dependency information?
 
 Not a problem... just change the dependency in project.xml.  Looks like it
 needs at least 1.2.2 to compile.  (It won't compile against Struts 1.1.
 Should it?)
 

Given that Struts changed incompatibly, I'm ok with 1.2.x as a
restriction.  But doesn't that mean we still need an independent
project.xml file instead of a shared one?

 If it makes sense, we can remove the extendbuild/project.xml/extend from
 project.xml, and that will make the build stand on its own.  That
 seems more appropriate for Tiles than Struts-Faces, though.

Yep ... but without disrupting all the subprojects that *do* want to
share dependencies.  Maybe another opportunity to use SVN externals
creatively.

 
 Thanks,
 Wendy

Craig

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r230968 - /struts/faces/trunk/project.xml

2005-08-08 Thread wsmoak
Author: wsmoak
Date: Mon Aug  8 21:53:40 2005
New Revision: 230968

URL: http://svn.apache.org/viewcvs?rev=230968view=rev
Log:
Changed dependency from Struts Core 1.3.0-dev to Struts 1.2.2, removed 
dependencies on struts-taglib and struts-tiles.

Modified:
struts/faces/trunk/project.xml

Modified: struts/faces/trunk/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/project.xml?rev=230968r1=230967r2=230968view=diff
==
--- struts/faces/trunk/project.xml (original)
+++ struts/faces/trunk/project.xml Mon Aug  8 21:53:40 2005
@@ -66,33 +66,14 @@
   
dependency
  groupIdstruts/groupId
- artifactIdstruts-core/artifactId
- version1.3.0-dev/version
+ artifactIdstruts/artifactId
+ version1.2.2/version
  properties
war.bundletrue/war.bundle
  /properties
   urlhttp://struts.apache.org//url
/dependency
 
-   dependency
- groupIdstruts/groupId
- artifactIdstruts-taglib/artifactId
- version1.3.0-dev/version
- properties
-   war.bundletrue/war.bundle
- /properties
-  urlhttp://struts.apache.org//url
-   /dependency
-
-   dependency
- groupIdstruts/groupId
- artifactIdstruts-tiles/artifactId
- version1.3.0-dev/version
- properties
-   war.bundletrue/war.bundle
- /properties
-  urlhttp://struts.apache.org//url
-   /dependency
 
dependency
  groupIdmyfaces/groupId



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Struts-Faces build (was Re: Vote: ANT or Maven ... )

2005-08-08 Thread Wendy Smoak

From: Craig McClanahan [EMAIL PROTECTED]


Still builds with Ant?  That's the key for the nightlies.


Did it ever NOT build with Ant?  This whole thing started because Faces 
wouldn't build for me with Maven.  (And how did it get on this thread? 
There was another one for Faces...)


There's a build.xml file in the Faces directory that I assume you are using. 
I haven't touched it.  (At least, I don't think so... it's not one of those 
Maven-generated Ant build files you were talking about, is it?)



PS:  Interestingly, one of the advantages Martin mentions for Maven
(generates the web site for you) was *not* a particular motivator for
the Struts web site, because we had already invested the pain and
agony to build reasonable stylesheets for transformations from our
xdocs files.  Good thing too ... if we had followed the conventional
wisdom at the time (Anakia) we would still not be done converting :-).


Funny you should mention that... integrating the existing website into the 
Maven-built multiproject site what I've been working on tonight.  More on 
that later.  Or maybe tomorrow. :)


--
Wendy 




-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Struts-Faces build (was Re: Vote: ANT or Maven ... )

2005-08-08 Thread Craig McClanahan
On 8/8/05, Wendy Smoak [EMAIL PROTECTED] wrote:
 From: Craig McClanahan [EMAIL PROTECTED]
 
  Still builds with Ant?  That's the key for the nightlies.
 
 Did it ever NOT build with Ant?  This whole thing started because Faces
 wouldn't build for me with Maven.  (And how did it get on this thread?
 There was another one for Faces...)
 
 There's a build.xml file in the Faces directory that I assume you are using.
 I haven't touched it.  (At least, I don't think so... it's not one of those
 Maven-generated Ant build files you were talking about, is it?)
 

Yes, the Ant script is what I use (for myself and for the nightly
builds).  There was talk on some thread or another ... hmm, I seem to
remember a certain advocate for a compromise position :-) ... that you
could have it both ways by simply generating a build.xml file from
Maven.

As pointed out in a couple of other emails, it's not clear to me
whether that is sufficient to provide the functionality that the
current struts-faces build.xml script provides -- particularly in
adapting to the differences between the JSF RI and MyFaces.  I may be
ignorant in this respect (especially with respect to recent
innovations in Maven land), but this is a dealbreaker for struts-faces
(or Shale, for that matter) to rely on Maven.

  PS:  Interestingly, one of the advantages Martin mentions for Maven
  (generates the web site for you) was *not* a particular motivator for
  the Struts web site, because we had already invested the pain and
  agony to build reasonable stylesheets for transformations from our
  xdocs files.  Good thing too ... if we had followed the conventional
  wisdom at the time (Anakia) we would still not be done converting :-).
 
 Funny you should mention that... integrating the existing website into the
 Maven-built multiproject site what I've been working on tonight.  More on
 that later.  Or maybe tomorrow. :)

Be interesting to see what you come up with.

 
 --
 Wendy
 

Craig

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r202117 - /struts/faces/trunk/project.xml

2005-06-27 Thread jmitchell
Author: jmitchell
Date: Mon Jun 27 19:38:05 2005
New Revision: 202117

URL: http://svn.apache.org/viewcvs?rev=202117view=rev
Log:
fix faces build - thanks Wendy\!

Modified:
struts/faces/trunk/project.xml

Modified: struts/faces/trunk/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/project.xml?rev=202117r1=202116r2=202117view=diff
==
--- struts/faces/trunk/project.xml (original)
+++ struts/faces/trunk/project.xml Mon Jun 27 19:38:05 2005
@@ -66,8 +66,18 @@
   
dependency
  groupIdstruts/groupId
- artifactIdstruts/artifactId
- version1.1/version
+ artifactIdstruts-core/artifactId
+ version1.3.0-dev/version
+ properties
+   war.bundletrue/war.bundle
+ /properties
+  urlhttp://struts.apache.org//url
+   /dependency
+
+   dependency
+ groupIdstruts/groupId
+ artifactIdstruts-taglib/artifactId
+ version1.3.0-dev/version
  properties
war.bundletrue/war.bundle
  /properties



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 35506] - [struts-faces] - core-library and example1-webapp patches

2005-06-26 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=35506.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35506


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Struts-Faces Library - core-|[struts-faces] - core-
   |library and example1-webapp |library and example1-webapp
   |patches |patches




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 35506] New: - Struts-Faces Library - core-library and example1-webapp patches

2005-06-25 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=35506.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35506

   Summary: Struts-Faces Library - core-library and example1-webapp
patches
   Product: Struts
   Version: Nightly Build
  Platform: Other
OS/Version: other
Status: NEW
  Severity: normal
  Priority: P2
 Component: Struts-Faces Library
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


A couple one line patches to work with tomcat 5.5

1)Changed the assumed action tour/ in the struts-config of the example1 
application to use an include on the the static document tour.htm.  The 
forward wants the target to be a jsp resource.

2)Change the signature of the call to get the ModuleConfig to use the 
ServletContext, another overload.  This is in the JavasrciptValidatorTag.  
This was raising a null pointer exception when not dispatched thru the request 
processor first.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 35179] - [struts-faces] JSF Conversion and validation errors under s:errors

2005-06-17 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=35179.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35179


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|JSF Conversion and  |[struts-faces] JSF
   |validation errors under |Conversion and validation
   |s:errors|errors under s:errors




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 35265] - [struts-faces] Example applications don't work?!

2005-06-17 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=35265.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35265


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Example applications don't  |[struts-faces] Example
   |work?!  |applications don't work?!




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 34682] - [struts-faces] Conversion errors are not showing up

2005-06-12 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34682.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=34682


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Conversion errors are not   |[struts-faces] Conversion
   |showing up  |errors are not showing up




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 35232] - [struts-faces] ErrorsRenderer assumes ActionErrors

2005-06-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=35232.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35232


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|ErrorsRenderer assumes  |[struts-faces]
   |ActionErrors|ErrorsRenderer assumes
   ||ActionErrors




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32165] - [struts-faces] FacesRequestProcessor bug when using prefix mapped Struts servlet extension mapped Faces servlet

2005-06-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=32165.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32165


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|FacesRequestProcessor bug   |[struts-faces]
   |when using prefix mapped|FacesRequestProcessor bug
   |Struts servlet  extension  |when using prefix mapped
   |mapped Faces servlet|Struts servlet  extension
   ||mapped Faces servlet




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r178996 - in /struts/faces/trunk: example1-webapp/build.xml example2-webapp/build.xml example2-webapp/src/web/loggedoff.jsp systest1-webapp/build.xml

2005-05-29 Thread craigmcc
Author: craigmcc
Date: Sun May 29 16:23:24 2005
New Revision: 178996

URL: http://svn.apache.org/viewcvs?rev=178996view=rev
Log:
Adapt build scripts to reflect the fact that commons-collections.jar is not
shipped with recent (i.e. 1.2.6+) versions of Struts.

Modified:
struts/faces/trunk/example1-webapp/build.xml
struts/faces/trunk/example2-webapp/build.xml
struts/faces/trunk/example2-webapp/src/web/loggedoff.jsp
struts/faces/trunk/systest1-webapp/build.xml

Modified: struts/faces/trunk/example1-webapp/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example1-webapp/build.xml?rev=178996r1=178995r2=178996view=diff
==
--- struts/faces/trunk/example1-webapp/build.xml (original)
+++ struts/faces/trunk/example1-webapp/build.xml Sun May 29 16:23:24 2005
@@ -66,6 +66,8 @@
 
 
   !-- Conditional Processing Flags --
+  available property=collections.present
+ file=${commons-collections.jar}/
   available property=jsfri.present
 classname=com.sun.faces.RIConstants
 classpath=${jsf-impl.jar}/
@@ -160,7 +162,16 @@
   /target
 
 
-  target name=static depends=prepare
+  target name=static-collections depends=prepare
+if=collections.present
+
+copytodir=${build.home}/${context.path}/WEB-INF/lib
+  file=${commons-collections.jar}/
+
+  /target
+
+
+  target name=static depends=static-collections
 
 !-- Copy static web resources --
 copytodir=${build.home}/${context.path}
@@ -180,8 +191,6 @@
   file=${struts-faces.jar}/
 copytodir=${build.home}/${context.path}/WEB-INF/lib
   file=${commons-beanutils.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-collections.jar}/
 copytodir=${build.home}/${context.path}/WEB-INF/lib
   file=${commons-digester.jar}/
 copytodir=${build.home}/${context.path}/WEB-INF/lib

Modified: struts/faces/trunk/example2-webapp/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/build.xml?rev=178996r1=178995r2=178996view=diff
==
--- struts/faces/trunk/example2-webapp/build.xml (original)
+++ struts/faces/trunk/example2-webapp/build.xml Sun May 29 16:23:24 2005
@@ -141,6 +141,7 @@
 echo  message=build.standalone = ${build.standalone}/
 echo  message=jsf-api.jar =  ${jsf-api.jar}/
 echo  message=jsf-impl.jar = ${jsf-impl.jar}/
+echo  message=struts.home =  ${struts.home}/
 echo  message=struts.jar =   ${struts.jar}/
 echo  message=struts-faces.jar = ${struts-faces.jar}/
 echo  message=jsfri.present =${jsfri.present}/
@@ -178,22 +179,10 @@
 mkdir dir=${build.home}/${context.path}/WEB-INF/lib/
 copytodir=${build.home}/${context.path}/WEB-INF/lib
   file=${struts-faces.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-beanutils.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-collections.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-digester.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-fileupload.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-logging.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${commons-validator.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${jakarta-oro.jar}/
-copytodir=${build.home}/${context.path}/WEB-INF/lib
-  file=${struts.jar}/
+copytodir=${build.home}/${context.path}/WEB-INF/lib
+  fileset dir=${struts.home}/lib
+  includes=*.jar/
+/copy
 
   /target
 

Modified: struts/faces/trunk/example2-webapp/src/web/loggedoff.jsp
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/src/web/loggedoff.jsp?rev=178996r1=178995r2=178996view=diff
==
--- struts/faces/trunk/example2-webapp/src/web/loggedoff.jsp (original)
+++ struts/faces/trunk/example2-webapp/src/web/loggedoff.jsp Sun May 29 
16:23:24 2005
@@ -24,6 +24,7 @@
 --
 
 
+%--
 h:formid=loggedoff
   h:panelGridcolumns=1
 h:commandLink id=register
@@ -38,3 +39,25 @@
 /h:commandLink
   /h:panelGrid
 /h:form
+--%
+
+h:panelGrid  columns=1
+
+  %-- FIXME - needs ?action=Create modifier --%
+  s:form  action=/editRegistration

svn commit: r168538 - /struts/faces/tags/STRUTS_1_2_7

2005-05-06 Thread niallp
Author: niallp
Date: Thu May  5 23:25:57 2005
New Revision: 168538

URL: http://svn.apache.org/viewcvs?rev=168538view=rev
Log:
faces - Version 1.2.7 Tagged

Added:
struts/faces/tags/STRUTS_1_2_7/
  - copied from r168537, struts/faces/trunk/


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r165410 - in /struts/faces/trunk: ./ core-library/src/java/org/apache/struts/faces/taglib/ core-library/src/java/org/apache/struts/faces/util/ sysclient-app/src/java/org/apache/struts/faces/sysclient/

2005-04-30 Thread jmitchell
Author: jmitchell
Date: Sat Apr 30 09:25:14 2005
New Revision: 165410

URL: http://svn.apache.org/viewcvs?rev=165410view=rev
Log:
bring faces current with all dependent subprojects(core, taglib, and tiles), 
upgrade to htmlunit 1.5

Modified:
struts/faces/trunk/build.properties.sample

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/taglib/JavascriptValidatorTag.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/util/StrutsContext.java

struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/AbstractTestCase.java

struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java

struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/SimpleTestCase.java

Modified: struts/faces/trunk/build.properties.sample
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/build.properties.sample?rev=165410r1=165409r2=165410view=diff
==
--- struts/faces/trunk/build.properties.sample (original)
+++ struts/faces/trunk/build.properties.sample Sat Apr 30 09:25:14 2005
@@ -142,7 +142,7 @@
 
 # The absolute or relative pathname to your installation of HtmlUnit 1.2.3
 # or later.  This code can be acquired at SourceForge.
-htmlunit.home=/usr/local/htmlunit-1.2.3
+htmlunit.home=/usr/local/htmlunit-1.5
 
 # The type of JSF state saving that should be used when the system integration
 # test web application is depoyed (client or server)

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/taglib/JavascriptValidatorTag.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/taglib/JavascriptValidatorTag.java?rev=165410r1=165409r2=165410view=diff
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/taglib/JavascriptValidatorTag.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/taglib/JavascriptValidatorTag.java
 Sat Apr 30 09:25:14 2005
@@ -28,6 +28,7 @@
 import javax.faces.component.UIComponent;
 import javax.faces.context.FacesContext;
 import javax.faces.webapp.UIComponentTag;
+import javax.servlet.http.HttpServletRequest;
 import javax.servlet.jsp.JspException;
 import javax.servlet.jsp.JspWriter;
 import javax.servlet.jsp.PageContext;
@@ -38,13 +39,14 @@
 import org.apache.commons.validator.Form;
 import org.apache.commons.validator.ValidatorAction;
 import org.apache.commons.validator.ValidatorResources;
-import org.apache.commons.validator.ValidatorUtil;
 import org.apache.commons.validator.Var;
+import org.apache.commons.validator.util.ValidatorUtils;
 import org.apache.struts.Globals;
 import org.apache.struts.config.ModuleConfig;
 import org.apache.struts.faces.component.FormComponent;
+import org.apache.struts.taglib.TagUtils;
 import org.apache.struts.util.MessageResources;
-import org.apache.struts.util.RequestUtils;
+import org.apache.struts.util.ModuleUtils;
 import org.apache.struts.validator.Resources;
 import org.apache.struts.validator.ValidatorPlugIn;
 
@@ -267,15 +269,16 @@
 public int doStartTag() throws JspException {
 StringBuffer results = new StringBuffer();
 
-ModuleConfig config = RequestUtils.getModuleConfig(pageContext);
+HttpServletRequest request = 
(HttpServletRequest)pageContext.getRequest();
+ModuleConfig config = 
ModuleUtils.getInstance().getModuleConfig(request);
 ValidatorResources resources =
 (ValidatorResources) pageContext.getAttribute(
 ValidatorPlugIn.VALIDATOR_KEY + config.getPrefix(),
 PageContext.APPLICATION_SCOPE);
 
-Locale locale = RequestUtils.retrieveUserLocale(this.pageContext, 
null);
+Locale locale = TagUtils.getInstance().getUserLocale(pageContext, 
null);
 
-Form form = resources.get(locale, formName);
+Form form = resources.getForm(locale, formName);
 if (form != null) {
 if (true.equalsIgnoreCase(dynamicJavascript)) {
 MessageResources messages =
@@ -290,7 +293,7 @@
 for (Iterator i = form.getFields().iterator(); i.hasNext();) {
 Field field = (Field) i.next();
 
-for (Iterator x = field.getDependencies().iterator(); 
x.hasNext();) {
+for (Iterator x = field.getDependencyList().iterator(); 
x.hasNext();) {
 Object o = x.next();
 
 if (o != null  !lActionMethods.contains(o)) {
@@ -338,7 +341,7 @@
  (va2.getDepends() != null  
va2.getDepends().length()  0)) {
 return -1;
 } else {
-return va1.getDependencies().size() - 
va2.getDependencies().size

svn commit: r160075 - struts/faces/trunk/project.xml

2005-04-04 Thread jmitchell
Author: jmitchell
Date: Mon Apr  4 11:30:06 2005
New Revision: 160075

URL: http://svn.apache.org/viewcvs?view=revrev=160075
Log:
bring faces up to date with latest available htmlunit on ibiblio

Modified:
struts/faces/trunk/project.xml

Modified: struts/faces/trunk/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/project.xml?view=diffr1=160074r2=160075
==
--- struts/faces/trunk/project.xml (original)
+++ struts/faces/trunk/project.xml Mon Apr  4 11:30:06 2005
@@ -108,7 +108,7 @@
 dependency
   groupIdhtmlunit/groupId
   artifactIdhtmlunit/artifactId
-  version1.2.3/version
+  version1.5/version
   urlhttp://htmlunit.sourceforge.net//url
   properties
 war.bundletrue/war.bundle



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r157452 - in struts/faces/trunk/core-library/src/java/org/apache/struts/faces: application/FacesRequestProcessor.java application/FacesTilesRequestProcessor.java util/HttpServletRequestWrapper.java

2005-03-14 Thread craigmcc
Author: craigmcc
Date: Mon Mar 14 11:46:14 2005
New Revision: 157452

URL: http://svn.apache.org/viewcvs?view=revrev=157452
Log:
Remove the problematic HttpServletRequestWrapper and references to it
in the request processor implementations.

Removed:

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/util/HttpServletRequestWrapper.java
Modified:

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java?view=diffr1=157451r2=157452
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
 Mon Mar 14 11:46:14 2005
@@ -128,11 +128,6 @@
 created = true;
 FacesContextFactory fcf = (FacesContextFactory)
 FactoryFinder.getFactory(FactoryFinder.FACES_CONTEXT_FACTORY);
-// HttpServletRequestWrapper wrapper = new 
HttpServletRequestWrapper(request, uri);
-// context = fcf.getFacesContext(servlet.getServletContext(), 
wrapper,
-//   response, lifecycle); 
-// Comment out the previous three lines and uncomment
-// the following two lines to test eliminating the wrapper
 context = fcf.getFacesContext(servlet.getServletContext(),
   request, response, lifecycle);
 }

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java?view=diffr1=157451r2=157452
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
 Mon Mar 14 11:46:14 2005
@@ -41,7 +41,6 @@
 import org.apache.struts.config.ForwardConfig;
 import org.apache.struts.faces.Constants;
 import org.apache.struts.faces.component.FormComponent;
-import org.apache.struts.faces.util.HttpServletRequestWrapper;
 import org.apache.struts.tiles.TilesRequestProcessor;
 
 
@@ -129,13 +128,8 @@
 created = true;
 FacesContextFactory fcf = (FacesContextFactory)
 FactoryFinder.getFactory(FactoryFinder.FACES_CONTEXT_FACTORY);
-HttpServletRequestWrapper wrapper = new 
HttpServletRequestWrapper(request, uri);
-context = fcf.getFacesContext(servlet.getServletContext(), wrapper,
-  response, lifecycle); 
-// Comment out the previous three lines and uncomment
-// the following two lines to test eliminating the wrapper
-// context = fcf.getFacesContext(servlet.getServletContext(),
-//   request, response, lifecycle);
+context = fcf.getFacesContext(servlet.getServletContext(),
+  request, response, lifecycle);
 }
 
 // Create a new view root



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32014] - HttpServletRequestWrapper in struts-faces broken for servlet 2.4

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=32014.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32014


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:49 ---
Fixed in nightly build 20050315.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672





--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:50 ---
Fixed in nightly build 20050315.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:50 ---
(Forgot to change bug resolution)


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-13 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672





--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 03:21 ---
This is great news!  I'll clean up the wrapper class stuff soon (still in a huge
email backlog).  By the way, is the Incubator CVS still the place to get the
latest MyFaces source code?


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-13 Thread Craig McClanahan
FYI, as soon as I apply some recent patches (to remove the wrapper
class) this whole area will become a non-issue for struts-faces.  In
the general case, however, it is not currently feasible to create a
wrapper that works across 2.3 and 2.4 unless you're willing to do a
little work with Java reflection to transparently deal with the 2.4
methods that are not there in 2.3 (and therefore you cannot delegate
to the underlying wrapper).

Craig



On Sat, 12 Mar 2005 09:16:39 -0500, Bill Siggelkow
[EMAIL PROTECTED] wrote:
 On 2005-03-11 00:25:35 -0500, Martin Cooper [EMAIL PROTECTED] said:
 
  On Tue, 1 Mar 2005 09:11:28 -0800, Craig McClanahan [EMAIL PROTECTED] 
  wrote:
  Doing the simple version of a change like this will cause the wrapper
  class to not be compilable against a 2.2 or 2.3 version of the servlet
  API any longer, which is a particular problem for the standard Struts
  release.
 
  I don't believe that's the case. The current implementation of
  MultipartRequestWrapper uses exactly the same technique that Ramiro
  mentions to allow compilation against the Servlets 2.3 API. Adding
  extra methods to the implementation class allows it to compile against
  the newer API, while the methods become just ordinary class methods
  when compiled against the older API.
 
  It's true that the dummy methods added to satisfy the newer API are
  not useful (without the use of reflection, as you suggest below), but
  that doesn't seem to have caused problems for people using the
  Servlets 2.3 API while we've been dummying up the methods new to that
  version over Servlets 2.2.
 
 Well, I ran into this the other day -- and while it seemed like I could
 just no-op (or wrapper) for the new methods in the Servlet API
 (getLocalAddr()  etc.) there are other problems in the taglibs against
 the JSP API -- because JSP 2.0 introduces new classes like
 ExpressionEvaluator.
 
 
 -
 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]



Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-13 Thread Niall Pemberton
Having the wrapper extend the HttpServletRequestWrapper (introduced in
Servlet 2.3) resolves this - just need to remove the dummy methods.

Niall

- Original Message - 
From: Craig McClanahan [EMAIL PROTECTED]
To: Struts Developers List dev@struts.apache.org
Sent: Monday, March 14, 2005 4:11 AM
Subject: Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0


 FYI, as soon as I apply some recent patches (to remove the wrapper
 class) this whole area will become a non-issue for struts-faces.  In
 the general case, however, it is not currently feasible to create a
 wrapper that works across 2.3 and 2.4 unless you're willing to do a
 little work with Java reflection to transparently deal with the 2.4
 methods that are not there in 2.3 (and therefore you cannot delegate
 to the underlying wrapper).

 Craig



 On Sat, 12 Mar 2005 09:16:39 -0500, Bill Siggelkow
 [EMAIL PROTECTED] wrote:
  On 2005-03-11 00:25:35 -0500, Martin Cooper [EMAIL PROTECTED] said:
 
   On Tue, 1 Mar 2005 09:11:28 -0800, Craig McClanahan
[EMAIL PROTECTED] wrote:
   Doing the simple version of a change like this will cause the wrapper
   class to not be compilable against a 2.2 or 2.3 version of the
servlet
   API any longer, which is a particular problem for the standard Struts
   release.
  
   I don't believe that's the case. The current implementation of
   MultipartRequestWrapper uses exactly the same technique that Ramiro
   mentions to allow compilation against the Servlets 2.3 API. Adding
   extra methods to the implementation class allows it to compile against
   the newer API, while the methods become just ordinary class methods
   when compiled against the older API.
  
   It's true that the dummy methods added to satisfy the newer API are
   not useful (without the use of reflection, as you suggest below), but
   that doesn't seem to have caused problems for people using the
   Servlets 2.3 API while we've been dummying up the methods new to that
   version over Servlets 2.2.
 
  Well, I ran into this the other day -- and while it seemed like I could
  just no-op (or wrapper) for the new methods in the Servlet API
  (getLocalAddr()  etc.) there are other problems in the taglibs against
  the JSP API -- because JSP 2.0 introduces new classes like
  ExpressionEvaluator.



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-13 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672





--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 08:20 ---
(In reply to comment #3)

 By the way, is the Incubator CVS still the place to get the
 latest MyFaces source code?

yes, the infra@ guys are working on scripts for movement to own CVS ;)

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-12 Thread Bill Siggelkow
On 2005-03-11 00:25:35 -0500, Martin Cooper [EMAIL PROTECTED] said:
On Tue, 1 Mar 2005 09:11:28 -0800, Craig McClanahan [EMAIL PROTECTED] wrote:
Doing the simple version of a change like this will cause the wrapper
class to not be compilable against a 2.2 or 2.3 version of the servlet
API any longer, which is a particular problem for the standard Struts
release.
I don't believe that's the case. The current implementation of
MultipartRequestWrapper uses exactly the same technique that Ramiro
mentions to allow compilation against the Servlets 2.3 API. Adding
extra methods to the implementation class allows it to compile against
the newer API, while the methods become just ordinary class methods
when compiled against the older API.
It's true that the dummy methods added to satisfy the newer API are
not useful (without the use of reflection, as you suggest below), but
that doesn't seem to have caused problems for people using the
Servlets 2.3 API while we've been dummying up the methods new to that
version over Servlets 2.2.
Well, I ran into this the other day -- and while it seemed like I could 
just no-op (or wrapper) for the new methods in the Servlet API 
(getLocalAddr()  etc.) there are other problems in the taglibs against 
the JSP API -- because JSP 2.0 introduces new classes like 
ExpressionEvaluator.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672





--- Additional Comments From [EMAIL PROTECTED]  2005-03-10 18:16 ---
I believe this has been resolved. I've been working with Matthias Wessendorf
(myfaces) and have contributed a couple patches to myfaces which have been
committed to cvs.

The first patch fixes the servlet mapping issue:
http://issues.apache.org/jira/browse/MYFACES-121

There is a catch with is patch, though, in that it may not perform correctly if
a user has multiple servlet mappings to jsf in their webapp. This is not too
common, though.

Then, a bug in myfaces was exposed, which has been committed:
http://issues.apache.org/jira/browse/MYFACES-122


I've tested both struts-faces example webapps against myfaces cvs and a modified
struts-faces (because bug 32014 has not yet been resolved in cvs).

All seems fine - both apps performed well. All that needs to be done is to
commit bug 32014 .

Mike


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672





--- Additional Comments From [EMAIL PROTECTED]  2005-03-10 19:16 ---
Mike-

Interessting, I did the same (I gues)
see here:
http://issues.apache.org/bugzilla/show_bug.cgi?id=33950

So it should be OK the remove the *legacy* Wrapper clazz ;)

-Matthias

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-10 Thread Martin Cooper
On Tue, 1 Mar 2005 09:11:28 -0800, Craig McClanahan [EMAIL PROTECTED] wrote:
 Doing the simple version of a change like this will cause the wrapper
 class to not be compilable against a 2.2 or 2.3 version of the servlet
 API any longer, which is a particular problem for the standard Struts
 release.

I don't believe that's the case. The current implementation of
MultipartRequestWrapper uses exactly the same technique that Ramiro
mentions to allow compilation against the Servlets 2.3 API. Adding
extra methods to the implementation class allows it to compile against
the newer API, while the methods become just ordinary class methods
when compiled against the older API.

It's true that the dummy methods added to satisfy the newer API are
not useful (without the use of reflection, as you suggest below), but
that doesn't seem to have caused problems for people using the
Servlets 2.3 API while we've been dummying up the methods new to that
version over Servlets 2.2.

--
Martin Cooper


 It will require a bit more work to use reflection to call
 through these methods if they exist, or return nothing if they do not
 exist.
 
 Craig
 
 
 On Tue, 01 Mar 2005 14:11:41 +0100, Ramiro Alba Queipo [EMAIL PROTECTED] 
 wrote:
  Hello everybody:
 
  I was using tomcat 4.0.24 to develop an application with struts, I and
  decided to migrate to tomcat 5.0.28 so as I could user EL natively from
  JSP 2.0. I thought it would be a good idea to compile STRUTS 1.2.4 under
  servlet 2.4 and JSP 2.0 from tomcat 5.0.28. Everything went wright
  except for one class:
 
  org/apache/struts/upload/MultipartRequestWrapper.java
 
  It is not implementing the following methods from interface
  javax.servlet.ServletRequest corresponding to servlet-api.jar of tomcat
  5.0.28:
 
  public String getLocalName() {}
  public String getLocalAddr() {}
  public int getLocalPort() {}
  public int getRemotePort() {}
 
  To bypass the problem I created trivial methods, and then I could
  complete the compilation process. Everything seams to work, but in order
  to work with tomcat 5.0.x:
 
  1) Is it a good idea to make this compilation o better to stuck with
 the official binary supplied struts.jar?
 
  2) Is this situation going to be solved in a near feature? (I have seen
  no changes in struts 1.2.6)
 
  I have found the same problem compiling struts-faces, which seams to be
  compiled with servlet and jsp apis from tomcat 5.0.25
 
  Thanks in advance
 
  Regards
 
  P.D: I have posted this e-mail at user@struts.apache.org more than a
  week ago and I have no answer, probably due to be a question out of
  scope for that list, so I decided to post here the same question. Sorry
  for the cross posting.
 
  -
  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]



DO NOT REPLY [Bug 32014] - HttpServletRequestWrapper in struts-faces broken for servlet 2.4

2005-03-09 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=32014.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32014





--- Additional Comments From [EMAIL PROTECTED]  2005-03-09 20:49 ---
Created an attachment (id=14449)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=14449action=view)
removes references to HttpServletRequestWrapper from FacesRequestProcessor.java
 FacesTilesRequestProcessor.java and removes HttpServletRequestWrapper


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32014] - HttpServletRequestWrapper in struts-faces broken for servlet 2.4

2005-03-09 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=32014.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32014


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|WONTFIX |




--- Additional Comments From [EMAIL PROTECTED]  2005-03-09 20:57 ---
FacesTilesRequestProcessor in svn is still not functional becuase it's using
HttpServletRequestWrapper. So, I've attached the above patch to speed up the
process of removing HttpServletRequestWrapper. I'd like to be able to use the
nightly builds without modification.

Thanks,
Mike

p.s. I'm reopening this bug because it is my understanding that notifications
are not sent out otherwise

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-02 Thread Ramiro Alba Queipo
On Tue, 2005-03-01 at 18:11, Craig McClanahan wrote:
 Doing the simple version of a change like this will cause the wrapper
 class to not be compilable against a 2.2 or 2.3 version of the servlet
 API any longer, which is a particular problem for the standard Struts
 release.  It will require a bit more work to use reflection to call
 through these methods if they exist, or return nothing if they do not
 exist.

So, in order to use STRUTS and STRUTS-FACES with JSP 2.0 and the
Expression Language, what should I do?

a) Compile against servlet 2.4 and JSP 2.0 and adding the mentioned
methods not implemented.

b) Using the oficial binary version for both STRUTS and STRUTS-FACES

I understand that both STRUTS and STRUTS-FACES are compiled against
servlet 2.2 and 2.3 and terster for servlet 2.4 and JSP 2.0.

Regards 


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: svn commit: r155889 - struts/faces/trunk/project.xml

2005-03-02 Thread Craig McClanahan
On Wed, 02 Mar 2005 06:11:55 -, [EMAIL PROTECTED]
[EMAIL PROTECTED] wrote:
 Author: jmitchell
 Date: Tue Mar  1 22:11:55 2005
 New Revision: 155889
 
 URL: http://svn.apache.org/viewcvs?view=revrev=155889
 Log:
 fix version (Craig is this what you want to see as the version number?)
 

Works for me.  Thanks for doing this.

Craig

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-01 Thread Ramiro Alba Queipo
Hello everybody:

I was using tomcat 4.0.24 to develop an application with struts, I and
decided to migrate to tomcat 5.0.28 so as I could user EL natively from
JSP 2.0. I thought it would be a good idea to compile STRUTS 1.2.4 under
servlet 2.4 and JSP 2.0 from tomcat 5.0.28. Everything went wright
except for one class:

org/apache/struts/upload/MultipartRequestWrapper.java

It is not implementing the following methods from interface
javax.servlet.ServletRequest corresponding to servlet-api.jar of tomcat
5.0.28:
 
public String getLocalName() {}
public String getLocalAddr() {}
public int getLocalPort() {} 
public int getRemotePort() {}

To bypass the problem I created trivial methods, and then I could
complete the compilation process. Everything seams to work, but in order
to work with tomcat 5.0.x:

1) Is it a good idea to make this compilation o better to stuck with
   the official binary supplied struts.jar?

2) Is this situation going to be solved in a near feature? (I have seen
no changes in struts 1.2.6)


I have found the same problem compiling struts-faces, which seams to be
compiled with servlet and jsp apis from tomcat 5.0.25

Thanks in advance

Regards


P.D: I have posted this e-mail at user@struts.apache.org more than a
week ago and I have no answer, probably due to be a question out of
scope for that list, so I decided to post here the same question. Sorry
for the cross posting.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Compiling STRUTS and STRUTS-FACES with servlet 2.4 and jsp 2.0

2005-03-01 Thread Craig McClanahan
Doing the simple version of a change like this will cause the wrapper
class to not be compilable against a 2.2 or 2.3 version of the servlet
API any longer, which is a particular problem for the standard Struts
release.  It will require a bit more work to use reflection to call
through these methods if they exist, or return nothing if they do not
exist.

Craig


On Tue, 01 Mar 2005 14:11:41 +0100, Ramiro Alba Queipo [EMAIL PROTECTED] 
wrote:
 Hello everybody:
 
 I was using tomcat 4.0.24 to develop an application with struts, I and
 decided to migrate to tomcat 5.0.28 so as I could user EL natively from
 JSP 2.0. I thought it would be a good idea to compile STRUTS 1.2.4 under
 servlet 2.4 and JSP 2.0 from tomcat 5.0.28. Everything went wright
 except for one class:
 
 org/apache/struts/upload/MultipartRequestWrapper.java
 
 It is not implementing the following methods from interface
 javax.servlet.ServletRequest corresponding to servlet-api.jar of tomcat
 5.0.28:
 
 public String getLocalName() {}
 public String getLocalAddr() {}
 public int getLocalPort() {}
 public int getRemotePort() {}
 
 To bypass the problem I created trivial methods, and then I could
 complete the compilation process. Everything seams to work, but in order
 to work with tomcat 5.0.x:
 
 1) Is it a good idea to make this compilation o better to stuck with
the official binary supplied struts.jar?
 
 2) Is this situation going to be solved in a near feature? (I have seen
 no changes in struts 1.2.6)
 
 I have found the same problem compiling struts-faces, which seams to be
 compiled with servlet and jsp apis from tomcat 5.0.25
 
 Thanks in advance
 
 Regards
 
 P.D: I have posted this e-mail at user@struts.apache.org more than a
 week ago and I have no answer, probably due to be a question out of
 scope for that list, so I decided to post here the same question. Sorry
 for the cross posting.
 
 -
 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]



svn commit: r155883 - in struts/faces/trunk/core-library/src: java/org/apache/struts/faces/component/ java/org/apache/struts/faces/renderer/ java/org/apache/struts/faces/taglib/ test/org/apache/struts/faces/util/

2005-03-01 Thread jmitchell
Author: jmitchell
Date: Tue Mar  1 22:03:13 2005
New Revision: 155883

URL: http://svn.apache.org/viewcvs?view=revrev=155883
Log:
Fix many small improvements thanks to our new maven reports

 - unused import(s)
 - useless object creation
 - unnecessary casting


Modified:

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/BaseComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/CommandLinkComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/ErrorsComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/HtmlComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/MessageComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/StylesheetComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/WriteComponent.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/renderer/AbstractRenderer.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/renderer/CommandLinkRenderer.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/renderer/MessageRenderer.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/taglib/JavascriptValidatorTag.java

struts/faces/trunk/core-library/src/test/org/apache/struts/faces/util/MessagesMapTestCase.java

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/BaseComponent.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/BaseComponent.java?view=diffr1=155882r2=155883
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/BaseComponent.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/BaseComponent.java
 Tue Mar  1 22:03:13 2005
@@ -17,9 +17,6 @@
 package org.apache.struts.faces.component;
 
 
-import java.io.IOException;
-import java.util.Iterator;
-
 import javax.faces.component.UIOutput;
 import javax.faces.context.FacesContext;
 import javax.faces.el.ValueBinding;

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/CommandLinkComponent.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/CommandLinkComponent.java?view=diffr1=155882r2=155883
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/CommandLinkComponent.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/CommandLinkComponent.java
 Tue Mar  1 22:03:13 2005
@@ -17,9 +17,6 @@
 package org.apache.struts.faces.component;
 
 
-import java.io.IOException;
-import java.util.Iterator;
-
 import javax.faces.component.UICommand;
 import javax.faces.context.FacesContext;
 import javax.faces.el.ValueBinding;

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/ErrorsComponent.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/ErrorsComponent.java?view=diffr1=155882r2=155883
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/ErrorsComponent.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/ErrorsComponent.java
 Tue Mar  1 22:03:13 2005
@@ -17,9 +17,6 @@
 package org.apache.struts.faces.component;
 
 
-import java.io.IOException;
-import java.util.Iterator;
-
 import javax.faces.component.UIOutput;
 import javax.faces.context.FacesContext;
 import javax.faces.el.ValueBinding;

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/HtmlComponent.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/HtmlComponent.java?view=diffr1=155882r2=155883
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/HtmlComponent.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/HtmlComponent.java
 Tue Mar  1 22:03:13 2005
@@ -17,9 +17,6 @@
 package org.apache.struts.faces.component;
 
 
-import java.io.IOException;
-import java.util.Iterator;
-
 import javax.faces.component.UIOutput;
 import javax.faces.context.FacesContext;
 import javax.faces.el.ValueBinding;

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/component/MessageComponent.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core

svn commit: r155884 - in struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example: EditRegistrationAction.java EditSubscriptionAction.java IndexBacking.java LogoffAction.java LogonAction.java MainMenuBacking.java RegistrationBacking.java SaveRegistrationAction.java

2005-03-01 Thread jmitchell
Author: jmitchell
Date: Tue Mar  1 22:03:56 2005
New Revision: 155884

URL: http://svn.apache.org/viewcvs?view=revrev=155884
Log:
Fix many small improvements thanks to our new maven reports

 - unused import(s)
 - useless object creation
 - unnecessary casting


Modified:

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditRegistrationAction.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditSubscriptionAction.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/IndexBacking.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/LogoffAction.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/LogonAction.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/MainMenuBacking.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/RegistrationBacking.java

struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/SaveRegistrationAction.java

Modified: 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditRegistrationAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditRegistrationAction.java?view=diffr1=155883r2=155884
==
--- 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditRegistrationAction.java
 (original)
+++ 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditRegistrationAction.java
 Tue Mar  1 22:03:56 2005
@@ -31,7 +31,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.util.MessageResources;
 
 
 /**
@@ -81,8 +80,6 @@
throws Exception {
 
// Extract attributes we will need
-   Locale locale = getLocale(request);
-   MessageResources messages = getResources(request);
HttpSession session = request.getSession();
String action = request.getParameter(action);
if (action == null)

Modified: 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditSubscriptionAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditSubscriptionAction.java?view=diffr1=155883r2=155884
==
--- 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditSubscriptionAction.java
 (original)
+++ 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/EditSubscriptionAction.java
 Tue Mar  1 22:03:56 2005
@@ -31,7 +31,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.util.MessageResources;
 
 
 /**
@@ -81,7 +80,6 @@
 
// Extract attributes we will need
Locale locale = getLocale(request);
-   MessageResources messages = getResources(request);
HttpSession session = request.getSession();
String action = request.getParameter(action);
if (action == null) {

Modified: 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/IndexBacking.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/IndexBacking.java?view=diffr1=155883r2=155884
==
--- 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/IndexBacking.java
 (original)
+++ 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/IndexBacking.java
 Tue Mar  1 22:03:56 2005
@@ -18,8 +18,6 @@
 package org.apache.struts.webapp.example;
 
 
-import java.io.IOException;
-import javax.faces.FacesException;
 import javax.faces.context.FacesContext;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;

Modified: 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/LogoffAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/LogoffAction.java?view=diffr1=155883r2=155884
==
--- 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/LogoffAction.java
 (original)
+++ 
struts/faces/trunk/example1-webapp/src/java/org/apache/struts/webapp/example/LogoffAction.java
 Tue Mar  1 22:03:56 2005
@@ -28,7 +28,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import

svn commit: r155885 - in struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient: ContextTestCase.java LogonTestCase.java SimpleTestCase.java

2005-03-01 Thread jmitchell
Author: jmitchell
Date: Tue Mar  1 22:04:30 2005
New Revision: 155885

URL: http://svn.apache.org/viewcvs?view=revrev=155885
Log:
Many small improvements thanks to our new maven reports

 - unused import(s)
 - useless object creation
 - unnecessary casting


Modified:

struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/ContextTestCase.java

struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java

struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/SimpleTestCase.java

Modified: 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/ContextTestCase.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/ContextTestCase.java?view=diffr1=155884r2=155885
==
--- 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/ContextTestCase.java
 (original)
+++ 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/ContextTestCase.java
 Tue Mar  1 22:04:30 2005
@@ -16,31 +16,11 @@
 
 package org.apache.struts.faces.sysclient;
 
-import com.gargoylesoftware.htmlunit.ElementNotFoundException;
-import com.gargoylesoftware.htmlunit.WebClient;
-import com.gargoylesoftware.htmlunit.html.HtmlAnchor;
-import com.gargoylesoftware.htmlunit.html.HtmlBase;
-import com.gargoylesoftware.htmlunit.html.HtmlBody;
-import com.gargoylesoftware.htmlunit.html.HtmlElement;
-import com.gargoylesoftware.htmlunit.html.HtmlForm;
-import com.gargoylesoftware.htmlunit.html.HtmlHead;
-import com.gargoylesoftware.htmlunit.html.HtmlHiddenInput;
-import com.gargoylesoftware.htmlunit.html.HtmlLink;
-import com.gargoylesoftware.htmlunit.html.HtmlPage;
-import com.gargoylesoftware.htmlunit.html.HtmlPasswordInput;
-import com.gargoylesoftware.htmlunit.html.HtmlResetInput;
-import com.gargoylesoftware.htmlunit.html.HtmlSpan;
-import com.gargoylesoftware.htmlunit.html.HtmlSubmitInput;
-import com.gargoylesoftware.htmlunit.html.HtmlTextInput;
-
-import java.net.URL;
-import java.util.ArrayList;
-import java.util.Iterator;
-import java.util.List;
-
 import junit.framework.Test;
-import junit.framework.TestCase;
 import junit.framework.TestSuite;
+
+import com.gargoylesoftware.htmlunit.html.HtmlSpan;
+import com.gargoylesoftware.htmlunit.html.HtmlSubmitInput;
 
 
 /**

Modified: 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java?view=diffr1=155884r2=155885
==
--- 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java
 (original)
+++ 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java
 Tue Mar  1 22:04:30 2005
@@ -16,32 +16,21 @@
 
 package org.apache.struts.faces.sysclient;
 
-import com.gargoylesoftware.htmlunit.ElementNotFoundException;
-import com.gargoylesoftware.htmlunit.WebClient;
+import java.util.List;
+
+import junit.framework.Test;
+import junit.framework.TestSuite;
+
 import com.gargoylesoftware.htmlunit.html.HtmlAnchor;
-import com.gargoylesoftware.htmlunit.html.HtmlBase;
-import com.gargoylesoftware.htmlunit.html.HtmlBody;
 import com.gargoylesoftware.htmlunit.html.HtmlElement;
 import com.gargoylesoftware.htmlunit.html.HtmlForm;
-import com.gargoylesoftware.htmlunit.html.HtmlHead;
 import com.gargoylesoftware.htmlunit.html.HtmlHiddenInput;
-import com.gargoylesoftware.htmlunit.html.HtmlLink;
-import com.gargoylesoftware.htmlunit.html.HtmlPage;
 import com.gargoylesoftware.htmlunit.html.HtmlPasswordInput;
 import com.gargoylesoftware.htmlunit.html.HtmlResetInput;
 import com.gargoylesoftware.htmlunit.html.HtmlSpan;
 import com.gargoylesoftware.htmlunit.html.HtmlSubmitInput;
 import com.gargoylesoftware.htmlunit.html.HtmlTextInput;
 
-import java.net.URL;
-import java.util.ArrayList;
-import java.util.Iterator;
-import java.util.List;
-
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
-
 
 /**
  * pTest case for a logon form that accepts a username and password./p
@@ -165,7 +154,7 @@
 
 HtmlSpan span = null;
 
-HtmlElement html = (HtmlElement) page;
+HtmlElement html = page;
 assertEquals(html, html.getTagName());
 assertEquals(http://www.w3.org/1999/xhtml;, 
html.getAttributeValue(xmlns));
 

Modified: 
struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/SimpleTestCase.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/sysclient-app/src/java/org/apache/struts/faces/sysclient/SimpleTestCase.java?view=diffr1=155884r2=155885

svn commit: r155886 - in struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2: EditRegistrationAction.java EditSubscriptionAction.java LogoffAction.java LogonAction.java

2005-03-01 Thread jmitchell
Author: jmitchell
Date: Tue Mar  1 22:04:47 2005
New Revision: 155886

URL: http://svn.apache.org/viewcvs?view=revrev=155886
Log:
Many small improvements thanks to our new maven reports

 - unused import(s)
 - useless object creation
 - unnecessary casting


Modified:

struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditRegistrationAction.java

struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditSubscriptionAction.java

struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogoffAction.java

struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogonAction.java

Modified: 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditRegistrationAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditRegistrationAction.java?view=diffr1=155885r2=155886
==
--- 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditRegistrationAction.java
 (original)
+++ 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditRegistrationAction.java
 Tue Mar  1 22:04:47 2005
@@ -31,7 +31,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.util.MessageResources;
 
 
 /**
@@ -82,7 +81,6 @@
 
// Extract attributes we will need
Locale locale = getLocale(request);
-   MessageResources messages = getResources(request);
HttpSession session = request.getSession();
String action = request.getParameter(action);
if (action == null)

Modified: 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditSubscriptionAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditSubscriptionAction.java?view=diffr1=155885r2=155886
==
--- 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditSubscriptionAction.java
 (original)
+++ 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/EditSubscriptionAction.java
 Tue Mar  1 22:04:47 2005
@@ -31,7 +31,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.util.MessageResources;
 
 
 /**
@@ -81,7 +80,6 @@
 
// Extract attributes we will need
Locale locale = getLocale(request);
-   MessageResources messages = getResources(request);
HttpSession session = request.getSession();
String action = request.getParameter(action);
if (action == null) {

Modified: 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogoffAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogoffAction.java?view=diffr1=155885r2=155886
==
--- 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogoffAction.java
 (original)
+++ 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogoffAction.java
 Tue Mar  1 22:04:47 2005
@@ -28,7 +28,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.util.MessageResources;
 
 
 /**
@@ -77,7 +76,6 @@
 
// Extract attributes we will need
Locale locale = getLocale(request);
-   MessageResources messages = getResources(request);
HttpSession session = request.getSession();
User user = (User) session.getAttribute(Constants.USER_KEY);
 

Modified: 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogonAction.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogonAction.java?view=diffr1=155885r2=155886
==
--- 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogonAction.java
 (original)
+++ 
struts/faces/trunk/example2-webapp/src/java/org/apache/struts/webapp/example2/LogonAction.java
 Tue Mar  1 22:04:47 2005
@@ -31,7 +31,6 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.apache.struts.util.MessageResources;
 import org.apache.struts.util.ModuleException;
 
 
@@ -80,7 +79,6 @@
 
// Extract attributes we will need
Locale

svn commit: r155887 - struts/faces/trunk/project.xml

2005-03-01 Thread jmitchell
Author: jmitchell
Date: Tue Mar  1 22:05:30 2005
New Revision: 155887

URL: http://svn.apache.org/viewcvs?view=revrev=155887
Log:
initial project.xml that builds the core-library (using myfaces)

Added:
struts/faces/trunk/project.xml   (with props)

Added: struts/faces/trunk/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/project.xml?view=autorev=155887
==
--- struts/faces/trunk/project.xml (added)
+++ struts/faces/trunk/project.xml Tue Mar  1 22:05:30 2005
@@ -0,0 +1,147 @@
+?xml version=1.0 encoding=UTF-8?
+project
+!-- 
+/*
+ * Copyright 2001-2004 The Apache Software Foundation.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the License);
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an AS IS BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+ --
+
+
+  extendbuild/project.xml/extend
+  idstruts-faces/id
+  nameStruts JSF Integration Library/name
+  
+  packageorg.apache.struts.faces/package
+  packageGroups
+packageGroup
+  titleStruts Faces/title
+  packagesorg.apache.struts.faces/packages
+/packageGroup
+  /packageGroups
+
+  shortDescriptionStruts JSF Integration Library./shortDescription
+
+  description
+  This package contains an add-on library that supports the use of
+  JavaServer Faces (JSF) user interface technology in a Struts based web
+  application, in place of the Struts custom tag libraries.  As a proof of
+  concept, it also includes the canonical struts-example example
+  web application, converted to use JSF tags, as well as tags from the
+  JSP Standard Tag Library (JSTL), version 1.0 or later.  It also includes
+  a very basic Tiles based application, modified in a similar manner.
+  
+  The Struts-Faces Integration Library should work with any implementation
+  of JavaServer Faces, version 1.0 or later.  It has primarily been tested
+  against version 1.1 of the JavaServer Faces reference implementation,
+  available at:
+  
+http://java.sun.com/j2ee/javaserverfaces/
+  
+  /description
+
+  repository
+
connectionscm|svn|http|//svn.apache.org/repos/asf/struts/faces/trunk/connection
+
developerConnectionscm|svn|https|//svn.apache.org/repos/asf/struts/faces/trunk/developerConnection
+urlhttp://svn.apache.org/repos/asf/struts/faces/trunk/url
+  /repository
+  
+  versions/
+  
+  branches/
+  
+  dependencies
+  
+   dependency
+ groupIdstruts/groupId
+ artifactIdstruts/artifactId
+ version1.1/version
+ properties
+   war.bundletrue/war.bundle
+ /properties
+  urlhttp://struts.apache.org//url
+   /dependency
+
+   dependency
+ groupIdstruts/groupId
+ artifactIdstruts-tiles/artifactId
+ version1.3.0-dev/version
+ properties
+   war.bundletrue/war.bundle
+ /properties
+  urlhttp://struts.apache.org//url
+   /dependency
+
+   dependency
+ groupIdmyfaces/groupId
+ artifactIdmyfaces-jsf-api/artifactId
+ version1.0.8-beta/version
+ properties
+   war.bundletrue/war.bundle
+ /properties
+  urlhttp://incubator.apache.org/myfaces/index.html/url
+   /dependency
+
+dependency
+  groupIdcommons-httpclient/groupId
+  artifactIdcommons-httpclient/artifactId
+  version2.0.1/version
+  urlhttp://jakarta.apache.org/commons/httpclient//url
+  properties
+war.bundletrue/war.bundle
+cactus.bundletrue/cactus.bundle
+  /properties
+/dependency
+
+dependency
+  groupIdhtmlunit/groupId
+  artifactIdhtmlunit/artifactId
+  version1.2.3/version
+  urlhttp://htmlunit.sourceforge.net//url
+  properties
+war.bundletrue/war.bundle
+cactus.bundletrue/cactus.bundle
+  /properties
+/dependency
+
+
+
+  /dependencies
+
+  build
+nagEmailAddress/
+sourceDirectory${basedir}/core-library/src/java/sourceDirectory
+
unitTestSourceDirectory${basedir}/core-library/src/test/unitTestSourceDirectory
+
+unitTest
+  includes
+include**/*TestCase/include
+  /includes
+/unitTest
+
+resources
+  resource
+directory${basedir}/core-library/src/java/directory
+includes
+  include**/*.properties/include
+/includes
+  /resource
+/resources
+  /build
+
+  reports
+   !-- reports specific to faces--
+   
+  /reports
+/project

Propchange: struts/faces/trunk/project.xml

svn commit: r155889 - struts/faces/trunk/project.xml

2005-03-01 Thread jmitchell
Author: jmitchell
Date: Tue Mar  1 22:11:55 2005
New Revision: 155889

URL: http://svn.apache.org/viewcvs?view=revrev=155889
Log:
fix version (Craig is this what you want to see as the version number?)

Modified:
struts/faces/trunk/project.xml

Modified: struts/faces/trunk/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/project.xml?view=diffr1=155888r2=155889
==
--- struts/faces/trunk/project.xml (original)
+++ struts/faces/trunk/project.xml Tue Mar  1 22:11:55 2005
@@ -22,6 +22,7 @@
   extendbuild/project.xml/extend
   idstruts-faces/id
   nameStruts JSF Integration Library/name
+  currentVersion1.0-dev/currentVersion
   
   packageorg.apache.struts.faces/package
   packageGroups



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] New: - struts-faces is incompatible with myfaces

2005-02-21 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33672.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672

   Summary: struts-faces is incompatible with myfaces
   Product: Struts
   Version: Nightly Build
  Platform: PC
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: P2
 Component: Struts-Faces Library
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


This bug is esentially the same as bug 32014 , but I believe the descriptions
lead one to believe that it only has to do with Tiles integration, whereas I
believe that FacesRequestProcessor is also incompatible with myfaces.

The example1 webapp included with struts-faces does not expose this bug because,
if I understand it correctly, it never directly invokes a struts action, but
instead struts actions are invoked through the tags-faces taglib, which is going
through FacesServlet first.

But, if you wrote an application which calls a struts action initially (as
opposed to being forwarded to from FacesServlet), FacesRequestProcessor will
fail against myfaces in the same manner that FacesTilesRequestProcessor fails
(as described in bug 32014).

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



  1   2   3   4   >