← Back to team overview

phpdevshell team mailing list archive

[Bug 596888] Re: Manually overriding settings for filemanager class not currently possible

 

Is this still relevant?

-- 
Manually overriding settings for filemanager class not currently possible
https://bugs.launchpad.net/bugs/596888
You received this bug notification because you are a member of
PHPDevShell, which is subscribed to PHPDevShell.

Status in Open Source php gui type development framework.: New

Bug description:
The filemanager class automatically loads its settings from the PHPDS database when the auto_upload() function is called, but due to a bug, specific settings can not be supplied on a case by case scenario. One would need to do this for example when you need to create differently sized thumbnails depending on the groups of images that you want to create thumbnails for.

The reason for this problem is as follows:

File: /includes/filemanager.class.php  (revision 348)

The auto_upload() function performs the following statements from line 473 and onwards:
if (empty($this->setting)) {
    $this->setting = $db->get_settings(array('log_uploads' ,....
}

The problem is that $this->setting will always be empty as it is never defined as a member of the class in the first place. The $setting member needs to defined as a public member in the first section of the class, i.e.: 

public $setting = array();

This will solve the above mentioned problem since one would then be able to override settings as need be.