Insteon: Clear No_Hop_Flag Even If No Debug; Don't Call Setby if Undef

Fixes errors pointed out by CDragon and MStovenour
Merge pull request #171 from krkeegan/fix_issue_83

Insteon: Fix Error in PLM Message Integrity Check Logic
Insteon: Catch and Log AllLink_Cleanup_Reports

Fix bug in which MH set the state of a device to 'link_cleanup_report.'
Log the results of the cleanup reports.

Fixes Issue hollie/misterhouse#168
Insteon: Catch Error if Device ID in All_Link_Failure Does Not Exist

Checks to make sure an object exists, if no object exists for that device_id then it sends a message to the log reminding the user to delete_orphan links.

Fixes mstovenour/misterhouse#17
Insteon: Catch Multiple Calls to Set_Receive in the Same Pass

Resolves one of the three loopholes discovered by @peloy which permitted duplicate messages received in the same pass to be processed.

Hopefully the final fix for hollie/misterhouse#169
Insteon: Catch Cleanup Messages Sent in the Same Pass as AllLink Messages

Issue discovered by @peloy.  MH should be ignoring the cleanup message received after an AllLink message.  However, both messages were previously processed if they were received in the same pass.  This resolves that discrepancy by checking what the state of the object will be at the start of the next pass.

Partially fixes hollie/misterhouse#185
Insteon: Log Output if PLM Parser Drops a Duplicate Message

As discovered by @peloy, the PLM Parser was silently dropping duplicate messages sequentially received in the same pass.

This adds a debug3 message to the log when such messages are dropped.

Partial fix for hollie/misterhouse#186
Insteon: Relocate PLM Decoder to Allow Decoding Multiple Messages Received at the Same Time

PLM Decoder will not decode each message if multiple are received in one check data request.

The only place that could create a hiccup, is if the PLM Parser has a bug.  Since we now rely on the PLM Parser to first identify a valid message before asking the decoder to decode it, we rely on the PLM Parser to accurately discover messages.

We still display the raw message, so the information is not lost, but we will not have a nice display of the message.  I don't see this as a big deal, if the PLM Parser is missing messages we would likely have other problems anyways.

Fixes the remainder of hollie/misterhouse#186
Return a web page instead of HTML code when visiting Phone
Calls/VoiceMail Msgs -> Phone List (starting at the top IA5 page).
Insteon: Move Failure_Reason into BaseObject from Message

The active_message must be cleared before calling the failure_callback, otherwise we get into a strange loop with sync and scan links
Once the active_message is cleared, we lose our ability to keep track of the failured message and corresponding failure_reason

To resolve this, moved failure_reason from the message object into BaseObject.
Merge pull request #166 from krkeegan/hop_tracking

Insteon: Clear No_Hop_Flag Even If No Debug; Don't Call Setby if Undef
Merge pull request #173 from krkeegan/fix_issue_168

Insteon: Catch and Log AllLink_Cleanup_Reports
Merge pull request #180 from krkeegan/fix_issue_83

Insteon: Fix Bug in PLM Sanity Check Which Caused Crash on X10 Messages
Merge pull request #189 from krkeegan/fix_issue_17

Insteon: Catch Error if Device ID in All_Link_Failure Does Not Exist
Merge pull request #190 from mstovenour/merge_i2_aldb_support

Merge i2 aldb support
Merge pull request #191 from krkeegan/fix_issue_177

Insteon: Update PLM Scene State on Change
Merge pull request #192 from krkeegan/fix_issue_185

Insteon: Catch Cleanup Messages Sent in the Same Pass as AllLink Message...
Merge pull request #194 from krkeegan/fix_issue_175

Insteon: Move Failure_Reason into BaseObject from Message
Merge branch 'master' into fix_issue_186

Conflicts:
	lib/Insteon_PLM.pm
Merge pull request #193 from krkeegan/fix_issue_186

Insteon: Log Output if PLM Parser Drops a Duplicate Messages