pagelist component doesn't respect unpublished content

[leachuk]

Bug Report

Current Behavior
When the pagelist component is set to list Child Pages and unpublishing one of the child nodes, it is still displayed on the author instance while correctly being removed from the publish instance list.

Expected behavior/code
The list on the author instance should not display the unpublished node so it provides an accurate and consistent authoring experience.

Environment

• AEM version and patch level (e.g. AEM 6.3 SP1 CFP2)
• Core Components version (e.g. 2.0.4)
• JRE version (e.g. Java(TM) SE Runtime Environment (build 1.8.0_152-b16))

Possible Solution
Code fix required.

Additional context / Screenshots
Add any other context about the problem here. If applicable, add screenshots to help explain.

[wildone]

so does this mean that on the Author pages that are unpublished should be hidden, similar behaviour to Hide in Navigation?

[wildone]

to do this the rule should be added as a filter for list collection that hides all pages that have cq:lastReplicationAction = Deactivated

[wildone]

this function can be updated to include this rule

static boolean includePageInList(Page page, boolean includeInvalid, boolean includeHidden) { return (includeHidden || !page.isHideInNav()) && (includeInvalid || page.isValid()) && page.getDeleted() == null; }

[wildone]

Also this is more a feature than a bug...

[wildone]

As an alternative I would recommend adding a flag for this to show to authors stuff that is in the CMS but has been unpublished this would provide some input in to figuring out why some content not appearing.