Skip to content

Comments

Fix(install): SQL warning#552

Merged
stonebuzz merged 2 commits intomainfrom
fix_install
Feb 23, 2026
Merged

Fix(install): SQL warning#552
stonebuzz merged 2 commits intomainfrom
fix_install

Conversation

@Rom1-B
Copy link
Contributor

@Rom1-B Rom1-B commented Feb 23, 2026

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

  • It fixes #N/A

Fixing this SQL warning during installation:

glpi.WARNING:   *** User Warning: MySQL query warnings:
  SQL: SELECT `go`.`entities_id` AS `entities_id`, `go`.`is_recursive` AS `is_recursive`, `goi`.`id` AS `items_id` FROM `glpi_plugin_order_orders` AS `go`, `glpi_plugin_order_orders_items` AS `goi` WHERE `goi`.`plugin_order_orders_id` = 'go.id'
  Warnings: 
1292: Truncated incorrect DOUBLE value: 'go.id' at DBmysql.php line 433
  Backtrace :
  ./src/DBmysql.php:433                              
  ./src/DBmysqlIterator.php:126                      DBmysql->doQuery()
  ./src/DBmysql.php:1074                             DBmysqlIterator->execute()
  ./marketplace/order/inc/order_item.class.php:2129  DBmysql->request()
  ./marketplace/order/hook.php:76                    PluginOrderOrder_Item::install()
  ./src/Plugin.php:1252                              plugin_order_install()
  ...Glpi/Console/Marketplace/UpgradeCommand.php:186 Plugin->install()
  ./vendor/symfony/console/Command/Command.php:326   Glpi\Console\Marketplace\UpgradeCommand->execute()
  ./vendor/symfony/console/Application.php:1106      Symfony\Component\Console\Command\Command->run()
  ./src/Glpi/Console/Application.php:330             Symfony\Component\Console\Application->doRunCommand()
  ./vendor/symfony/console/Application.php:324       Glpi\Console\Application->doRunCommand()
  ./vendor/symfony/console/Application.php:175       Symfony\Component\Console\Application->doRun()
  ./bin/console:144                                  Symfony\Component\Console\Application->run()

Screenshots (if appropriate):

@Rom1-B Rom1-B requested a review from stonebuzz February 23, 2026 07:25
@stonebuzz stonebuzz merged commit 385c6ff into main Feb 23, 2026
3 checks passed
@stonebuzz stonebuzz deleted the fix_install branch February 23, 2026 08:08
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.

2 participants