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

Lpd 41036 DRAFT #194

Draft
wants to merge 51 commits into
base: master
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
51 commits
Select commit Hold shift + click to select a range
d7c134d
LPD-40091 Allow out of date publications to be enabled or disabled
brookedalton Nov 7, 2024
0c6f98c
LPD-40091 Configuration supersedes check
brookedalton Nov 11, 2024
a254f2e
LPD-40091 Add and adjust integration test since default behavior changed
brookedalton Nov 7, 2024
b5d3c65
LPD-40091 The schema version is the now latest
brookedalton Nov 8, 2024
face16b
LPD-40091 Don't expire the ctCollection if it is the latest version
brookedalton Nov 7, 2024
b26dc16
LPD-40091 buildLang
brookedalton Nov 11, 2024
d4bf98e
LPD-40091 ant update-portal-osgi-configuration-properties
brookedalton Nov 11, 2024
a94a2e0
LPD-40091 Remove poshi tests
brookedalton Nov 12, 2024
673e139
LPD-40091 Wordsmith
brianchandotcom Nov 13, 2024
b72df8f
LPD-40091 Regen
brianchandotcom Nov 13, 2024
c8d90ca
LPD-40091 SF
brianchandotcom Nov 13, 2024
60ce854
LPD-40091 Regen
brianchandotcom Nov 13, 2024
469fc77
LPD-40091 Prep next
brianchandotcom Nov 13, 2024
134236f
LPD-40091 Regen
brianchandotcom Nov 13, 2024
af23e56
LPD-41985 Do these when we run SF for current branch
ling-alan-huang Nov 13, 2024
8b11a10
LPD-41260 Replace usages of onclick inline event in <liferay-ui:quick…
izaera Oct 7, 2024
5c299f6
LPD-41985 prep next
brianchandotcom Nov 13, 2024
d8e6105
LPD-41985 prep next
brianchandotcom Nov 13, 2024
e9c7e5a
LPD-41548 Adds fetchLayoutUtilityPageEntryByExternalReferenceCode met…
ealonso Nov 12, 2024
ffb459b
LPD-41548 Adds a DTO Converter for UtilityPage
ealonso Nov 12, 2024
1b37357
LPD-41548 Adds Get UtilityPages endpoint
ealonso Nov 12, 2024
49717f6
LPD-41548 Adds POST UtilityPage endpoint
ealonso Nov 12, 2024
0a940ef
LPD-41548 Adds GET, DELETE and PUT endpoints for a specific UtilityPage
ealonso Nov 12, 2024
dcbffc8
LPD-41548 Ignore not implemented tests
ealonso Nov 12, 2024
374328b
LPD-41548 Implements tests
ealonso Nov 12, 2024
74c2e84
LPD-41548 Auto
ealonso Nov 12, 2024
dc8a3df
LPD-41548 SF
brianchandotcom Nov 13, 2024
39c0014
LPD-41548 I only see this one pattern for now
brianchandotcom Nov 13, 2024
ad262a4
LPD-41769 Apply hidden overflow to body only in desktop
sandrodw3 Nov 12, 2024
1cc0340
LPD-41769 Don't add extra pixel
sandrodw3 Nov 12, 2024
56d5f5e
LPD-41769 Add hidden overflow to wrapper in non-desktop viewports
georgel-pop-lr Nov 12, 2024
f32fa8d
LPD-41769 Add playwright test 'Checks that the layout has scroll when…
georgel-pop-lr Nov 12, 2024
5fdd886
LPD-25204 integration test
jgarcialr Oct 1, 2024
06fbfb8
LPD-25204 use testIntegrationImplementation
jgarcialr Oct 1, 2024
f8fd80f
LPD-25204 Relates to a previous approach, no longer needed
stian-sigvartsen Nov 8, 2024
1b6d154
LPD-25204 SF where there is a 2 there is a 1
stian-sigvartsen Nov 8, 2024
b975590
LPD-25204 I don't think we went to pass the test upon an exception?
stian-sigvartsen Nov 8, 2024
f180daf
LPD-25204 Follow expected module structure. See log*n-web-test
stian-sigvartsen Nov 8, 2024
e9b96dd
LPD-25204 SF randomString
jgarcialr Nov 12, 2024
b3151f9
LPD-25204 SF
brianchandotcom Nov 13, 2024
2aac033
LPD-25204 SF
brianchandotcom Nov 13, 2024
ca41f43
LPD-25204 Does not need to be static
brianchandotcom Nov 13, 2024
ef69f87
LPD-25204 SF
brianchandotcom Nov 13, 2024
9788a95
LPD-25204 SF
brianchandotcom Nov 13, 2024
da5a49d
LPD-25204 SF
brianchandotcom Nov 13, 2024
699654a
LPD-41701 Fix poshi test
boton Nov 13, 2024
62aeef4
LPD-41935 correcting broken url
rafaellajordao Nov 12, 2024
6cc313f
LPD-25204 SF
brianchandotcom Nov 13, 2024
97fb36d
LPD-41036 - check against current companyId
wanderlast Nov 13, 2024
277d11e
LPD-41036 - this looks wrong?
wanderlast Nov 13, 2024
9bbf0e7
LPD-41036 - add test
wanderlast Nov 18, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
artifact.git.id=1382281d80babcd31d90abbb3f080392b1ca2fd3
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.defaults/10.1.203/com.liferay.gradle.plugins.defaults-10.1.203-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.defaults/10.1.203/com.liferay.gradle.plugins.defaults-10.1.203-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.defaults/10.1.203/com.liferay.gradle.plugins.defaults-10.1.203.jar
artifact.git.id=5c299f67868a1350af6d7a8f90aa85be086fbc99
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.defaults/10.1.204/com.liferay.gradle.plugins.defaults-10.1.204-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.defaults/10.1.204/com.liferay.gradle.plugins.defaults-10.1.204-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.defaults/10.1.204/com.liferay.gradle.plugins.defaults-10.1.204.jar
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
artifact.git.id=3484725f9c38a4d2e7391eac15959d973bab2a51
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.source.formatter/5.2.173/com.liferay.gradle.plugins.source.formatter-5.2.173-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.source.formatter/5.2.173/com.liferay.gradle.plugins.source.formatter-5.2.173-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.source.formatter/5.2.173/com.liferay.gradle.plugins.source.formatter-5.2.173.jar
artifact.git.id=8b11a10ecdd2e98dc4c9f685eddbbc0b0cd439b6
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.source.formatter/5.2.174/com.liferay.gradle.plugins.source.formatter-5.2.174-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.source.formatter/5.2.174/com.liferay.gradle.plugins.source.formatter-5.2.174-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.source.formatter/5.2.174/com.liferay.gradle.plugins.source.formatter-5.2.174.jar
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
artifact.git.id=1382281d80babcd31d90abbb3f080392b1ca2fd3
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.workspace/12.1.3/com.liferay.gradle.plugins.workspace-12.1.3-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.workspace/12.1.3/com.liferay.gradle.plugins.workspace-12.1.3-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.workspace/12.1.3/com.liferay.gradle.plugins.workspace-12.1.3.jar
artifact.git.id=60ce8540b01cb71380b678d5f3acd019face8621
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.workspace/12.1.4/com.liferay.gradle.plugins.workspace-12.1.4-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.workspace/12.1.4/com.liferay.gradle.plugins.workspace-12.1.4-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins.workspace/12.1.4/com.liferay.gradle.plugins.workspace-12.1.4.jar
8 changes: 4 additions & 4 deletions modules/.releng/sdk/gradle-plugins/artifact.properties
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
artifact.git.id=1382281d80babcd31d90abbb3f080392b1ca2fd3
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins/15.0.114/com.liferay.gradle.plugins-15.0.114-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins/15.0.114/com.liferay.gradle.plugins-15.0.114-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins/15.0.114/com.liferay.gradle.plugins-15.0.114.jar
artifact.git.id=5c299f67868a1350af6d7a8f90aa85be086fbc99
artifact.javadoc.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins/15.0.115/com.liferay.gradle.plugins-15.0.115-javadoc.jar
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins/15.0.115/com.liferay.gradle.plugins-15.0.115-sources.jar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.gradle.plugins/15.0.115/com.liferay.gradle.plugins-15.0.115.jar
10 changes: 5 additions & 5 deletions modules/.releng/util/source-formatter/artifact.properties
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
artifact.git.id=3484725f9c38a4d2e7391eac15959d973bab2a51
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1456/com.liferay.source.formatter-1.0.1456-sources.jar
artifact.tar.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1456/com.liferay.source.formatter-1.0.1456.tar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1456/com.liferay.source.formatter-1.0.1456.jar
artifact.zip.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1456/com.liferay.source.formatter-1.0.1456.zip
artifact.git.id=8b11a10ecdd2e98dc4c9f685eddbbc0b0cd439b6
artifact.sources.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1457/com.liferay.source.formatter-1.0.1457-sources.jar
artifact.tar.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1457/com.liferay.source.formatter-1.0.1457.tar
artifact.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1457/com.liferay.source.formatter-1.0.1457.jar
artifact.zip.url=https://repository-cdn.liferay.com/nexus/content/repositories/liferay-public-releases/com/liferay/com.liferay.source.formatter/1.0.1457/com.liferay.source.formatter-1.0.1457.zip
Original file line number Diff line number Diff line change
Expand Up @@ -7,30 +7,44 @@

import com.liferay.arquillian.extension.junit.bridge.junit.Arquillian;
import com.liferay.blogs.constants.BlogsPortletKeys;
import com.liferay.exportimport.kernel.configuration.ExportImportConfigurationParameterMapFactoryUtil;
import com.liferay.exportimport.kernel.lar.PortletDataContext;
import com.liferay.exportimport.kernel.lar.PortletDataHandlerKeys;
import com.liferay.exportimport.kernel.lar.StagedModelDataHandler;
import com.liferay.exportimport.kernel.lar.StagedModelDataHandlerRegistryUtil;
import com.liferay.exportimport.kernel.service.StagingLocalServiceUtil;
import com.liferay.exportimport.kernel.staging.StagingUtil;
import com.liferay.exportimport.kernel.staging.constants.StagingConstants;
import com.liferay.exportimport.portlet.preferences.processor.ExportImportPortletPreferencesProcessor;
import com.liferay.exportimport.test.util.ExportImportTestUtil;
import com.liferay.layout.test.util.LayoutTestUtil;
import com.liferay.petra.lang.SafeCloseable;
import com.liferay.petra.string.StringPool;
import com.liferay.portal.kernel.model.Group;
import com.liferay.portal.kernel.model.Layout;
import com.liferay.portal.kernel.model.Organization;
import com.liferay.portal.kernel.model.Portlet;
import com.liferay.portal.kernel.portlet.PortletPreferencesFactoryUtil;
import com.liferay.portal.kernel.service.GroupLocalServiceUtil;
import com.liferay.portal.kernel.service.OrganizationLocalServiceUtil;
import com.liferay.portal.kernel.service.PortletLocalService;
import com.liferay.portal.kernel.service.ServiceContext;
import com.liferay.portal.kernel.service.persistence.GroupUtil;
import com.liferay.portal.kernel.test.rule.AggregateTestRule;
import com.liferay.portal.kernel.test.rule.DeleteAfterTestRun;
import com.liferay.portal.kernel.test.util.GroupTestUtil;
import com.liferay.portal.kernel.test.util.OrganizationTestUtil;
import com.liferay.portal.kernel.test.util.PropsValuesTestUtil;
import com.liferay.portal.kernel.test.util.ServiceContextTestUtil;
import com.liferay.portal.kernel.test.util.TestPropsValues;
import com.liferay.portal.kernel.test.util.UserTestUtil;
import com.liferay.portal.kernel.util.GetterUtil;
import com.liferay.portal.kernel.util.PortalUtil;
import com.liferay.portal.test.rule.Inject;
import com.liferay.portal.test.rule.LiferayIntegrationTestRule;

import java.util.HashMap;
import java.util.Map;

import javax.portlet.PortletPreferences;

Expand Down Expand Up @@ -153,6 +167,78 @@ public void testProcessOrganizationId() throws Exception {
GetterUtil.getLong(importedOrganizationId));
}

@Test
public void testRemoteStagedOrganizationMissingRefValidation()
throws Exception {

Group remoteLiveGroup = GroupTestUtil.addGroup();

try {
_enableRemoteStaging(remoteLiveGroup, _group);

remoteLiveGroup = GroupLocalServiceUtil.getGroup(
remoteLiveGroup.getGroupId());

Map<String, String[]> parameters =
ExportImportConfigurationParameterMapFactoryUtil.
buildFullPublishParameterMap();

StagingUtil.publishLayouts(
TestPropsValues.getUserId(), _group.getGroupId(),
remoteLiveGroup.getGroupId(), false, parameters);

Assert.assertEquals(1, remoteLiveGroup.getPublicLayoutsPageCount());
}
finally {
GroupTestUtil.deleteGroup(remoteLiveGroup);
}
}

private void _enableRemoteStaging(
Group remoteLiveGroup, Group remoteStagingGroup)
throws Exception {

try (SafeCloseable safeCloseable1 =
PropsValuesTestUtil.swapWithSafeCloseable(
"TUNNELING_SERVLET_SHARED_SECRET",
"F0E1D2C3B4A5968778695A4B3C2D1E0F");
SafeCloseable safeCloseable2 =
PropsValuesTestUtil.swapWithSafeCloseable(
"TUNNELING_SERVLET_SHARED_SECRET_HEX", true)) {

ServiceContext serviceContext =
ServiceContextTestUtil.getServiceContext();

serviceContext.setAddGroupPermissions(true);
serviceContext.setAddGuestPermissions(true);
serviceContext.setScopeGroupId(remoteStagingGroup.getGroupId());

_setStagingAttribute(
serviceContext, PortletDataHandlerKeys.PORTLET_DATA_ALL, false);
_setStagingAttribute(
serviceContext, PortletDataHandlerKeys.PORTLET_SETUP_ALL,
false);

UserTestUtil.setUser(TestPropsValues.getUser());

StagingLocalServiceUtil.enableRemoteStaging(
TestPropsValues.getUserId(), remoteStagingGroup, false, false,
"localhost", PortalUtil.getPortalServerPort(false),
PortalUtil.getPathContext(), false,
remoteLiveGroup.getGroupId(), serviceContext);

GroupUtil.clearCache();
}
}

