[CMS-4244] ContentFilterHelper optimisation - JIRA Ametys

Type: Improvement
Resolution: Fixed
Priority: Major
Fix Version/s: 3.4.2, 3.5
Affects Version/s: None
Component/s: None
Labels:
None

ContentFilterHelper.saxMatchingContents

            boolean orphan = isOrphan(content);
            if (!maskOrphan || (maskOrphan && !orphan))
            {

should be

            if (!maskOrphan || (maskOrphan && !isOrphan(content)))
            {

to avoid a useless isOrphan call

Nicolas Gavalda (Inactive) added a comment - 22/Feb/13 14:19

Should be

if (!maskOrphan || !isOrphan(content))
{

to avoid unnecessary test too.

Nicolas Gavalda (Inactive) added a comment - 22/Feb/13 14:19 Should be if (!maskOrphan || !isOrphan(content)) { to avoid unnecessary test too.

Assignee:: Unassigned

Reporter:: Raphaël Franchet

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Created:: 22/Feb/13 11:50

Updated:: 14/May/13 16:19

Resolved:: 04/Mar/13 13:59