Fix element queries ignoring disabled elements

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Release Notes for Blitz
 
+## 4.5.2 - 2023-08-14
+
+### Fixed
+
+- Fixed a bug in which tracked element queries were ignoring disabled elements when determining which cached pages to refresh ([#527](https://github.com/putyourlightson/craft-blitz/issues/527)).
+
 ## 4.5.1 - 2023-08-09
 
 ### Fixed

composer.json

@@ -1,7 +1,7 @@
 {
   "name": "putyourlightson/craft-blitz",
   "description": "Intelligent static page caching for creating lightning-fast sites.",
-  "version": "4.5.1",
+  "version": "4.5.2",
   "type": "craft-plugin",
   "homepage": "https://putyourlightson.com/plugins/blitz",
   "license": "proprietary",

src/helpers/RefreshCacheHelper.php

@@ -140,6 +140,10 @@ public static function getElementQueryCacheIds(ElementQueryRecord $elementQueryR
         /** @var ElementQuery $elementQuery */
         $elementQuery = $elementType::find();
 
+        // Get elements with all statuses
+        // https://github.com/putyourlightson/craft-blitz/issues/527
+        $elementQuery->status(null);
+
         $params = Json::decodeIfJson($elementQueryRecord->params);
 
         // If json decode failed

tests/TESTS.md

@@ -104,6 +104,7 @@ This document outlines the test specification for the Blitz plugin.
 - Element query type records are returned when an entry is changed by custom fields used in the query.
 - Element query type records are not returned when an entry is changed by custom fields not used in the query.
 - Element query type records are returned when an entry is changed with the date updated used in the query.
+- Element query cache IDs returned when a disabled entry is changed.
 
 ### [Site Uri](pest/Feature/SiteUriTest.php)
 

tests/pest/Feature/RefreshCacheTest.php

@@ -205,6 +205,17 @@
         ->toHaveCount(0);
 });
 
+test('Element query cache IDs are returned when a disabled entry is changed', function() {
+    $entry = createEntry(enabled: false);
+    Blitz::$plugin->generateCache->addElementQuery(Entry::find());
+    Blitz::$plugin->generateCache->save(createOutput(), createSiteUri());
+    $refreshData = RefreshDataModel::createFromElement($entry);
+    $elementQueryRecord = ElementQueryRecord::find()->orderBy('id DESC')->one();
+
+    expect(RefreshCacheHelper::getElementQueryCacheIds($elementQueryRecord, $refreshData))
+        ->toHaveCount(1);
+});
+
 test('Element query type records are returned when an entry is changed', function() {
     $entry = createEntry();
     Blitz::$plugin->generateCache->addElementQuery(Entry::find());