[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-1009) optionally login-require in a more specific page should be able to override a wildcard login-require
by Leo Baschy (JIRA)
optionally login-require in a more specific page should be able to override a wildcard login-require
----------------------------------------------------------------------------------------------------
Key: JBSEAM-1009
URL: http://jira.jboss.com/jira/browse/JBSEAM-1009
Project: JBoss Seam
Issue Type: Patch
Components: Security
Affects Versions: 1.2.0.GA
Environment: all
Reporter: Leo Baschy
This should be optional to switch on, so no one's existing expectations of security get broken.
The point is about having a generic wildcard <page view-id="*" scheme="http" login-required="true"> to secure the whole site, and then allowing specific pages or specific wildcards to have login-required="false". E.g. for a registration (with preview) section as one cannot be logged in if one isn't registered yet.
Some may suggest instead forcing pages into dedicated secure and not-secure directories, but in reality if there are multiple reasons to force pages into directories different ways (security, hyperlink management, publishability of URLs, etc.), one cannot serve all of them.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 3 months
[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-808) StatckOverflowError if factory specifies it's component
by John Ray (JIRA)
StatckOverflowError if factory specifies it's component
-------------------------------------------------------
Key: JBSEAM-808
URL: http://jira.jboss.com/jira/browse/JBSEAM-808
Project: JBoss Seam
Issue Type: Bug
Environment: Latest from CVS
Reporter: John Ray
I accidently specified the name of my component instead of a context variable and this resulted in a StatckOverflowError. For example
@Stateful
@Name("foo")
@Scope(ScopeType.EVENT)
public class FooAction implements Foo {
@Out("bar")
private String bar;
@Factory("foo")
public void initBar() {
...
}
}
I'd sugest changing the code in org.jboss.seam.Component at line 490 to include an if/throw. I highlighted the 2 new lines with an asterisk
if ( method.isAnnotationPresent(org.jboss.seam.annotations.Factory.class) )
{
Init init = (Init) applicationContext.get( Seam.getComponentName(Init.class) ); //can't use Init.instance() here 'cos of unit tests
String contextVariable = toName( method.getAnnotation(org.jboss.seam.annotations.Factory.class).value(), method );
* if (contextVariable.equals(name))
* throw new IllegalStateException("@Factory method can not be for it's own component: " + name);
init.addFactoryMethod(contextVariable, method, this);
}
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 4 months
[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-1000) Propagation of conversations with pageflow in testing environment.
by Denis Karpov (JIRA)
Propagation of conversations with pageflow in testing environment.
------------------------------------------------------------------
Key: JBSEAM-1000
URL: http://jira.jboss.com/jira/browse/JBSEAM-1000
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 1.2.0.GA
Environment: windows 2000 jboss-4.0.5.GA
Reporter: Denis Karpov
I think there is bug in testing environment (propagation of conversations with pageflow does not work)
In this test there is 2 requests .
In the first I start conversation and pageflow
In the second conversation is lost.
If I do not start pageflow then conversation propagates.
Code:
public class TOrdTest extends SeamTest{
private static final Log log = LogFactory.getLog( TOrdTest.class );
@Test
public void Test1() throws Exception
{
String id = new FacesRequest(){
@Override
protected void invokeApplication() throws Exception {
Conversation.instance().begin();
Pageflow.instance().begin("val_buy"); // uses start-state
setOutcome("browse");
log.info("##### 1 Conversation_ID "+ Conversation.instance().getId());
assert isLongRunningConversation();
}
@Override
protected void renderResponse() throws Exception {
}
}.run();
id = new FacesRequest("/exchange/buy_readonly.xhtml",id){
@Override
protected void beforeRequest()
{
log.info("#####beforeRequest Conversation_ID "+ getConversationId());
}
@Override
protected void applyRequestValues() throws Exception {
log.info("##### 2 Conversation_ID "+ Conversation.instance().getId()+" "+getConversationId());
}
@Override
protected void invokeApplication() throws Exception {
//Conversation.instance().beginNested();
log.info("##### 3 Conversation_ID "+ Conversation.instance().getId()+" "+getConversationId());
assert isLongRunningConversation();
}
}.run();
}
}
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 5 months
[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-829) ICEfaces Tree Component needs reworking
by Gregory Dick (JIRA)
ICEfaces Tree Component needs reworking
---------------------------------------
Key: JBSEAM-829
URL: http://jira.jboss.com/jira/browse/JBSEAM-829
Project: JBoss Seam
Issue Type: Bug
Components: ICE Faces Integration
Reporter: Gregory Dick
Assigned To: Gregory Dick
Some comments
Plain tree component with label:
<ice:tree value="#{nodeTree.model}" var="item">
<ice:treeNode>
<f:facet name="content">
<ice:commandLink actionListener="#
{item.userObject.nodeClicked}">
<ice:outputText value="#
{item.userObject.text}"/>
</ice:commandLink>
</f:facet>
</ice:treeNode>
</ice:tree>
Renders HTML with whitespace between <img> and <a> tags, resulting in
broken <div> padding and unwanted spaces:
<div class="iceTreeRow" id="_id21:_id22-div-2"
name="treeNodeDiv"><img alt="" border="0" src="./xmlhttp/css/xp/css-
images/tree_line_blank.gif"></img>
<img alt="" border="0" src="./xmlhttp/css/xp/css-images/
tree_line_last_node.gif"></img>
Should be:
<div class="iceTreeRow" id="_id21:_id22-div-2"
name="treeNodeDiv"><img alt="" border="0" src="./xmlhttp/css/xp/css-
images/tree_line_blank.gif"></img><img alt="" border="0" src="./
xmlhttp/css/xp/css-images/tree_line_last_node.gif"></img>
and so on... the only whitespaces should (could) be between </
div><div> tags, nowhere else. The trick with whitespace:nowrap in the
CSS is not going to cut it, there are still spaces left that I don't
want.
---------------------------------------------------
The xp/tree_nav_top_close_no_siblings.gif GIF is broken, it is not
transparent and the icon is not centered on the canvas.
---------------------------------------------------
After clicking on the tree and playing with the XHTML a little, I got
this:
03:36:56,599 ERROR [[Blocking Servlet]] Servlet.service() for servlet
Blocking Servlet threw exception
java.lang.RuntimeException: viewNumber 1 update queue exceeded 26
at
com.icesoft.faces.webapp.xmlhttp.BlockingResponseState.writeElement
(BlockingResponseState.java:183)
at com.icesoft.faces.context.DOMResponseWriter.writeDOM
(DOMResponseWriter.java:375)
at com.icesoft.faces.application.D2DViewHandler.renderView
(D2DViewHandler.java :161)
Had to restart the server.
---------------------------------------------------
To collapse a node I can click on the [-] icon of the node. Expanding
with the then displayed [+] icon does not work, although I see a
submit on the server log. If I click on the commandLink (the label)
instead, the node expands but I also get the select event for that
node. This is either a simple copy/paste bug or inconsistent UI
behavior.
---------------------------------------------------
Did not find a way (documentation doesn't mention it) to customize
the images that are rendered by components without CSS, for example,
the expand/collapse icons of a tree. After reading the source and API
doc of the Tree component, I found the setImageDir() method, this
should be documented better in the reference.
---------------------------------------------------
The data binding for the tree is questionable. First, the idea of
using the Swing datamodels is good, in fact, JSF should not have its
own UI datamodels at all and every UI component should be able to use
the Swing datamodels. This allows us to expose a UI datamodel on
backing beans (EJBs in Seam) without requiring extension or inclusion
of any vendor-specific API or JAR in the EJB deployment module. Most
JSF component providers do not realize this and require custom
datamodels.
I can't list everything that is wrong with the tree databinding,
short list:
- Finally, the UI component does no longer require DefaultTreeModel
but the TreeModel interface (good)
- Every object in the TreeModel is cast to a DefaultMutableTreeNode,
this is not an interface! I want to use my own TreeNode implementation!
- Every DefaultMutableTreeNode needs a wrapped IceUserObject, this is
completely unecessary if I'd be able to use my own TreeNode
implementation (maybe I can also implement an IceTreeNode interface
for some UI binding).
- This would also solve the lazy loading FAQs on the Icefaces forum,
the approach with event listeners in the IceUserObject is a non-
starter, my TreeModel/TreeNode implementation could do that just fine
- There are unused classes such as TreeDataModel hanging around the
icefaces package, these only add to the confusion
Summary: This needs a serious redesign before it is usable.
---------------------------------------------------
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 7 months
[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-715) Allow multiple pages.xml files
by Stephan Bublava (JIRA)
Allow multiple pages.xml files
------------------------------
Key: JBSEAM-715
URL: http://jira.jboss.com/jira/browse/JBSEAM-715
Project: JBoss Seam
Issue Type: Feature Request
Components: JSF
Affects Versions: 1.1.1.GA
Reporter: Stephan Bublava
Currently it's possible to use one global pages.xml file or one file per page. I believe that both options are not well-suited for large applications.
Consider an application like:
/catalog/foo.xhtml
/bar.xhtml
/...
/basket/foo.xhtml
/bar.xhtml
/...
/admin/foo.xhtml
/bar.xhtml
/...
In this case using a global file would be too convoluted, but at the same time one file per page is too fine grained.
It would be great to allow one file per directory ("module"), either using a similar naming convention to page.xml or by adding an "import" tag to pages.xml.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 8 months