Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Insteon: ALDB_PLM sub add_link improperly sets message callback #90

Closed
krkeegan opened this issue Feb 21, 2013 · 0 comments
Closed

Insteon: ALDB_PLM sub add_link improperly sets message callback #90

krkeegan opened this issue Feb 21, 2013 · 0 comments
Assignees
@krkeegan
Labels
bug

Comments

@krkeegan
Copy link
Collaborator

Similar to Issue #61 which was solved in pull request #63.

The sub add_link in the aldb_plm object sets the $message->callback to _process_sync_queue. This results in _process_sync_queue getting called twice, once when the message is sent, and then again when the ACK is processed. This causes sync_links to process out of order.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@krkeegan krkeegan

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@krkeegan