[Phpmyadmin-devel] Proposed fix for bug #3464377

Alex UKF alex.ukf at gmail.com
Mon Jan 23 17:51:21 CET 2012

Hello phpmyadmins,

I am new around here, and I would like to submit a *small bug fix for [0]*.

I have read some pages on the website [1], wiki [2] and took a look at
the patch tracker [3]. I understand that I have to either submit a patch
( created with git format-patch, or with git diff between two branches),
or I can post the link to the branch solving the bug on my forked
phpMyAdmin repo [4].

I did not add a patch on the tracker, because I am not sure if that is
the good way to do it, but if I were to do that, I would probably classify
it as "Category -> Debugging", and "Group -> Finished. Needs basic
tests". Is that ok?

The other option would be to post a comment on the bug tracker [0]
with a link to the branch containing *my proposed fix on my forked *
*PMA repo [5]*. But on my 3464377 branch, I made more commits to
reach the final desired code : a few test commits, then the actual
code fix commit + a white space delete, then a coding style commit
and finally an undo white space deletion commit :). And I'm not sure
if that's ok. Should I try and merge all those commits into one somehow?
So just in case my repo is not good to merge from,* I copied the final *
*diff on pastebin [6]*.

And also, I edited the code in netbeans, so I hope there are no other
differences in the white spaces, or any other coding style related issues,
than the ones I spotted. I think that it should be ok, but again, please
correct me if I am wrong. And, of course, any comments about the
actual fix are more than welcome.

[0] *
[1] http://www.phpmyadmin.net/home_page/improve.php
[2] http://wiki.phpmyadmin.net/pma/Development
[3] https://sourceforge.net/tracker/?func=add&group_id=23067&atid=377410
[4] http://repo.or.cz/w/phpmyadmin/alexukf.git
*[5] http://repo.or.cz/w/phpmyadmin/alexukf.git/shortlog/refs/heads/3464377*
*[6] http://pastebin.com/g0CGrhZv*

Kind regards,
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20120123/637a03d5/attachment.html>

More information about the Developers mailing list