Why all the unnecessary code? Why not?
Public Function ProtectStatus() As String
If ActiveSheet.Protect Then
ProtectStatus = "Protected"
Else
ProtectStatus = "Unprotected"
End If
End Function
And you never need code like "pStatus = True" as pStatus is already a
boolean.
Don <
www.donwiss.com> (e-mail link at home page bottom).