Performance due to loading product collection twice

[Swahjak]

Issue Brief

The product collection is being loaded twice due to calling getItems() early to check results.

Environment

• PHP Version: 8.1
• Magento Version: not relevant
• Tweakwise Version: 8.2.0
• Tweakwise Export Version: not relevant
• Magento Deploy Mode: production|development
• Third party modules:

Steps to reproduce

1. Load Tweakwise product collection

Actual result

The load method is called twice due to the collection being cleared in Model/Catalog/Layer/ItemCollectionProvider.php:99 which causes all related event listeners (catalog_product_collection_load_after) and plugins being fired twice. Resulting in a performance hit.

Expected result

When loading a collection the load method should only be fired once to prevent unnecessary overhead.

[Swahjak]

In general I wonder what this https://github.com/EmicoEcommerce/Magento2Tweakwise/blob/master/Model/Catalog/Layer/ItemCollectionProvider.php#L99-L105 section actually does. Since the getSize() is also being overwritten by Tweakwise and triggers a load that isn't an option either. Would it be sufficient to have https://github.com/EmicoEcommerce/Magento2Tweakwise/blob/master/Model/Catalog/Product/AbstractCollection.php#L20 as a public and count those?