HomePhorge

IRC Macro Handler - check result from conduit call before operating on it

Description

IRC Macro Handler - check result from conduit call before operating on it

Summary: 'cuz github issue 114 came into existence. instead, just return false early here. note i am not sure if I should phlog that this is happening or not but its not exception worthy IMO.

Test Plan: lint-only 'cuz i don't want to setup an IRC server locally / somehow get my local phabricator instance accessible out there. happy to test end to end if there's an easier way...!

Reviewers: vrana, epriestley

Reviewed By: vrana

CC: aran, Koolvin

Differential Revision: https://secure.phabricator.com/D2490

Details

Provenance
Bob TrahanAuthored on May 17 2012, 2:55 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP60696de09511: Fix default limit in elasticsearch
Branches
Unknown
Tags
Unknown

Event Timeline