Ask Questions and Find Answers
Important:
Ask is now read-only. You can review any existing questions and answers, but not add anything new.
But - don't panic! While ask is no more, we've replaced it with discuss - the new Liferay Discussion Forum! Read more here here or just visit the site here:
discuss.liferay.com
RE: Method isCheckedOut in DLFileEntryImpl.java bad implementation?
As of version 7.4.3.25-ga25 and up (I didn't checked older versions) the implementation of "isCheckedOut" method in DLFileEntryImpl.java uses: DLFileEntryServiceUtil.isFileEntryCheckedOut
This means it will check the permissions - that is not desirable I think on that low level.
Shouldn't be used: DLFileEntryLocalServiceUtil.isFileEntryCheckedOut to skip premission checking ?
for example hasLock() or getContentStream(String version) methods in the same class "DLFileEntryImpl.java " use "DLFileEntryLocalServiceUtil".
I'm inclined to say this is intentional, only because there's another
instance of a remote service call in the DLFileEntryImpl
class: DLFileVersionServiceUtil
is also used. As someone
who's never looked at this code before today, I can't explain why it's
designed this way, but I don't think it's a bug. I could be wrong, of course.
Powered by Liferay™