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

LPS-32718 #13862

Closed

Conversation

caorongjin
Copy link

No description provided.

Sergio González added 8 commits September 13, 2013 21:48
…inheriting the document type restrictions or not
…ed in that folder. If we inherit we return the document types allowed in the folder we are inheriting from (even it if it's empty). We don't want to always return at least one document type, because if the folder is restricted to only one document type and the user cannot add that document type, he won't be able to add any document in that folder.
…ct amongst the document types supported by that folder, considering that either he has VIEW permission on the doc types or it's the basic document, or it's the current document type of the document.
… we will get duplicates when applying permission to the query
… the permissions. As "Basic Document" is an entity that doesn't have any permission (because it's a 'fake' entity) we need to extract it from the query affected by the permission and do a UNION ALL to make sure that it's always returned.
…FileEntryTypeServiceUtil.getFolderFileEntryTypes except when we are in a external repo that doesn't allow fileEntryTypes that we return Basic Document. The option to upload multiple documents will be displayed if the user has permission to upload a fileEntryType in that folder (upload multiple documents is not supported in external repos)
@liferay-dev-ci
Copy link

Some tests have FAILED. ❌
Please use the following link to check if any of them are related to your changes.
Refer to this link for build results: http://ci.liferay.org.es/jenkins/job/brianchandotcom/4795/

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 83360d1...4b3e090

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.

3 participants