updated readme

Author: Zerg00s

ConvertPackage.ps1

@@ -27,7 +27,7 @@ $resources = Import-Csv  -Path .\resourceMapping.csv
 
 # Create dist folder if it does not exist
 if (-not (Test-Path .\dist)) {
-    $suppress = New-Item -ItemType Directory -Path .\dist
+    New-Item -ItemType Directory -Path .\dist | Out-Null
 }
 
 for ($k = 0; $k -lt $packages.count; $k++) {

MISC/Move-Lists.ps1

@@ -109,8 +109,8 @@ if ($MigrationType -eq "Export") {
     # Select all extended columns nodes based on the Group attribute
     # We don't want to migrate extended columns
     $extendedColumnNodes = $xml.SelectNodes("//*[@Group='Extended Columns']")
-    if($extendedColumnNodes -ne $null) {
-        Write-host "Extended columns count: " $extendedColumnNodes.Count
+    if ($extendedColumnNodes -ne $null) {
+        Write-Host "Extended columns count: " $extendedColumnNodes.Count
         # Remove each node
         foreach ($extendedColumnNode in $extendedColumnNodes) {
             $extendedColumnNode.ParentNode.RemoveChild($extendedColumnNode) | Out-Null
@@ -119,27 +119,34 @@ if ($MigrationType -eq "Export") {
 
     # Get all 'Field' nodes with attribute Hidden='TRUE'
     $hiddenFields = $xml.SelectNodes("//Field[@Hidden='TRUE']")
-    if($null -ne $hiddenFields) {
-        Write-host "Hidden fields count: " $hiddenFields
+    if ($null -ne $hiddenFields) {
+        Write-Host "Hidden fields count: " $hiddenFields
         # Remove all 'Field' nodes with attribute Hidden='TRUE'
-        foreach($field in $hiddenFields)
-        {
+        foreach ($field in $hiddenFields) {
             $field.ParentNode.RemoveChild($field) | Out-Null
         }
 
     }
 
-    $siteFields = $xml.GetElementsByTagName("pnp:SiteFields")
+    $siteFields = $xml.GetElementsByTagName("pnp:SiteFields")   
+   
     # Check if 'SiteFields' node is empty
-    if ($siteFields.ChildNodes.Count -eq 0) {
-        # Remove the 'SiteFields' node completely
-        $siteFields.ParentNode.RemoveChild($siteFields)
+    if ($null -ne $siteFields) {
+        if($siteFields.GetType().Name -eq "XmlElementList"){
+            $siteFields = $siteFields[0]
+        }
+
+        if ( $siteFields.ChildNodes.Count -eq 0) {
+            # Remove the 'SiteFields' node completely
+            $siteFields.ParentNode.RemoveChild($siteFields) | Out-Null
+        }
     }
 
+    
     $propertyBagEntries = $xml.GetElementsByTagName('pnp:PropertyBagEntries')
     if ($propertyBagEntries -ne $null -and $propertyBagEntries.Count -gt 0) {
         for ($i = $propertyBagEntries.Count - 1; $i -gt -1 ; $i--) {
-            $supress = $propertyBagEntries[$i].ParentNode.RemoveChild($propertyBagEntries[$i])
+            $propertyBagEntries[$i].ParentNode.RemoveChild($propertyBagEntries[$i])  | Out-Null
         }
     }
 
@@ -150,7 +157,13 @@ if ($MigrationType -eq "Export") {
     foreach ($title in $titles) {
         # Get the latest list item form layout. Footer, Header and the Body:
         $list = Get-PnPList $title -Includes ContentTypes
+        if ($list -is [array]) {
+            $list = $list[0]
+        }        
         $contentType = $list.ContentTypes | Where-Object { $_.Name -eq "Item" }
+        if($null -eq $contentType){
+            continue
+        }
         $contentType.ClientFormCustomFormatter | Set-Content .\$title.json
     }
 }

MISC/PS-Forms.ps1

@@ -126,7 +126,7 @@ function Get-FormArrayItem {
 
         $ComboBox.DropDownStyle  = [System.Windows.Forms.ComboBoxStyle]::DropDown;                      
         foreach($item in $items){
-            $suppress = $ComboBox.Items.add($item)
+            $ComboBox.Items.add($item) | Out-Null
         }
         $ComboBox.SelectedIndex = 1        
 
@@ -144,7 +144,7 @@ function Get-FormArrayItem {
         $Panel.Height += $spaceBetweenControls    
 
         function Combobox{
-            if ($ComboBox.SelectedItem -eq $null){
+            if ($null -eq $ComboBox.SelectedItem){
                 $OKButton.Enabled = $false
             }else{
                 $OKButton.Enabled = $true

README.md

@@ -1,6 +1,6 @@
 ![SharePoint Online](https://img.shields.io/badge/SharePoint-Online-yellow.svg) 
 ![Windows](https://img.shields.io/static/v1?label=OS&message=Windows&color=green)
-![Version](https://img.shields.io/static/v1?label=Version&message=3.9.5&color=white)
+![Version](https://img.shields.io/static/v1?label=Version&message=3.9.6&color=white)
 
 
 # Flow & Power Apps Migrator
@@ -182,7 +182,15 @@ SharePoint List forms customized with Power Apps can be migrated if you follow t
 
 # Latest Updates
 
-### 3.9.4 Version - 2023-08-6
+### 3.9.6 Version - 2023-09-06
+#### Changes
+- Addressed issue #66 [You cannot call a method on a null-valued expression](https://github.com/Zerg00s/FlowPowerAppsMigrator/issues/66)
+
+### 3.9.5 Version - 2023-08-30
+#### Changes
+- Removed the REST API call that was causing 403 error for some users.
+
+### 3.9.4 Version - 2023-08-06
 #### Changes
 - The pnp:SiteFields node is now automatically removed from the Lists.xml PnP provisioning template if it's empty. An empty node was causing the "invalid template" errors.