Merge pull request #3109 from pnp/revert-3093-fix/aliased-parameters-v3

Revert "Improved aliased parameter parsing v3"
pnp · Aug 12, 2024 · 208cfa9 · 208cfa9
2 parents 5ef2f7f + 2e4dabc
commit 208cfa9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 48 deletions.
diff --git a/packages/sp/spqueryable.ts b/packages/sp/spqueryable.ts
@@ -81,11 +81,17 @@ export class _SPQueryable<GetType = any> extends Queryable<GetType> {
 
         const aliasedParams = new URLSearchParams(this.query);
 
-        // this regex is designed to locate aliased parameters within url paths
-        let url = this.toUrl().replace(/'!(@.+?)::((?:[^']|'')+)'/ig, (match, labelName, value) => {
+        // this regex is designed to locate aliased parameters within url paths. These may have the form:
+        // /something(!@p1::value)
+        // /something(!@p1::value, param=value)
+        // /something(param=value,!@p1::value)
+        // /something(param=value,!@p1::value,param=value)
+        // /something(param=!@p1::value)
+        // there could be spaces or not around the boundaries
+        let url = this.toUrl().replace(/([( *| *, *| *= *])'!(@.*?)::(.*?)'([ *)| *, *])/ig, (match, frontBoundary, labelName, value, endBoundary) => {
             this.log(`Rewriting aliased parameter from match ${match} to label: ${labelName} value: ${value}`, 0);
-            aliasedParams.set(labelName, `'${value}'`);
-            return labelName;
+            aliasedParams.set(labelName,`'${value}'`);
+            return `${frontBoundary}${labelName}${endBoundary}`;
         });
 
         const query = aliasedParams.toString();

diff --git a/test/sp/alias.ts b/test/sp/alias.ts
@@ -6,7 +6,6 @@ import "@pnp/sp/files/web";
 import "@pnp/sp/files/folder";
 import "@pnp/sp/lists/web";
 import { combine } from "@pnp/core";
-import { SPQueryable } from "@pnp/sp";
 
 describe("Alias Parameters", function () {
 
@@ -29,49 +28,6 @@ describe("Alias Parameters", function () {
         await ler.list.rootFolder.files.addUsingPath("text.txt", "Some file content!");
     });
 
-    it("Parameter parsing", function() {
-        /** Values to test */
-        const values = [
-            "value",
-            "value's",
-            "value with space",
-            "value with space' and apostrophe",
-            "ending with apostrophe'",
-            "'staring with apostrophe",
-            "'staring and ending with apostrophe'",
-            "with,' comma",
-        ];
-        /** Aliased parameters to test */
-        const tests = values.reduce<Record<string, Record<string, string>>>((obj, value)=>{
-            // Escape apostrophe in value
-            value = value.replace(/'/g, "''");
-
-            obj[`something('!@p1::${value}')`] = {"@p1": `'${value}'`};
-            obj[`something('!@p1::${value}','!@p2::${value}2')`] = {"@p1": `'${value}'`, "@p2": `'${value}2'`};
-            obj[`something('!@p1::${value}', param=value)`] = {"@p1": `'${value}'`};
-            obj[`something('!@p1::${value}', param=value, '!@p2::${value}2')`] = {"@p1": `'${value}'`, "@p2": `'${value}2'`};
-            obj[`something(param=value,'!@p1::${value}')`] = {"@p1": `'${value}'`};
-            obj[`something(param=value,'!@p1::${value}','!@p2::${value}2')`] = {"@p1": `'${value}'`, "@p2": `'${value}2'`};
-            obj[`something(param=value,'!@p1::${value}',param=value)`] = {"@p1": `'${value}'`};
-            obj[`something(param=value,'!@p1::${value}',param=value,'!@p2::${value}2')`] = {"@p1": `'${value}'`, "@p2": `'${value}2'`};
-            obj[`something(param='!@p1::${value}')`] = {"@p1": `'${value}'`};
-            obj[`something(param='!@p1::${value}',param2='!@p2::${value}2')`] = {"@p1": `'${value}'`, "@p2": `'${value}2'`};
-            return obj;
-        }, {});
-
-        // Test all aliased parameters
-        for(const [alias, params] of Object.entries(tests)) {
-            const requestUrl = SPQueryable(this.pnp.sp.web, alias).toRequestUrl();
-            const searchParams = Object.fromEntries(new URL(requestUrl).searchParams.entries());
-
-            // eslint-disable-next-line guard-for-in
-            for(const param in params) {
-                expect(searchParams, `Failed to parse "${alias}"`).to.have.property(param);
-                expect(searchParams[param], `Failed to parse "${alias}"`).to.equal(params[param]);
-            }
-        }
-    });
-
     it("Folders", function () {
 
         return expect(this.pnp.sp.web.getFolderByServerRelativePath(`!@p1::/${combine(webRelativeUrl, "AliasTestLib/MyTestFolder")}`)()).to.eventually.be.fulfilled;