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

some bug fix. #71

Merged

Conversation

fjcondy
Copy link
Contributor

@fjcondy fjcondy commented Oct 30, 2024

1.FormManager,java load workgroupForm: xstream have Type limit.must set allowTypes when xstream update 1.4.20
2.Request.java: when chatbot.enabled =false default. chatBot is null so throw NLP exception.
3.WorkgroupUtils.java when set custom dispatchers must in the first of dispatcher List. because RequestQueue.java gets first dispather to the default dispather class in dispatcher List.

…et allowTypes when xstream update 1.4.20

2.Request.java: when chatbot.enabled =false default. chatBot is null so throw NLP exception.
3.WorkgroupUtils.java when set custom dispatchers must in the first of dispatcher List. because RequestQueue.java get first dispather to default dispather class  in dispatcher List.
@guusdk
Copy link
Member

guusdk commented Oct 30, 2024

Thank you!

I believe that this fixes the following issues:

Can you please modify your change to include this in the changelog.html file?

@fjcondy
Copy link
Contributor Author

fjcondy commented Oct 31, 2024

already add fix #59 #72 #73 in changelog.html

@guusdk guusdk merged commit ae71812 into igniterealtime:main Oct 31, 2024
3 checks passed
@guusdk
Copy link
Member

guusdk commented Oct 31, 2024

Thanks for your fixes! You can now download the latest SNAPSHOT release from https://www.igniterealtime.org/projects/openfire/plugin-archive.jsp?plugin=fastpath to test your changes!

@fjcondy fjcondy deleted the getAvailableDispatcherClasses_xstream_check branch November 1, 2024 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants