On Sat, Aug 4, 2012 at 3:00 PM, Rouslan Placella <rouslan@placella.com> wrote:
On 04/08/2012 01:39, Thilina Buddika Abeyrathna wrote:
>
>
> On Fri, Aug 3, 2012 at 10:24 PM, Rouslan Placella <rouslan@placella.com
> <mailto:rouslan@placella.com>> wrote:
>
>     On 03/08/2012 17:45, Rouslan Placella wrote:
>      > On 03/08/2012 17:43, Marc Delisle wrote:
>      >> Marc Delisle a écrit :
>      >>> Rouslan Placella a écrit :
>      >>>> On 03/08/2012 14:19, Marc Delisle wrote:
>      >>>>> Michal Čihař a écrit :
>      >>>>>> Hi
>      >>>>>>
>      >>>>>> Dne Fri, 03 Aug 2012 11:28:31 +0200
>      >>>>>> Rouslan Placella <rouslan@placella.com
>     <mailto:rouslan@placella.com>> napsal(a):
>      >>>>>>
>      >>>>>>> Quick question: why do we have two forms for renaming a
>     table on
>      >>>>>>> tbl_operations.php? Can one be dropped? Please see attached
>     screenshot
>      >>>>>>> for the location of forms.
>      >>>> Well, I have a dilemma about this form in my branch. With the
>     "rename"
>      >>>> functionality in the table options form, a dedicated ajax
>     handler is
>      >>>> necessary for it to work. Without, the form can be handled by
>     a generic
>      >>>> page loading mechanism, which means less code and a more
>     maintainable
>      >>>> solution.
>      >>>>
>      >>>>>> The difference is probably in implementation - in same
>     database, you
>      >>>>>> can use just RENAME, for moving data to other database, this
>     might
>      >>>>>> fail, so basically it's CREATE TABLE and INSERT .. SELECT.
>      >>>> I see, it sounds a bit more complicated than I was hoping for.
>      >>>>
>      >>>>>> Still it would be better to merge these and use RENAME when
>     possible
>      >>>>>> and fallback to complex method if needed.
>      >>>>> I'm not sure these should be merged. In the mind of some
>     users, renaming
>      >>>>> and moving are different actions.
>      >>>> Then we could call it "rename/move"...
>      >>>
>      >>> I would be happy with that. It would have the added benefit of
>     removing
>      >>> the rename table dialog from the table options fieldset, where
>     it does
>      >>> not really belong.
>      >>>
>      >> By the way, Thilina is refactoring db and tbl operations these
>     days...
>      >
>      > Sounds like one of us will have a few conflicts to resolve. I already
>      > have some committed changes in those areas. When does he expect to
>      > finish up with that?
>
>     Or did you mean asking Thilina to look into this?
>
> Hi,
> For the moment, I have done some refactoring for both rename table and
> move table operations in my tree,

Which repo/branch/commits? I don't see anything like that in
https://github.com/thilinaa/phpmyadmin/commits/master

Some commits, 
[1] - https://github.com/thilinaa/phpmyadmin/commit/d9933223977c52426edcfc9f558fc754baa6217b
[2] - https://github.com/thilinaa/phpmyadmin/commit/83401195710de736dc6217f885313b5dad9071ae

But some functions has changed for further refactoring.
 

--
Regards.

Thilina Abeyrathna
Gtalk : thilinaabeyrathna
skype: thilinabuddika88
thilinaa.wordpress.com