On 03/08/2012 19:48, Marc Delisle wrote:
Rouslan Placella a écrit :
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 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?
I only meant that you two guys should talk to each other, especially if you already have changes in your tree about the operations stuff.
It's up to him if he wants to merge these two dialogs; it was not planned in his schedule.
Fair enough, as he said that he already did refactoring work on these forms, I think that I'll just write a dedicated ajax handler for them as to avoid further merge conflicts. I'll look into this issue once again after all the merges, when the table operations code will ve stable.