Skip to content

Conversation

@stonebuzz
Copy link
Contributor

@stonebuzz stonebuzz commented Dec 18, 2025

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

Fix !41115

[2025-12-18 11:11:48] glpi.ERROR:   *** Caught InvalidArgumentException: Invalid criteria type. Expected `array`, `Glpi\DBAL\QueryExpression` received.
  Backtrace :
  ./src/DBmysqlIterator.php:532                      
  ./src/DBmysqlIterator.php:327                      DBmysqlIterator->analyseCrit()
  ./src/DBmysqlIterator.php:124                      DBmysqlIterator->buildQuery()
  ./src/DBmysql.php:1061                             DBmysqlIterator->execute()
  ./plugins/credit/inc/entity.class.php:438          DBmysql->request()
  ./src/CronTask.php:897                             PluginCreditEntity::cronLowCredits()
  ./front/crontask.form.php:55                       CronTask::launch()
  ...Glpi/Controller/LegacyFileLoadController.php:64 require()
  ./vendor/symfony/http-kernel/HttpKernel.php:181    Glpi\Controller\LegacyFileLoadController->__invoke()
  ./vendor/symfony/http-kernel/HttpKernel.php:76     Symfony\Component\HttpKernel\HttpKernel->handleRaw()
  ./vendor/symfony/http-kernel/Kernel.php:197        Symfony\Component\HttpKernel\HttpKernel->handle()
  ./public/index.php:70                              Symfony\Component\HttpKernel\Kernel->handle()

Screenshots (if appropriate):

@stonebuzz stonebuzz requested a review from Rom1-B December 18, 2025 11:18
@Rom1-B
Copy link
Contributor

Rom1-B commented Dec 19, 2025

Please fix CI

@stonebuzz stonebuzz merged commit bcf8f55 into main Dec 19, 2025
3 checks passed
@stonebuzz stonebuzz deleted the fix_sql_cron branch December 19, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants