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-19861 | master #336

Open
wants to merge 5,322 commits into
base: master
Choose a base branch
from
Open

LPD-19861 | master #336

wants to merge 5,322 commits into from

Conversation

marcosapmf
Copy link
Owner

AlmirFe and others added 30 commits April 17, 2024 17:45
…ojects/LPP/components Localization seems to be part of core infra team, so changing it accordingly
…M", update test.properties properly and sets the correct main component
…olbar that are filtering without opening a modal window
luisacmn and others added 29 commits April 18, 2024 17:34
# breaking

## What modules/apps/layout/layout-utility-page-service/src/main/java/com/liferay/layout/utility/page/service/impl/LayoutUtilityPageEntryLocalServiceImpl.java

Adding parameters String friendlyURL and boolean privateLayout to method addLayoutUtilityPageEntry, changing its signature from
    
    public LayoutUtilityPageEntry addLayoutUtilityPageEntry(
                            String externalReferenceCode, long userId, long groupId,
                            long plid, long previewFileEntryId,
                            boolean defaultLayoutUtilityPageEntry, String name, String type,
                            long masterLayoutPlid, ServiceContext serviceContext)
                    throws PortalException
    
    to
    
    public LayoutUtilityPageEntry addLayoutUtilityPageEntry(
                            String externalReferenceCode, long userId, long groupId,
                            String friendlyURL, long plid, long previewFileEntryId,
                            boolean defaultLayoutUtilityPageEntry, String name, String type,
                            long masterLayoutPlid, boolean privateLayout,
                            ServiceContext serviceContext)
                    throws PortalException
                    
## Why

This change is required to have the ability to create a utility page with a public layout.


----

# breaking

## What modules/apps/layout/layout-utility-page-service/src/main/java/com/liferay/layout/utility/page/service/impl/LayoutUtilityPageEntryServiceImpl.java

Adding parameters String friendlyURL and boolean privateLayout to method addLayoutUtilityPageEntry, changing its signature from

 public LayoutUtilityPageEntry addLayoutUtilityPageEntry(
                            String externalReferenceCode, long groupId, long plid, long previewFileEntryId,
                            boolean defaultLayoutUtilityPageEntry, String name, String type,
                            long masterLayoutPlid, ServiceContext serviceContext)
                    throws PortalException
    
    to
    
    public LayoutUtilityPageEntry addLayoutUtilityPageEntry(
                            String externalReferenceCode, long groupId, String friendlyURL,
                            long plid, long previewFileEntryId,
                            boolean defaultLayoutUtilityPageEntry, String name, String type,
                            long masterLayoutPlid, boolean privateLayout,
                            ServiceContext serviceContext)
                    throws PortalException

## Why

This change is required to have the ability to create a utility page with a public layout.


----
@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.