You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2013/10/22 00:47:49 UTC
svn commit: r1534403 - in /myfaces/shared/trunk_4.0.x/core: ./
src/main/java/org/apache/myfaces/shared/context/flash/
src/main/java/org/apache/myfaces/shared/renderkit/html/
src/main/java/org/apache/myfaces/shared/renderkit/html/util/
src/test/java/org...
Author: lu4242
Date: Mon Oct 21 22:47:49 2013
New Revision: 1534403
URL: http://svn.apache.org/r1534403
Log:
synch shared
Removed:
myfaces/shared/trunk_4.0.x/core/src/test/java/org/apache/myfaces/shared/context/flash/FlashImplTest.java
Modified:
myfaces/shared/trunk_4.0.x/core/pom.xml
myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java
myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlButtonRendererBase.java
myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java
myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java
myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/util/HTMLEncoder.java
Modified: myfaces/shared/trunk_4.0.x/core/pom.xml
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_4.0.x/core/pom.xml?rev=1534403&r1=1534402&r2=1534403&view=diff
==============================================================================
--- myfaces/shared/trunk_4.0.x/core/pom.xml (original)
+++ myfaces/shared/trunk_4.0.x/core/pom.xml Mon Oct 21 22:47:49 2013
@@ -143,7 +143,7 @@
<profile>
<id>synch-myfaces-impl-shared</id>
<properties>
- <myfaces.impl.shared.version>2.0.18-SNAPSHOT</myfaces.impl.shared.version>
+ <myfaces.impl.shared.version>2.0.19-SNAPSHOT</myfaces.impl.shared.version>
</properties>
<activation>
<property>
Modified: myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java?rev=1534403&r1=1534402&r2=1534403&view=diff
==============================================================================
--- myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java (original)
+++ myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java Mon Oct 21 22:47:49 2013
@@ -538,7 +538,10 @@ public class FlashImpl extends Flash
final PhaseId currentPhaseId = facesContext.getCurrentPhaseId();
boolean lastPhaseNormalRequest = PhaseId.RENDER_RESPONSE.equals(currentPhaseId);
- boolean lastPhaseIfRedirect = PhaseId.INVOKE_APPLICATION.equals(currentPhaseId)
+ // According to the spec, if there is a redirect, responseComplete()
+ // has been called, and Flash.setRedirect() has been called too,
+ // so we just need to check both are present.
+ boolean lastPhaseIfRedirect = facesContext.getResponseComplete()
&& _isRedirectTrueOnThisRequest(facesContext);
return lastPhaseNormalRequest || lastPhaseIfRedirect;
Modified: myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlButtonRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlButtonRendererBase.java?rev=1534403&r1=1534402&r2=1534403&view=diff
==============================================================================
--- myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlButtonRendererBase.java (original)
+++ myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlButtonRendererBase.java Mon Oct 21 22:47:49 2013
@@ -68,7 +68,9 @@ public class HtmlButtonRendererBase
facesContext, uiComponent, UICommand.class);
//super.decode must not be called, because value is handled here
- if (!isReset(uiComponent) && isSubmitted(facesContext, uiComponent))
+ boolean disabled = isDisabled(facesContext, uiComponent);
+ if (!isReset(uiComponent) && isSubmitted(facesContext, uiComponent) &&
+ !disabled)
{
uiComponent.queueEvent(new ActionEvent(uiComponent));
@@ -77,7 +79,7 @@ public class HtmlButtonRendererBase
}
if (uiComponent instanceof ClientBehaviorHolder &&
- !HtmlRendererUtils.isDisabled(uiComponent))
+ !disabled)
{
HtmlRendererUtils.decodeClientBehaviors(facesContext, uiComponent);
}
Modified: myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java?rev=1534403&r1=1534402&r2=1534403&view=diff
==============================================================================
--- myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java (original)
+++ myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java Mon Oct 21 22:47:49 2013
@@ -84,7 +84,8 @@ public abstract class HtmlLinkRendererBa
{
String clientId = component.getClientId(facesContext);
FormInfo formInfo = findNestingForm(component, facesContext);
- if (formInfo != null)
+ boolean disabled = HtmlRendererUtils.isDisabled(component);
+ if (formInfo != null && !disabled)
{
String reqValue = (String) facesContext.getExternalContext().getRequestParameterMap().get(
HtmlRendererUtils.getHiddenCommandLinkFieldName(formInfo, facesContext));
@@ -97,7 +98,7 @@ public abstract class HtmlLinkRendererBa
}
}
if (component instanceof ClientBehaviorHolder &&
- !HtmlRendererUtils.isDisabled(component))
+ !disabled)
{
HtmlRendererUtils.decodeClientBehaviors(facesContext, component);
}
Modified: myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java?rev=1534403&r1=1534402&r2=1534403&view=diff
==============================================================================
--- myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java (original)
+++ myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java Mon Oct 21 22:47:49 2013
@@ -483,6 +483,7 @@ public class HtmlTableRendererBase exten
}
List children = null;
+ int columnStyleIndex = 0;
for (int j = 0, size = getChildCount(component); j < size; j++)
{
if (children == null)
@@ -496,16 +497,18 @@ public class HtmlTableRendererBase exten
if (columnRendering)
{
- beforeColumn(facesContext, uiData, j);
+ beforeColumn(facesContext, uiData, columnStyleIndex);
}
encodeColumnChild(facesContext, writer, uiData, child,
- styles, nc * uiData.getChildCount() + j);
+ styles, nc * uiData.getChildCount() + columnStyleIndex);
if (columnRendering)
{
- afterColumn(facesContext, uiData, j);
+ afterColumn(facesContext, uiData, columnStyleIndex);
}
+ columnStyleIndex = columnStyleIndex +
+ getColumnCountForComponent(facesContext, uiData, child);
}
}
@@ -582,6 +585,7 @@ public class HtmlTableRendererBase exten
renderRowStart(facesContext, writer, uiData, styles, currentRow);
List<UIComponent> children = null;
+ int columnStyleIndex = 0;
for (int j = 0, size = getChildCount(component); j < size; j++)
{
if (children == null)
@@ -595,16 +599,18 @@ public class HtmlTableRendererBase exten
if (columnRendering)
{
- beforeColumn(facesContext, uiData, j);
+ beforeColumn(facesContext, uiData, columnStyleIndex);
}
encodeColumnChild(facesContext, writer, uiData, child,
- styles, j);
+ styles, columnStyleIndex);
if (columnRendering)
{
- afterColumn(facesContext, uiData, j);
+ afterColumn(facesContext, uiData, columnStyleIndex);
}
+ columnStyleIndex = columnStyleIndex +
+ getColumnCountForComponent(facesContext, uiData, child);
}
}
@@ -821,6 +827,22 @@ public class HtmlTableRendererBase exten
{
}
/**
+ * Indicates the number of columns the component represents. By default each UIColumn instance
+ * is 1 column
+ * @param facesContext
+ * @param uiData
+ * @param child
+ * @return
+ */
+ protected int getColumnCountForComponent(FacesContext facesContext, UIData uiData, UIComponent child)
+ {
+ if (child instanceof UIColumn)
+ {
+ return 1;
+ }
+ return 0;
+ }
+ /**
*Perform any operations necessary immediately before each column child's header or footer is rendered
*
* @param facesContext the <code>FacesContext</code>.
Modified: myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/util/HTMLEncoder.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/util/HTMLEncoder.java?rev=1534403&r1=1534402&r2=1534403&view=diff
==============================================================================
--- myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/util/HTMLEncoder.java (original)
+++ myfaces/shared/trunk_4.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/util/HTMLEncoder.java Mon Oct 21 22:47:49 2013
@@ -103,6 +103,14 @@ public abstract class HTMLEncoder
default:
break;
}
+ // http://www.w3.org/MarkUp/html3/specialchars.html
+ // From C0 extension U+0000-U+001F only U+0009, U+000A and
+ // U+000D are valid control characters
+ if (c <= 0x1F && c != 0x09 && c != 0x0A && c != 0x0D)
+ {
+ // Ignore escape character
+ app = "";
+ }
}
else if (encodeNonLatin && (int)c > 0x80)
{
@@ -226,6 +234,14 @@ public abstract class HTMLEncoder
default:
break;
}
+ // http://www.w3.org/MarkUp/html3/specialchars.html
+ // From C0 extension U+0000-U+001F only U+0009, U+000A and
+ // U+000D are valid control characters
+ if (c <= 0x1F && c != 0x09 && c != 0x0A && c != 0x0D)
+ {
+ // Ignore escape character
+ app = "";
+ }
}
else if (encodeNonLatin && (int)c > 0x80)
{
@@ -377,6 +393,14 @@ public abstract class HTMLEncoder
default:
break;
}
+ // http://www.w3.org/MarkUp/html3/specialchars.html
+ // From C0 extension U+0000-U+001F only U+0009, U+000A and
+ // U+000D are valid control characters
+ if (c <= 0x1F && c != 0x09 && c != 0x0A && c != 0x0D)
+ {
+ // Ignore escape character
+ app = "";
+ }
}
else if (encodeNonLatin && (int)c > 0x80)
{