[RC] Attachments in subfolders

[I-FR70]

Modification Name: Attachments in subfolders
Author: I-FR70

Modification Description: Saves the attachments in sub-folders instead of in only one folder. This means that the server works quickly finding the attachments and the FTP backup will be facilitated.

Modification Version: RC 0.9.9

Modification Download: http://www.rambaudi.net/attachments_in_subfolders.zip

[I-FR70]

As I wrote in this topic:

http://www.phpbb.com/community/viewtopi ... &t=2130165

I'm searching someone who deeply knows the PHPBB core, to check my MOD and verify if I forgot to apply my modification to some file that handles the attachments...

Thank you in advance.

[I-FR70]

Sorry, the URL was wrong... I've fixed it.

[4_seven]

similar to this or different and with a new, selfprogrammed code-base?

[I-FR70]

I was not aware of the MOD you linked... my MOD is a new, selfprogrammed code-base.

I don't think that is similar, even though both the 2 download links within the topic you've linked are related to the same file, that contains only the two "Included files", so I can't check the code... but reading the description, the method I've used to create the subfolders is different: is based on the attachment ID.

[4_seven]

I have take a closer look now. It's a good work and also a very important Mod.

Some little things yet:
File_Exists() < Functions should always be written lowercase
PHP/Html Mix in One File < Should have phpbb3 PHP/Template-Syntax** (*.php > *.html)
$_GET < Should be request_var() (if possible)
mysql_select_db < Use always phpBB3 dbal (if possible),
** so there is also no need, to add db_name, db_password etc to your main php

But i know you can fix that, so congrats and good luck with your further developement

[I-FR70]

4_seven, thank you very much for checking the MOD

My biggest concern was to forget to modify some important file... read your comments make me relieved.

About your comments:

File_Exists() < Functions should always be written lowercase

Yes... true... I have the bad habit to use the uppercase for functions too... ... but now I fixed the code.

PHP/Html Mix in One File < Should have phpbb3 PHP/Template-Syntax** (*.php > *.html)
$_GET < Should be request_var() (if possible)
mysql_select_db < Use always phpBB3 dbal (if possible),
** so there is also no need, to add db_name, db_password etc to your main php

These 3 comments are related to the file sposta_allegati_nelle_subdir.php that I wrote outside of PHPBB3 because I know nothing of PHPBB3 core. I don't know to write an embedded file from scratch, so this was the shortest and easiest way for me.

But now I have take a quick look to the interesting document you've linked to me and I'll try to rewrite the file in the right way.

Again thank you very much for the time you've spent to check the MOD.

[I-FR70]

OK, I've update the MOD according the 4_seven's suggestions (on the first post).

If someone want to give me any other suggestion or comment, I'll be delighted to get it.

[4_seven]

Looks much better now

In all cases avoid the using of Super- or Server-Globals, like $_GET or $_SERVER * (if possible anyway).
In all cases try to use genuine phpBB3-Functions, which are global.
And if some is not global out of the box, than try to make them so
All other things code as you are, looks like a good style.

* so may change your

function current_page_url()

to this already present phpBB3 function

Code: Select all

/**
* Generate board url (example: http://www.example.com/phpBB)
* @param bool $without_script_path if set to true the script path gets not appended (example: http://www.example.com)
*/
function generate_board_url($without_script_path = false)

you find it in includes/functions.php (~line 2249)

This is an example, so take a look on other things too.

Here you have some different stuff from the first link, may for later usage..
http://wiki.phpbb.com/Viewtopic_Template_Variables

[I-FR70]

Thank you very much again, 4_seven.

In all cases avoid the using of $_GET or $_SERVER * (if possible anyway).

I've tried to avoid the use of $_SERVER, but I didn't found any other option ... but now I think I can avoid the use of the server variables because I can use the function you've suggested to me: generate_board_url.

I'll try to do this tomorrow morning.

I’m infinitely grateful to you for these tips...

[I-FR70]

OK, I cleaned the code using the PHPBB3 framework function generate_board_url() instead of my function current_page_url().

I updated the link on the first topic with the last MOD version 0.9.92.

Any other comment?

[4_seven]

@ a first glance it looks good now, so await more user-feedback.
Congrats for working out this awesome mod

[Peter77sx]

Hello, does this create the folder based on a user's ID?

[4_seven]

Seems to be No. Best thing (for me) is, to generate link-structure like this files/user_id/date/*.* like in jQuery [img] Upload.

[I-FR70]

@ a first glance it looks good now, so await more user-feedback.
Congrats for working out this awesome mod

Thank you to you for all comments and suggestions you gave to me.

Now I know the 1% of PHPBB3 framework...