Yaay, my first accepted patch :)<br><div><div><br></div><div>Regarding the mentioned fix, I was actually also thinking that it </div><div>would be better to move the "Bookmarked SQL Query" section </div><div>into a separate form, so maybe I shall try doing that in the future.</div>
<div><br></div><div>Rouslan, thanks for the tip, I'm getting jiggy with git.. :P</div><div><br></div><div><br></div><div>Alex</div></div><div><br></div><div><div>// And btw, those *stars* in my first mail look pretty weird, </div>
<div>// I meant to bold the text from Gmail.</div></div><div><br><br><div class="gmail_quote">On Mon, Jan 23, 2012 at 8:28 PM, Rouslan Placella <span dir="ltr"><<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 23/01/12 17:52, Marc Delisle wrote:<br>
> Alex UKF a écrit :<br>
>> Hello phpmyadmins,<br>
>><br>
>> I am new around here, and I would like to submit a *small bug fix for [0]*.<br>
>><br>
>> I have read some pages on the website [1], wiki [2] and took a look at<br>
>> the patch tracker [3]. I understand that I have to either submit a patch<br>
>> ( created with git format-patch, or with git diff between two branches),<br>
>> or I can post the link to the branch solving the bug on my forked<br>
>> phpMyAdmin repo [4].<br>
>><br>
>> I did not add a patch on the tracker, because I am not sure if that is<br>
>> the good way to do it, but if I were to do that, I would probably classify<br>
>> it as "Category ->  Debugging", and "Group ->  Finished. Needs basic<br>
>> tests". Is that ok?<br>
>><br>
>> The other option would be to post a comment on the bug tracker [0]<br>
>> with a link to the branch containing *my proposed fix on my forked *<br>
>> *PMA repo [5]*. But on my 3464377 branch, I made more commits to<br>
>> reach the final desired code : a few test commits, then the actual<br>
>> code fix commit + a white space delete, then a coding style commit<br>
>> and finally an undo white space deletion commit :). And I'm not sure<br>
>> if that's ok. Should I try and merge all those commits into one somehow?<br>
<br>
</div>A way of "merging" several commits into one is using interactive git<br>
rebase. You just point rebase to your oldest commit and start squashing<br>
then. So if you had 5 commits on your branch you could run:<br>
<br>
git checkout mybranch<br>
git rebase -i HEAD~4<br>
<br>
Then "pick" your first commit and "squash" all the other ones in the<br>
interactive screen.<br>
<span class="HOEnZb"><font color="#888888"><br>
Rouslan<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
>> So just in case my repo is not good to merge from,* I copied the final *<br>
>> *diff on pastebin [6]*.<br>
>><br>
>> And also, I edited the code in netbeans, so I hope there are no other<br>
>> differences in the white spaces, or any other coding style related issues,<br>
>> than the ones I spotted. I think that it should be ok, but again, please<br>
>> correct me if I am wrong. And, of course, any comments about the<br>
>> actual fix are more than welcome.<br>
>><br>
>><br>
>> [0]<br>
>> *<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=3464377&group_id=23067&atid=377408" target="_blank">https://sourceforge.net/tracker/index.php?func=detail&aid=3464377&group_id=23067&atid=377408</a><br>

>> <<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=3464377&group_id=23067&atid=377408" target="_blank">https://sourceforge.net/tracker/index.php?func=detail&aid=3464377&group_id=23067&atid=377408</a>>*<br>

>> [1] <a href="http://www.phpmyadmin.net/home_page/improve.php" target="_blank">http://www.phpmyadmin.net/home_page/improve.php</a><br>
>> [2] <a href="http://wiki.phpmyadmin.net/pma/Development" target="_blank">http://wiki.phpmyadmin.net/pma/Development</a><br>
>> [3] <a href="https://sourceforge.net/tracker/?func=add&group_id=23067&atid=377410" target="_blank">https://sourceforge.net/tracker/?func=add&group_id=23067&atid=377410</a><br>
>> <<a href="https://sourceforge.net/tracker/?func=add&group_id=23067&atid=377410" target="_blank">https://sourceforge.net/tracker/?func=add&group_id=23067&atid=377410</a>><br>
>> [4] <a href="http://repo.or.cz/w/phpmyadmin/alexukf.git" target="_blank">http://repo.or.cz/w/phpmyadmin/alexukf.git</a><br>
>> *[5] <a href="http://repo.or.cz/w/phpmyadmin/alexukf.git/shortlog/refs/heads/3464377*" target="_blank">http://repo.or.cz/w/phpmyadmin/alexukf.git/shortlog/refs/heads/3464377*</a><br>
>> *[6] <a href="http://pastebin.com/g0CGrhZv*" target="_blank">http://pastebin.com/g0CGrhZv*</a><br>
>> *<br>
>> *<br>
>><br>
>> Kind regards,<br>
>> Alex<br>
><br>
> Hi Alex and welcome,<br>
><br>
> I was able to generate a final patch by adding your repository to my<br>
> local one and run this command:<br>
><br>
> git diff origin/master..alexukf/bug_3464377 js/sql.js<br>
><br>
> A long-term fix for this problem would probably be to avoid having one<br>
> big form for these two dialogs.<br>
><br>
> I am happy with the patch and will merge it for 3.5.0-beta1.<br>
><br>
<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">------------------------------------------------------------------------------<br>
Try before you buy = See our experts in action!<br>
The most comprehensive online learning library for Microsoft developers<br>
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,<br>
Metro Style Apps, more. Free future releases when you subscribe now!<br>
<a href="http://p.sf.net/sfu/learndevnow-dev2" target="_blank">http://p.sf.net/sfu/learndevnow-dev2</a><br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
</div></div></blockquote></div><br></div>