[Phpmyadmin-devel] tbl_operation - Bug Fixing - Developers

List overview All Threads
Download

newer

[Phpmyadmin-devel] tbl_operation - Bug Fixing

older

[Phpmyadmin-devel] Issues with...

[Phpmyadmin-devel] Bug with...

Thilina Buddika Abeyrathna

15 Aug 2012 15 Aug '12

8:24 p.m.

Hi Marc, I'm looking [1] bug in the table operation page, I 'git checkout' to very old commit and check the same bug in there, But still this bug was there, I found that, first bad commit was 4d2fcd910bc7e07114fcc76f55705c1a6e8c685e (This is not a my commit) I can't figure out where is this error happen. [1] - http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23… -- Regards. Thilina Abeyrathna Gtalk : thilinaabeyrathna skype: thilinabuddika88 thilinaa.wordpress.com

Attachments:

attachment.html (text/html — 1.1 KB)

Show replies by date

Marc Delisle

15 Aug 15 Aug

9:25 p.m.

Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit :

...

Thilina, please use "git bisect" [2] to find the problematic commits. I used git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad commit commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com> Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2] http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit -- Marc Delisle http://infomarc.info

Thilina Buddika Abeyrathna

9:27 p.m.

On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle <marc(a)infomarc.info> wrote:

...

Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit :

Hi Marc, I'm looking [1] bug in the table operation page, I 'git checkout' to very old commit and check the same bug in there, But still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23… Thilina, please use "git bisect" [2] to find the problematic commits. I used git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad commit commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com> Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2] http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit

Thank you Marc. -- Regards. Thilina Abeyrathna Gtalk : thilinaabeyrathna skype: thilinabuddika88 thilinaa.wordpress.com

Thilina Buddika Abeyrathna

16 Aug 16 Aug

3:06 p.m.

On Thu, Aug 16, 2012 at 2:57 AM, Thilina Buddika Abeyrathna < thilinaabeyrathna(a)gmail.com> wrote:

...

On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle <marc(a)infomarc.info> wrote:

Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit :

Hi Marc, I'm looking [1] bug in the table operation page, I 'git checkout' to very old commit and check the same bug in there, But still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

Thank you Marc. Hi Rouslan,

Can you look at this error? -- Regards. Thilina Abeyrathna Gtalk : thilinaabeyrathna skype: thilinabuddika88 thilinaa.wordpress.com

Rouslan Placella

7:46 p.m.

On 16/08/2012 17:06, Thilina Buddika Abeyrathna wrote:

...

On Thu, Aug 16, 2012 at 2:57 AM, Thilina Buddika Abeyrathna <thilinaabeyrathna(a)gmail.com <mailto:thilinaabeyrathna@gmail.com>> wrote: On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle <marc(a)infomarc.info <mailto:marc@infomarc.info>> wrote: Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit :

Hi Marc, I'm looking [1] bug in the table operation page, I 'git checkout' to very old commit and check the same bug in

there, But

still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23… Thilina, please use "git bisect" [2] to find the problematic commits. I used git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad commit commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com <mailto:rouslan@placella.com>> Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2] http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit Thank you Marc. Hi Rouslan, Can you look at this error?

Problem #1 is already fixed in my branch [0], so I guess that you can ignore it. Problem #2 has nothing to do with my code. Also keep in mind that the issue comes up when changing from innodb to myisam, but not the other way around. Here's the results of a bisect: 83401195710de736dc6217f885313b5dad9071ae is the first bad commit commit 83401195710de736dc6217f885313b5dad9071ae Author: Thilina Buddika <thilinaabeyrathna(a)gmail.com> Date: Wed Aug 1 13:52:49 2012 +0530 further refactoring for PMA_getTableOptionFieldset() :040000 040000 f7dba249f1ec47de37b26a2df7fc4562dd717b80 7b7945eceffca79186916f9ed7e4e30f0d4cb0d7 M libraries Bye, Rouslan [0]: https://github.com/roccivic/phpmyadmin/tree/frames-removal

Thilina Buddika Abeyrathna

7:56 p.m.

On Fri, Aug 17, 2012 at 1:16 AM, Rouslan Placella <rouslan(a)placella.com>wrote;wrote:

