Pnpjs: Search bug: searchWithCaching - TypeError: Cannot read property 'RelevantResults' of undefined

[Gernouille]

Category

• [ ] Enhancement
• [x] Bug
• [ ] Question
• [ ] Documentation gap/issue

Version

Please specify what version of the library you are using: [2.0.2-5]
Please specify what version(s) of SharePoint you are targeting: [SPO]

Expected / Desired Behavior / Question

I want to execute a search with caching in an otherwise blank webpart.

Observed Behavior

I created a blank new SPFx webpart using the latest beta package as suggested in issue https://github.com/pnp/pnpjs/issues/1027.
When performing a search using await await sp.searchWithCaching("contentclass:STS_Site path:https://[tenant].sharepoint.com/sites/[prefix]*", cacheOptions), the search request is executed but the SearchResults object contains only undefined properties. When attempting to use the search results the browser console window reports the error: TypeError: Cannot read property 'RelevantResults' of undefined.

When the same search is executed with sp.search("contentclass:STS_Site path:https://[tenant].sharepoint.com/sites/[prefix]*") with the same parameter (except for the caching options), everything goes smoothly.

Steps to Reproduce

Create a new blank SPFx webpart using the latest Yeoman generator
Add the relevant imports (@pnp/[email protected] in particular), setup and add a search using await sp.searchWithCaching("[your query]", cacheOptions).
Run the webpart in a hosted workbench

[patrick-rodgers]

Can you try with the latest published version, 2.0.4? That beta is old at this point.

[Gernouille]

I updated to 2.0.4 and it's still happening.

To be sure that I'm running on 2.0.4 package, I did the whole thing:

• npm i @pnp/[email protected] --save
• gulp clean
• node_modules folder deletion
• npm install

[patrick-rodgers]

Ok, thanks. We'll have a look when we have a chance.

[patrick-rodgers]

Yep, this was a bug. The url wasn't getting set right so it wasn't hitting search at all, which when you're trying to search doesn't help. Fixed and will go out in next release. Thanks for reporting this!

[unnieayilliath]

@patrick-rodgers thanks for fixing this. Do you when is the next release? Any way for me to get the build with this fix?

[juliemturner]

@unnieayilliath We intend to release in the coming week but I cannot guarantee that will happen. You can, of course, fork the repo and build it yourself, so it is doable just maybe not ideal.

[unnieayilliath]

Thanks @juliemturner , for the time being I will use the sp.search as everything is working smoothly in that method