PowerShell to crete HTML report

I am using the attached PowerShell scripts (XeAapp_Check.ps1, Ctx_functions.ps1) to export the XenApp 4.5 Zone details to HTML file as a report. On executing the script for the first time I am getting the desired result but unfortunately everytime I execute the script, the zone details gets appended to that report.

Here is my server details:

Farm Name: MyFarm

Server Name              zone Name
==========               ==========
XEN-1                           192.168.1.0
XEN-2                           TEST ZONE

Please assist me to get the correct result so that I can get the desired output by correctly modifying that script.
Ctx-Functions.txt
XenApp-Check.txt
XenApp-Check--Farm-Name--MyFarm-.mht
hchabriaAsked:
Who is Participating?
 
SubsunCommented:
Did you try setting $OutputEnteteZone1 to $null
$OutputEnteteZone1 = $null
If ($Check_OutputEnteteZone -eq "TRUE")
{
	
	Foreach ($Zones in $Zones) {
	$OutputEnteteZone1 += @($Zones | Select @{N="Zone Name";E={if($Zones.ZoneName -eq $Null){"Error"}else{$Zones.ZoneName}}}, @{N="Zone Data Collector";E={if($Zones.DataCollector -eq $Null){"Error"}else{$Zones.DataCollector}}}) }
	$XenAppReport += Get-CustomHeader ("Zone(s) Details : "+($Zones_Count))
	$XenAppReport += Get-HTMLTable $OutputEnteteZone1
	$XenAppReport += Get-CustomHeaderClose
	
}

Open in new window

0
 
Meir RivkinFull stack Software EngineerCommented:
the problem is that its appended to the html result instead of generating fresh one?
can u describe expected result?
0
 
hchabriaAuthor Commented:
Yes, that is the exact problem. Can you please guide me to rectify this?
0
Making Bulk Changes to Active Directory

Watch this video to see how easy it is to make mass changes to Active Directory from an external text file without using complicated scripts.

 
hchabriaAuthor Commented:
Please find the attached expected result.
XenApp-Check--Farm-Name--MyFarm-.mht
0
 
Meir RivkinFull stack Software EngineerCommented:
in XenApp-Check.txt script just before you call out-file to save as .htm (scroll all the way down the script), you wanna delete them if already exists:

if(Test-Path  $FileXenApp_Check){
remove-item $FileXenApp_Check
}

if(Test-Path  $FileXenApp_CheckBackup){
remove-item  $FileXenApp_CheckBackup
	
}

$XenAppReport | out-file -encoding ASCII -filepath $FileXenApp_Check
$XenAppReport | out-file -encoding ASCII -filepath $FileXenApp_CheckBackup
	
echo "DONE"

Open in new window

0
 
Meir RivkinFull stack Software EngineerCommented:
i saw that you hard coded:
$Check_OutputEnteteZone = "TRUE"

