Insteon: Prevent Crash When PLM Busy but no Active Message

The PLM may report a busy code 15, even when there is no active_message.  Added logic to test for an active message before trying to resend it.

Partial fix for issue #154
Insteon: Prevent Crashing if AllLink_Failure_Report Receive and No Active Message

I am not clear how this could be occuring, but it was reported as a crash.  Added simple logic to test for an active_message before trying to call the active message.

Fixes Issue #154
X10/Insteon: Fix error in parsing of x10 Receive Messages in Insteon_PLM

Patch submitted by Chris Dragon

Fixes issue #156
Insteon_Irrigation: Move Old Irrigation Code to Insteon Lib
Insteon_Irrigation: Update Perldoc
Insteon_Irrigation: Update Initialization Settings
Insteon_Irrigation: Update new Subroutine
Insteon_Irrigation:Convert all calls to _send_cmd to use Message Class
Insteon_Irrigation: Update Print_Log to match class name
Insteon_Irrigation: Add Irrigation Message Types
Insteon_Irrigation: Update Read Table A Definition
Insteon_Irrigation: Fix Message Creation

Need to pass object not device
Insteon_Irrigation: Add call to SUPER is_info_request

Don't override the whole function, just supplement it.
Insteon_Irrigation: Fix Typo in Read Table A

Require of class name doesn't require quotes
Insteon_Irrigation: Set Message_Types in Self Hash
Insteon_PLM: Fix Error in Message Parser Dumping Buffer on Busy

Solves issue #158 identified by Chris Dragon

This will trim the leading "15" bytes from the data.  If anything is left it will be placed into the message fragment to be read next time.
Insteon: Set Device State on a Received AllLink Broadcast Message

Fixes issue #153

The set routine is now called when a broadcase message is received.  The flag _pending_cleanup is also set.

When a Direct Cleanup Message is received, if the state of the device matches the pending state and the _pending_cleanup flag is set, we assume that this command was already processed.

This code is all adjacent to each other, so using a flag should not be confusing.

Much of the code was already there, but was commented out by Gregg in his first commit of the new insteon code.  This initial commit included a huge number of changes and no notes were added as to why this was disabled in the redux.
Insteon: Flip Logic on Received AllLink Broadcast Message

Had the logic backwards, dont set if state the same and pending_cleanup is set.
Merge pull request #162 from krkeegan/fix_issue_153

Insteon: Set Device State on a Received AllLink Broadcast Message
Merge pull request #159 from krkeegan/fix_issue_158

Fix issue 158