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

Bug 11426 #1070

Closed
wants to merge 2 commits into from
Closed

Bug 11426 #1070

wants to merge 2 commits into from

Conversation

mmoqui
Copy link
Member

@mmoqui mmoqui commented Mar 31, 2020

  • Social Networking has been lightly refactored
  • Fix the bug by adding a new property in chat.properties settings file: chat.xmpp.domain.default. When set, this property gives the XMPP domain to use for all user domains in Silverpeas that aren't mapped to a given XMPP domain. The mappings are now explicitly set through the properties in the form of chat.xmp.domain..

Don't forget to merge after PR Silverpeas/Silverpeas-Components#693

Refactor the SocialNetwork code to lower its complexity and to remove the
use of the old and deprecated Silverpeas Date objects.
Resolve the issue by taking into account only the user domains
in Silverpeas that have a mapping to an XMPP domain defined in the
chat.properties settings file. The rendering of silverchat and the
connection opening with the remote XMPP server is only performed now if the
chat service is enabled for the current user (that is if the both the chat
service is enabled and the domain the user belongs to has a mapping to an XMPP
domain).

A default XMPP domain mapping can be now explictly set with the new property in
the chat.properties settings: chat.xmpp.domain.default. If this property is
set, any non explicitly mapped user domains in Silverpeas will be mapped to this
default XMPP domain. If not set, then the users in the non-mapped domains in
Silverpeas won't have the chat enabled for them.

Add unit tests on the user account registration in the chat server.
@sonarcloud
Copy link

sonarcloud bot commented Apr 1, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 2 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@SilverYoCha SilverYoCha self-assigned this Apr 21, 2020
@SilverYoCha
Copy link
Member

Merged.
Closed manually because of manual git operations performed during integration.

@mmoqui mmoqui deleted the bug-11426 branch May 28, 2020 07:32
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.

None yet

2 participants