[opensuse] hylafax/avantfax with php 5.4+ (any fixes or interest in helping fix it?)
All, The elegant fax handling solution of hylafax/avantfax will not work with php
= 5.4. I have discussed the issue with the developer on the hylafax list and understand that avantfax is basically broken until it can be rewritten but the developer hasn't, and probably won't, have time to fix/rewrite avantfax over the next few months. The purpose of the post here is to ask:
(1) are there any users of avantfax that have patched avantfax to work with php 5.4+; (2) any interest in doing so? If there is anyone fluent in php that is interested, I'm interested in helping and have a couple of installs to test with. However, I don't have the time or php fluency required to shoulder the rewrite. -- David C. Rankin, J.D.,P.E. -- To unsubscribe, e-mail: opensuse+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse+owner@opensuse.org
El 20/08/14 a las #4, David C. Rankin escribió:
All,
The elegant fax handling solution of hylafax/avantfax will not work with php >= 5.4. I have discussed the issue with the developer on the hylafax list and understand that avantfax is basically broken until it can be rewritten but the developer hasn't, and probably won't, have time to fix/rewrite avantfax over the next few months. The purpose of the post here is to ask:
(1) are there any users of avantfax that have patched avantfax to work with php 5.4+;
(2) any interest in doing so?
If there is anyone fluent in php that is interested, I'm interested in helping and have a couple of installs to test with. However, I don't have the time or php fluency required to shoulder the rewrite.
What exact errors/warnings are you getting ? after which PHP version did the app stopped working ? -- Cristian "I don't know the key to success, but the key to failure is trying to please everybody." -- To unsubscribe, e-mail: opensuse+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse+owner@opensuse.org
On 08/20/2014 06:38 PM, Cristian Rodríguez wrote:
El 20/08/14 a las #4, David C. Rankin escribió:
All,
The elegant fax handling solution of hylafax/avantfax will not work with php >= 5.4. I have discussed the issue with the developer on the hylafax list and understand that avantfax is basically broken until it can be rewritten but the developer hasn't, and probably won't, have time to fix/rewrite avantfax over the next few months. The purpose of the post here is to ask:
(1) are there any users of avantfax that have patched avantfax to work with php 5.4+;
(2) any interest in doing so?
If there is anyone fluent in php that is interested, I'm interested in helping and have a couple of installs to test with. However, I don't have the time or php fluency required to shoulder the rewrite.
What exact errors/warnings are you getting ? after which PHP version did the app stopped working ?
Christian, I'll put together a summary of my communications with David Mimms, but the crux of the issue was that login no longer provided access to the application. The error from the apache log was: [Fri Jan 10 18:47:47 2014] [error] [client 192.168.6.102] PHP Fatal error: Class 'Smarty' not found in /srv/http/avantfax/includes/classes.php on line 30 The problem was with autoloading smarty classes includes. The developer suggested several fixes, and the login problem has been partially solved. I've put up the latest patch to the autoload function in classes.php as a text file on my sever: [7.6k] http://www.3111skyline.com/dl/bugs/openSuSE/131/classes.php.txt The latest autoload function was: function __autoload($class_name) { if (file_exists(AVFX_LIB_DIR.$class_name.'.php')) require_once $class_name.'.php'; } The problem was first noted on update of php 5.4.4-1 -> 5.4.5-1. While login is now possible, there are a number of additional warnings that are produced that need to be addressed. The warnings/error have to do with the remaining smarty includes that are handled by the autoload function. I'll create a summary of the current errors/warnings in a follow-up post. Avantfax is an incredible web interface to hylafax (auto tiff->pdf conversion, inbound/outbound fax management, e-mail fax notification, fax pdf/tiff download, reply, etc...). It has been a critical part of my work flow for quite some time. It needs update to insure it is compatible (and secure) with current php. That will provide an upgrade path when my 11.0 fax server finally croaks. :) I'll get the summary together in a day or so. -- David C. Rankin, J.D.,P.E. -- To unsubscribe, e-mail: opensuse+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse+owner@opensuse.org
El 21/08/14 a las #4, David C. Rankin escribió: m/dl/bugs/openSuSE/131/classes.php.txt
The latest autoload function was:
function __autoload($class_name) { if (file_exists(AVFX_LIB_DIR.$class_name.'.php')) require_once $class_name.'.php'; }
The problem was first noted on update of php 5.4.4-1 -> 5.4.5-1.
So, the autoloader changed behaviour in a minor version update ? hrmm..that smells bad..the original __autoload is : function __autoload($class_name) { if (file_exists($class_name.'.php')) require_once $class_name . '.php'; } Well..that's some very fishy code there ;-) Hrmm.. what's the current working directory value at the time __autoload is executed .. ? ( getcwd() ) hint: that's some unspecified behaviour :-) It might have changed to the CWD of the file actually requesting the class to be autoloaded instead of the CWD of the file itself.. or it is SERVER_ROOT like in __destruct() .. -- Cristian "I don't know the key to success, but the key to failure is trying to please everybody." -- To unsubscribe, e-mail: opensuse+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse+owner@opensuse.org
On 08/21/2014 01:13 PM, Cristian Rodríguez wrote:
El 21/08/14 a las #4, David C. Rankin escribió: m/dl/bugs/openSuSE/131/classes.php.txt
The latest autoload function was:
function __autoload($class_name) { if (file_exists(AVFX_LIB_DIR.$class_name.'.php')) require_once $class_name.'.php'; }
The problem was first noted on update of php 5.4.4-1 -> 5.4.5-1.
So, the autoloader changed behaviour in a minor version update ? hrmm..that smells bad..the original __autoload is :
function __autoload($class_name) {
if (file_exists($class_name.'.php')) require_once $class_name . '.php'; }
Well..that's some very fishy code there ;-)
Hrmm.. what's the current working directory value at the time __autoload is executed .. ? ( getcwd() )
hint: that's some unspecified behaviour :-)
It might have changed to the CWD of the file actually requesting the class to be autoloaded instead of the CWD of the file itself.. or it is SERVER_ROOT like in __destruct() ..
Christian, The directory layout for avantfax is: /srv/http/avantfax index.php <snip> /admin /ajax /css /images /includes /includes/langs /includes/Smarty /includes/templates /js /tools The primary directories relevant to login and application framework are: /srv/http/avantfax index.php <snip> /admin /includes The very first call in index.php is: require_once 'includes/classes.php'; So I suspect the cwd is /srv/http/avantfax when autoload is called. Where/what would I look for to see if there was a further change of the working directory between the http call to index.php and the first call to classes.php in index.php? (if anywhere). If you don't already have a copy of avantfax, you can get it at: http://www.avantfax.com/download.php It is a small 700k download. I will have more time this weekend to find all my notes and get a fresh set of logs. I'll also need to update another box to current php 5.5.14 to make sure I'm not missing any errors that are waiting in the new version. Thanks for looking at this. -- David C. Rankin, J.D.,P.E. -- To unsubscribe, e-mail: opensuse+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse+owner@opensuse.org
participants (2)
-
Cristian Rodríguez -
David C. Rankin