-
1. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
tremes May 20, 2013 3:33 AM (in response to larshe)Hi!
It really looks like that this bug was introduced with the fix for issue you mentioned. I created https://issues.jboss.org/browse/JBSEAM-5097 and I'll do further investigation.
-
2. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
larshe May 20, 2013 4:05 AM (in response to tremes)Hi Tomas,
Thank you for your help.
/Lars
-
3. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
manarh May 20, 2013 4:25 AM (in response to larshe)Lars, I was fixing rewriting of URL in Seam UI command components. The URL of action method should not be encoded. So there is %3A instead of ':' in URL generated in s:link/button.
Unfortunately SafeActions is a pure letter to letter comparison and it doesn' expect encoded URL letter. I have the integration test, which passes for JBSEAM-5085's fix, so I need to look deeper why it was encoded where it shouldn't be.
Thanks for catching the problem. I appreciate your post.
Cheers.
-
4. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
larshe May 21, 2013 1:34 AM (in response to manarh)Marek, glad to help and thanks for the explanation. /Lars
-
5. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
hdu May 21, 2013 3:49 PM (in response to manarh)This error also seems to happen when s:link has an action:
<s:link action="#{authenticator.logout()}" value="Logout" />
15:14:51,682 ERROR [org.jboss.seam.exception.Exceptions] (http-localhost-127.0.0.1-8443-4) handled and logged exception: java.lang.IllegalArgumentException: Invalid action method home.xhtml%3Aauthenticator.logout%28%29
at org.jboss.seam.navigation.SafeActions.isActionSafe(SafeActions.java:62) [jboss-seam-2.3.1-SNAPSHOT.jar:2.3.1-SNAPSHOT]
The same thing is happening with the s:button action.
-
6. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
manarh May 22, 2013 2:32 AM (in response to hdu)yes, if I wasn't clear enough, it was related to checking encoded action URL. So it was not related to taskInstance or other attributes in s:link/button, but only to action attribute of Seam command UI controls ;-)
-
7. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
larshe May 22, 2013 2:07 PM (in response to manarh)Many thanks for solving this so quickly. I Just built using the new snapshot and it now working fine again :-) /Lars
-
8. Re: Latest 2.3.1.CR1 snapshot - IllegalArgumentException with s:link
hdu May 22, 2013 3:46 PM (in response to manarh)Yes, thanks for the quick fix.