Skip to content

fix: Bypass filtering on the groups assignment option#300

Merged
Rom1-B merged 4 commits intopluginsGLPI:mainfrom
Rom1-B:support_36789
Mar 13, 2025
Merged

fix: Bypass filtering on the groups assignment option#300
Rom1-B merged 4 commits intopluginsGLPI:mainfrom
Rom1-B:support_36789

Conversation

@Rom1-B
Copy link
Copy Markdown
Contributor

@Rom1-B Rom1-B commented Mar 13, 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

  • It fixes !36789
  • Here is a brief description of what this PR does

Since #291, when a user is created, this error appears in sql-error.log:

glpisqllog.ERROR: DBmysql::doQuery() in .../src/DBmysql.php line 395
  *** MySQL query error:
  SQL: INSERT INTO glpi_plugin_escalade_users (`users_id`, `use_filter_assign_group`)
                  VALUES (217023, 0)
  Error: Unknown column 'use_filter_assign_group' in 'field list'
  Backtrace :
  marketplace/escalade/hook.php:476                  DBmysql->doQuery()
  src/Plugin.php:1713                                plugin_escalade_item_add_user()
  src/CommonDBTM.php:1404                            Plugin::doHook()
  src/AuthLDAP.php:2955                              CommonDBTM->add()
  src/Console/Ldap/SynchronizeUsersCommand.php:387   AuthLDAP::ldapImportUserByServerId()
  vendor/symfony/console/Command/Command.php:298     Glpi\Console\Ldap\SynchronizeUsersCommand->execute()
  vendor/symfony/console/Application.php:1040        Symfony\Component\Console\Command\Command->run()
  src/Console/Application.php:286                    Symfony\Component\Console\Application->doRunCommand()
  vendor/symfony/console/Application.php:301         Glpi\Console\Application->doRunCommand()
  vendor/symfony/console/Application.php:171         Symfony\Component\Console\Application->doRun()
  bin/console:134                                    Symfony\Component\Console\Application->run()

Screenshots (if appropriate):

@Rom1-B Rom1-B merged commit cbc7973 into pluginsGLPI:main Mar 13, 2025
3 checks passed
@Rom1-B Rom1-B deleted the support_36789 branch March 13, 2025 14:56
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