jsf.js_next_gen 4.0.0-RC.2 → 4.0.0-RC.20

package/src/main/typescript/impl/xhrCore/ResponseProcessor.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import {Config, DomQuery, DQ, Lang, Stream, XMLQuery} from "mona-dish";
+import {Config, DomQuery, DomQueryCollector, DQ, DQ$, Lang, LazyStream, Stream, XMLQuery} from "mona-dish";
 import {Implementation} from "../AjaxImpl";
 import {Assertions} from "../util/Assertions";
 import {IResponseProcessor} from "./IResponseProcessor";
@@ -23,6 +23,7 @@ import {StateHolder} from "../core/ImplTypes";
 import {EventData} from "./EventData";
 
 import {
+    $faces,
     $nsp,
     APPLIED_CLIENT_WINDOW,
     APPLIED_VST,
@@ -30,32 +31,40 @@ import {
     ATTR_NAME,
     ATTR_URL,
     ATTR_VALUE,
+    DEFERRED_HEAD_INSERTS,
     EMPTY_FUNC,
     EMPTY_STR,
     ERROR_MESSAGE,
     ERROR_NAME,
-    HTML_VIEWSTATE,
+    HTML_TAG_BODY,
+    HTML_TAG_FORM,
+    HTML_TAG_HEAD,
+    HTML_TAG_LINK,
+    HTML_TAG_SCRIPT,
+    HTML_TAG_STYLE, IDENT_ALL, IDENT_NONE,
     ON_ERROR,
-    ON_EVENT, P_CLIENT_WINDOW,
+    ON_EVENT,
+    P_CLIENT_WINDOW,
+    P_EXECUTE,
     P_PARTIAL_SOURCE,
+    P_RENDER,
+    P_RENDER_OVERRIDE,
     P_VIEWSTATE,
-    RESPONSE_XML, SEL_CLIENT_WINDOW_ELEM,
-    SEL_SCRIPTS_STYLES,
+    PARTIAL_ID,
+    RESPONSE_XML,
+    SEL_CLIENT_WINDOW_ELEM,
     SEL_VIEWSTATE_ELEM,
     SOURCE,
     SUCCESS,
-    TAG_AFTER,
-    TAG_ATTR,
-    TAG_BEFORE,
-    TAG_BODY,
-    TAG_FORM,
-    TAG_HEAD,
     UPDATE_ELEMS,
     UPDATE_FORMS,
-    DEFERRED_HEAD_INSERTS
+    XML_TAG_AFTER,
+    XML_TAG_ATTR,
+    XML_TAG_BEFORE
 } from "../core/Const";
-import trim = Lang.trim;
 import {ExtConfig, ExtDomQuery} from "../util/ExtDomQuery";
+import {HiddenInputBuilder} from "../util/HiddenInputBuilder";
+import trim = Lang.trim;
 
 
 /**
@@ -82,31 +91,38 @@ export class ResponseProcessor implements IResponseProcessor {
      * the data incoming must represent the html representation of the head itself one way or the other
      */
     replaceHead(shadowDocument: XMLQuery | DQ) {
-        let shadowHead = shadowDocument.querySelectorAll(TAG_HEAD);
+        const shadowHead = shadowDocument.querySelectorAll(HTML_TAG_HEAD);
         if (!shadowHead.isPresent()) {
             return;
         }
+        const head = ExtDomQuery.querySelectorAll(HTML_TAG_HEAD);
+        // full replace we delete everything
+        head.childNodes.delete();
+        this.addToHead(shadowHead);
+        //we copy the attributes as well (just in case myfaces introduces the id in head)
+        head.copyAttrs(shadowHead);
+    }
 
-        let oldHead = ExtDomQuery.querySelectorAll(TAG_HEAD);
-
-        //delete all to avoid script and style overlays
-        oldHead.querySelectorAll(SEL_SCRIPTS_STYLES).delete();
+    addToHead(shadowHead: XMLQuery | DQ) {
+        const mappedHeadData = new ExtDomQuery(shadowHead);
+        const scriptTags = [HTML_TAG_SCRIPT];
+        const nonExecutables = mappedHeadData.filter(item => scriptTags.indexOf(item.tagName.orElse("").value) == -1);
+        nonExecutables.runHeadInserts(true);
 
-        // we cannot replace new elements in the head, but we can eval the elements
-        // eval means the scripts will get attached (eval script attach method)
-        // but this is done by DomQuery not in this code
-        this.storeForEval(shadowHead);
         //incoming either the outer head tag or its children
-        //shadowHead = (shadowHead.tagName.value === "HEAD") ? shadowHead.childNodes : shadowHead;
-        //this.addToHead(shadowHead);
+        const nodesToAdd = (shadowHead.tagName.value === "HEAD") ? shadowHead.childNodes : shadowHead;
+        // this is stored for "post" processing
+        // after the rest of the "physical build up", head before body
+        const scriptElements = nodesToAdd.stream
+            .filter(item => scriptTags.indexOf(item.tagName.orElse("").value) != -1).collect(new DomQueryCollector());
+
+        this.addToHeadDeferred(scriptElements);
     }
 
-    addToHead(newElements: XMLQuery | DQ) {
+    addToHeadDeferred(newElements: XMLQuery | DQ) {
         this.internalContext.assign(DEFERRED_HEAD_INSERTS).value.push(newElements);
     }
 
-
-
     /**
      * replaces the body in the expected manner
      * which means the entire body content is refreshed
@@ -117,21 +133,21 @@ export class ResponseProcessor implements IResponseProcessor {
      */
     replaceBody(shadowDocument: XMLQuery | DQ) {
 
-        let shadowBody = shadowDocument.querySelectorAll(TAG_BODY);
+        const shadowBody = shadowDocument.querySelectorAll(HTML_TAG_BODY);
         if (!shadowBody.isPresent()) {
             return;
         }
 
-        let shadowInnerHTML: string = <string>shadowBody.html().value;
+        const shadowInnerHTML: string = <string>shadowBody.innerHTML;
 
-        let resultingBody = <DQ>ExtDomQuery.querySelectorAll(TAG_BODY).html(shadowInnerHTML);
-        let updateForms = resultingBody.querySelectorAll(TAG_FORM);
+        const resultingBody = <DQ>ExtDomQuery.querySelectorAll(HTML_TAG_BODY);
+        const updateForms = resultingBody.querySelectorAll(HTML_TAG_FORM);
 
         // main difference, we cannot replace the body itself, but only its content
         // we need a separate step for post-processing the incoming
         // attributes, like classes, styles etc...
-        resultingBody.copyAttrs(shadowBody);
-
+        (resultingBody.html(shadowInnerHTML) as DQ).copyAttrs(shadowBody);
+        this.externalContext.assign($nsp(P_RENDER_OVERRIDE)).value = "@all";
         this.storeForPostProcessing(updateForms, resultingBody);
     }
 
@@ -158,18 +174,18 @@ export class ResponseProcessor implements IResponseProcessor {
          * <error>
          */
 
-        let mergedErrorData = new ExtConfig({});
+        const mergedErrorData = new ExtConfig({});
         mergedErrorData.assign(SOURCE).value = this.externalContext.getIf(P_PARTIAL_SOURCE).get(0).value;
         mergedErrorData.assign(ERROR_NAME).value = node.querySelectorAll(ERROR_NAME).textContent(EMPTY_STR);
         mergedErrorData.assign(ERROR_MESSAGE).value = node.querySelectorAll(ERROR_MESSAGE).cDATAAsString;
 
-        let hasResponseXML = this.internalContext.get(RESPONSE_XML).isPresent();
+        const hasResponseXML = this.internalContext.get(RESPONSE_XML).isPresent();
 
         //we now store the response xml also in the error data for further details
         mergedErrorData.assignIf(hasResponseXML, RESPONSE_XML).value = this.internalContext.getIf(RESPONSE_XML).value.get(0).value;
 
         // error post-processing and enrichment (standard messages from keys)
-        let errorData = ErrorData.fromServerError(mergedErrorData);
+        const errorData = ErrorData.fromServerError(mergedErrorData);
 
         // we now trigger an internally stored onError function which might be an attached to the context
         // either we do not have an internal on error, or an on error has been based via params from the outside.
@@ -187,7 +203,7 @@ export class ResponseProcessor implements IResponseProcessor {
     redirect(node: XMLQuery) {
         Assertions.assertUrlExists(node);
 
-        let redirectUrl = trim(node.attr(ATTR_URL).value);
+        const redirectUrl = trim(node.attr(ATTR_URL).value);
         if (redirectUrl != EMPTY_STR) {
             window.location.href = redirectUrl;
         }
@@ -199,8 +215,8 @@ export class ResponseProcessor implements IResponseProcessor {
      * @param cdataBlock the cdata block with the new html code
      */
     update(node: XMLQuery, cdataBlock: string) {
-        let result = ExtDomQuery.byId(node.id.value, true).outerHTML(cdataBlock, false, false);
-        let sourceForm = result?.parents(TAG_FORM).orElseLazy(() => result.byTagName(TAG_FORM, true));
+        const result = ExtDomQuery.byId(node.id.value, true).outerHTML(cdataBlock, false, false);
+        const sourceForm = result?.firstParent(HTML_TAG_FORM).orElseLazy(() => result.byTagName(HTML_TAG_FORM, true));
         if (sourceForm) {
             this.storeForPostProcessing(sourceForm, result);
         }
@@ -220,9 +236,9 @@ export class ResponseProcessor implements IResponseProcessor {
      * @param node
      */
     attributes(node: XMLQuery) {
-        let elem = DQ.byId(node.id.value, true);
+        const elem = DQ.byId(node.id.value, true);
 
-        node.byTagName(TAG_ATTR).each((item: XMLQuery) => {
+        node.byTagName(XML_TAG_ATTR).each((item: XMLQuery) => {
             elem.attr(item.attr(ATTR_NAME).value).value = item.attr(ATTR_VALUE).value;
         });
     }
@@ -243,17 +259,17 @@ export class ResponseProcessor implements IResponseProcessor {
     insert(node: XMLQuery) {
         //let insertId = node.id; //not used atm
 
-        let before = node.attr(TAG_BEFORE);
-        let after = node.attr(TAG_AFTER);
+        const before = node.attr(XML_TAG_BEFORE);
+        const after = node.attr(XML_TAG_AFTER);
 
-        let insertNodes = DQ.fromMarkup(<any>node.cDATAAsString);
+        const insertNodes = DQ.fromMarkup(<any>node.cDATAAsString);
 
         if (before.isPresent()) {
             DQ.byId(before.value, true).insertBefore(insertNodes);
             this.internalContext.assign(UPDATE_ELEMS).value.push(insertNodes);
         }
         if (after.isPresent()) {
-            let domQuery = DQ.byId(after.value, true);
+            const domQuery = DQ.byId(after.value, true);
             domQuery.insertAfter(insertNodes);
 
             this.internalContext.assign(UPDATE_ELEMS).value.push(insertNodes);
@@ -266,12 +282,12 @@ export class ResponseProcessor implements IResponseProcessor {
      * @param node the node hosting the insert data
      */
     insertWithSubTags(node: XMLQuery) {
-        let before = node.querySelectorAll(TAG_BEFORE);
-        let after = node.querySelectorAll(TAG_AFTER);
+        const before = node.querySelectorAll(XML_TAG_BEFORE);
+        const after = node.querySelectorAll(XML_TAG_AFTER);
 
         before.each(item => {
-            let insertId = item.attr(ATTR_ID);
-            let insertNodes = DQ.fromMarkup(<any>item.cDATAAsString);
+            const insertId = item.attr(ATTR_ID);
+            const insertNodes = DQ.fromMarkup(<any>item.cDATAAsString);
             if (insertId.isPresent()) {
                 DQ.byId(insertId.value, true).insertBefore(insertNodes);
                 this.internalContext.assign(UPDATE_ELEMS).value.push(insertNodes);
@@ -279,8 +295,8 @@ export class ResponseProcessor implements IResponseProcessor {
         });
 
         after.each(item => {
-            let insertId = item.attr(ATTR_ID);
-            let insertNodes = DQ.fromMarkup(<any>item.cDATAAsString);
+            const insertId = item.attr(ATTR_ID);
+            const insertNodes = DQ.fromMarkup(<any>item.cDATAAsString);
             if (insertId.isPresent()) {
                 DQ.byId(insertId.value, true).insertAfter(insertNodes);
                 this.internalContext.assign(UPDATE_ELEMS).value.push(insertNodes);
@@ -295,7 +311,7 @@ export class ResponseProcessor implements IResponseProcessor {
      */
     processViewState(node: XMLQuery): boolean {
         if (ResponseProcessor.isViewStateNode(node)) {
-            let state = node.cDATAAsString;
+            const state = node.cDATAAsString;
             this.internalContext.assign(APPLIED_VST, node.id.value).value = new StateHolder($nsp(node.id.value), state);
             return true;
         }
@@ -304,7 +320,7 @@ export class ResponseProcessor implements IResponseProcessor {
 
     processClientWindow(node: XMLQuery): boolean {
         if (ResponseProcessor.isClientWindowNode(node)) {
-            let state = node.cDATAAsString;
+            const state = node.cDATAAsString;
             this.internalContext.assign(APPLIED_CLIENT_WINDOW, node.id.value).value = new StateHolder($nsp(node.id.value), state);
             return true;
         }
@@ -333,29 +349,29 @@ export class ResponseProcessor implements IResponseProcessor {
      */
     fixViewStates() {
         Stream.ofAssoc<StateHolder>(this.internalContext.getIf(APPLIED_VST).orElse({}).value)
-            .each((item: Array<any>) => {
-                let value: StateHolder = item[1];
-                let nameSpace = DQ.byId(value.nameSpace, true).orElse(document.body);
-                let affectedForms = nameSpace.byTagName(TAG_FORM);
-                let affectedForms2 = nameSpace.filter(item => item.tagName.orElse(EMPTY_STR).value.toLowerCase() == TAG_FORM);
+            .each(([, value]) => {
+                const namingContainerId = this.internalContext.getIf(PARTIAL_ID);
+                const affectedForms = this.getContainerForms(namingContainerId)
+                    .filter(affectedForm => this.isInExecuteOrRender(affectedForm));
 
-                this.appendViewStateToForms(new DomQuery(affectedForms, affectedForms2), value.value);
+                this.appendViewStateToForms(affectedForms, value.value, namingContainerId.orElse("").value);
             });
     }
 
+
+
     /**
      * same as with view states before applies the incoming client windows as last step after the rest of the processing
      * is done.
      */
     fixClientWindow() {
         Stream.ofAssoc<StateHolder>(this.internalContext.getIf(APPLIED_CLIENT_WINDOW).orElse({}).value)
-            .each((item: Array<any>) => {
-                let value: StateHolder = item[1];
-                let nameSpace = DQ.byId(value.nameSpace, true).orElse(document.body);
-                let affectedForms = nameSpace.byTagName(TAG_FORM);
-                let affectedForms2 = nameSpace.filter(item => item.tagName.orElse(EMPTY_STR).value.toLowerCase() == TAG_FORM);
+            .each(([, value]) => {
+                const namingContainerId = this.internalContext.getIf(PARTIAL_ID);
+                const affectedForms = this.getContainerForms(namingContainerId)
+                    .filter(affectedForm => this.isInExecuteOrRender(affectedForm));
 
-                this.appendClientWindowToForms(new DomQuery(affectedForms, affectedForms2), value.value);
+                this.appendClientWindowToForms(affectedForms, value.value, namingContainerId.orElse("").value);
             });
     }
 
@@ -363,10 +379,10 @@ export class ResponseProcessor implements IResponseProcessor {
      * all processing done we can close the request and send the appropriate events
      */
     done() {
-        let eventData = EventData.createFromRequest(this.request.value, this.externalContext, SUCCESS);
+        const eventData = EventData.createFromRequest(this.request.value, this.externalContext, SUCCESS);
 
         //because some frameworks might decorate them over the context in the response
-        let eventHandler = this.externalContext.getIf(ON_EVENT).orElseLazy(() => this.internalContext.getIf(ON_EVENT).value).orElse(EMPTY_FUNC).value;
+        const eventHandler = this.externalContext.getIf(ON_EVENT).orElseLazy(() => this.internalContext.getIf(ON_EVENT).value).orElse(EMPTY_FUNC).value;
         Implementation.sendEvent(eventData, eventHandler);
     }
 
@@ -375,9 +391,10 @@ export class ResponseProcessor implements IResponseProcessor {
      *
      * @param forms the forms to append the viewState to
      * @param viewState the final viewState
+     * @param namingContainerId
      */
-    private appendViewStateToForms(forms: DQ, viewState: string) {
-        this.assignState(forms, $nsp(SEL_VIEWSTATE_ELEM), viewState);
+    private appendViewStateToForms(forms: DQ, viewState: string, namingContainerId = "") {
+        this.assignState(forms, $nsp(SEL_VIEWSTATE_ELEM), viewState, namingContainerId);
     }
 
 
@@ -386,9 +403,10 @@ export class ResponseProcessor implements IResponseProcessor {
      *
      * @param forms the forms to append the viewState to
      * @param clientWindow the final viewState
+     * @param namingContainerId
      */
-    private appendClientWindowToForms(forms: DQ, clientWindow: string) {
-        this.assignState(forms, $nsp(SEL_CLIENT_WINDOW_ELEM), clientWindow);
+    private appendClientWindowToForms(forms: DQ, clientWindow: string, namingContainerId = "") {
+        this.assignState(forms, $nsp(SEL_CLIENT_WINDOW_ELEM), clientWindow, namingContainerId);
     }
 
     /**
@@ -398,29 +416,28 @@ export class ResponseProcessor implements IResponseProcessor {
      * @param selector the selector for the state
      * @param state the state itself which needs to be assigned
      *
+     * @param namingContainerId
      * @private
      */
-    private assignState(forms: DQ, selector: string, state: string) {
-        forms.each((form: DQ) => {
-            let stateHolders = form.querySelectorAll(selector)
-                .orElseLazy(() => ResponseProcessor.newViewStateElement(form));
-
-            stateHolders.attr("value").value = state;
+    private assignState(forms: DQ, selector: string, state: string, namingContainerId: string) {
+        /**
+         * creates the viewState or client window id element
+         * @param form
+         */
+        const createAndAppendHiddenInput = (form: DomQuery)  => {
+            return new HiddenInputBuilder(selector)
+                .withNamingContainerId(namingContainerId)
+                .withParent(form)
+                .build();
+        };
+
+        forms.each(form => {
+            const hiddenInput = form.querySelectorAll(selector)
+                .orElseLazy(() => createAndAppendHiddenInput(form));
+            hiddenInput.val = state;
         });
     }
 
-    /**
-     * Helper to Create a new JSF ViewState Element
-     *
-     * @param parent, the parent node to attach the viewState element to
-     * (usually a form node)
-     */
-    private static newViewStateElement(parent: DQ): DQ {
-        let newViewState = DQ.fromMarkup($nsp(HTML_VIEWSTATE));
-        newViewState.appendTo(parent);
-        return newViewState;
-    }
-
     /**
      * Stores certain aspects of the dom for later post-processing
      *
@@ -457,10 +474,10 @@ export class ResponseProcessor implements IResponseProcessor {
      * @returns if it is a viewState node
      */
     private static isViewStateNode(node: XMLQuery): boolean {
-        let separatorChar = (window?.faces ?? window?.jsf).separatorchar;
+        const SEP = $faces().separatorchar;
         return "undefined" != typeof node?.id?.value && (node?.id?.value == $nsp(P_VIEWSTATE) ||
-            node?.id?.value?.indexOf([separatorChar, $nsp(P_VIEWSTATE)].join(EMPTY_STR)) != -1 ||
-            node?.id?.value?.indexOf([$nsp(P_VIEWSTATE), separatorChar].join(EMPTY_STR)) != -1);
+            node?.id?.value?.indexOf([SEP, $nsp(P_VIEWSTATE)].join(EMPTY_STR)) != -1 ||
+            node?.id?.value?.indexOf([$nsp(P_VIEWSTATE), SEP].join(EMPTY_STR)) != -1);
     }
 
     /**
@@ -470,15 +487,59 @@ export class ResponseProcessor implements IResponseProcessor {
      * @returns true of it ii
      */
     private static isClientWindowNode(node: XMLQuery): boolean {
-        let separatorChar = (window?.faces ?? window?.jsf).separatorchar;
+        const SEP =  $faces().separatorchar;
         return "undefined" != typeof node?.id?.value && (node?.id?.value == $nsp(P_CLIENT_WINDOW) ||
-            node?.id?.value?.indexOf([separatorChar, $nsp(P_CLIENT_WINDOW)].join(EMPTY_STR)) != -1 ||
-            node?.id?.value?.indexOf([$nsp(P_CLIENT_WINDOW), separatorChar].join(EMPTY_STR)) != -1);
+            node?.id?.value?.indexOf([SEP, $nsp(P_CLIENT_WINDOW)].join(EMPTY_STR)) != -1 ||
+            node?.id?.value?.indexOf([$nsp(P_CLIENT_WINDOW), SEP].join(EMPTY_STR)) != -1);
     }
 
     private triggerOnError(errorData: ErrorData) {
-        this.externalContext.getIf(ON_ERROR).orElse(this.internalContext.getIf(ON_ERROR).value).orElse(EMPTY_FUNC).value(errorData);
+        this.externalContext.getIf(ON_ERROR).orElseLazy(() => this.internalContext.getIf(ON_ERROR).value).orElse(EMPTY_FUNC).value(errorData);
     }
 
+    /**
+     * filters the forms according to being member of the "execute" or "render" cycle
+     * @param affectedForm
+     * @private
+     */
+    private isInExecuteOrRender(affectedForm) {
+        const executes = this.externalContext.getIf($nsp(P_EXECUTE)).orElse("@none").value.split(/\s+/gi);
+        const renders = this.externalContext.getIf(P_RENDER_OVERRIDE)
+            .orElseLazy(() => this.externalContext.getIf($nsp(P_RENDER)).value)
+            .orElse(IDENT_NONE).value.split(/\s+/gi);
+        const executeAndRenders = executes.concat(...renders);
+        return LazyStream.of(...executeAndRenders).filter(nameOrId => {
+            if ([IDENT_ALL, IDENT_NONE].indexOf(nameOrId) != -1) {
+                return true;
+            }
+
+            const NAME_OR_ID = this.getNameOrIdSelector(nameOrId);
+            //either the form directly is in execute or render or one of its children or one of its parents
+            return affectedForm.matchesSelector(NAME_OR_ID) ||
+                affectedForm.querySelectorAll(NAME_OR_ID).isPresent() ||
+                affectedForm.firstParent(NAME_OR_ID).isPresent();
+        }).first().isPresent();
+    }
 
+    /**
+     * gets all forms under a single naming container id
+     * @param namingContainerId
+     * @private
+     */
+    private getContainerForms(namingContainerId: Config) {
+        if (namingContainerId.isPresent()) {
+            //naming container mode, all forms under naming container id must be processed
+            return DQ$(this.getNameOrIdSelector(namingContainerId.value))
+                // missing condition if the naming container is not present we have to
+                // use the body as fallback
+                .orElseLazy(() => DQ.byTagName(HTML_TAG_BODY))
+                .byTagName(HTML_TAG_FORM, true);
+        } else {
+            return DQ.byTagName(HTML_TAG_FORM);
+        }
+    }
+
+    private getNameOrIdSelector(nameOrId) {
+        return `[id='${nameOrId}'], [name='${nameOrId}']`;
+    }
 }

package/src/main/typescript/impl/xhrCore/XhrFormData.ts

@@ -109,7 +109,7 @@ export class XhrFormData extends Config {
      * @param form the form holding the view state value
      */
     private applyViewState(form: DQ) {
-        let viewState = form.byId(P_VIEWSTATE, true).inputValue;
+        let viewState = form.querySelectorAllDeep(`[name*='${P_VIEWSTATE}'`).inputValue;
         this.appendIf(viewState.isPresent(), P_VIEWSTATE).value = viewState.value;
     }
 

package/src/main/typescript/impl/xhrCore/XhrRequest.ts

@@ -23,20 +23,21 @@ import {ErrorData} from "./ErrorData";
 import {EventData} from "./EventData";
 import {ExtLang} from "../util/Lang";
 import {
+    $faces,
     BEGIN,
     COMPLETE,
     CONTENT_TYPE,
     CTX_PARAM_MF_INTERNAL,
-    CTX_PARAM_PASS_THR,
+    CTX_PARAM_REQ_PASS_THR,
     ERROR,
     HEAD_FACES_REQ,
     MALFORMEDXML,
     NO_TIMEOUT,
     ON_ERROR,
-    ON_EVENT, P_EXECUTE,
+    ON_EVENT, P_EXECUTE, P_PARTIAL_SOURCE,
     REQ_ACCEPT,
     REQ_TYPE_GET,
-    REQ_TYPE_POST,
+    REQ_TYPE_POST, SOURCE,
     STATE_EVT_TIMEOUT,
     STD_ACCEPT,
     URL_ENCODED,
@@ -44,6 +45,7 @@ import {
 } from "../core/Const";
 import {resolveFinalUrl, resolveHandlerFunc} from "./RequestDataResolver";
 import failSaveExecute = ExtLang.failSaveExecute;
+import {ExtConfig} from "../util/ExtDomQuery";
 
 /**
  * Faces XHR Request Wrapper
@@ -86,7 +88,7 @@ export class XhrRequest implements AsyncRunnable<XMLHttpRequest> {
     constructor(
         private source: DQ,
         private sourceForm: DQ,
-        private requestContext: Config,
+        private requestContext: ExtConfig,
         private internalContext: Config,
         private partialIdsArray = [],
         private timeout = NO_TIMEOUT,
@@ -111,12 +113,12 @@ export class XhrRequest implements AsyncRunnable<XMLHttpRequest> {
         let xhrObject = this.xhrObject;
 
         let executesArr = () => {
-            return this.requestContext.getIf(CTX_PARAM_PASS_THR, P_EXECUTE).get("none").value.split(/\s+/gi);
+            return this.requestContext.getIf(CTX_PARAM_REQ_PASS_THR, P_EXECUTE).get("none").value.split(/\s+/gi);
         };
         try {
 
             let formElement = this.sourceForm.getAsElem(0).value;
-            let viewState = (window?.faces ?? window?.jsf).getViewState(formElement);
+            let viewState = $faces().getViewState(formElement);
             // encoded we need to decode
             // We generated a base representation of the current form
             // in case someone has overloaded the viewState with additional decorators we merge
@@ -131,15 +133,22 @@ export class XhrRequest implements AsyncRunnable<XMLHttpRequest> {
             this.contentType = formData.isMultipartRequest ? "undefined" : this.contentType;
 
             // next step the pass through parameters are merged in for post params
+            this.requestContext.$nspEnabled = false;
             let requestContext = this.requestContext;
-            let passThroughParams = requestContext.getIf(CTX_PARAM_PASS_THR);
-
+            let requestPassThroughParams = requestContext.getIf(CTX_PARAM_REQ_PASS_THR) as ExtConfig;
+            requestPassThroughParams.$nspEnabled = false;
             // this is an extension where we allow pass through parameters to be sent down additionally
             // this can be used and is used in the impl to enrich the post request parameters with additional
             // information
-            formData.shallowMerge(passThroughParams, true, true);
 
-            this.responseContext = passThroughParams.deepCopy;
+            try {
+                formData.shallowMerge(requestPassThroughParams, true, true);
+            } finally {
+                this.requestContext.$nspEnabled = true;
+                requestPassThroughParams.$nspEnabled = true;
+            }
+
+            this.responseContext = requestPassThroughParams.deepCopy;
 
             // we have to shift the internal passthroughs around to build up our response context
             let responseContext = this.responseContext;
@@ -290,7 +299,7 @@ export class XhrRequest implements AsyncRunnable<XMLHttpRequest> {
             return;
         }
 
-        (window?.faces ?? window.jsf).ajax.response(this.xhrObject, this.responseContext.value ?? {});
+        $faces().ajax.response(this.xhrObject, this.responseContext.value ?? {});
     }
 
     private handleMalFormedXML(resolve: Function) {
@@ -300,9 +309,9 @@ export class XhrRequest implements AsyncRunnable<XMLHttpRequest> {
             status: MALFORMEDXML,
             responseCode: 200,
             responseText: this.xhrObject?.responseText,
-            source: {
-                id: this.source.id.value
-            }
+            // we remap the element just in case it gets replaced
+            // it will be unremapped
+            source:  this.source.id.value
         };
         try {
             this.handleError(errorData, true);
@@ -352,6 +361,7 @@ export class XhrRequest implements AsyncRunnable<XMLHttpRequest> {
 
             Implementation.sendEvent(eventData, eventHandler);
         } catch (e) {
+            e.source = e?.source ?? this.requestContext.getIf(SOURCE).value;
             this.handleError(e);
             throw e;
         }