ProjeQtOr free project management software - Cron.php notification timing check error - ProjeQtOr

ProjeQtOr au Paris Open Source Summit | 10-11/12/2019

ProjeQtOr sera présent au salon

les 10 et 11 décembre 2019

de 9H00 à 18H00 

 

Venez nous rencontrer sur le Stand B23  

 

L'accès au salon se fait sur inscription

Demandez gratuitement votre badge sur le site du salon

 

Cron.php notification timing check error

More
12 Oct 2019 14:48 - 12 Oct 2019 14:58 #1 by burgerzs
Hi,

I found this error when cron is started and I would like to send notifications in email.

The same problem seems to be in code version 8.2.2. as well.

2019-10-12 14:29:09.224 ***** ERROR ***** [V8.1.5] EXCEPTION *****
2019-10-12 14:29:09.224 ***** ERROR ***** [V8.1.5] on file '/usr/share/nginx/html/projeqtor/model/Cron.php' at line (549)
2019-10-12 14:29:09.224 ***** ERROR ***** [V8.1.5] cause = Call to a member function format() on string
2019-10-12 14:29:09.224 ***** ERROR ***** [V8.1.5] => #0 /usr/share/nginx/html/projeqtor/model/Cron.php (491) -> checkNotificatio
ns()
2019-10-12 14:29:09.224 ***** ERROR ***** [V8.1.5] => #1 /usr/share/nginx/html/projeqtor/tool/cronRun.php (31) -> run()

Regards,
Zsolt
Last edit: 12 Oct 2019 14:58 by burgerzs.

Please Connexion or Create an account to join the conversation.

More
12 Oct 2019 14:57 #2 by burgerzs
I also think that the condition operation (...>=...) has to be changed as well. This seems to be the correct logic:
if ($notif->notificationTime<=$currentDate->format('H:i:s')) {

Regards,
Zsolt

Please Connexion or Create an account to join the conversation.

More
14 Oct 2019 12:28 #3 by babynus
Issue #4229 recorded.

Babynus
Administrator of ProjeQtOr web site

Please Connexion or Create an account to join the conversation.

More
29 Oct 2019 16:47 #4 by burgerzs
It seems to me that in 8.2.3 the selection criteria is wrong:
$lstNotif = $notif->getSqlElementsFromCriteria($crit);
But it should be something like this:
$lstNotif = $notif->getSqlElementsFromCriteria(null,false,$crit);

Please Connexion or Create an account to join the conversation.

More
29 Oct 2019 18:39 #5 by babynus
You're right.
Issue #4249 reopened, will be fixed on 8.2.4

Babynus
Administrator of ProjeQtOr web site

Please Connexion or Create an account to join the conversation.

Moderators: babynusprotion
Time to create page: 0.031 seconds