Repository: microsoft/CSS-Exchange
File: Calendar/Get-CalendarDiagnosticObjectsSummary.ps1 (PowerShell)

Code before:
    }
    if ($ClientInfoString -like "*InternalCalendarSharing*" -and $ClientInfoString -like "*Outlook*") {
        $ShortClientName = "Outlook-ModernCalendarSharing"
    }
    if ($ClientInfoString -like "Client=ActiveSync*" -and $ClientInfoString -like "*Outlook*") {
        $ShortClientName = "Outlook-ModernCalendarSharing"
    }

    if ($ShortClientName -eq "") {
        $ShortClientName = "[NoShortNameFound]"
    }

    return $ShortClientName
}

<#
.SYNOPSIS
Checks to see if the Calendar Log is Ignorable.
Many updates are not interesting in the Calendar Log, marking these as ignorable. 99% of the time this is correct.
#>
function SetIsIgnorable {
    param(
        $CalLog
    )

    if ($ShortClientName -like "TBA*SharingSyncAssistant" `
            -or $ShortClientName -eq "CalendarReplication" `
            -or $CalendarItemTypes.($CalLog.ItemClass) -eq "SharingCFM" `
            -or $CalendarItemTypes.($CalLog.ItemClass) -eq "SharingDelete") {
        return "Sharing"
    } elseif ($CalendarItemTypes.($CalLog.ItemClass) -eq "RespAny" `
            -or $CalendarItemTypes.($CalLog.ItemClass) -eq "AttendeeList") {
        return "Resp"
    } elseif ($ShortClientName -like "EBA*" `
            -or $ShortClientName -like "TBA*" `
            -or $ShortClientName -eq "LocationProcessor" `
            -or $ShortClientName -eq "GriffinRestClient" `
            -or $ShortClientName -eq "RestConnector" `
            -or $ShortClientName -eq "TimeService" ) {
        return "True"
    } else {
        return "False"
    }
}

<#
.SYNOPSIS
Replaces a value of NotFound with a blank string.
#>
function ReplaceNotFound {
    param (

Review comment (suggestion):
Let's avoid using back ticks here for a new line. Instead just place your `-or` or `-and` on the previous line and PowerShell knows to continue to the next line. 

Code after:
    if ($ClientInfoString -like "*InternalCalendarSharing*" -and
        $ClientInfoString -like "*OWA*" -and
        $ClientInfoString -notlike "*OneOutlook*") {
        $ShortClientName = "Owa-ModernCalendarSharing"
    }
    if ($ClientInfoString -like "*InternalCalendarSharing*" -and $ClientInfoString -like "*MacOutlook*") {
        $ShortClientName = "MacOutlook-ModernCalendarSharing"
    }
    if ($ClientInfoString -like "*InternalCalendarSharing*" -and $ClientInfoString -like "*Outlook*") {
        $ShortClientName = "Outlook-ModernCalendarSharing"
    }
    if ($ClientInfoString -like "Client=ActiveSync*" -and $ClientInfoString -like "*Outlook*") {
        $ShortClientName = "Outlook-ModernCalendarSharing"
    }
    if ($ClientInfoString -like "*OneOutlook*") {
        $ShortClientName = "OneOutlook"
    }
    if ($ShortClientName -eq "") {
        $ShortClientName = "[NoShortNameFound]"
    }

    return $ShortClientName
}

<#
.SYNOPSIS
Checks to see if the Calendar Log is Ignorable.
Many updates are not interesting in the Calendar Log, marking these as ignorable. 99% of the time this is correct.
#>
function SetIsIgnorable {
    param(
        $CalLog
    )

    if ($ShortClientName -like "TBA*SharingSyncAssistant" -or
        $ShortClientName -eq "CalendarReplication" -or
        $CalendarItemTypes.($CalLog.ItemClass) -eq "SharingCFM" -or
        $CalendarItemTypes.($CalLog.ItemClass) -eq "SharingDelete") {
        return "Sharing"
    } elseif (($CalendarItemTypes.($CalLog.ItemClass) -like "*Resp*" -and
            $CalLog.CalendarLogTriggerAction -ne "Create" ) -or
        $CalendarItemTypes.($CalLog.ItemClass) -eq "AttendeeList") {
        return "Resp"
    } elseif ($ShortClientName -like "EBA*" -or
        $ShortClientName -like "TBA*" -or
        $ShortClientName -eq "LocationProcessor" -or
        $ShortClientName -eq "GriffinRestClient" -or
        $ShortClientName -eq "RestConnector" -or
        $ShortClientName -eq "ELC-B2" -or
        $ShortClientName -eq "TimeService" ) {
        return "True"