Skip to content
Succès

Changes

Summary

  1. Bug #14471 (details)
  2. Bug #14478 (details)
  3. Bug #14503 (details)
  4. bug #14334: add parameter action on method call addArrrayCellCheckbox (details)
  5. bug #14470: the id of the space is not displayed when several looks are availabled (details)
Commit cb4e4b0fab7f6fa131857a6396ec980ddbedf48a by Miguel Moquillon
Bug #14471

A publication is a peculiar contribution in Silverpeas as it can be
located in several locations in the same or in different component
instances. In this case, we have the original publication along with its
all aliases. This capability is used in the Kmelia application. As
consequence, the publication is only referred uniquely by its local
identifier (its local identifier is its global one), whereas the others
types of contributions are identified uniquely by their global
identifier that is made up of their local identifier and of the
identifier of the application instance they belong to.

Despite this peculiarity, the indexation, the search and the
access rights computation treat the publications only by their local
identifier, and this can cause troubles when the aliases of a
publication (with the original one) are implied. For example, if an
alias was removed, then the index in which the alias is referred is
simply deleted, despite the fact there is others aliases or, worse,
there is the original publication referred by the index.

In the fix, we ensure now that all the treatments are done on both the
local identifier of the publication and the identifier of the
application instance it belongs to. And, in the management of the
indexes of the publications, they are deleted only if there is no
more publications (original one/aliases) referred by them.
And, in the access rights computation, in the case of a search,
we look for the access rights of nodes with specific rights in which
is linked the in-checking alias.
The file was modifiedcore-library/src/integration-test/java/org/silverpeas/core/contribution/publication/service/PublicationDAOIT.java (diff)
The file was modifiedcore-library/src/test/java/org/silverpeas/core/security/authorization/PublicationDetail4Test.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/contribution/publication/model/PublicationPK.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/index/search/SimpleSearchEngine.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/contribution/publication/model/PublicationIdentifier.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/contribution/publication/service/DefaultPublicationService.java (diff)
The file was modifiedcore-api/src/main/java/org/silverpeas/core/ResourceReference.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/contribution/publication/model/PublicationDetail.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/contribution/publication/dao/PublicationDAO.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/security/authorization/PublicationAccessController.java (diff)
Commit 5217b3dba611e3c3fac876c4361a175d60bb660d by Miguel Moquillon
Bug #14478

In several very peculiar locations in the code, both in JS and in Java,
the filename, description or title of an attachment (or of a publication),
aren't correctly escaped for HTML rendering. Fix those omissions.
The file was modifiedcore-web/src/main/java/org/silverpeas/core/web/attachment/tag/SimpleDocumentContextualMenu.java (diff)
The file was modifiedcore-war/src/main/webapp/portlets/jsp/myLastPubliRead/portlet.jsp (diff)
The file was modifiedcore-web/src/main/java/org/silverpeas/core/webapi/attachment/SimpleDocumentResource.java (diff)
The file was modifiedcore-war/src/main/webapp/util/javaScript/silverpeas-fileUpload.js (diff)
The file was modifiedcore-war/src/main/webapp/WEB-INF/tags/silverpeas/util/commonDragAndDrop.tag (diff)
The file was modifiedcore-war/src/main/webapp/socialNetwork/jsp/js/newsfeed.js (diff)
The file was modifiedcore-war/src/main/java/org/silverpeas/web/notificationuser/servlets/UserNotificationRequestRouter.java (diff)
The file was modifiedcore-war/src/main/webapp/portlets/jsp/lastPublications/portlet.jsp (diff)
The file was modifiedcore-war/src/main/webapp/util/javaScript/silverpeas.js (diff)
The file was modifiedcore-web/src/main/java/org/silverpeas/core/webapi/attachment/SimpleDocumentResourceCreator.java (diff)
The file was modifiedcore-war/src/main/webapp/attachment/jsp/displayAttachedFiles.jsp (diff)
The file was modifiedcore-war/src/main/webapp/WEB-INF/tags/silverpeas/util/attachmentDragAndDrop.tag (diff)
The file was modifiedcore-war/src/main/webapp/util/javaScript/silverpeas-ddUpload.js (diff)
Commit 6b57cb3c63b11c16b97a43e2e06f41c06f83d8d1 by Miguel Moquillon
Bug #14503

The move of a node was done by deleting and then recreating in database
the node with its new location, and this without updating its
translations to refer the correct new node.

Replace the deletion and then the creation of the moved node by a simple
suitable update: in fact, a move method already existed in the NodeDAO
but it wasn't anymore used. This method update accordingly a node
details in the database for a node move. The translations of the node
are then not lost.

