Learning through bug fixing

Learning through bug fixing

por Jun Yamog -
Número de respostas: 5
Hi,

I am new to moodle, I am trying to learn moodle.  I figured the best way for me to learn is to help on some bugs.

I picked this bug for me to learn some stuff about moodle

http://moodle.org/bugs/bug.php?op=show&bugid=4122

I have created a patch for this, Martin Langhoff told me the patch is fine.  I have tested it to run on both pgsql and mysql, clean install and running the upgrade scripts.

Do you guys think the patch is ok?

Jun
Média das avaliações:  -
Em resposta à Jun Yamog

Re: Learning through bug fixing

por Tim Hunt -
Imagem de Core developers Imagem de Documentation writers Imagem de Particularly helpful Moodlers Imagem de Peer reviewers Imagem de Plugin developers
Minor nit:

In Lines 10 and 11 of your patch, you are using tabs instead of spaces to indent the lines. Moodle coding standard says use spaces.

Ditto, lines 41-45.

It's not important, but do you have any particular reason for setting $version to 2006022001 rather than 2006022000 in version.php?
Em resposta à Tim Hunt

Re: Learning through bug fixing

por Jun Yamog -
Hi Tim,

Sorry about the tabs, I must have set emacs to use spaces after making the patch.  I set it 2006022001 since I revised my upgrade script, anyway its internal to me.  I have now set the version to 2006022000 in version.php and the respective upgrade scripts.

Reattached another patch file, I will commit maybe tomorrow if the patch is ok.  As well as add a comment on the bugs.moodle.org

Jun
Em resposta à Jun Yamog

Re: Learning through bug fixing

por Tim Hunt -
Imagem de Core developers Imagem de Documentation writers Imagem de Particularly helpful Moodlers Imagem de Peer reviewers Imagem de Plugin developers
The patch looks fine to me now. Thanks for taking such care over the code you contribute.
Em resposta à Tim Hunt

Re: Learning through bug fixing

por Jun Yamog -
Thanks Tim.  I have updated the bug report to include a link here and attached the patch.  I will just commit next week at catalyst to allow more time.
Em resposta à Jun Yamog

Re: Learning through bug fixing

por Martin Dougiamas -
Imagem de Core developers Imagem de Documentation writers Imagem de Moodle HQ Imagem de Particularly helpful Moodlers Imagem de Plugin developers Imagem de Testers
Minor incremental bumps in the version number like this is no problem, that's why we have those last two digits.