Skip to content
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

"Show this query here again" checkbox does not work #14144

Closed
barkermn01 opened this issue Mar 29, 2018 · 4 comments · Fixed by #14166
Closed

"Show this query here again" checkbox does not work #14144

barkermn01 opened this issue Mar 29, 2018 · 4 comments · Fixed by #14166
Assignees
Labels
Bug A problem or regression with an existing feature
Milestone

Comments

@barkermn01
Copy link

Steps to reproduce

  1. Login to PHP MyAdmin
  2. Go to SQL Tab Write a none default query E.G select * from table where 2=2
  3. Uncheck the "Show this query here again"
  4. Submit query
  5. go to SQL Tab again

Expected behavior

The "Show this query here again" checkbox should remain unchecked and old query should have been replaced with the default select * from table where 1

Actual behavior

It does not save the setting of "Show this query here again" checkbox so it's re-ticked and due to this auto-populates query box with the last run query

Server configuration

Operating system:
Ubuntu 16.10

Web server:
Apache 2

Database:
MariaDB 10.2

PHP version:
PHP 7.2 FPM

phpMyAdmin version:
Version information: 4.9.0-dev, latest stable version: 4.7.9
Git revision: 695fe3a from master branch,
committed on Mar 28, 2018 at 01:48 PM by Weblate

Client configuration

Browser:
Google Chrome - Version 64.0.3282.186 (Official Build) (64-bit)

Operating system:
Windows 10 v1709 - build 16299.248

@Piyush3079
Copy link
Contributor

I have started working on this issue

@Piyush3079
Copy link
Contributor

I have raised on issue related to this #14148 but need some discussion regarding this @ibennetch @MauricioFauth before solving this issue or that issue.

@aroralakshya
Copy link
Contributor

aroralakshya commented Apr 2, 2018

Exactly opposite of what you have described happens in the stable branch. There, the entered query is never showed again whatsoever. I am not sure but think it is related to #14118.
After a quick look, I think there is no code written for "Show this query here again". Correct me if I'm wrong

@Piyush3079
Copy link
Contributor

Piyush3079 commented Apr 2, 2018

I have checked the current stable. You are right, the auto save sql is not working there and no code is written for "Show this query here again" in either stable or in current master, just a form is there.
I think in stable branch they are trying to get saved queries using get saved query button but in current master the implemented auto save sql feature doesn't require get saved query button as it always load saved query from local-storage.
I think the flow and logic of auto save sql needs to be changed for implementing show this query again.
Well although I have started working on this and will made a pr soon with the aforementioned features.

@MauricioFauth MauricioFauth self-assigned this May 3, 2018
@MauricioFauth MauricioFauth added this to the 5.0.0 milestone May 3, 2018
@MauricioFauth MauricioFauth added the Bug A problem or regression with an existing feature label May 3, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature
Projects
None yet
4 participants