πŸ‡ΊπŸ‡ΈUnited States @Ec1ipsis

Account created on 24 October 2011, over 12 years ago
#

Merge Requests

Recent comments

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

I can't tell whether the automated test needs to change or if the behavior itself has a bug. I've merged in the most recent release to the 8.1 branch, and the error that I'm seeing from PHPUnit is:

1) Drupal\Tests\redis\Kernel\RedisCacheTest::testSetGet
Failed asserting that false is of type "object".
/builds/issue/redis-3179757/vendor/phpunit/phpunit/src/Framework/Constraint/Constraint.php:122

I believe that the automated test itself needs to be updated, but I'm not certain. Anyone have a firm answer on that?

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

This is resolved by the 2.2.0 release. Thank you so much for the PR!

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

Ec1ipsis β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

Just a quick heads up, we've released two versions (8.1.7 and 8.1.8) which resolve bugs that caused this issue. Let us know if you still see it after updates!

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

I just tested this on a fresh Drupal 10 install from the Pantheon upstream and composer require drupal/search_api_pantheon:^8 completed as expected. Can you provide some additional context around Drupal version, specific version attempted to install, and whether or not there are other modules which require kint?

Thank you!

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

Issue resolved as of the 8.1.4 release.

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

Understood, thank you for that information, I need to double check our current branch/release strategy and verify the PR, but I don't imagine any issues getting this in shortly unless something goes terribly awry.

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

I'm testing and verifying this patch for merge into the project. The patch itself looks good, but I'm having some trouble reproducing the original issue. After spinning up a fresh Drupal 10 site, enabling the various Search modules, exporting configuration, making a small change, and then importing configuration, I get no errors. Can you shed some additional light on what steps cause the issue? Thank you!

πŸ‡ΊπŸ‡ΈUnited States Ec1ipsis

To clarify, is it still failing after the PR? If so, is it a different error or something that was easily worked around?

Production build 0.69.0 2024