Re: Change maximum number of backups kept!!!

by Naaman Fallouh.  

Thanks guys,

I'm totally aware of your fears of a bad user behavior in the settings UI, but I thing it could be avoided by:

• A special validation on a text box, e.g. allow digits less than 50.
• OR, keep it as drop down list but narrowing the seed range, e.g. 1 5 10 15 20 25 ...

And about hacking the code, the list is located in the file moodle/admin/settings/courses.php line 251:

$maxkeptoptions = array(
 0 => new lang_string('all'), 1 => '1',
 2 => '2',
 5 => '5',
 10 => '10',
 20 => '20',
 30 => '30',
 40 => '40',
 50 => '50',
 100 => '100',
 200 => '200',
 300 => '300',
 400 => '400',
 500 => '500');

But I really don't like the idea of hacking the core code.

Now, what do you think of posting a request in the tracker for this issue, as Emma suggested?

Best regards,