Sequel Pro makes wrong assumption about table content

vieten / sequel-pro

Automatically exported from code.google.com/p/sequel-pro

Other

0 stars 0 forks source link

Sequel Pro makes wrong assumption about table content #1166

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

1. Create a table and insert one row
2. Use mysql client to insert another row
3. Without refreshing select the one row added in SP and press Delete.

=> SP will create a Query like "DELETE FROM tbl1" actually dropping both rows!
=> SP will also not inform the user that is has deleted more rows than should 
exist in the table!

Original issue reported on code.google.com by schlabbe...@gmail.com on 26 Aug 2011 at 10:37

GoogleCodeExporter commented 9 years ago

Original comment by stuart02 on 30 Aug 2011 at 9:01

Added labels: Milestone-Release0.9

GoogleCodeExporter commented 9 years ago

This has been addressed by checking that at least 50 rows are selected before 
using the remove-all-rows codepath, which should ensure the user is attempting 
to empty the table rather than delete just the selected rows.  Committed in 
r3416.

Original comment by rowanb@gmail.com on 7 Sep 2011 at 9:40

Changed state: Fixed