Enabled setting folder property bags on noscript sites

[jensotto]

Type

• Bug Fix
• New Feature
• Sample

Related Issues?

N/A

What is in this Pull Request ?

Setting properties on folders in document libraries is supported even on no script sites.

Web.IsNoScriptSite() extension method is not a definitive answer to if the site in fact is enabled for NoScript. This can also return true for sites that does not have NoScript enabled and the current user (or app only credentials) does not have required permissions to modify data.

[KoenZomers]

Thanks @jensotto. I've given this PR a try on a modern teamsite, but am running into the ServerUnauthorizedAccessException even though I'm site collection admin. Can you provide more information in which scenario you managed so set a property bag value on a folder located in a site with noscript enabled?

[jensotto]

@KoenZomers
This is what I did for a site with DenyAddAndCustomizePages enabled both OnPrem and Online:

Connect-PnPOnline <siteUrl> 
$folder = Get-PnPFolder "/<server relative URL to site>/shared documents"
$folder.Properties["Test"] = "Test"
$folder.Update()
$folder.Context.ExecuteQuery()

[KoenZomers]

Thanks @jensotto. Key here was that you're not trying to set a propertybag value on the root folder but on any subfolder (Shared Documents in your sample). Then it indeed does work fine with this fix. Merging it through PR #2567