Feedback on asterisak@home extensions_custom.conf?

okay, so i finally figured out how to get zapateller and privacy manager to work for VOIP and PSTN lines. one complication is that voicepulse is incorrectly sending “Unknown” for the CID so i wanted to null out the CID info so that zapateller and privacy manager could do their thing. drove me nuts for awhile, since SetCallerID() would NOT accept a null input, and since it wants CIDNUM and CIDNAME, i had the idea of passing (,) and it worked. anyway, here’s my [from-pstn-custom] context. any feeback?

[from-pstn-custom]
exten => s,1,Answer
exten => s,n,GotoIf($["${CALLERIDNUM}" = “Unknown”]:CallerIDOkay)
exten => s,n,SetCallerID(,)
exten => s,n(CallerIDOkay),Zapateller(nocallerid)
exten => s,n(PrivacyManager),PrivacyManager
exten => s,PrivacyManager+101,Background(tt-allbusy)
exten => s,n,Background(tt-somethingwrong)
exten => s,n,Background(tt-monkeysintro)
exten => s,n,Background(tt-weasels)
exten => s,n,Background(tt-monkeys-intro)
exten => s,n,Background(lots-o-monkeys)
exten => s,n,Background(the-monkeys-twice)
exten => s,n,Background(tt-monkeys)