Forum

Cron.php notificati...
 
Notifications
Retirer tout

Cron.php notification timing check error

5 Posts
2 Utilisateurs
0 Reactions
1,761 Vu
(@burgerzs)
Posts: 24
Active Member
Début du sujet
 
[#6289]

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


 
Posté : 12 Oct PM 16:1010
(@burgerzs)
Posts: 24
Active Member
Début du sujet
 

I also think that the condition operation (...>=...) has to be changed as well. This seems to be the correct logic:
if ($notif->notificationTimeformat('H:i:s')) {

Regards,
Zsolt


 
Posté : 12 Oct PM 16:1010
(@babynus)
Posts: 14952
Membre Admin
 

Issue #4229 recorded.


 
Posté : 14 Oct PM 14:1010
(@burgerzs)
Posts: 24
Active Member
Début du sujet
 

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);


 
Posté : 29 Oct PM 18:1010
(@babynus)
Posts: 14952
Membre Admin
 

You're right.
Issue #4249 reopened, will be fixed on 8.2.4


 
Posté : 29 Oct PM 20:1010
Share:
Retour en haut