Two bugs for Newbie Devs

[ninetydegrees]

http://bugs.dwscoalition.org/show_bug.cgi?id=4151: easy fix is literally changing one line in core2.s2 as explained in the comments. How's that for a non-scary style thing?

http://bugs.dwscoalition.org/show_bug.cgi?id=5160: cleanup! If you're into time-travelling removing a few *very* old lines of code this one's for you.

Comments

[kaberett]

Thank you!

Dear everyone: super-willing to go into more detail on either of these. <3

[poulpette]

I think I might be able to do 4151 and give a go at the harder fix after fixing the id thing first. I have a vague idea how to go about it?

Though I'll probably need a bit of hand holding with Git (I don't know why but the thing scares me to bits).

Caveat: probably won't have time to work on this before, uh, this coming firday so if anyone else wants it, have at it.

[kaberett]

I was absolutely terrified of git as well - took me 8 months to migrate my hack - and I now mostly swear at it good-naturedly rather than in actual hatred. Would be delighted to do hand-holding, and by next Friday I'll have got rid of my houseguest so will have the time to do so :-)

Regarding bug 4151

[abhi]

Hello guys
I have began working on 4151(easy fix). This is my first bug. Hope, I will be able go through the whole process in correct manner :)

Re: Regarding bug 4151

[denise]

Good luck! If you get stuck, feel free to ask :)

Re: Regarding bug 4151

[ninetydegrees]

Hey abhi! That's awesome! Thanks!