You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sk...@apache.org on 2008/07/13 16:11:49 UTC
svn commit: r676323 - in
/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra:
connectionManager/ conversation/ conversation/jsf/
conversation/jsf/components/facelets/ conversation/spring/
frameworkAdapter/ frameworkAda...
Author: skitching
Date: Sun Jul 13 07:11:48 2008
New Revision: 676323
URL: http://svn.apache.org/viewvc?rev=676323&view=rev
Log:
Code style fixes only
Modified:
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/connectionManager/DisconnectableConnectionFactory.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/AccessScopeManager.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationContext.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationManager.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationRequestParameterProvider.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/_JsfConversationUtils.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/components/facelets/ConverterTagHandler.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/AbstractSpringOrchestraScope.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/BeanDefinitionConversationNameAttrDecorator.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/OrchestraAdvisorBeanPostProcessor.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/PersistenceContextConversationInterceptor.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/ScopedBeanTargetSource.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/_SpringUtils.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/FrameworkAdapter.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapter.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapterFilter.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/jsf/JsfFrameworkAdapterFilter.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/springBasic/SpringBasicFrameworkAdapterFilter.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/_ReentrantLock.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/ConfigUtils.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/FrameworkAdapterRequestHandler.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterProviderManager.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterResponseWrapper.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterServletFilter.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/jsf/RequestParameterFacesContextFactory.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/CompositeViewControllerNameMapper.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerPhaseListener.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerVariableResolver.java
myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/spring/SpringViewControllerScope.java
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/connectionManager/DisconnectableConnectionFactory.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/connectionManager/DisconnectableConnectionFactory.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/connectionManager/DisconnectableConnectionFactory.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/connectionManager/DisconnectableConnectionFactory.java Sun Jul 13 07:11:48 2008
@@ -95,7 +95,8 @@
}
catch (SQLException e)
{
- LogFactory.getLog(DisconnectableConnectionFactory.class).warn(e.getLocalizedMessage(), e);
+ LogFactory.getLog(DisconnectableConnectionFactory.class)
+ .warn(e.getLocalizedMessage(), e);
}
}
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/AccessScopeManager.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/AccessScopeManager.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/AccessScopeManager.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/AccessScopeManager.java Sun Jul 13 07:11:48 2008
@@ -75,7 +75,8 @@
//
// This is a little dodgy as settings made through the new AccessScopeManage
// bean will will now be silently ignored.
- FlashScopeManagerConfiguration cfg = (FlashScopeManagerConfiguration) fa.getBean(FlashScopeManagerConfiguration.class.getName());
+ FlashScopeManagerConfiguration cfg = (FlashScopeManagerConfiguration) fa.getBean(
+ FlashScopeManagerConfiguration.class.getName());
if (cfg != null)
{
manager = new AccessScopeManager();
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationContext.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationContext.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationContext.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationContext.java Sun Jul 13 07:11:48 2008
@@ -308,7 +308,8 @@
{
touch();
- Conversation[] convs = (Conversation[]) conversations.values().toArray(new Conversation[conversations.size()]);
+ Conversation[] convs = (Conversation[]) conversations.values().toArray(
+ new Conversation[conversations.size()]);
return Arrays.asList(convs).iterator();
}
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationManager.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationManager.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationManager.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationManager.java Sun Jul 13 07:11:48 2008
@@ -108,7 +108,8 @@
}
}
- ConversationManager conversationManager = (ConversationManager) frameworkAdapter.getSessionAttribute(CONVERSATION_MANAGER_KEY);
+ ConversationManager conversationManager = (ConversationManager) frameworkAdapter.getSessionAttribute(
+ CONVERSATION_MANAGER_KEY);
if (conversationManager == null && create)
{
// TODO: do not call new directly here, as it makes it impossible to configure
@@ -140,8 +141,10 @@
{
if (fa.containsRequestParameterAttribute(CONVERSATION_CONTEXT_PARAM))
{
- String urlConversationContextId = fa.getRequestParameterAttribute(CONVERSATION_CONTEXT_PARAM).toString();
- conversationContextId = new Long(Long.parseLong(urlConversationContextId, Character.MAX_RADIX));
+ String urlConversationContextId = fa.getRequestParameterAttribute(
+ CONVERSATION_CONTEXT_PARAM).toString();
+ conversationContextId = new Long(
+ Long.parseLong(urlConversationContextId, Character.MAX_RADIX));
}
}
return conversationContextId;
@@ -247,7 +250,8 @@
{
synchronized (this)
{
- ConversationContext conversationContext = (ConversationContext) conversationContexts.get(conversationContextId);
+ ConversationContext conversationContext = (ConversationContext) conversationContexts.get(
+ conversationContextId);
if (conversationContext == null)
{
conversationContext = new ConversationContext(null, conversationContextId.longValue());
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationRequestParameterProvider.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationRequestParameterProvider.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationRequestParameterProvider.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/ConversationRequestParameterProvider.java Sun Jul 13 07:11:48 2008
@@ -39,7 +39,7 @@
ConversationManager.CONVERSATION_CONTEXT_PARAM
};
- private final static ThreadLocal inSeparationMode = new ThreadLocal();
+ private final static ThreadLocal inSeparationModeThreadLocal = new ThreadLocal();
/**
* Update a threadlocal flag indicating whether URLs written to the
@@ -58,7 +58,7 @@
{
// TODO: consider using a request-scope variable rather than a
// ThreadLocal; less damage if the flag is not reset..
- inSeparationMode.set(separationMode ? Boolean.TRUE : Boolean.FALSE);
+ inSeparationModeThreadLocal.set(separationMode ? Boolean.TRUE : Boolean.FALSE);
}
/**
@@ -67,7 +67,7 @@
*/
public static boolean isInSeparationMode()
{
- return Boolean.TRUE.equals(inSeparationMode.get());
+ return Boolean.TRUE.equals(inSeparationModeThreadLocal.get());
}
public String getFieldValue(String field)
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/_JsfConversationUtils.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/_JsfConversationUtils.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/_JsfConversationUtils.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/_JsfConversationUtils.java Sun Jul 13 07:11:48 2008
@@ -64,7 +64,8 @@
/**
* Find a child start or end conversation component for the given conversation name
*/
- public static AbstractConversationComponent findEndConversationComponent(UIComponent component, String conversationName)
+ public static AbstractConversationComponent findEndConversationComponent(
+ UIComponent component, String conversationName)
{
Iterator iterComponents = component.getFacetsAndChildren();
while (iterComponents.hasNext())
@@ -96,7 +97,8 @@
/**
* end and restart a conversation
*/
- public static void endAndRestartConversation(FacesContext context, String conversationName, Boolean restart, MethodBinding restartAction)
+ public static void endAndRestartConversation(FacesContext context,
+ String conversationName, Boolean restart, MethodBinding restartAction)
{
ConversationManager conversationManager = ConversationManager.getInstance();
Conversation conversation = conversationManager.getConversation(conversationName);
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/components/facelets/ConverterTagHandler.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/components/facelets/ConverterTagHandler.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/components/facelets/ConverterTagHandler.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/components/facelets/ConverterTagHandler.java Sun Jul 13 07:11:48 2008
@@ -47,7 +47,8 @@
useWrapper = getAttribute("useWrapper"); // NON-NLS
}
- public void apply(FaceletContext faceletContext, UIComponent parent) throws IOException, FacesException, FaceletException, ELException
+ public void apply(FaceletContext faceletContext, UIComponent parent)
+ throws IOException, FacesException, FaceletException, ELException
{
if (parent.getParent() == null)
{
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/AbstractSpringOrchestraScope.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/AbstractSpringOrchestraScope.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/AbstractSpringOrchestraScope.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/AbstractSpringOrchestraScope.java Sun Jul 13 07:11:48 2008
@@ -65,7 +65,8 @@
Scope, BeanFactoryAware, ApplicationContextAware
{
private final Log log = LogFactory.getLog(AbstractSpringOrchestraScope.class);
- private static final String POST_PROCESSOR_BEAN_NAME = AbstractSpringOrchestraScope.class.getName() + "_BeanPostProcessor";
+ private static final String POST_PROCESSOR_BEAN_NAME =
+ AbstractSpringOrchestraScope.class.getName() + "_BeanPostProcessor";
private ConfigurableApplicationContext applicationContext;
private Advice[] advices;
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/BeanDefinitionConversationNameAttrDecorator.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/BeanDefinitionConversationNameAttrDecorator.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/BeanDefinitionConversationNameAttrDecorator.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/BeanDefinitionConversationNameAttrDecorator.java Sun Jul 13 07:11:48 2008
@@ -43,7 +43,8 @@
* A unique key used to store the orchestra conversationName within the attributes map
* of a spring bean definition.
*/
- public final static String CONVERSATION_NAME_ATTRIBUTE = "org.apache.myfaces.orchestra.spring.conversationName"; // NON-NLS
+ public final static String CONVERSATION_NAME_ATTRIBUTE =
+ "org.apache.myfaces.orchestra.spring.conversationName"; // NON-NLS
public BeanDefinitionHolder decorate(
Node node,
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/OrchestraAdvisorBeanPostProcessor.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/OrchestraAdvisorBeanPostProcessor.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/OrchestraAdvisorBeanPostProcessor.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/OrchestraAdvisorBeanPostProcessor.java Sun Jul 13 07:11:48 2008
@@ -189,7 +189,9 @@
// ok, this is not an orchestra-scoped bean
if (log.isDebugEnabled())
{
- log.debug("scope associated with bean " + beanName + " is not orchestra:" + scopeObj.getClass().getName());
+ log.debug(
+ "scope associated with bean " + beanName +
+ " is not orchestra:" + scopeObj.getClass().getName());
}
return null;
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/PersistenceContextConversationInterceptor.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/PersistenceContextConversationInterceptor.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/PersistenceContextConversationInterceptor.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/PersistenceContextConversationInterceptor.java Sun Jul 13 07:11:48 2008
@@ -45,8 +45,10 @@
*/
public class PersistenceContextConversationInterceptor implements MethodInterceptor
{
- private final static String PERSISTENCE_CONTEXT_CONV_ATTRIBUTE = PersistenceContextConversationInterceptor.class.getName() + ".PERSISTENCE_CONTEXT";
- public final static String REQUEST_ATTRIBUTE = PersistenceContextConversationInterceptor.class.getName() + ".USED_PERSISTENCE_CONTEXTS";
+ private final static String PERSISTENCE_CONTEXT_CONV_ATTRIBUTE =
+ PersistenceContextConversationInterceptor.class.getName() + ".PERSISTENCE_CONTEXT";
+ public final static String REQUEST_ATTRIBUTE =
+ PersistenceContextConversationInterceptor.class.getName() + ".USED_PERSISTENCE_CONTEXTS";
private PersistenceContextFactory persistenceContextFactory;
@@ -66,7 +68,8 @@
Conversation conversation = Conversation.getCurrentInstance();
if (conversation != null)
{
- PersistenceContextCloser persistenceContextCloser = (PersistenceContextCloser) conversation.getAttribute(PERSISTENCE_CONTEXT_CONV_ATTRIBUTE);
+ PersistenceContextCloser persistenceContextCloser = (PersistenceContextCloser)
+ conversation.getAttribute(PERSISTENCE_CONTEXT_CONV_ATTRIBUTE);
if (persistenceContextCloser != null)
{
persistenceContext = persistenceContextCloser.getPersistenceContext();
@@ -76,7 +79,9 @@
{
persistenceContext = persistenceContextFactory.create();
- conversation.setAttribute(PERSISTENCE_CONTEXT_CONV_ATTRIBUTE, new PersistenceContextCloser(persistenceContext));
+ conversation.setAttribute(
+ PERSISTENCE_CONTEXT_CONV_ATTRIBUTE,
+ new PersistenceContextCloser(persistenceContext));
}
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/ScopedBeanTargetSource.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/ScopedBeanTargetSource.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/ScopedBeanTargetSource.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/ScopedBeanTargetSource.java Sun Jul 13 07:11:48 2008
@@ -43,7 +43,8 @@
*
* @since 1.1
*/
-public class ScopedBeanTargetSource extends AbstractBeanFactoryBasedTargetSource {
+public class ScopedBeanTargetSource extends AbstractBeanFactoryBasedTargetSource
+{
private AbstractSpringOrchestraScope scope;
private String conversationName;
private String targetBeanName;
@@ -54,8 +55,8 @@
String conversationName,
String targetBeanName,
ObjectFactory objectFactory,
- BeanFactory beanFactory) {
-
+ BeanFactory beanFactory)
+ {
this.scope = scope;
this.conversationName = conversationName;
this.targetBeanName = targetBeanName;
@@ -65,7 +66,8 @@
super.setBeanFactory(beanFactory);
}
- public Object getTarget() throws Exception {
+ public Object getTarget() throws Exception
+ {
final Log log = LogFactory.getLog(ScopedBeanTargetSource.class);
log.debug("getRealBean for " + targetBeanName);
return scope.getRealBean(conversationName, targetBeanName, objectFactory);
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java Sun Jul 13 07:11:48 2008
@@ -53,7 +53,8 @@
* </bean>
*
*
- * <bean id="persistentContextConversationInterceptor" class="org.apache.myfaces.orchestra.conversation.spring.PersistenceContextConversationInterceptor">
+ * <bean id="persistentContextConversationInterceptor"
+ * class="org.apache.myfaces.orchestra.conversation.spring.PersistenceContextConversationInterceptor">
* <property name="persistenceContextFactory" ref="yourPersistentContextFactory" />
* </bean>
* </pre>
@@ -188,7 +189,8 @@
{
super.notifyAccessConversation(conversation);
- ConversationAccessLifetimeAspect aspect = (ConversationAccessLifetimeAspect) conversation.getAspect(ConversationAccessLifetimeAspect.class);
+ ConversationAccessLifetimeAspect aspect = (ConversationAccessLifetimeAspect)
+ conversation.getAspect(ConversationAccessLifetimeAspect.class);
if (aspect != null)
{
aspect.markAsAccessed();
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/_SpringUtils.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/_SpringUtils.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/_SpringUtils.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/conversation/spring/_SpringUtils.java Sun Jul 13 07:11:48 2008
@@ -159,7 +159,8 @@
pf.setTargetSource(scopedTargetSource);
Class beanType = beanFactory.getType(targetBeanName);
- if (beanType == null) {
+ if (beanType == null)
+ {
throw new IllegalStateException("Cannot create scoped proxy for bean '" + targetBeanName +
"': Target type could not be determined at the time of proxy creation.");
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/FrameworkAdapter.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/FrameworkAdapter.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/FrameworkAdapter.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/FrameworkAdapter.java Sun Jul 13 07:11:48 2008
@@ -36,7 +36,7 @@
*/
public abstract class FrameworkAdapter
{
- private final static ThreadLocal instance = new ThreadLocal();
+ private final static ThreadLocal instanceThreadLocal = new ThreadLocal();
private ConversationMessager conversationMessager;
private boolean prepared = false;
@@ -52,7 +52,7 @@
{
if (frameworkAdapter == null)
{
- instance.remove();
+ instanceThreadLocal.remove();
return;
}
@@ -64,7 +64,7 @@
}
}
- instance.set(frameworkAdapter);
+ instanceThreadLocal.set(frameworkAdapter);
}
/**
@@ -74,7 +74,7 @@
*/
public static FrameworkAdapter getCurrentInstance()
{
- return (FrameworkAdapter) instance.get();
+ return (FrameworkAdapter) instanceThreadLocal.get();
}
/**
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapter.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapter.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapter.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapter.java Sun Jul 13 07:11:48 2008
@@ -60,8 +60,8 @@
{
private final static String ISE_MESSAGE="No request/response data available"; // NON-NLS
- private final static ThreadLocal httpServletRequest = new ThreadLocal();
- private final static ThreadLocal httpServletResponse = new ThreadLocal();
+ private final static ThreadLocal httpServletRequestThreadLocal = new ThreadLocal();
+ private final static ThreadLocal httpServletResponseThreadLocal = new ThreadLocal();
private final Log log = LogFactory.getLog(BasicFrameworkAdapter.class);
@@ -100,12 +100,12 @@
private HttpServletRequest getRequest()
{
- return (HttpServletRequest) httpServletRequest.get();
+ return (HttpServletRequest) httpServletRequestThreadLocal.get();
}
private HttpServletResponse getResponse()
{
- return (HttpServletResponse) httpServletResponse.get();
+ return (HttpServletResponse) httpServletResponseThreadLocal.get();
}
public void beginRequest(ServletRequest req, ServletResponse rsp)
@@ -113,11 +113,11 @@
log.debug("Beginning request");
if (req instanceof HttpServletRequest)
{
- httpServletRequest.set(req);
+ httpServletRequestThreadLocal.set(req);
}
if (rsp instanceof HttpServletResponse)
{
- httpServletResponse.set(rsp);
+ httpServletResponseThreadLocal.set(rsp);
}
FrameworkAdapter.setCurrentInstance(this);
@@ -127,8 +127,8 @@
{
log.debug("Ending request");
FrameworkAdapter.setCurrentInstance(null);
- httpServletRequest.set(null);
- httpServletResponse.set(null);
+ httpServletRequestThreadLocal.set(null);
+ httpServletResponseThreadLocal.set(null);
}
public String getInitParameter(String key)
@@ -269,10 +269,16 @@
Object obj;
obj = getRequestAttribute(name);
- if (obj != null) return obj;
+ if (obj != null)
+ {
+ return obj;
+ }
obj = getSessionAttribute(name);
- if (obj != null) return obj;
+ if (obj != null)
+ {
+ return obj;
+ }
// TODO: look up application-scoped objects.
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapterFilter.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapterFilter.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapterFilter.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/basic/BasicFrameworkAdapterFilter.java Sun Jul 13 07:11:48 2008
@@ -57,7 +57,8 @@
adapter = new BasicFrameworkAdapter(filterConfig.getServletContext(), conversationMessager);
}
- public void doFilter(ServletRequest req, ServletResponse rsp, FilterChain filterChain) throws IOException, ServletException
+ public void doFilter(ServletRequest req, ServletResponse rsp, FilterChain filterChain)
+ throws IOException, ServletException
{
log.debug("doFilter");
try
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/jsf/JsfFrameworkAdapterFilter.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/jsf/JsfFrameworkAdapterFilter.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/jsf/JsfFrameworkAdapterFilter.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/jsf/JsfFrameworkAdapterFilter.java Sun Jul 13 07:11:48 2008
@@ -65,7 +65,8 @@
adapter = new JsfFrameworkAdapter(conversationMessager);
}
- public void doFilter(ServletRequest req, ServletResponse rsp, FilterChain filterChain) throws IOException, ServletException
+ public void doFilter(ServletRequest req, ServletResponse rsp, FilterChain filterChain)
+ throws IOException, ServletException
{
log.debug("doFilter");
try
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/springBasic/SpringBasicFrameworkAdapterFilter.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/springBasic/SpringBasicFrameworkAdapterFilter.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/springBasic/SpringBasicFrameworkAdapterFilter.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/frameworkAdapter/springBasic/SpringBasicFrameworkAdapterFilter.java Sun Jul 13 07:11:48 2008
@@ -45,7 +45,8 @@
* beanname defined in the dependency-injection framework, or an absolute
* classname of a type implementing ConversationMessager.
* <p>
- * This filter access the Spring framework to lookup the bean in its {@link org.apache.myfaces.orchestra.frameworkAdapter.FrameworkAdapter#getBean} method
+ * This filter access the Spring framework to lookup the bean in its
+ * {@link org.apache.myfaces.orchestra.frameworkAdapter.FrameworkAdapter#getBean} method
*/
public class SpringBasicFrameworkAdapterFilter implements Filter
{
@@ -61,7 +62,8 @@
adapter = new SpringBasicFrameworkAdapter(filterConfig.getServletContext(), conversationMessager);
}
- public void doFilter(ServletRequest req, ServletResponse rsp, FilterChain filterChain) throws IOException, ServletException
+ public void doFilter(ServletRequest req, ServletResponse rsp, FilterChain filterChain)
+ throws IOException, ServletException
{
log.debug("doFilter");
try
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/_ReentrantLock.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/_ReentrantLock.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/_ReentrantLock.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/_ReentrantLock.java Sun Jul 13 07:11:48 2008
@@ -38,7 +38,8 @@
*
* @since 1.1
*/
-public class _ReentrantLock implements java.io.Serializable {
+public class _ReentrantLock implements java.io.Serializable
+{
private static final long serialVersionUID = 7373984872572414699L;
private static final long WAIT_WARN_MILLIS = 30L * 1000L; // 30 seconds
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/ConfigUtils.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/ConfigUtils.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/ConfigUtils.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/ConfigUtils.java Sun Jul 13 07:11:48 2008
@@ -36,9 +36,13 @@
{
List rhl = (List) reqScope.get(RH_KEY);
if (rhl != null)
+ {
return rhl;
+ }
else
+ {
return Collections.EMPTY_LIST;
+ }
}
/**
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/FrameworkAdapterRequestHandler.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/FrameworkAdapterRequestHandler.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/FrameworkAdapterRequestHandler.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/lib/jsf/FrameworkAdapterRequestHandler.java Sun Jul 13 07:11:48 2008
@@ -35,7 +35,8 @@
*/
class FrameworkAdapterRequestHandler implements RequestHandler
{
- private final static String CONVERSATION_MESSAGER_CLASS = "org.apache.myfaces.orchestra.CONVERSATION_MESSAGER"; // NON-NLS
+ private final static String CONVERSATION_MESSAGER_CLASS =
+ "org.apache.myfaces.orchestra.CONVERSATION_MESSAGER"; // NON-NLS
private final Log log = LogFactory.getLog(FrameworkAdapterRequestHandler.class);
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterProviderManager.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterProviderManager.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterProviderManager.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterProviderManager.java Sun Jul 13 07:11:48 2008
@@ -79,7 +79,8 @@
public static RequestParameterProviderManager getInstance()
{
RequestParameterProviderManager manager =
- (RequestParameterProviderManager) FrameworkAdapter.getCurrentInstance().getSessionAttribute(PARAMETER_PROVIDER_MANAGER_KEY);
+ (RequestParameterProviderManager) FrameworkAdapter.getCurrentInstance()
+ .getSessionAttribute(PARAMETER_PROVIDER_MANAGER_KEY);
if (manager == null)
{
@@ -122,7 +123,10 @@
{
if (!isResponseIntercepted())
{
- throw new IllegalStateException("RequestParameterServletFilter not called. Please configure the filter " + RequestParameterServletFilter.class.getName() + " in your web.xml to cover your faces requests.");
+ throw new IllegalStateException(
+ "RequestParameterServletFilter not called. Please configure the filter " +
+ RequestParameterServletFilter.class.getName() +
+ " in your web.xml to cover your faces requests.");
}
if (url == null)
@@ -191,8 +195,10 @@
{
FrameworkAdapter fa = FrameworkAdapter.getCurrentInstance();
- return Boolean.TRUE.equals(fa.getRequestAttribute(RequestParameterServletFilter.REQUEST_PARAM_FILTER_CALLED))
- || Boolean.TRUE.equals(fa.getRequestAttribute(RequestParameterServletFilter.REQUEST_PARAM_RESPONSE_WRAPPED));
+ return Boolean.TRUE.equals(
+ fa.getRequestAttribute(RequestParameterServletFilter.REQUEST_PARAM_FILTER_CALLED))
+ || Boolean.TRUE.equals(
+ fa.getRequestAttribute(RequestParameterServletFilter.REQUEST_PARAM_RESPONSE_WRAPPED));
}
protected List getProviders()
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterResponseWrapper.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterResponseWrapper.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterResponseWrapper.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterResponseWrapper.java Sun Jul 13 07:11:48 2008
@@ -22,7 +22,8 @@
import javax.servlet.http.HttpServletResponseWrapper;
/**
- * This wrapper intercepts encodeURL and pass it to the {@link RequestParameterProviderManager} which attaches the url parameters
+ * This wrapper intercepts encodeURL and pass it to the {@link RequestParameterProviderManager}
+ * which attaches the url parameters
*/
public class RequestParameterResponseWrapper extends HttpServletResponseWrapper
{
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterServletFilter.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterServletFilter.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterServletFilter.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/RequestParameterServletFilter.java Sun Jul 13 07:11:48 2008
@@ -35,8 +35,10 @@
*/
public class RequestParameterServletFilter implements Filter
{
- public final static String REQUEST_PARAM_FILTER_CALLED = RequestParameterServletFilter.class.getName() + ".CALLED";
- public final static String REQUEST_PARAM_RESPONSE_WRAPPED = RequestParameterServletFilter.class.getName() + ".WRAPPED";
+ public final static String REQUEST_PARAM_FILTER_CALLED =
+ RequestParameterServletFilter.class.getName() + ".CALLED";
+ public final static String REQUEST_PARAM_RESPONSE_WRAPPED =
+ RequestParameterServletFilter.class.getName() + ".WRAPPED";
public RequestParameterServletFilter()
{
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/jsf/RequestParameterFacesContextFactory.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/jsf/RequestParameterFacesContextFactory.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/jsf/RequestParameterFacesContextFactory.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/requestParameterProvider/jsf/RequestParameterFacesContextFactory.java Sun Jul 13 07:11:48 2008
@@ -55,18 +55,22 @@
* to that FacesContext instance is a modified one that tweaks every url that is
* processed by the ServletResponse.encodeUrl method.
*/
- public FacesContext getFacesContext(Object context, Object request, Object response, Lifecycle lifecycle) throws FacesException
+ public FacesContext getFacesContext(Object context, Object request, Object response, Lifecycle lifecycle)
+ throws FacesException
{
if (response instanceof HttpServletResponse)
{
HttpServletRequest httpServletRequest = (HttpServletRequest) request;
- // Wrap this request only if something else (eg a RequestParameterServletFilter) has not already wrapped it.
- if (!Boolean.TRUE.equals(httpServletRequest.getAttribute(RequestParameterServletFilter.REQUEST_PARAM_FILTER_CALLED)))
+ // Wrap this request only if something else (eg a RequestParameterServletFilter)
+ // has not already wrapped it.
+ if (!Boolean.TRUE.equals(httpServletRequest.getAttribute(
+ RequestParameterServletFilter.REQUEST_PARAM_FILTER_CALLED)))
{
- // No servlet filter has wrapped the response, so do it now for the response referenced by this FacesContext.
- // Note that this wrapper will therefore apply to all output generated via the FacesContext, but not to
- // anything that might be written by filters etc.
+ // No servlet filter has wrapped the response, so do it now for the response
+ // referenced by this FacesContext. Note that this wrapper will therefore
+ // apply to all output generated via the FacesContext, but not to anything
+ // that might be written by filters etc.
response = new RequestParameterResponseWrapper((HttpServletResponse) response);
// We now need to reassure the RequestParameterProviderManager that the response has indeed been
@@ -79,7 +83,8 @@
// Therefore we have two separate flags; RequestParameterProviderManager checks whether either
// REQUEST_PARAM_FILTER_CALLED or REQUEST_PARAM_RESPONSE_WRAPPED has been set.
- httpServletRequest.setAttribute(RequestParameterServletFilter.REQUEST_PARAM_RESPONSE_WRAPPED, Boolean.TRUE);
+ httpServletRequest.setAttribute(
+ RequestParameterServletFilter.REQUEST_PARAM_RESPONSE_WRAPPED, Boolean.TRUE);
}
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/CompositeViewControllerNameMapper.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/CompositeViewControllerNameMapper.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/CompositeViewControllerNameMapper.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/CompositeViewControllerNameMapper.java Sun Jul 13 07:11:48 2008
@@ -33,7 +33,10 @@
public CompositeViewControllerNameMapper(ViewControllerNameMapper[] viewControllerNameMappers)
{
this.viewControllerNameMappers = new ViewControllerNameMapper[viewControllerNameMappers.length];
- System.arraycopy(viewControllerNameMappers, 0, this.viewControllerNameMappers, 0, viewControllerNameMappers.length);
+ System.arraycopy(
+ viewControllerNameMappers, 0,
+ this.viewControllerNameMappers, 0,
+ viewControllerNameMappers.length);
}
public String mapViewId(String viewId)
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerPhaseListener.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerPhaseListener.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerPhaseListener.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerPhaseListener.java Sun Jul 13 07:11:48 2008
@@ -205,11 +205,14 @@
protected ViewControllerPhaseListenerState getState(FacesContext facesContext)
{
- ViewControllerPhaseListenerState state = (ViewControllerPhaseListenerState) facesContext.getExternalContext().getRequestMap().get(ViewControllerPhaseListenerState.class.getName());
+ ViewControllerPhaseListenerState state = (ViewControllerPhaseListenerState)
+ facesContext.getExternalContext().getRequestMap()
+ .get(ViewControllerPhaseListenerState.class.getName());
if (state == null)
{
state = new ViewControllerPhaseListenerState();
- facesContext.getExternalContext().getRequestMap().put(ViewControllerPhaseListenerState.class.getName(), state);
+ facesContext.getExternalContext().getRequestMap().put(
+ ViewControllerPhaseListenerState.class.getName(), state);
}
return state;
}
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerVariableResolver.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerVariableResolver.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerVariableResolver.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/jsf/ViewControllerVariableResolver.java Sun Jul 13 07:11:48 2008
@@ -54,7 +54,9 @@
static ViewControllerManager getViewControllerManager(FacesContext context)
{
- ViewControllerManager manager = (ViewControllerManager) context.getApplication().getVariableResolver().resolveVariable(context, ViewControllerManager.VIEW_CONTROLLER_MANAGER_NAME);
+ ViewControllerManager manager = (ViewControllerManager) context.getApplication()
+ .getVariableResolver().resolveVariable(
+ context, ViewControllerManager.VIEW_CONTROLLER_MANAGER_NAME);
if (manager != null)
{
return manager;
Modified: myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/spring/SpringViewControllerScope.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/spring/SpringViewControllerScope.java?rev=676323&r1=676322&r2=676323&view=diff
==============================================================================
--- myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/spring/SpringViewControllerScope.java (original)
+++ myfaces/orchestra/branches/prepare_core12/src/main/java/org/apache/myfaces/orchestra/viewController/spring/SpringViewControllerScope.java Sun Jul 13 07:11:48 2008
@@ -71,7 +71,9 @@
public Conversation createConversation(ConversationContext context, String conversationName)
{
- throw new IllegalStateException("the viewController scope is not supposed to start a conversation on its own. conversation to start: " + conversationName);
+ throw new IllegalStateException(
+ "The viewController scope is not supposed to start a conversation on its own. " +
+ "Conversation to start: " + conversationName);
}
protected void assertSameScope(String beanName, Conversation conversation)