Refine the update of a node in DefaultNodeService by avoiding to
retrieve again from the database the current node before update; Get it
at first, then use it all along the update process.
The file was modifiedcore-library/src/main/java/org/silverpeas/core/node/dao/NodeDAO.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/node/service/DefaultNodeService.java (diff)
The file was modifiedcore-library/src/main/java/org/silverpeas/core/node/model/NodeDetail.java (diff)
Commit 643923de5b643a0211d9910c02a4d47b87939a1d by Miguel Moquillon
bug #14334: add parameter action on method call addArrrayCellCheckbox
The file was modifiedcore-war/src/main/webapp/workflowDesigner/jsp/editQualifiedUsers.jsp (diff)
The file was modifiedcore-war/src/main/webapp/workflowDesigner/jsp/editItem.jsp (diff)
The file was modifiedcore-war/src/main/webapp/workflowDesigner/jsp/editInput.jsp (diff)
The file was modifiedcore-war/src/main/webapp/workflowDesigner/jsp/editConsequence.jsp (diff)
The file was modifiedcore-war/src/main/webapp/workflowDesigner/jsp/editColumns.jsp (diff)
The file was modifiedcore-war/src/main/webapp/workflowDesigner/jsp/editState.jsp (diff)
Commit 29f7701844350394d6481e72b9e28fc7614a0b79 by Miguel Moquillon
bug #14470: the id of the space is not displayed when several looks are availabled
The file was modifiedcore-war/src/main/webapp/jobStartPagePeas/jsp/startPageInfo.jsp (diff)

Summary

  1. Bug #14471 (details)
  2. Bug #14478 (details)
  3. Bug #14503 (details)
  4. bug #14483: Wrong label (details)
Commit 365fe050c8e88e813d7658049f672ad45d931650 by Miguel Moquillon
Bug #14471

A publication is a pueculiar contribution in Silverpeas as iti can be
located in several locations in the same or in different component
instances. In this case, we have the original publication along with
its all aliases. This capability is used in the Kmelia application.
As consequence, the publication is only referred uniquely by its
local identifier (its local identifier is its global one), whereas
the others types of contributions are identified uniquely by their
global identifier that is made up of their local identifier and of
the identifier of the application instance they belong to.

Despite this peculiarity, the authorization mechanism in Kmelia takes
the publication by their local identifier and, in a such context, any
access right validation can be incorrect on publication aliases
located in a topic for which specific access rights have been set.

Now, the publications, whatever they are (original one or aliases) are
taken in charge by both their local identifier and the identifier of the
component instance they belong to.
The file was modifiedkmelia/kmelia-library/src/main/java/org/silverpeas/components/kmelia/KmeliaComponentAuthorization.java (diff)
Commit 9ca1c3dce1c250c4b67763a09b44060e6696431a by Miguel Moquillon
Bug #14478

In several peculiar locations, the filename, title or description of an
attachment aren't correctly escaped for HTML rendering. Fix those
issues.

Refactor the rendering process in AjaxPublicationListServlet in order
first to separate in a way some responsibilities, and to fix the issues
sent back by Sonar.
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/importOneFile.jsp (diff)
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/RenderingContext.java
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/ForbiddenAccessRenderer.java
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/defermentMotive.jsp (diff)
The file was removedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/PublicationFragmentSettings.java
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/LastPublicationsRenderer.java
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/importMultiFiles.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/publicationManager.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/publication.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/kmax_axisReport.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/topicReport.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/publicationsList.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/AjaxPublicationsListServlet.java (diff)
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/NothingRenderer.java
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/Renderer.java
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/PublicationsRenderer.java
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/checkKmelia.jsp (diff)
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/styleSheets/kmelia.css (diff)
The file was addedkmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/servlets/renderers/PublicationFragmentSettings.java
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/orderPublications.jsp (diff)
Commit efd66d5813aef8ff834029bfb5ce14fc636488e6 by Miguel Moquillon
Bug #14503

By testing the fix of the bug, I discovered a nasty hidden bug in js
when adding a new translation for a given topic: when the properties of a
topic is rendered, an HTML select element is updated with the
translations of the node's properties. If we ask to see the properties
of a topic for which its name and description were translated for all of
the supported languages, the HTML select element is correctly updated.
But, if after we ask to see the properties of another topic for which a
translation is missed, then the HTML select element isn't correctly
updated for the missed translation: instead of having for the missing
translation an identifier of -1, it kept the id of the translation of
the previous selected topic for the same language.

Fix this bug.
The file was modifiedkmelia/kmelia-war/src/main/webapp/kmelia/jsp/javaScript/navigation.js (diff)
The file was modifiedkmelia/kmelia-configuration/src/main/config/xmlcomponents/kmelia.xml (diff)
The file was modifiedkmelia/kmelia-configuration/src/main/config/xmlcomponents/toolbox.xml (diff)

Summary

  1. Bug #14478 (details)
  2. Fix  Bug #14238 (details)
Commit bd8ea645bb70f16fcb8394b13dcdb469e92343ee by Miguel Moquillon
Bug #14478

Escape for HTML rendering the title and the description of a publication
in the block of the last authored publications.
The file was modifiedaurora/aurora-war/src/main/webapp/WEB-INF/tags/silverpeas/look/displayPublications.tag (diff)
Commit 1b5959dc0d02d280e80ea82fb032e9171831da09 by Miguel Moquillon
Fix  Bug #14238

add missing parameters on this both skin
The file was modifiedaurora/aurora-configuration/src/main/config/properties/org/silverpeas/util/viewGenerator/settings/prima.properties (diff)
The file was modifiedaurora/aurora-configuration/src/main/config/properties/org/silverpeas/util/viewGenerator/settings/waves.properties (diff)

Summary

  1. Remove the WebSite Designer application from Silverpeas. WebSite (details)
Commit 1440d7740962c63078022e654d906f65764c9fef by Miguel Moquillon
Remove the WebSite Designer application from Silverpeas. WebSite
Designer is an old application that hasn't involved for a long time and
that isn't more used by any of ours customers and users. By doing so,
fix the vulnerability #14511.
The file was modifiedpom.xml (diff)