when u generate the html, you iterate the zones:
If ($Check_OutputEnteteZone -eq "TRUE")
{
Foreach ($Zones in $Zones) {

so the if statement is redundant (cause its always true).
0
 
hchabriaAuthor Commented:
I've added the code you provided but still having the same result (please see the attached file). The report gets deleted but when recreated, the restult gets appended in the report.

I am not that much expert on PowerShell. Can you please guide me what should I do in case if I set $Check_OutputEnteteZone = "TRUE".

It would be very helpful of me if you could provide me the rectified code.
XenApp-Check--Farm-Name--MyFarm-.mht
0
 
hchabriaAuthor Commented:
Event I've deleted the "If ($Check_OutputEnteteZone -eq "TRUE")" condition and then executed the script, but getting same result.

#Farm Name
$XenAppReport = Get-CustomHTML "Farm Name : $Farm"
$XenAppReport += Get-CustomHeaderClose

#Write-CustomOut "..Adding Zones Details Info to the report"
	Foreach ($Zones in $Zones) {
	$OutputEnteteZone2 += @($Zones | Select @{N="Zone Name";E={if($Zones.ZoneName -eq $Null){"Error"}else{$Zones.ZoneName}}}, @{N="Zone Data Collector";E={if($Zones.DataCollector -eq $Null){"Error"}else{$Zones.DataCollector}}}) }
	$XenAppReport += Get-CustomHeader ("Zone(s) Details : "+($Zones_Count))
	$XenAppReport += Get-HTMLTable $OutputEnteteZone2
	$XenAppReport += Get-CustomHeaderClose
	

$Date = Get-Date -Format "yyyyMMddHH"
$FileXenApp_Check = $Path + $FileName + ".htm"
$FileXenApp_CheckBackup = $PathBackup + $FileName + "_" + $date + ".htm"


if(Test-Path  $FileXenApp_Check){
remove-item $FileXenApp_Check
}

if(Test-Path  $FileXenApp_CheckBackup){
remove-item  $FileXenApp_CheckBackup
	
}

$XenAppReport | out-file -encoding ASCII -filepath $FileXenApp_Check
$XenAppReport | out-file -encoding ASCII -filepath $FileXenApp_CheckBackup
	
echo "DONE"

Open in new window

0
 
hchabriaAuthor Commented:
What I have seen that, if I change the following variable name everytime then, I am getting the desired result.

$OutputEnteteZone

Also, I there is any way to delete the value of that variable then I think the result will come but I don't have any clue about how to do this.
0
 
Meir RivkinFull stack Software EngineerCommented:
change this:
If ($Check_OutputEnteteZone -eq "TRUE")
{
	Foreach ($Zones in $Zones) {
	$OutputEnteteZone1 += @($Zones | Select @{N="Zone Name";E={if($Zones.ZoneName -eq $Null){"Error"}else{$Zones.ZoneName}}}, @{N="Zone Data Collector";E={if($Zones.DataCollector -eq $Null){"Error"}else{$Zones.DataCollector}}}) }
	$XenAppReport += Get-CustomHeader ("Zone(s) Details : "+($Zones_Count))
	$XenAppReport += Get-HTMLTable $OutputEnteteZone1
	$XenAppReport += Get-CustomHeaderClose
}

Open in new window


to this:
	$OutputEnteteZone1 += @($Zones | Select @{N="Zone Name";E={if($Zones.ZoneName -eq $Null){"Error"}else{$Zones.ZoneName}}}, @{N="Zone Data Collector";E={if($Zones.DataCollector -eq $Null){"Error"}else{$Zones.DataCollector}}}) 
	$XenAppReport += Get-CustomHeader ("Zone(s) Details : "+($Zones_Count))
	$XenAppReport += Get-HTMLTable $OutputEnteteZone1
	$XenAppReport += Get-CustomHeaderClose

Open in new window


--> remove if stetment and foreach stetment.
the if is redundant and the foreach already takes place when you call:
$OutputEnteteZone1 += @($Zones | ....
0
 
Meir RivkinFull stack Software EngineerCommented:
btw, can u debug your script?
0
 
hchabriaAuthor Commented:
Please find below the code as you instructed and the attached result. The result is getting appended everytime and the ouput is producing an error as $Zones.ZoneName value is producing NULL value.

$XenAppReport = Get-CustomHTML "Farm Name : $Farm"
$XenAppReport += Get-CustomHeaderClose

$OutputEnteteZone1 += @($Zones | Select @{N="Zone Name";E={if($Zones.ZoneName -eq $Null){"Error"}else{$Zones.ZoneName}}}, @{N="Zone Data Collector";E={if($Zones.DataCollector -eq $Null){"Error"}else{$Zones.DataCollector}}}) 
$XenAppReport += Get-CustomHeader ("Zone(s) Details : "+($Zones_Count))
$XenAppReport += Get-HTMLTable $OutputEnteteZone1
$XenAppReport += Get-CustomHeaderClose

$Date = Get-Date -Format "yyyyMMddHH"
$FileXenApp_Check = $Path + $FileName + ".htm"
$FileXenApp_CheckBackup = $PathBackup + $FileName + "_" + $date + ".htm"

if(Test-Path  $FileXenApp_Check){
remove-item $FileXenApp_Check
}
if(Test-Path  $FileXenApp_CheckBackup){
remove-item  $FileXenApp_CheckBackup
}

$XenAppReport | out-file -encoding ASCII -filepath $FileXenApp_Check
$XenAppReport | out-file -encoding ASCII -filepath $FileXenApp_CheckBackup
	
echo "DONE"

Open in new window

Report.PNG
0
 
Meir RivkinFull stack Software EngineerCommented:
the problem is not appending the file but the $zones creation.
there's a function called GET-XAZone i assume which returns aggregated $Zones.
i couldn't;t find source of this function in the files you've attached.
please check $Zones.Count when code is back from GET-XAZone.
i guess that from second run you get aggregated result.
but again do not have the source of GET-XAZone.
0
 
Meir RivkinFull stack Software EngineerCommented:
btw, you can remove the test-path code i've suggested before cause its not the issue.
0
 
hchabriaAuthor Commented:
I posted the above result after debugging my script only.
0
 
hchabriaAuthor Commented:
Excellent Subsun. It's working perfectly now after setting $OutputEnteteZone1 to $null.

Thank you so much.
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.