Email Issues in patient reminder

arnabnaha wrote on Saturday, January 19, 2013:

HI Brady…
was testing out my system…and trying out sending emails from the patient reminder screen…(I mean sending  the patient reminders). After clicking process and send button, t showed 2/2 patients (As I have set up two patients only in my test site) and then it was processing for infinite time…There were total 9 reminders and the mails were like never sent as the processing time was too much…please check this issue…

you can test this issue in my test site:

http://nahahealthclinic.dyndns.org/openemrtest

username: admin
password: pass

arnabnaha wrote on Saturday, January 19, 2013:

I have set up the offsite portal and mail server is running…you can test the email function…just change the contact email id of the patients in the demographics.

bradymiller wrote on Sunday, January 20, 2013:

Hi,

What are you now seeing at Reports->Clinic->‘Report Results’ ?

-brady
OpenEMR

bradymiller wrote on Sunday, January 20, 2013:

Hi,

I just checked this on your demo. Note when you click on the Report link in above screen, you get an error regarding not having CURL installed. So, your server needs the CURL php extension installed (in ubuntu, this is the ‘php5-curl’ package).

-brady
OpenEMR

arnabnaha wrote on Sunday, January 20, 2013:

Hi…
I enabled the curl extension in php.ini file…now after going to report results it shows that the emails are pending…(screenshot attached). Anything more to do or install??

Link to screenshot: http://nahahealthclinic.dyndns.org/public/report.jpg

arnabnaha wrote on Sunday, January 20, 2013:

still the emails are not send and it is continuously processing….

arnabnaha wrote on Monday, January 21, 2013:

Any fixes to this issue??

bradymiller wrote on Monday, January 21, 2013:

Hi,

Are you seeing anything in the php errror log (check it after you try to do the report)?

thanks,
-brady
OpenEMR

arnabnaha wrote on Monday, January 21, 2013:

This is the only things which are shown:

   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\interface\\patient_file\\summary\\shot_record.php on line 245, referer: http://nahahealthclinic.dyndns.org/openemrtest/interface/patient_file/summary/immunizations.php?mode=add&id=&pid=2&cvx_code=33&administered_date=2013-01-21&manufacturer=Glaxo&lot_number=IN11233&administered_by=Naha%2C+Dr.+Arnab&administered_by_id=1&education_date=2013-01-21&vis_date=2013-01-21&note=
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 318
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 566
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 924
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 925
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 179
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 211
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 268
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 318
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 566
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 924
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 925
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 179
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 211
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 268
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 318
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 566
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 924
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\controllers\\C_Document.class.php on line 925
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 179
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 211
   PHP Deprecated:  Assigning the return value of new by reference is deprecated in C:\\xampp\\htdocs\\openemrtest\\library\\classes\\TreeMenu.php on line 268

arnabnaha wrote on Monday, January 21, 2013:

Got it:

Here is the problem I guess:

   PHP Fatal error:  Class ‘MyMailer’ not found in C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php on line 365, referer: http://nahahealthclinic.dyndns.org/openemrtest/interface/patient_file/reminder/patient_reminders.php?mode=admin&patient_id=

bradymiller wrote on Monday, January 21, 2013:

Hi,

Yep, you pinpointed it. Try this fix and let me know if it works:
http://github.com/bradymiller/openemr/commit/0712f8eeeb37b25682e9d7f7a79331a0c69d6f65

thanks,
-brady
OpenEMR

arnabnaha wrote on Monday, January 21, 2013:

Hi…
The issue is solved…thanks for the fix but I have to press the process and send button 3 times…the emails dont go at one go….why is this…you can check on my demo…

bradymiller wrote on Tuesday, January 22, 2013:

Hi arnabnaha,

Looks like your demo is down. Let me know when it’s back up, so I can play around with the one at a time email issue. Also, check your php error log (it’s possible that an a php error is being thrown after sending the email, which is then breaking it out of the loop that should be sending all of the emails).

thanks,
-brady
OpenEMR

arnabnaha wrote on Tuesday, January 22, 2013:

The demo is UP…The php log shows this now:

Tue Jan 22 16:46:55 2013]   client denied by server configuration: C:/xampp/htdocs/xampp/
   File does not exist: C:/xampp/htdocs/favicon.ico
   File does not exist: C:/xampp/htdocs/favicon.ico
   File does not exist: C:/xampp/htdocs/robots.txt
   PHP Fatal error:  Class ‘MaviqClient’ not found in C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php on line 395, referer: http://nahahealthclinic.dyndns.org/openemrtest/interface/patient_file/reminder/patient_reminders.php?mode=admin&patient_id=
   PHP Fatal error:  Class ‘MaviqClient’ not found in C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php on line 395, referer: http://nahahealthclinic.dyndns.org/openemrtest/interface/patient_file/reminder/patient_reminders.php?mode=admin&patient_id=

bradymiller wrote on Tuesday, January 22, 2013:

oh,
That should be an easy fix. Just need to include the maviq class. I’ve added this fix and the previous fix (for the mail class) to sourceforge and they’ll be mirrored to github in 13 minutes. Let me know if it works with these fixes.
-brady
OpenEMR

bradymiller wrote on Tuesday, January 22, 2013:

To avoid confusion, note there are two different commits. Here they on on sourceforge (they will be on github in 10 minutes):
http://openemr.git.sourceforge.net/git/gitweb.cgi?p=openemr/openemr;a=commitdiff;h=956f1268ac95885de6af944afbdc26193cf9f8d2
http://openemr.git.sourceforge.net/git/gitweb.cgi?p=openemr/openemr;a=commitdiff;h=2220d50a0f28310aaf25e135cca9ef03baf8242a

arnabnaha wrote on Tuesday, January 22, 2013:

got ur commits…thanks…but the problem still persists….the apache log shows the following now:

   PHP Fatal error:  Uncaught exception ‘Exception’ with message ‘Curl failed with error <url> malformed’ in C:\\xampp\\htdocs\\openemrtest\\library\\maviq_phone_api.php:82\nStack trace:\n#0 C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php(410): MaviqClient->sendRequest(‘appointment’, ‘POST’, Array)\n#1 C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php(171): send_reminders()\n#2 C:\\xampp\\htdocs\\openemrtest\\library\\ajax\\execute_pat_reminder.php(55): update_reminders_batch_method(’’, ‘’, ‘24’, true)\n#3 {main}\n  thrown in C:\\xampp\\htdocs\\openemrtest\\library\\maviq_phone_api.php on line 82, referer: http://nahahealthclinic.dyndns.org/openemrtest/interface/patient_file/reminder/patient_reminders.php?mode=admin&patient_id=
   PHP Fatal error:  Uncaught exception ‘Exception’ with message ‘Curl failed with error <url> malformed’ in C:\\xampp\\htdocs\\openemrtest\\library\\maviq_phone_api.php:82\nStack trace:\n#0 C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php(410): MaviqClient->sendRequest(‘appointment’, ‘POST’, Array)\n#1 C:\\xampp\\htdocs\\openemrtest\\library\\reminders.php(171): send_reminders()\n#2 C:\\xampp\\htdocs\\openemrtest\\library\\ajax\\execute_pat_reminder.php(55): update_reminders_batch_method(’’, ‘’, ‘25’, true)\n#3 {main}\n  thrown in C:\\xampp\\htdocs\\openemrtest\\library\\maviq_phone_api.php on line 82, referer: http://nahahealthclinic.dyndns.org/openemrtest/interface/patient_file/reminder/patient_reminders.php?mode=admin&patient_id=

bradymiller wrote on Tuesday, January 22, 2013:

Hi arnabnaha,

Try this fix now (this is in addition to the first two fixes):
http://github.com/bradymiller/openemr/commit/e1c2b79470c9035d72362d41d7314b22812378aa

Let me know if it now works.

Thanks,
-brady
OpenEMR

bradymiller wrote on Tuesday, January 22, 2013:

Hi Tony,

Can I get a status on the Maviq feature. Is this dead? This is important because I have essentially commented out this feature in the above fix because it appears to break on Windows.

thanks,
-brady
OpenEMR

arnabnaha wrote on Wednesday, January 23, 2013:

Hi Brady…
EVERYTHING WORKS VERY WELL NOW….Thanks for the fix!!