Store images by date folders

[Stepashka]

Need help with the code.
I modified the script a little to store images by date folder, everything work fine except the images/thumbs/
the folder is created but nothing is uploaded to there

Here's what i did:

in condig.php

$folderdate = date("d.m.y");

$config['folder_images'] = 'images/' . $folderdate;
$config['folder_thumbs'] = 'images/thumbs/' . $folderdate;

in class.upload.php

public function process()
{

if (!is_dir('$img_upload_path')) {
mkdir($img_upload_path , 0777);
chmod($img_upload_path, 0777);
}

if (!is_dir('$thumb_upload_path')) {
mkdir($thumb_upload_path , 0777);
chmod($thumb_upload_path , 0777);
}

 

[Danny.Domb]

I don't know if it will change anything, but here some correction to your code:

First of all, you should activate the error_reporting in chevereto configs, so if you have an error, you will see it

find

$config['error_reporting'] = false; // Values: true|false

replace with

$config['error_reporting'] = true; // Values: true|false

Chevereto is always ending path variables with an / so... in config you must do the same...

$config['folder_images'] = 'images/' . $folderdate . '/';
$config['folder_thumbs'] = 'images/thumbs/' . $folderdate . '/';

// a php var should not be between '' it can be between "", but it is not recommended, only text should be. 
// you should be using file_exists instead of is_dir (doesn't change anything... but using the rights functions is always better)
// also $img_upload_path and thumb... are class variables, you can not access them using the normal way
// you need to put $this-> before to specify i'm using this class, and then the variable name without the ($)
if (!file_exists($this->img_upload_path)) {

    //mkdir already chmod the directory when creating it. 
    // the third parameters tell mkdir to create the directories recursively instead of stopping if it doesnt find one.
    mkdir($this->img_upload_path , 0755, true);
}

if (!file_exists($this->thumb_upload_path)) {
    // giving the permissions set 0777 is not recommended for security reasons, listing and downloading all images of your folder 
    //become much easier for an hacker
    mkdir($this->thumb_upload_path , 0755, true);
}

 

[Stepashka]

Than it's comes out with double splashes.


The code below fixed the thumbs isuue. Thanks for the detailed answer.

// a php var should not be between '' it can be between "", but it is not recommended, only text should be.
// you should be using file_exists instead of is_dir (doesn't change anything... but using the rights functions is always better)
// also $img_upload_path and thumb... are class variables, you can not access them using the normal way
// you need to put $this-> before to specify i'm using this class, and then the variable name without the ($)
if (!file_exists($this->img_upload_path)) {

//mkdir already chmod the directory when creating it.
// the third parameters tell mkdir to create the directories recursively instead of stopping if it doesnt find one.
mkdir($this->img_upload_path , 0755, true);
}

if (!file_exists($this->thumb_upload_path)) {
// giving the permissions set 0777 is not recommended for security reasons, listing and downloading all images of your folder
//become much easier for an hacker
mkdir($this->thumb_upload_path , 0755, true);
}

 

[mecho]

sounds really an good idea . is it possible to have images on Monthly folders ? if yes , would anybody share the whole code step by step ?

Regards

 

[Rodolfo]

Don't worry, 2.1 will store everything in datefolders 😉

 

[mecho]

sounds good Rodolfo , when do u think it will be release ?

 

[Rodolfo]

March

 

[CMedia_de]

March

sounds very good! 😀