Aleksander Adamowsky already developed a sms notification tool. I started to
integrate it, but I’m not finished yet (and atm I have no time at all to get
it done, sorry). If you want, I can send you that plugin or you contact
Aleksander. Just search for sms notification in nabble.
Hi!
I’ve taken time to integrate my SMS notification backend with the
latest version of notifications app.
Attaching the revised version.
As to the problem with setlocale
(http://www.nabble.com/how-to-handle-setlocale-to14730139s3741.html#a14730139),
I think the best way we can afford is to store the old locale, do the
thing that requires changed locale as quickly as possible, and revert
to the original locale when done.
This is how the new version of my SMS backend works.
Note that this is a problem only on web servers that use threads
instead of processes.
So this will be a problem only on MS Windows+IIS or on very uncommon
configurations of Apache that use the threaded “worker” MPM.
Since PHP is considered to fail under worker MPM, almost nobody uses
it for PHP apps. Multithreaded servers cause lots of other problems as
well, so our problem caused by SMS notification backend under some
specific unlikely conditions (this is a very tight race condition)
will probably be the most minor of them all.
That leaves us only with MS Windows + IIS, and this platform is quite
uncommon (at least for open source apps). Using it is asking for
trouble anyway. So I think you can quite safely apply my patch and
just note the potential problem in the documentation or online help.
I think we could also try to detect presence of a multithreaded server
environment from our PHP scripts (anybody knows how?) and display a
big red warning in the admin/notifications configuration page next to
the SMS backend enable/disable option.
–
Best Regards,
Aleksander Adamowski
http://olo.org.pl