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.

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

properly handle subscribed topics with + wildcard #3

Merged
merged 2 commits into from
Oct 11, 2019
Merged

properly handle subscribed topics with + wildcard #3

merged 2 commits into from
Oct 11, 2019

Conversation

cjaikaeo
Copy link
Contributor

@cjaikaeo cjaikaeo commented Oct 8, 2019

Currently subscribe panels can handle wildcard topics containing # properly, but not topics containing +. This pull request contains a fix that allows a subscribe panel with + as part of the topic name to correctly receive incoming messages by employing a topic matching function available in Paho's MQTT implementation.

@adrianromero adrianromero self-assigned this Oct 8, 2019
Copy link
Owner

@adrianromero adrianromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix. I have to keep Java 8 sources compatibility due to the mobile (Android) version. Please replace 'var' by 'Entry<String, List>' before approving the pull request.

@cjaikaeo
Copy link
Contributor Author

cjaikaeo commented Oct 9, 2019

I already replaced var with Map.Entry<String, List<TopicSubscription>>. However, gradle complained about org.openjfx.javafxplugin class not compatible with Java 8, so I couldn't verify the code against Java 8. (The code compiled with Java 13 though.)

@adrianromero adrianromero merged commit ed4a683 into adrianromero:master Oct 11, 2019
@adrianromero
Copy link
Owner

Merged!

As you can see it is needed to merge your changes into the branch mobile to make the gradle script work for Android and Java 8. I have two different gradle build files. One for desktop and java 11 in the master branch and other for android and java 8 in the branch mobile. Maybe I should create one single gradle build file for both environments.

Thanks

@cjaikaeo
Copy link
Contributor Author

Thank you so much for accepting the PR, and for creating such wonderful work.

@cjaikaeo cjaikaeo deleted the bug/wildcard branch October 12, 2019 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants