Skip to content

Commit 255e637

Browse files
authored
Merge pull request #826 from volosied/prep-2311-release
Prep for a 2.3.11 release
2 parents 0c3ae37 + 749f49c commit 255e637

File tree

7 files changed

+12
-11
lines changed

7 files changed

+12
-11
lines changed

api/src/main/javascript/META-INF/resources/myfaces/_impl/core/Impl.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -343,6 +343,9 @@ _MF_SINGLTN(_PFX_CORE + "Impl", _MF_OBJECT, /** @lends myfaces._impl.core.Impl.
343343
var delayTimeout = options.delay || this._RT.getLocalOrGlobalConfig(context, "delay", false);
344344

345345
if (!!delayTimeout) {
346+
if(delayTimeout.toLowerCase && delayTimeout.toLowerCase() === "none"){
347+
delayTimeout = 0;
348+
}
346349
if(!(delayTimeout >= 0)) {
347350
// abbreviation which covers all cases of non positive values,
348351
// including NaN and non-numeric strings, no type equality is deliberate here,

api/src/main/javascript/META-INF/resources/myfaces/_impl/xhrCore/_AjaxUtils.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,10 @@ _MF_SINGLTN(_PFX_XHR+"_AjaxUtils", _MF_OBJECT,
194194
form = this._Dom.byId(form);
195195
var _t = this;
196196
var foundNames = this._Dom.findAll(form, function(node) {
197-
var name = node.getAttribute("name");
197+
var name = null;
198+
if(node.getAttribute && node.getAttribute("name")) {
199+
name = node.getAttribute("name");
200+
}
198201
if(!name || name.indexOf(_t.P_VIEWSTATE)) {
199202
return false;
200203
}

impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -588,7 +588,6 @@ public final void addELContextListener(final ELContextListener listener)
588588
public void publishEvent(FacesContext facesContext, Class<? extends SystemEvent> systemEventClass,
589589
Class<?> sourceBaseType, Object source)
590590
{
591-
checkNull(facesContext, "facesContext");
592591
checkNull(systemEventClass, "systemEventClass");
593592
checkNull(source, "source");
594593

@@ -647,7 +646,6 @@ public void publishEvent(FacesContext facesContext, Class<? extends SystemEvent>
647646
@Override
648647
public void publishEvent(FacesContext facesContext, Class<? extends SystemEvent> systemEventClass, Object source)
649648
{
650-
checkNull(source, "source");
651649
publishEvent(facesContext, systemEventClass, source.getClass(), source);
652650
}
653651

impl/src/main/java/org/apache/myfaces/cdi/JsfArtifactProducer.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import javax.faces.annotation.RequestParameterValuesMap;
3232
import javax.faces.annotation.SessionMap;
3333
import javax.faces.annotation.ViewMap;
34+
import javax.faces.application.Application;
3435
import javax.faces.application.ResourceHandler;
3536
import javax.faces.component.UIViewRoot;
3637
import javax.faces.context.ExternalContext;
@@ -45,9 +46,9 @@ public class JsfArtifactProducer
4546
@Produces
4647
@Named("application")
4748
@ApplicationScoped
48-
public Object getApplication()
49+
public Application getApplication()
4950
{
50-
return FacesContext.getCurrentInstance().getExternalContext().getContext();
51+
return FacesContext.getCurrentInstance().getApplication();
5152
}
5253

5354
@Produces

impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -590,10 +590,6 @@ public void redirect(final String url) throws IOException
590590
PartialViewContext partialViewContext = facesContext.getPartialViewContext();
591591
if (partialViewContext.isPartialRequest())
592592
{
593-
if (_servletResponse instanceof HttpServletResponse && facesContext.getResponseComplete())
594-
{
595-
throw new IllegalStateException();
596-
}
597593
PartialResponseWriter writer = partialViewContext.getPartialResponseWriter();
598594
this.setResponseContentType("text/xml");
599595
this.setResponseCharacterEncoding("UTF-8");

impl/src/test/java/org/apache/myfaces/component/search/SearchExpressionImplTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,7 @@ public void testMyFaces4695() throws Exception
423423

424424
SearchExpressionHandler handler = facesContext.getApplication().getSearchExpressionHandler();
425425

426-
Assertions.assertEquals("form1:one", handler.resolveClientId(searchContext, "form1:one"));
426+
Assert.assertEquals("form1:one", handler.resolveClientId(searchContext, "form1:one"));
427427

428428
processRemainingPhases();
429429
}

shared/src/main/java/org/apache/myfaces/shared/application/FacesServletMappingUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ else if (allowExactMatch && mapping.startsWith("/") && mapping.equals(servletPat
292292
}
293293
if (mapping.equals(servletPath))
294294
{
295-
return FacesServletMapping.createExactMapping(mapping); // MYFACES-4524
295+
return FacesServletMapping.createPrefixMapping(mapping);
296296
}
297297
}
298298
}

0 commit comments

Comments
 (0)