peterborkuti/liferay-portal — pull request #7 by vilmospapp (2014-05-12 12:55:31+00:00)
Title: LPS-46756 Avoid infinite Loop in Asset Publsiher's manual configuration

Hi @peterborkuti,

I've got this issue as it was marked as related to operations, but it turned out that it's related to AssetPublisher's modification with SearchContainer pattern modification. The problem is that during the SearchContainerRow processing, we call continue; when the processed asset is not visible, hence we will not call the doAfterBody() method of SearchContainerTag which is responible to increment the index for processing the results in SearchContainer, hence we will process the same entry again, again and again. Though the compiled js will print new lines into the response infinitely, so StringBundler will allocate more and more memory what ends up in a OutOfMemoryException: java heap space message in the logs.

I twisted condition checking to prevent the infinite loop on non visible assetEntry.

Please review my changes.

Cheers,
Vilmos