Admin Tool Kit

[EXreaction]

To not repeat a topic...
http://www.phpbb.com/community/viewtopi ... &t=1089145

[igorw]

Very interesting, i still need to test this!

Some things i encountered when peeking into the files.

index.php
Use $phpEx:

Code: Select all

if (substr($file, -(strlen($phpEx) + 1)) != '.' . $phpEx || $file == 'tutorial.php')

Use semantic variable names:

Code: Select all

$t = '';

root/atk/tools/fix_modules.php
I don't like how this is just in the file like that:

Code: Select all

if (!defined('IN_PHPBB'))
{
   exit;
}

// Add the required language file
$user->add_lang('atk/tools/fix_modules');

class fix_modules

root/atk/tools/make_founder.php
utf_normalize_nfc()?

Code: Select all

$user_req = request_var('user_to_founder', '', true);

I really like the idea of this and the way it's structured. I may even contribute something myself, for example something like "merge users". Nice work nathan, keep it up!

[EXreaction]

I fixed all but the language addition.

That is just the best place for it unless I were to have it automatically load all language files in atk/tools/ (which I may do, I am not sure yet). Otherwise it has to be repeated in the 3 functions (and can possibly be called twice).

What would be great is to have a Developer tool for auto creating install instructions or something. You could get the path to the unedited files by having an input box for the relative url.

Perhaps a tool to run SQL queries as well.

[igorw]

Okay, i've tested it, and i't a nice tool.

merge_groups gives me a notice. reparse_bbcodes gives me "BBCODE_REPARSE_CONFIRM".

[EXreaction]

Yep, the reparse bbcode needs to be looked over. It's on my to-do list once I get some time.

I am not sure about the merge groups, I haven't really looked at it. Could you bring it up in the topic at phpbb.com?

[Erik Frèrejean]

I've just fixed the problem in the merge group tool.

[aminetm]

that's very good