-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #17398 - Fixing clicking on JSON
columns triggers update
query
#17976
Conversation
Codecov ReportBase: 47.22% // Head: 47.73% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## QA_5_2 #17976 +/- ##
============================================
+ Coverage 47.22% 47.73% +0.50%
Complexity 16825 16825
============================================
Files 602 602
Lines 71576 71576
============================================
+ Hits 33805 34164 +359
+ Misses 37771 37412 -359
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
…e query it also fixes the issue where the JSON columns are saved with spaces and new lines, after being updated from the UI. Fixes: phpmyadmin#17398 Signed-off-by: iifawzi <iifawzie@gmail.com>
… JS Function Signed-off-by: iifawzi <iifawzie@gmail.com>
b8c4e7d
to
cb6012c
Compare
Signed-off-by: iifawzi <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
7fe40f5
to
b860117
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
Description
Hi, This PR is fixing the two issues mentioned at #17398
1- An
update
query is executed when clicking on theJSON
columns2- The
JSON
columns values are saved with unneeded spaces and lines when updatedScreen.Recording.2022-12-19.at.2.23.13.AM.mov
Fixes #17398