Code review comment for lp:~jontai/openvista-gtm-integration/bug429699

Revision history for this message
Jon Tai (jontai) wrote :

> We already have
> ZRULE=$G(ZRULE) in the routine
>
> I like Jon am wondering how we got here without ZRULE defined.
>
> Although the change will not hurt anything and the code would fix the bug as
> described.

I discussed this with Jeff and we decided to drop this patch. Under normal circumstances, ZRULE should be set to $G(ZRULE) already -- we must have been calling EXIT directly during debugging when we ran into this. We can re-open the bug if we run into a real-world situation where EXIT is called and ZRULE is undefined.

« Back to merge proposal