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.

Sign up for GitHub

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

Jump to bottom

$cfg['NaturalOrder'] is ignored in Create new trigger modal #18964

Open
liviuconcioiu opened this issue Feb 2, 2024 · 5 comments · May be fixed by #18966
Open

$cfg['NaturalOrder'] is ignored in Create new trigger modal #18964

liviuconcioiu opened this issue Feb 2, 2024 · 5 comments · May be fixed by #18966

Comments

@liviuconcioiu
Copy link
Contributor

Describe the bug

$cfg['NaturalOrder'] is ignored in Create new trigger modal.

To Reproduce

Steps to reproduce the behavior:

  1. Set $cfg['NaturalOrder'] to true.
  2. Go to Triggers
  3. Click on Create new trigger
  4. See issue

Expected behavior

Select should respect the $cfg['NaturalOrder'] value.

Screenshots

trigger

Server configuration

  • phpMyAdmin version: 5.2.2-dev, 6.0.0-dev
@SachinBahukhandi
Copy link
Contributor

Is it possible for us to select triggers without selecting databases?

@liviuconcioiu
Copy link
Contributor Author

Is it possible for us to select triggers without selecting databases?

I don't think this is possible.

@SachinBahukhandi
Copy link
Contributor

Is it possible for us to select triggers without selecting databases?

I don't think this is possible.

Sorry, if I misunderstood but isn't it respecting already as it shows in an order somewhat described here: https://docs.phpmyadmin.net/en/latest/config.html#cfg_NaturalOrder

@liviuconcioiu
Copy link
Contributor Author

Sorry, if I misunderstood but isn't it respecting already as it shows in an order somewhat described here: https://docs.phpmyadmin.net/en/latest/config.html#cfg_NaturalOrder

It should be:

tbl_1
tbl_2
tbl_3
tbl_4
tbl_5
tbl_6
tbl_7
tbl_8
tbl_9
tbl_10
tbl_11
...

@SachinBahukhandi SachinBahukhandi linked a pull request Feb 3, 2024 that will close this issue
Open
@SachinBahukhandi
Copy link
Contributor

Sorry, if I misunderstood but isn't it respecting already as it shows in an order somewhat described here: https://docs.phpmyadmin.net/en/latest/config.html#cfg_NaturalOrder

It should be:

tbl_1
tbl_2
tbl_3
tbl_4
tbl_5
tbl_6
tbl_7
tbl_8
tbl_9
tbl_10
tbl_11
...

Thank you for the clarification!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added the fix for natural order sorting if present SachinBahukhandi/phpmyadmin
2 participants
@SachinBahukhandi @liviuconcioiu