tomolimo / mailanalyzer

Mail Analyzer GLPI Plugin may be used to combine CC mails into one Ticket
25 stars 10 forks source link

GLPI 10 #66

Closed epigraphe closed 2 years ago

epigraphe commented 2 years ago

Good day. Is the plugin planned to work with glpi 10? I installed it but I get the error:

*** PHP User Warning (512): Unknown attributes "classname" used in "PluginMailAnalyzer" class registration in /var/www/glpi/src/Plugin.php at line 1410

Is it possible to run it under version 10, since it does not have this functionality?

tomolimo commented 2 years ago

Hello @epigraphe Yes this is planned, but currently not scheduled :) Thank you, Regards, Tomolimo

epigraphe commented 2 years ago

Thanks. We are very, very much looking forward to it. You have the most important plugin for GLPI to work

sghebuz commented 2 years ago

Hello @tomolimo I think this plugin is a must for glpi, do you think we can sponsor the compatibility with version 10? thanks

tomolimo commented 2 years ago

Hello @sghebuz Currently, this a problem of time, not a problem of money 😃 Thank you for your proposal, but I would prefer if you propose a pull request for the compatibility :) Thank you, Regards, Tomolimo

sghebuz commented 2 years ago

eheh same here :-D it was worth a shot

jlangone commented 2 years ago

Hi @tomolimo as @sghebuz said this plugin is one of the most important tools for glpi to work. Can you estimate when it will be a compatible version with glpi 10? Thank u very much, regards from UY.

tomolimo commented 2 years ago

Hello @jlangone Probably for September. But in the meanwhile, you propose a PR to update it to GLPI 10.0 Thank you, Regards, Tomolimo

shuriban commented 2 years ago

Hello @tomolimo For GLPI 10.0. replace $iterator->next(); to $iterator->current(); https://github.com/glpi-project/glpi/issues/6010#issuecomment-1062857744

empereira commented 2 years ago

Hello @tomolimo For GLPI 10.0. replace $iterator->next(); to $iterator->current(); glpi-project/glpi#6010 (comment)

Hi, In which file should this change take place?

tomolimo commented 2 years ago

Fixed by https://github.com/tomolimo/mailanalyzer/releases/tag/3.0.0