9.4.3 recurrent ticket is not created when periodicity is hours/days

grajek · 2019-07-10 09:41:21

Hello!
After upgrade from 9.1.6 and MySql to MariaDB I had a problem with recurrent tickets that weren't created when periodicity was set to hour or days. It didn't come up in tests, because first occurence did show up, problem was with next occurrences. After investigation I found out that in database glpi_ticketrecurrents.next_creation_date field was not being set. Updating this field manually triggered ticket creation, but only once. After inserting some debug lines in inc/ticketrecurrent.class.php I found out that method computeNextCreationDate() was failing periodicity check:

      
343      if (!is_int($periodicity) && !preg_match($periodicity_pattern, $periodicity)) {
344         // Invalid periodicity.
345         return 'NULL';
346      }

The culprit is is_int() function, that checks if variable is of type int, which is not - it's a string containing valid int.
After replacing it with is_number() everything went back to normal

Langundo · 2019-07-13 14:58:53

Hello grajek,
thank for your work to find the bug.
Have trouble with my recurrent tickets too. So I decide to use your solution but then I edit recurrent tickets or start automatic action ticketcurrent the web IF hangs (shows a blank page)
After a little try and error I found that the (!is_string($periodicity) seems to work for me.
But I'm not fit in php so maybe you (or someone else) can have a look at this.

Thanks.

grajek · 2019-07-13 22:28:12

Oh, you're right, I made a mistake in function name writing this post. Correct function name is: is_numeric() . Sorry.

Langundo · 2019-07-15 13:28:48

Have edit the function.

After update to 9.4.3 the ticketrecurrent works (as you wrote) one time. So i didn't think there is a problem with glpi and was loocking for some mismatch configs in my settings.
Never I had the idea to look in database.

So I have to say well done and thank you again ;-)

alientm · 2019-11-19 08:32:44

I also have problems with the recurrent ticket version 9.4.4.

I have several recurrent tickets that should be repeated every 1 month, since version 9.4.3 it will not work correctly for me.

Langundo · 2019-11-19 21:31:39

Hello alientm,

because some changes in my work actually I don't work with glpi ;-(
Only "play" a little with the 9.5 at home. As I can see this little bug is still present in 9.5.

Did you have made the little changes in inc/ticketrecurrent.class.php?
Pls edit the line 343 as described.

alientm · 2019-11-21 08:47:34

Should be:
is_number()
or
is_numeric()
in place of is_int() ?

alientm · 2019-12-05 13:34:32

Any help ?

Phorta · 2020-01-15 17:02:52

Langundo wrote:

Hello alientm,
because some changes in my work actually I don't work with glpi ;-(
Only "play" a little with the 9.5 at home. As I can see this little bug is still present in 9.5.
Did you have made the little changes in inc/ticketrecurrent.class.php?
Pls edit the line 343 as described.

hello guys,
I'm using GLPI 9.4.4 with MariaDB.
and I am exactly with the same problem, during testing the recurring tickets worked, but in production the date is not respected ...

I tried to fix the problem as described above, but in my version it is a little different and I am not php programmer, could they help me ?
@Lagundo and @grajek

below I copied the contents of line 343 to 365 of the file inc/ticketrecurrent.class.php

if ($periodicity <> 0) {
// Standard time computation
$timestart = strtotime($begin_date) - $create_before;
$now = time();
if ($now > $timestart) {
$value = $periodicity;
$step = "second";
if (preg_match('/([0-9]+)MONTH/', $periodicity, $matches)) {
$value = $matches[1];
$step = 'MONTH';
} else if (preg_match('/([0-9]+)YEAR/', $periodicity, $matches)) {
$value = $matches[1];
$step = 'YEAR';
} else {
if (($value%DAY_TIMESTAMP)==0) {
$value = $value/DAY_TIMESTAMP;
$step = "DAY";
} else {
$value = $value/HOUR_TIMESTAMP;
$step = "HOUR";

alientm · 2020-01-16 13:44:11

343 to 347 but code is from glpi

if (!is_string($periodicity) && !preg_match('/^\d+$/', $periodicity)
&& !preg_match($periodicity_pattern, $periodicity)) {
// Invalid periodicity.
return 'NULL';
}

and work for me

Forum GLPI-Project

Announcement

#1 2019-07-10 09:41:21

9.4.3 recurrent ticket is not created when periodicity is hours/days

#2 2019-07-13 14:58:53

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#3 2019-07-13 22:28:12

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#4 2019-07-15 13:28:48

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#5 2019-11-19 08:32:44

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#6 2019-11-19 21:31:39

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#7 2019-11-21 08:47:34

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#8 2019-12-05 13:34:32

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#9 2020-01-15 17:02:52

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

#10 2020-01-16 13:44:11

Re: 9.4.3 recurrent ticket is not created when periodicity is hours/days

Board footer