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 94cbefe83121c298a42bb43dc3761b13f2dd84ac 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 308b220953b9dba5d97599d0072143375751b799 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-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)
The file was modifiedcore-configuration/src/main/config/resources/StringTemplates/core/comment/commented_fr.st (diff)
Commit d01724cc2c13109a93e629aa28de3c37b8ca4988 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 e5baf5fa3182f8a5721d4e7df6cd4bcc4e2e756b 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 46e34630a40dfbbe6305eb8700fae0efe162780d 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)