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
But some functions has changed for further refactoring.