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

Schema browser search/filter #247

Closed
jkdavew opened this issue Jul 18, 2024 · 4 comments · Fixed by #249
Closed

Schema browser search/filter #247

jkdavew opened this issue Jul 18, 2024 · 4 comments · Fixed by #249
Labels
enhancement New feature or request

Comments

@jkdavew
Copy link

jkdavew commented Jul 18, 2024

Type: Bug

When using a schema browse/filter, the help says "show objects that start with this value" but I'm finding it's doing a "contains" search instead.

Also noticed that underscores aren't handled too well. If I search for v_ (my goal finding views that start with that pattern), it's finding many more things (and the _ isn't factored in). I have a feeling it could be due to the underscore in a LIKE statement. The snippet below "may" help resolve the issue using v_ as the search example.

Presumed this is currently being done:
like '%V_%'

Adjusted:
like 'V_%' escape ''

Suggestion: if underscores appears in the browse/filter, prepend with a backslash.

Keep up the good work!!

Extension version: 1.0.0
VS Code version: Code 1.91.1 (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:06:49.809Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info
Item Value
CPUs 13th Gen Intel(R) Core(TM) i7-13800H (20 x 2918)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
Load (avg) undefined
Memory (System) 31.59GB (11.38GB free)
Process Argv --crash-reporter-id 2f019281-5e42-4fb5-8cc5-2cecaf354ddc
Screen Reader no
VM 0%
A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonnoceb:30805159
asynctok:30898717
pythonregdiag2:30936856
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
chatpanelc:31048052
dsvsc021:30996838
945dj816:31013170
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecation:31068756
dwnewjupyter:31046869
impr_priority:31094925
refactorc:31098050
ccplc:31098109

@worksofliam
Copy link
Contributor

@jkdavew

the help says "show objects that start with this value"

Please share where this is referenced. Is in our docs or in the extension itself?

I will look into solving this for you this week.

@worksofliam worksofliam added the enhancement New feature or request label Jul 22, 2024
@jkdavew
Copy link
Author

jkdavew commented Jul 26, 2024 via email

@worksofliam
Copy link
Contributor

@jkdavew I just realized while working on a security fix I may have broken this in the next release. Here is how we're building the filter clause now:

https://github.com/codefori/vscode-db2i/pull/252/files#diff-3c7d0b76e5f5800890e460fe8b325034da2d2e64c748c47a17eff692e56374abR30

@jkdavew
Copy link
Author

jkdavew commented Aug 2, 2024

Ok, yeah I see it was a pretty significant change and that Julia is working on a fix related to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants