Skip to content
Sign in
Succès

Changes

Summary

  1. Bug #14872 (details)
  2. Bug #14807 (details)
  3. Update silverpeas-main.css to allow all the comments to go up in order (details)
Commit 7edcc0ff9e6bca89445daf6b49087ed0d2085b46 by Miguel Moquillon
Bug #14872

In InvitationResource, fix validation in checkInvitation method: the
requester must be either the sender or the receiver.
The file was modifiedcore-web/src/main/java/org/silverpeas/core/webapi/socialnetwork/invitation/InvitationResource.java (diff)
Commit a7f50e3595d16733aa7476969c15971344d494f5 by Miguel Moquillon
Bug #14807

In the commenting mechanism, when the content of the message and the
additional message are the same, the additional message is dismissed in
the sent notification. But, in the case of the notification about a
comment, the additional message is the comment as is whereas the content
of the message is the comment escaped for its printing in HTML! So, the
two becomes different.
In the case of a comment, there is no need to have both the additional
message and the content of the notification message as they are
identical. The fix here is to remove the extra message setting in the
case of a notification about a comment.
The file was modifiedcore-configuration/src/main/config/resources/StringTemplates/core/comment/commented_fr.st (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/notification/user/builder/AbstractTemplateUserNotificationBuilder.java (diff)
The file was modifiedcore-services/comment/src/main/java/org/silverpeas/core/comment/service/CommentUserNotification.java (diff)
Commit 5ffdf36cee17db375d72bd46c5c2ef9b484a294b by Miguel Moquillon
Update silverpeas-main.css to allow all the comments to go up in order
to cover the hole created by the deletion of a comment.
The file was modifiedcore-war/src/main/webapp/util/styleSheets/silverpeas-main.css (diff)

Summary

  1. bug #14898 : Do not add the web context in the link of this notificationBuilder, because he is added after this step. (details)
  2. The abstract user notification builders in Kmelia have now their (details)
Commit a51dda1d068504604cb7ee230ddebe4b8ee6f394 by Miguel Moquillon
bug #14898 : Do not add the web context in the link of this notificationBuilder, because he is added after this step.
The file was modifiedcommunity/community-library/src/main/java/org/silverpeas/components/community/notification/user/MembershipRequestUserNotificationBuilder.java (diff)
The file was modifiedcommunity/community-library/src/main/java/org/silverpeas/components/community/notification/user/AbstractCommunityUserNotificationBuilder.java (diff)
Commit 04c4ed9472af8eca158e4692563d08df0fde71f5 by Miguel Moquillon
The abstract user notification builders in Kmelia have now their
constructor protected as they are used only for the concrete
implementation.
The file was modifiedkmelia/kmelia-library/src/main/java/org/silverpeas/components/kmelia/notification/AbstractKmeliaActionPublicationUserNotification.java (diff)
The file was modifiedkmelia/kmelia-library/src/main/java/org/silverpeas/components/kmelia/notification/AbstractKmeliaUserNotification.java (diff)