...

On 16/08/2012 17:06, Thilina Buddika Abeyrathna wrote:

On Thu, Aug 16, 2012 at 2:57 AM, Thilina Buddika Abeyrathna <thilinaabeyrathna(a)gmail.com <mailto:thilinaabeyrathna@gmail.com>>

wrote:

On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle <marc(a)infomarc.info <mailto:marc@infomarc.info>> wrote: Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit :

Hi Marc, I'm looking [1] bug in the table operation page, I 'git checkout' to very old commit and check the same bug in

there, But

still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23…

Thilina, please use "git bisect" [2] to find the problematic commits. I

used

git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad commit commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com <mailto:rouslan@placella.com>> Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2]

http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit

Thank you Marc. Hi Rouslan, Can you look at this error?

Thanks Rouslan. -- Regards. Thilina Abeyrathna Gtalk : thilinaabeyrathna skype: thilinabuddika88 thilinaa.wordpress.com

Rouslan Placella

17 Aug 17 Aug

7:20 p.m.

On 16/08/2012 21:56, Thilina Buddika Abeyrathna wrote:

...

On Fri, Aug 17, 2012 at 1:16 AM, Rouslan Placella <rouslan(a)placella.com <mailto:rouslan@placella.com>> wrote: On 16/08/2012 17:06, Thilina Buddika Abeyrathna wrote:

On Thu, Aug 16, 2012 at 2:57 AM, Thilina Buddika Abeyrathna <thilinaabeyrathna(a)gmail.com <mailto:thilinaabeyrathna@gmail.com>

<mailto:thilinaabeyrathna@gmail.com <mailto:thilinaabeyrathna@gmail.com>>> wrote:

On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle

<marc(a)infomarc.info <mailto:marc@infomarc.info>

<mailto:marc@infomarc.info <mailto:marc@infomarc.info>>> wrote: Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit : > Hi Marc, > I'm looking [1] bug in the table operation page, > I 'git checkout' to very old commit and check the same

bug in

there, But

still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23…

Thilina, please use "git bisect" [2] to find the problematic

commits. I used

git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad

commit

commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com

<mailto:rouslan@placella.com>

<mailto:rouslan@placella.com <mailto:rouslan@placella.com>>> Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2]

http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit

Thank you Marc. Hi Rouslan, Can you look at this error?

Problem #1 is already fixed in my branch [0], so I guess that you can ignore it. Problem #2 has nothing to do with my code. Also keep in mind that the issue comes up when changing from innodb to myisam, but not the other way around. Here's the results of a bisect: 83401195710de736dc6217f885313b5dad9071ae is the first bad commit commit 83401195710de736dc6217f885313b5dad9071ae Author: Thilina Buddika <thilinaabeyrathna(a)gmail.com <mailto:thilinaabeyrathna@gmail.com>> Date: Wed Aug 1 13:52:49 2012 +0530 further refactoring for PMA_getTableOptionFieldset() :040000 040000 f7dba249f1ec47de37b26a2df7fc4562dd717b80 7b7945eceffca79186916f9ed7e4e30f0d4cb0d7 M libraries Bye, Rouslan [0]: https://github.com/roccivic/phpmyadmin/tree/frames-removal Problem 2# has been fixed. Thanks Rouslan.

Hi Thilina, Same problem in copy table form: https://github.com/phpmyadmin/phpmyadmin/blob/master/libraries/operations.l… Could you please do a quick grep to see if there are more instances of the same elsewhere? Lastly, IMO, the ternary operator operator does no favours to us in terms of code readability there. Bye, Rouslan

Thilina Buddika Abeyrathna

18 Aug 18 Aug

3:16 a.m.

On Sat, Aug 18, 2012 at 12:50 AM, Rouslan Placella <rouslan(a)placella.com>wrote;wrote:

...

On 16/08/2012 21:56, Thilina Buddika Abeyrathna wrote:

On Fri, Aug 17, 2012 at 1:16 AM, Rouslan Placella <rouslan(a)placella.com <mailto:rouslan@placella.com>> wrote: On 16/08/2012 17:06, Thilina Buddika Abeyrathna wrote:

