Allow editing values of SELECT query Result about heidisql

[ansgarbecker]

Quite dangerous, this "Use All (selected) columns" button. You probably update more than one row then. Worst case would be you overwrite an important setting with that, and you even don't know you did it.

Is that "Use All columns" really something HeidiSQL should provide?

[ansgarbecker]

This is already possible. The only requirement is that HeidiSQL is able to determine the table name of the result, and that all columns derive from a single table.

[Eddcapone]

@ansgarbecker I can't confirm it. It knows the name but I still can't change data:

[ansgarbecker]

Ok, then please select the primary key column(s) as well.

[Eddcapone]

Yes, that workaround works. But it would consume less time, if I could just change it without having to figure out the primary key column(s) first and changing my query accordingly. Another problem is, that people need to have that knowledge, otherwise they will think it is a bug / missing feature, like me...

[ansgarbecker]

You say in DBeaver it works. How does it work there in such a situation? How does an UPDATE look like when you edit a value there?

[Eddcapone]

Well, I just execute my SELECT e.g.

SELECT path, value FROM
core_config_data 

Then I can simply edit the entries and click on save at the end.

[ansgarbecker]

My question was how the UPDATE query looks like, after you edited an entry. Can you see that somewhere in the app, perhaps in a log panel?
I can only tell you with these both column names, the UPDATE would break your data, as this Magento table has additional columns in its unique key which are missing here.

[Eddcapone]

I see. Initially this window popped up:

I clicked on Use All Columns. Then after saving it executed these queries:

UPDATE company_2024.core_config_data
    SET value='http://company.vm/xy'
WHERE `path`='web/secure/base_url' AND value='http://company.vm/x'

And then a SELECT afterwards:

SELECT path, value FROM core_config_data WHERE (`path` = 'web/secure/base_url') AND (value = 'http://company.vm/xy')

[Eddcapone]

Im not sure about that. Maybe you could name the button "Use All Colums (use at your own risk)" or you omit it completly.

[ansgarbecker]

I'm against such an addition. It leads to data loss. In my experience, even if a dialog says "use at your own risk", it's something many users don't understand and/or don't read.

But I just made the warning message visible in the log in such cases: