ForoFyL/forofyl.com.ar — pull request #1 by Canta (2013-11-07 18:01:52+00:00)
Title: Added a default database user for use in development

I've added a database user ("forofyl_admin") to the database scripts, in order to avoid common mistakes as commiting our own custom db connection config, or bad practices like using root everywhere.

I've also added a "drop database if exists" statement to the database script, so we can just run the script from command line every time we want to restart the database (i do that all the time).

NOTE: As there's no "drop user if exists" in mysql (see http://bugs.mysql.com/bug.php?id=15287), and given that the script will throw an error when trying to drop an unexistent user OR when trying to create an already existent one, i've added that logic to another script, which should be executed before the one with the database. So, i also named the script, adding an order number prefix. For example, "00-pre_database.sql","01-database.sql", etc. This policy is taken from debian's scripts ordering policy.

This is also my first test of pulling requests. Never done it before. So, i took the liberty to make a somewhat aggresive change like adding a second database script, in order to see what happens when the pull request is accepted or not.