diff --git a/notes-service/src/main/java/org/exoplatform/wiki/service/impl/NoteServiceImpl.java b/notes-service/src/main/java/org/exoplatform/wiki/service/impl/NoteServiceImpl.java index 90fc3cb428..2194f21104 100644 --- a/notes-service/src/main/java/org/exoplatform/wiki/service/impl/NoteServiceImpl.java +++ b/notes-service/src/main/java/org/exoplatform/wiki/service/impl/NoteServiceImpl.java @@ -1846,8 +1846,7 @@ private void checkToRemoveDomainInUrl(Page note) { private boolean canImportNotes(String authenticatedUser, Space space, Page page) throws WikiException { if (space != null) { - return (spaceService.isSuperManager(authenticatedUser) || spaceService.isManager(space, authenticatedUser) - || spaceService.isRedactor(space, authenticatedUser)); + return (spaceService.canRedactOnSpace(space, authenticatedUser)); } else if (StringUtils.equals(page.getOwner(), IdentityConstants.SYSTEM)) { return cmsService.hasAccessPermission(Utils.getIdentity(authenticatedUser), NotePageViewService.CMS_CONTENT_TYPE, page.getName()); } else { diff --git a/notes-service/src/main/java/org/exoplatform/wiki/utils/Utils.java b/notes-service/src/main/java/org/exoplatform/wiki/utils/Utils.java index 9c8e87b64e..fcd03dea79 100644 --- a/notes-service/src/main/java/org/exoplatform/wiki/utils/Utils.java +++ b/notes-service/src/main/java/org/exoplatform/wiki/utils/Utils.java @@ -739,10 +739,7 @@ public static boolean canManageNotes(String authenticatedUser, Space space, Page SpaceService spaceService = CommonsUtils.getService(SpaceService.class); CMSService cmsService = CommonsUtils.getService(CMSService.class); if (space != null) { - return (spaceService.isSuperManager(authenticatedUser) - || spaceService.isManager(space, authenticatedUser) - || spaceService.isRedactor(space, authenticatedUser) - || spaceService.isMember(space, authenticatedUser) && ArrayUtils.isEmpty(space.getRedactors())); + return spaceService.canRedactOnSpace(space, authenticatedUser); } else if (StringUtils.equals(page.getOwner(), IdentityConstants.SYSTEM)) { return cmsService.hasEditPermission(getIdentity(authenticatedUser), NotePageViewService.CMS_CONTENT_TYPE, page.getName()); } else { diff --git a/notes-service/src/test/java/org/exoplatform/wiki/mock/MockSpaceService.java b/notes-service/src/test/java/org/exoplatform/wiki/mock/MockSpaceService.java index 0a81b8896d..0eff30b070 100644 --- a/notes-service/src/test/java/org/exoplatform/wiki/mock/MockSpaceService.java +++ b/notes-service/src/test/java/org/exoplatform/wiki/mock/MockSpaceService.java @@ -100,11 +100,11 @@ public Space updateSpace(Space existingSpace) { return null; } - public Space updateSpaceAvatar(Space existingSpace) { + public Space updateSpaceAvatar(Space existingSpace, String username) { return null; } - public Space updateSpaceBanner(Space existingSpace) { + public Space updateSpaceBanner(Space existingSpace, String username) { return null; } @@ -248,7 +248,7 @@ public void renameSpace(Space space, String newDisplayName) throws SpaceExceptio } - public void renameSpace(String remoteId, Space space, String newDisplayName) throws SpaceException { + public void renameSpace(Space space, String newDisplayName, String remoteId) throws SpaceException { } @@ -468,14 +468,6 @@ public void unregisterSpaceLifeCycleListener(SpaceLifeCycleListener listener) { } - public void setPortletsPrefsRequired(PortletPreferenceRequiredPlugin portletPrefsRequiredPlugin) { - - } - - public String[] getPortletsPrefsRequired() { - return null; - } - public ListAccess getVisitedSpaces(String remoteId, String appId) { return null; }