mateusznitka / protocolsmanager

GLPI Plugin to report users inventory
8 stars 12 forks source link

GLPI v10 #45

Closed Alexk05m closed 2 years ago

Alexk05m commented 2 years ago

Hello,

Could you add compatibility for glpi v10 and php 7.4 ?

daviox commented 2 years ago

I'd be more than happy if I'll be able to continue using this plugin in v10

Wolvverine commented 2 years ago

@daviox

Look to my repositories, you have last patched version.

Alexk05m commented 2 years ago

@Wolvverine

I use your last patched version but I still have access denied

Wolvverine commented 2 years ago

Which branch?

Look on develop-v10

Alexk05m commented 2 years ago

I just did a test with a new installation (debian 10 - GLPI v10 - php7.4) and the develop-v10 branch but I have an error message when I install the plugin :

- Erreur durant l'éxecution de la requête : ALTER TABLE glpi_plugin_protocolsmanager_emailconfig MODIFY COLUMN email_content TEXT - L'erreur est Table 'db_glpi.glpi_plugin_protocolsmanager_emailconfig' doesn't exist

But even removing the problematic if I get the message : access denied

brusilva84 commented 2 years ago

Yes the access denied message also affects me... Can't really test if you repo works @Wolvverine since this is blocking the usage.

Would be glad on helping developing but not very expert on php

LuizRicarte commented 2 years ago

@Wolvverine

After downloading develo-10, performing the installation, it displays the message of access denied. Can you help me? image

Wolvverine commented 2 years ago

In hook.php, change: if (!$DB->FieldExists('glpi_plugin_protocolsmanager_emailconfig', 'email_content')) { to: if ($DB->FieldExists('glpi_plugin_protocolsmanager_emailconfig', 'email_content')) {

LuizRicarte commented 2 years ago

@Wolvverine in line 157? Not effect to me, still access denied.

LuizRicarte commented 2 years ago

@Wolvverine Dou you Help Me?

image

Wolvverine commented 2 years ago

Check now, and I will add new dompdf but in other branch.

Alexk05m commented 2 years ago

@Wolvverine

Thank you for the time spent, on my side it works perfectly with branch (develop-v10-dompdf)!

thanks again

Wolvverine commented 2 years ago

@LuizRicarte @MC-brunomendes @daviox @dayflare @pistokcad ?

pistokcad commented 2 years ago

@LuizRicarte @MC-brunomendes @daviox @Dayflare @pistokcad ?

I don't have any issue (your modification good) - I written another topic.

daviox commented 2 years ago

I'll check the updates as soon as I'll be able to.

I guess that will do it, since I too have a problem with permissions.

brusilva84 commented 2 years ago

@Wolvverine repo with dompdf works great on 10.0

Dayflare commented 2 years ago

@LuizRicarte @MC-brunomendes @daviox @Dayflare @pistokcad ?

i'll check soon with the new 10.0.1 version of GLPI

Dayflare commented 2 years ago

@LuizRicarte @MC-brunomendes @daviox @Dayflare @pistokcad ?

runs great on GLPI 10.0.1, having a few issues with the image in the header of the pdf, but beside of this it's running good and everything works. a big thanks for your work!