We selected bug #2485. This is an issue either when a user first signs up for a Sugar user account, or when a user updates his/her user name in the settings. Greenfeld created this ticket because he believes the user name should be trimmed if there are leading or trailing spaces.
We found the bug by emailing the developers' mailing list and simply asking. At first we had tried a search in the code, but our search returned many results, and we didn't really have a good idea of what words to search for. It was much easier just to ask, and the developers were very responsive.
It was fairly easy to fix the bug in the code once we found it (in the two different files). Then we rebuilt the project and ran it to make sure it changed what we were intending.
Although our bug seemed straightforward and easy to fix, the contributors on the mailing list brought up some issues and potential negatives effects that this fix might cause. We think that there is a compromise for fixing this bug (i.e., in the initial setting but not in the update settings section). In order to test the bug, we'll need to learn more about creating a user account with a "fancy name" since that is the feature that might be affected by changing the code.
No comments:
Post a Comment