WSOD on new client create submit

[TGates71]

When you add a new client and hit 'save' the WSOD appears yet the user is still added. Seems to be OS non specific.

[danclarkau]

yeh I get that too..

[Caffe1neAdd1ct]

This one reminds me of a bug quite a while back... looks like we have a regression..

[Caffe1neAdd1ct]

Only seems to happen when the email user tickbox is selected.

[Caffe1neAdd1ct]

[Tue Jul 29 02:53:10 2014] [error] [client xx.xx.xx.xx] PHP Fatal error: Class 'ui_templateparser' not found in /etc/zpanel/panel/inc/init.inc.php on line 143, referer: http://xx.xx.xx.xx/?module=manage_clients

[danclarkau]

This is on a brand new installation on Ubuntu..

I created another user just now, the WSOD page URL is...

http://x.x.x.x/?module=manage_clients&action=CreateClient

I can't see anything in the logs?

Cheers
Dan

On Tue, Jul 29, 2014 at 12:53 PM, Kevin Andrews [email protected]
wrote:

[Tue Jul 29 02:53:10 2014] [error] [client xx.xx.xx.xx] PHP Fatal error:
Class 'ui_templateparser' not found in /etc/zpanel/panel/inc/init.inc.php
on line 143, referer: http://xx.xx.xx.xx/?module=manage_clients

—
Reply to this email directly or view it on GitHub
#166 (comment).

Kind Regards

Dan Clark

43 Lord Street
Gladstone, QLD
Australia 4680
AU: +61 4 8719 5948
NZ: +64 3 974 8267
SKYPE: danclar
Google Talk: [email protected]

[TGates71]

Confirmed, I created a new account on my windows machine, no email send, and works just fine. Only happens if send email is checked.

[Caffe1neAdd1ct]

Its definitely failing on including a few files from dryden framework... if i manually add the files it doesn't parse the template correctly and i get a load of placeholders instead of text...

Must be a bug stopping the autoloading of required files after the post client create action...

[Caffe1neAdd1ct]

Blimey that was a hard bug to find... had to fire up xdebug in the end.

The PHPMailerAutoload class (which is only included when sending emails is taking over the autoloading of classes!

We need to register our autoloader differently i think...

[Caffe1neAdd1ct]

To fix we need to change:

function __autoload($class_name)
{
    $path = 'dryden/' . str_replace('_', '/', $class_name) . '.class.php';
    if (file_exists($path)) {
        require_once $path;
    }
}

to

function __autoload($class_name)
{
    $path = 'dryden/' . str_replace('_', '/', $class_name) . '.class.php';
    if (file_exists($path)) {
        require_once $path;
    }
}

spl_autoload_register('__autoload');

in dryden/loader.inc.php

[Caffe1neAdd1ct]

#239