On Thu, Aug 16, 2012 at 2:57 AM, Thilina Buddika Abeyrathna <thilinaabeyrathna(a)gmail.com <mailto:thilinaabeyrathna@gmail.com>

<mailto:thilinaabeyrathna@gmail.com <mailto:thilinaabeyrathna@gmail.com>>> wrote:

On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle

<marc(a)infomarc.info <mailto:marc@infomarc.info> > <mailto:marc@infomarc.info <mailto:marc@infomarc.info>>>

wrote:

Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit : > Hi Marc, > I'm looking [1] bug in the table operation page, > I 'git checkout' to very old commit and check the same

bug in

there, But

still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23…

Thilina, please use "git bisect" [2] to find the problematic

commits. I used

git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad

commit

commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com

<mailto:rouslan@placella.com>

<mailto:rouslan@placella.com <mailto:rouslan@placella.com

Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2]

http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit

Thank you Marc. Hi Rouslan, Can you look at this error?

Problem #1 is already fixed in my branch [0], so I guess that you can ignore it. Problem #2 has nothing to do with my code. Also keep in mind that the issue comes up when changing from innodb to myisam, but not the other way around. Here's the results of a bisect: 83401195710de736dc6217f885313b5dad9071ae is the first bad commit commit 83401195710de736dc6217f885313b5dad9071ae Author: Thilina Buddika <thilinaabeyrathna(a)gmail.com <mailto:thilinaabeyrathna@gmail.com>> Date: Wed Aug 1 13:52:49 2012 +0530 further refactoring for PMA_getTableOptionFieldset() :040000 040000 f7dba249f1ec47de37b26a2df7fc4562dd717b80 7b7945eceffca79186916f9ed7e4e30f0d4cb0d7 M libraries Bye, Rouslan [0]: https://github.com/roccivic/phpmyadmin/tree/frames-removal Problem 2# has been fixed. Thanks Rouslan.

Thanks Rouslan, I did the changes.

...

Lastly, IMO, the ternary operator operator does no favours to us in terms of code readability there.

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

Marc Delisle

16 Aug 16 Aug

8:01 p.m.

Rouslan Placella a écrit :

...

On 16/08/2012 17:06, Thilina Buddika Abeyrathna wrote:

Hi Marc, I'm looking [1] bug in the table operation page, I 'git checkout' to very old commit and check the same bug in

there, But

still this bug was there, I found that, first bad commit was

4d2fcd910bc7e07114fcc76f55705c1a6e8c685e

(This is not a my commit) I can't figure out where is this error happen. [1] -

http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23… Thilina, please use "git bisect" [2] to find the problematic commits. I used git bisect start HEAD QA_3_5 and found this: a16e723048940f87b4be4b4097d16c4376826f2c is the first bad commit commit a16e723048940f87b4be4b4097d16c4376826f2c Author: Rouslan Placella <rouslan(a)placella.com <mailto:rouslan@placella.com>> Date: Mon Jun 11 17:58:35 2012 +0100 Dropped PMA_ajaxResponse() [2] http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit Thank you Marc. Hi Rouslan, Can you look at this error?

Rouslan, sorry, when I wrote about the results of my git bisect, this was only intended for problem #1 of the bug report. Problem #2 is already fixed in Thilina's repo and upstream master.

...

83401195710de736dc6217f885313b5dad9071ae is the first bad commit commit 83401195710de736dc6217f885313b5dad9071ae Author: Thilina Buddika <thilinaabeyrathna(a)gmail.com> Date: Wed Aug 1 13:52:49 2012 +0530 further refactoring for PMA_getTableOptionFieldset() :040000 040000 f7dba249f1ec47de37b26a2df7fc4562dd717b80 7b7945eceffca79186916f9ed7e4e30f0d4cb0d7 M libraries Bye, Rouslan [0]: https://github.com/roccivic/phpmyadmin/tree/frames-removal ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel(a)lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel

-- Marc Delisle http://infomarc.info

4306

days inactive

4309

days old

developers@phpmyadmin.net

Manage subscription

8 comments

3 participants

tags (0)

participants (3)

Marc Delisle
Rouslan Placella
Thilina Buddika Abeyrathna