-
Notifications
You must be signed in to change notification settings - Fork 18
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
Missing brackets for SELECT from Table / View #9
Comments
Ok I did it! Does anyone know how to compile it to setup file? |
On what Visual studio do you want to compile? I'm compiling it in 2017 and for that I had to install add-in: https://marketplace.visualstudio.com/items?itemName=VisualStudioProductTeam.MicrosoftVisualStudio2017InstallerProjects. |
And how should I create installer? Since now I just copying |
Just rebuild all projects in solution including "HuntingDogInstall.vdproj".
In bin folder of " HuntingDogInstall" you will find the compiled installer.
czw., 21 cze 2018 o 09:40 pklejnowski <notifications@github.com> napisał(a):
… And how should I create installer? Since now I just copying
C:\GIT\sql-hunting-dog2\HuntingDog2014\bin\Debug\HuntingDog.dll to
C:\Program Files (x86)\Hunting Dog\SSMS2014 after rebuild. But I can't
share with this dll, it doesn't work on my friend's computer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALRGZbojG85aUjku6Q1lwfTYTFSn3T4Uks5t-03xgaJpZM4UvWyb>
.
|
Still this folder is empty. |
Is the project selected in the "Configuration manager"?
pon., 25 cze 2018 o 14:38 pklejnowski <notifications@github.com> napisał(a):
… Still this folder is empty.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALRGZTBl-smszbyQ7lXquUlW_M3RMq20ks5uANm0gaJpZM4UvWyb>
.
|
Can you give me screenshot? |
Yes but still nothing. In Output I have result Rebuild All: 2 succeeded, 2 failed, 1 skipped:
|
I noticed that there is missing brackets when you're trying to select some table or view. So it might be problematic when you have tables like: dbo.User
I don't know how to create PR so here is how to fix:
File: ManagmentStudioController.cs
Line: 74:
builder.AppendFormat("\tSELECT TOP {0} {1}\r\n\tFROM [{2}].[{3}] {4}", selectTopX, selectColumns, view.Schema, view.Name, noLockHint);
Line 105:
builder.AppendFormat("\tSELECT TOP {0} {1}\r\n\tFROM [{2}].[{3}] {4}", selectTopXTable, selectColumns, tbl.Schema, tbl.Name, noLockHint);
Or can anyone tell me how to compile it to the install file? I'd like it to use it...
The text was updated successfully, but these errors were encountered: