Add parameter ExtractWebParts to command Add-PnPFileToProvisioningTemplate

[stevebeauge]

Before creating a pull request, make sure that you have read the contribution file located at

https://github.com/SharePoint/PnP-PowerShell/blob/dev/CONTRIBUTING.md

Type

• Bug Fix
• New Feature
• Sample

Related Issues?

What is in this Pull Request ?

This PR adds a switch parameter ExtractWebPart to the command Add-PnPFileToProvisioningTemplate. When set, if the file is a classic page, its webparts are included in the target provisioning template

[KoenZomers]

Thanks for this contribution @stevebeauge. Trying to validate your submission. Can you elaborate why the code change in Commands/Utilities/StringExtensions.cs would be necessary? You write to make it more reliable. Is there any concrete scenario where the current code fails to do what it needs to do? Otherwise I would say don't try to fix something that isn't broken. Can you elaborate on your thoughts on this?

[stevebeauge]

Thanks for this contribution @stevebeauge. Trying to validate your submission. Can you elaborate why the code change in Commands/Utilities/StringExtensions.cs would be necessary? You write to make it more reliable. Is there any concrete scenario where the current code fails to do what it needs to do? Otherwise I would say don't try to fix something that isn't broken. Can you elaborate on your thoughts on this?

@KoenZomers : it's in fact a performance improvement. My comment was maybe a bit confusing. The background of this optimization came from a stackoverflow answer.