Skip to content

Change WriteDrillthroughJSON method#79

Closed
vberkutovv wants to merge 1 commit intointersystems-community:masterfrom
vberkutovv:method-WriteDrillthroughJSON
Closed

Change WriteDrillthroughJSON method#79
vberkutovv wants to merge 1 commit intointersystems-community:masterfrom
vberkutovv:method-WriteDrillthroughJSON

Conversation

@vberkutovv
Copy link
Copy Markdown

No description provided.

@jakcpto
Copy link
Copy Markdown

jakcpto commented Nov 21, 2022

This pull request fixing issue discovered in DSW intersystems-community/DeepSeeWeb#264

@eduard93
Copy link
Copy Markdown

Can you give me SQL examples for do ##class(%ZEN.Auxiliary.jsonSQLProvider).%WriteJSONFromSQL(,SQL) please?

@jakcpto
Copy link
Copy Markdown

jakcpto commented Nov 22, 2022

@eduard93 here is debug output:

IRISAPP>d ##class(MDX2JSON.Utils).WriteDrillthroughJSON("DRILLTHROUGH SELECT FROM [HOLEFOODS] %FILTER ([DateOfSale].[Actual].[YearSold].&[2017]) %LISTING [ListingX]")

MDX: DRILLTHROUGH SELECT FROM [HOLEFOODS] %FILTER ([DateOfSale].[Actual].[YearSold].&[2017]) %LISTING [ListingX]

SQL: SELECT ID,UnitsSold,Outlet->City FROM HoleFoods.SalesTransaction source WHERE source.%ID IN (SELECT _DSsourceId FROM HoleFoods_Cube.Listing WHERE _DSqueryKey = 'en767687656')

@eduard93
Copy link
Copy Markdown

eduard93 commented Nov 22, 2022

Could you try to construct a schemaless example please, or share the class?

i.e. SELECT '0.5', '.6'

@jakcpto
Copy link
Copy Markdown

jakcpto commented Nov 23, 2022

@eduard93
Copy link
Copy Markdown

Case:

s SQL="SELECT '5125.72','1714.50'"
do ##class(%ZEN.Auxiliary.jsonSQLProvider).%WriteJSONFromSQL(,SQL)
{
"children":[
{"HostVar_1":5125.72,"HostVar_2":"1714.50"}
]
}

@eduard93 eduard93 closed this in 78f29b7 Nov 25, 2022
@eduard93
Copy link
Copy Markdown

Fixed here

@jakcpto
Copy link
Copy Markdown

jakcpto commented Nov 30, 2022

Yes, looks like fixed ) Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants