Forum

Welcome to ProjeQtOr new Forum. We migrated old forum to the new website.
You will find all your posts here, with your usual account.
Just one point : you’ll have to reinitialize your password. Use “Lost password” feature.

Cron.php notificati...
 
Notifications
Clear all

Cron.php notification timing check error

5 Posts
2 Users
0 Reactions
1,797 Views
(@burgerzs)
Posts: 24
Active Member
Topic starter
 
[#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


 
Posted : 12 Oct 2019 16H48
(@burgerzs)
Posts: 24
Active Member
Topic starter
 

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


 
Posted : 12 Oct 2019 16H57
babynus
(@babynus)
Posts: 14966
Main Contributor Admin
 

Issue #4229 recorded.


 
Posted : 14 Oct 2019 14H28
(@burgerzs)
Posts: 24
Active Member
Topic starter
 

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


 
Posted : 29 Oct 2019 18H47
babynus
(@babynus)
Posts: 14966
Main Contributor Admin
 

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


 
Posted : 29 Oct 2019 20H39
Share:

Scroll to Top