private void _setStagingAttribute(
ServiceContext serviceContext, String key, Object value) {

serviceContext.setAttribute(
StagingConstants.STAGED_PREFIX + key + StringPool.DOUBLE_DASH,
String.valueOf(value));
}

@Inject(filter = "javax.portlet.name=" + BlogsPortletKeys.BLOGS_AGGREGATOR)
private ExportImportPortletPreferencesProcessor
_exportImportPortletPreferencesProcessor;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,4 +29,11 @@ public interface CTConflictConfiguration {
)
public boolean modificationDeletionConflictCheckEnabled();

@Meta.AD(
deflt = "false",
description = "schema-version-check-enabled-description",
name = "schema-version-check-enabled", required = false
)
public boolean schemaVersionCheckEnabled();

}
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,12 @@
public class CTSchemaVersionHelper {

public void expireCTCollection(CTCollection ctCollection) {
if (_ctSchemaVersionLocalService.isLatestCTSchemaVersion(
ctCollection.getSchemaVersionId())) {

return;
}

try {
Map<Long, List<ConflictInfo>> conflictMap =
_ctCollectionLocalService.checkConflicts(ctCollection);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,17 @@

package com.liferay.change.tracking.service.impl;

import com.liferay.change.tracking.configuration.CTConflictConfiguration;
import com.liferay.change.tracking.model.CTSchemaVersion;
import com.liferay.change.tracking.service.base.CTSchemaVersionLocalServiceBaseImpl;
import com.liferay.portal.aop.AopService;
import com.liferay.portal.configuration.module.configuration.ConfigurationProvider;
import com.liferay.portal.kernel.dao.orm.QueryUtil;
import com.liferay.portal.kernel.log.Log;
import com.liferay.portal.kernel.log.LogFactoryUtil;
import com.liferay.portal.kernel.model.Release;
import com.liferay.portal.kernel.module.configuration.ConfigurationException;
import com.liferay.portal.kernel.security.auth.CompanyThreadLocal;
import com.liferay.portal.kernel.service.ReleaseLocalService;
import com.liferay.portal.kernel.version.Version;

Expand All @@ -27,6 +33,7 @@
* @author Preston Crary
*/
@Component(
configurationPid = "com.liferay.change.tracking.configuration.CTConflictConfiguration",
property = "model.class.name=com.liferay.change.tracking.model.CTSchemaVersion",
service = AopService.class
)
Expand Down Expand Up @@ -74,6 +81,24 @@ public CTSchemaVersion getLatestCTSchemaVersion(long companyId) {
public boolean isLatestCTSchemaVersion(
CTSchemaVersion ctSchemaVersion, boolean strict) {

try {
CTConflictConfiguration ctConflictConfiguration =
_configurationProvider.getCompanyConfiguration(
CTConflictConfiguration.class,
CompanyThreadLocal.getCompanyId());

if (!ctConflictConfiguration.schemaVersionCheckEnabled()) {
return true;
}
}
catch (ConfigurationException configurationException) {
_log.error(configurationException);
}

if (ctSchemaVersion == null) {
return false;
}

Map<String, Serializable> schemaContext =
ctSchemaVersion.getSchemaContext();

Expand Down Expand Up @@ -119,13 +144,15 @@ public boolean isLatestCTSchemaVersion(long ctSchemaVersionId) {
CTSchemaVersion ctSchemaVersion =
ctSchemaVersionPersistence.fetchByPrimaryKey(ctSchemaVersionId);

if (ctSchemaVersion == null) {
return false;
}

return isLatestCTSchemaVersion(ctSchemaVersion, false);
}

private static final Log _log = LogFactoryUtil.getLog(
CTSchemaVersionLocalServiceImpl.class);

@Reference
private ConfigurationProvider _configurationProvider;

@Reference
private ReleaseLocalService _releaseLocalService;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ public void testPortalInstanceRegistered() throws Exception {
ctCollection = _ctCollectionLocalService.fetchCTCollection(
ctCollection.getCtCollectionId());

Assert.assertNotEquals(
Assert.assertEquals(
ctSchemaVersion.getSchemaVersionId(),
ctCollection.getSchemaVersionId());
Assert.assertEquals(
Expand Down
Loading