-
Notifications
You must be signed in to change notification settings - Fork 17
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
SQL TRG`INSTALL not installing sql tables. #12
Comments
Ack, I'm in the middle of a revamp to the SQL code... there's been some
stupidities on my end. The fixed version should be:
&SQL`FORMAT
[u(cobj,mco)]=localize(regeditalli(v(Q`%0),v(REG`SQL`TABLES),u(SQL`TABLENAME,$0),v(REG`SQL`ENTRY),if(isint(u(setr`%va,set,v(u(setr`%va,num,add(%q<num>,1))))),%q<set>,switch($0,!,%q<set>,'[sqlescape(%q<set>)]'))))
…On Fri, May 5, 2017 at 11:55 PM, g33kcub ***@***.***> wrote:
u(SQL`FORMAT,%0,%1,%2,%3,%4,%5,%6,%7,%8,%9) => #-1 FUNCTION (REGEDITALLI)
EXPECTS AT LEAST 3 ARGUMENTS BUT GOT 2
This is the issue. I am not sure what the fix would be here, looks like
the updated code does not have the correct formatting in the SQL`FORMAT
function call.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#12>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEPp4kUmu3XhiRgUi8dU4IMcw13m01u_ks5r2-8fgaJpZM4NSnFu>
.
|
God. Uh. This is caused by some of the SQL functions on the MCO having been
removed...
Use Github's Commits history to grab an earlier version of the MCO for the
missing SQL functions. Or I can drop in and help out. This should be solved
soon.
…On Sat, May 6, 2017 at 7:48 PM, g33kcub ***@***.***> wrote:
So I inserted this and ran the install triggers on the objects and it
installed the TABLES but didn't do them correctly compared to other DBs
that I've seen. So... I'm curious what went wrong.
[image: sqlerror]
<https://cloud.githubusercontent.com/assets/7422497/25776725/e810b506-3294-11e7-9064-3fd0ed625691.png>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEPp4rYityTT5-Cn17HNZ0Ts9N95LnJ2ks5r3Qa1gaJpZM4NSnFu>
.
|
f93e674#diff-c3ff4a464f27cf7ddc80fb5ba4d6ee54
The red hilighted spots are the lines that were stupidly removed. You can
install those and it should work.
…On Sat, May 6, 2017 at 8:11 PM, Andrew Bastien ***@***.***> wrote:
God. Uh. This is caused by some of the SQL functions on the MCO having
been removed...
Use Github's Commits history to grab an earlier version of the MCO for the
missing SQL functions. Or I can drop in and help out. This should be solved
soon.
On Sat, May 6, 2017 at 7:48 PM, g33kcub ***@***.***> wrote:
> So I inserted this and ran the install triggers on the objects and it
> installed the TABLES but didn't do them correctly compared to other DBs
> that I've seen. So... I'm curious what went wrong.
> [image: sqlerror]
> <https://cloud.githubusercontent.com/assets/7422497/25776725/e810b506-3294-11e7-9064-3fd0ed625691.png>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <#12 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEPp4rYityTT5-Cn17HNZ0Ts9N95LnJ2ks5r3Qa1gaJpZM4NSnFu>
> .
>
|
Come on over 52.205.251.189 4210 create a bit and I'll wiz you |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
u(SQL`FORMAT,%0,%1,%2,%3,%4,%5,%6,%7,%8,%9) => #-1 FUNCTION (REGEDITALLI) EXPECTS AT LEAST 3 ARGUMENTS BUT GOT 2
This is the issue. I am not sure what the fix would be here, looks like the updated code does not have the correct formatting in the SQL`FORMAT function call.
The text was updated successfully, but these errors were encountered: