You are not logged in.

Announcement

 Téléchargez la dernière version stable de GLPI      -     Et vous, que pouvez vous faire pour le projet GLPI ? :  Contribuer
 Download last stable version of GLPI                      -     What can you do for GLPI ? :  Contribute

#1 2015-07-27 18:07:26

jgobet
Member
Registered: 2013-06-14
Posts: 72

Notifications by default in Entities

Hello,

I have problem in entities administration with the view Notifications.
When I try to save this view and that the field Enable notifications by default is Inheritance of the parent entity value, I've got the following error At least one field has an incorrect value: Enable notifications by default.
So it's impossible to save my updates.

I use the last GLPI version (0.85.4) and I view in the Roadmap but this bug isn't listed.

Thank you for your answer.

Offline

#2 2015-07-30 15:50:36

blerohellec
Member
From: Paris
Registered: 2015-04-08
Posts: 84

Re: Notifications by default in Entities

Hello,

Indeed, I could reproduce this bug.

In inc/entity.class.php, comparing to another field of the same type (eg 'Alarm on contracts whose possible values are also Yes/No/inheritance), it seems that the type of the 'Enable notifs by default' field is incorrect.
Type is 'bool' although it should be 'specific'.
I think that type 'bool' only accepts Yes/No, thus generating an error when value is inheritance.

I've made a test, replacing 'bool' by 'specific' in line 570 of inc/entity.class.php and it works much better.
I cannot guarantee there is no other impact.

The core developers team will have to investigate.

Regards,


Benoit Le Rohellec
blerohellec@tlsconsulting.eu
www.tlsconsulting.eu

Offline

#3 2015-07-31 10:45:51

blerohellec
Member
From: Paris
Registered: 2015-04-08
Posts: 84

Re: Notifications by default in Entities

I've created a GLPI issue refering to this post.
According to my tests, the changes in this tab (of any field) are correctly saved. The only thing you can't do is to go back to value 'Inheritance' for 'Enable notifs ...'
Regards


Benoit Le Rohellec
blerohellec@tlsconsulting.eu
www.tlsconsulting.eu

Offline

#4 2015-07-31 12:30:36

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,278

Re: Notifications by default in Entities

I have no problem in 0.85.5 svn or in 0.90 beta.
On which entity are you during modification? sub-entity?
Which fields are you trying to update?


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

#5 2015-07-31 16:45:53

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,278

Re: Notifications by default in Entities


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

#6 2015-08-06 11:13:22

blerohellec
Member
From: Paris
Registered: 2015-04-08
Posts: 84

Re: Notifications by default in Entities

Correction validated.
Issue to close.
Thx a lot.


Benoit Le Rohellec
blerohellec@tlsconsulting.eu
www.tlsconsulting.eu

Offline

#7 2015-08-07 10:34:09

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,278

Re: Notifications by default in Entities

Thanks for this test. I close this post.


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

Board footer

Powered by FluxBB