Compatibility with Servlet API 2.4
--
Key: MYFACES-3374
URL: https://issues.apache.org/jira/browse/MYFACES-3374
Project: MyFaces Core
Issue Type: Task
Reporter: Udo Schnurpfeil
Hi Gregor,
This issue seems to be a race condition resulting of the combination
of xerces closing every InputStream it gets and Jetty failing to
handle this.
In the stacktrace you can see that there is a call to
ServletContext.getResource() via ExternalContext. Also you can see
that it currently
Hi!
Currently we have really long and very descriptive variable names in MyFaces.
I personally like that, but due to that we are really often exceeding the 120
character per line.
Thus my question: should we extend this from 120 to 160 characters being
allowed per line?
[+1] yup make 160
-1
Am 26.10.2011 14:18, schrieb Mark Struberg:
Hi!
Currently we have really long and very descriptive variable names in MyFaces.
I personally like that, but due to that we are really often exceeding the 120
character per line.
Thus my question: should we extend this from 120 to 160
MyFaces can incorrectly determine that Portlets are not supported
-
Key: MYFACES-3375
URL: https://issues.apache.org/jira/browse/MYFACES-3375
Project: MyFaces Core
Issue Type:
Hello Everyone,
I've opened MYFACES-3375 and would like to propose a solution and receive
some feedback. The proposed solution is as follows:
1) New Class PortletBridgeDetector
- bridgeEnabled() - This method will use the Application Class loader
to determine if a Portlet Bridge Class
Hi!
checkstyle just found another kind of gems
for example in AbstractAttributeMap.java
private CollectionV _values;
@Override
public CollectionV values()
{
return (_values != null) ? _values : (_values = new Values());
}
This code is not thread safe!
_values
Hello,
I think I have found a bug, but rather than open an issue I prefer to ask it
here first.
I'm writing a custom version of UIData component extending the default UIData
class. I want to register a DataModelListener in order to be notified on row
index changes.
I have overridden the
ValueExpressions for SelectItems are evaluated too often for checboxes or
radios with spread layout
---
Key: TOMAHAWK-1605
URL:
Hi folks!
I see a lot of commented out code which is many years old.
I'm highly in favour to just delete code we don't need anymore!
IF you only temporarily comment out something, then please mark it with
/*X TODO some comment
or
//X TODO some comment
otherwise comments must ONLY be
+1 since you can always get it back out of svn if you need it.
On Wed, Oct 26, 2011 at 12:08 PM, Mark Struberg strub...@yahoo.de wrote:
Hi folks!
I see a lot of commented out code which is many years old.
I'm highly in favour to just delete code we don't need anymore!
IF you only
+1
Am 26.10.2011 18:20, schrieb Mike Kienenberger:
+1 since you can always get it back out of svn if you need it.
On Wed, Oct 26, 2011 at 12:08 PM, Mark Strubergstrub...@yahoo.de wrote:
Hi folks!
I see a lot of commented out code which is many years old.
I'm highly in favour to just
CSS of tc:button needs to be refactored
-
Key: TOBAGO-1042
URL: https://issues.apache.org/jira/browse/TOBAGO-1042
Project: MyFaces Tobago
Issue Type: Bug
Affects Versions: 1.5.0-beta-1
Problem with internal dispaching (e.g. to error pages) when using CDI/OWB with
old containers
-
Key: TOBAGO-1043
URL: https://issues.apache.org/jira/browse/TOBAGO-1043
[
https://issues.apache.org/jira/browse/TOBAGO-1043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13136153#comment-13136153
]
Udo Schnurpfeil commented on TOBAGO-1043:
-
I need it for Tomcat 5.5 and WebSphere
[
https://issues.apache.org/jira/browse/TOBAGO-1043?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Udo Schnurpfeil resolved TOBAGO-1043.
-
Resolution: Fixed
Problem with internal dispaching (e.g. to error pages) when
[
https://issues.apache.org/jira/browse/TOBAGO-1042?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Udo Schnurpfeil resolved TOBAGO-1042.
-
Resolution: Fixed
Fix Version/s: 1.5.0-beta-2
CSS of tc:button needs to be
+1
On Wed, Oct 26, 2011 at 7:23 AM, Michael Kurz michi.k...@gmx.at wrote:
-1
Am 26.10.2011 14:18, schrieb Mark Struberg:
Hi!
Currently we have really long and very descriptive variable names in
MyFaces.
I personally like that, but due to that we are really often exceeding the
120
Hi folks!
I'm down to 24 checkstyle issues (with LineLength=160, otherwise ~600 more)
which are really non-trivial.
A few of them are 'equals() without hashCode()' which are definitely problematic
Then we have a few variable namings which I didn't want to change without
talking to you.
And of
+1
regards,
gerhard
http://www.irian.at
Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German
Professional Support for Apache MyFaces
2011/10/26 Michael Kurz michi.k...@gmx.at
+1
Am 26.10.2011 18:20, schrieb Mike Kienenberger:
+1 since you can always get
Hi
I'm keeping an eye on the changes done. I don't remember any problem
about 'double lock' on myfaces. Could you please be more specific? In
my understanding everything works as expected.
regards,
Leonardo Uribe
2011/10/26 Mark Struberg strub...@yahoo.de:
Hi folks!
I'm down to 24
+0
2011/10/26 Grant Smith work.gr...@gmail.com:
+1
On Wed, Oct 26, 2011 at 7:23 AM, Michael Kurz michi.k...@gmx.at wrote:
-1
Am 26.10.2011 14:18, schrieb Mark Struberg:
Hi!
Currently we have really long and very descriptive variable names in
MyFaces.
I personally like that, but due
Again (think I posted this before already), a very good source for java mem
spec behaviour is the comment of Brian Goetz
http://www.cs.umd.edu/~pugh/java/memoryModel/jsr-133-faq.html
This also explains why you need to declare the field volatile - don't wanna
stress this, but I told this also
Yes I know that, but where in myfaces we have a problem like that? so
we can discuss it in deep.
2011/10/26 Mark Struberg strub...@yahoo.de:
Again (think I posted this before already), a very good source for java mem
spec behaviour is the comment of Brian Goetz
For example
NavigationHandlerImpl#_navigationCases
I already fixed a few occurrences.
There are also a few other checkstyle errors waiting to get fixed like methods
which are 450 lines long..
Would be cool if you could take a look at it - txs!
LieGrue,
strub
- Original Message -
Create abstract test classes that runs MyFaces Core as in a container
-
Key: MYFACES-3376
URL: https://issues.apache.org/jira/browse/MYFACES-3376
Project: MyFaces Core
[
https://issues.apache.org/jira/browse/MYFACES-3376?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Leonardo Uribe resolved MYFACES-3376.
-
Resolution: Fixed
Fix Version/s: 2.1.4
2.0.10
Create
Hi
In NavigationHandlerImpl#_navigationCases, that var should be
volatile, but note if it is not nothing happens, just the code will be
executed more times than necessary.
regards,
Leonardo Uribe
2011/10/26 Mark Struberg strub...@yahoo.de:
For example
NavigationHandlerImpl#_navigationCases
This is an argument which I hear often. But it's actually wrong. Let's assume:
Map getX() {
if (x=null) {
synchronized(this) {
if (x=null) {
x=new HashMap();
}
}
}
return x;
}
People argue that the worst what can happen is that x will get created twice.
Hi
It is true, but the case we have does not manipulate _navigationCases
directly, instead an new map is created and filled each time and only
before end the synchronized block it is assigned. In this case we
don't have any additional manipulation of _navigationCases after that
too.
regards,
+1 Be ruthless !
On Wed, Oct 26, 2011 at 11:35 AM, Gerhard Petracek
gerhard.petra...@gmail.com wrote:
+1
regards,
gerhard
http://www.irian.at
Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German
Professional Support for Apache MyFaces
2011/10/26
[
https://issues.apache.org/jira/browse/TRINIDAD-2140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13136720#comment-13136720
]
Pavitra Subramaniam commented on TRINIDAD-2140:
---
Another recommendation
Hi
There are two classes:
org.apache.myfaces.util.AbstractAttributeMap
and
org.apache.myfaces.util.AbstractThreadSafeAttributeMap
You're right, there is a bug, but apply them please on
AbstractThreadSafeAttributeMap. Note ApplicationMap and SessionMap
wrappers used by ExternalContext inherits
Hi
The problem with this is we are talking about two different things.
The code that detects a portlet environment was designed for:
- JSR-301: Portlet 1.0 Bridge for JavaServer Faces 1.2
- JSR-329: Portlet 2.0 Bridge for JavaServer Faces 1.2
- Portlet 2.0 Bridge for JavaServer Faces 2.0
But
34 matches
Mail list logo
