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

Hello, I encountered an issue with a plugin dynamically modifying SQL #519

Open
TTTq opened this issue Apr 26, 2024 · 4 comments
Open

Hello, I encountered an issue with a plugin dynamically modifying SQL #519

TTTq opened this issue Apr 26, 2024 · 4 comments
Labels
help wanted Extra attention is needed

Comments

@TTTq
Copy link

TTTq commented Apr 26, 2024

          Thinks! But I encountered another plugin executing the modified SQL, and at the end of the execution, it was not the modified SQL, but the SQL before the modification. May I ask why this is?

image
image

Originally posted by @TTTq in #512 (comment)

@TTTq TTTq changed the title Thinks! But I encountered another plugin executing the modified SQL, and at the end of the execution, it was not the modified SQL, but the SQL before the modification. May I ask why this is? Hello, I encountered an issue with a plugin dynamically modifying SQL Apr 26, 2024
@zhuxiujia
Copy link
Member

All plugins execute their before function first before executing the actual SQL operation. Finally, execute their after function uniformly

@zhuxiujia zhuxiujia added the help wanted Extra attention is needed label Apr 29, 2024
@TTTq
Copy link
Author

TTTq commented May 1, 2024

After reviewing the source code, it was found that the SQL was modified when executing before, but the original SQL was still executed when executing the query. This should be a bug, and the modified SQL should be executed to correct the issue

@TTTq
Copy link
Author

TTTq commented May 1, 2024

1714539846814
1714539951770
1714539984608
1714539997073
1714540007191
1714540145634

@Issues-translate-bot
Copy link
Collaborator

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


1714539846814
1714539951770
1714539984608
1714539997073
1714540007191
1714540145634

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants