From 7d8b8a06842fc94a6d12566ca2f6c78c708f6c3c Mon Sep 17 00:00:00 2001 From: cnathe Date: Fri, 20 Sep 2024 13:39:26 -0500 Subject: [PATCH 01/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 8 ++++---- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 20 insertions(+), 20 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index c668cf790e7..38a1c8005f4 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.2" + "@labkey/components": "5.5.7-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.2.tgz", - "integrity": "sha512-oEoa5r5dd9CgA8LnJczVFJetjlwT/9tlljNaJCKexjM+B67ALdB4U7/00MyxvrmSfBblMGQ66Uco2bVudSEysg==", + "version": "5.5.7-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.7-fb-charts2410.0.tgz", + "integrity": "sha512-sKsTBgCUWcIY0VXtFQ5e+6rVFy6ih5CK2JbfJrjnvmDVbNwjRaOqxXfq0pEJ64j6/RN/WKbvipcJPXrCPy512Q==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/assay/package.json b/assay/package.json index da3b1056d56..e0e8418d60d 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.5.2" + "@labkey/components": "5.5.7-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 7cb19311233..970394c6981 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.3", + "@labkey/components": "5.5.7-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3488,9 +3488,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.3", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.3.tgz", - "integrity": "sha512-bCvmiuMyx7uasiv95CCtS25GE1heHUKzqEGmR8AXKP9kmmilIJIsz1a9+9hL/AAoFDFSK8It5/hA9le0mgUxEA==", + "version": "5.5.7-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.7-fb-charts2410.0.tgz", + "integrity": "sha512-sKsTBgCUWcIY0VXtFQ5e+6rVFy6ih5CK2JbfJrjnvmDVbNwjRaOqxXfq0pEJ64j6/RN/WKbvipcJPXrCPy512Q==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/core/package.json b/core/package.json index 26a545f0912..ab9097e9cd6 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.5.3", + "@labkey/components": "5.5.7-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index 916ee3ce2d7..f6e8cfbe296 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.2" + "@labkey/components": "5.5.7-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.2.tgz", - "integrity": "sha512-oEoa5r5dd9CgA8LnJczVFJetjlwT/9tlljNaJCKexjM+B67ALdB4U7/00MyxvrmSfBblMGQ66Uco2bVudSEysg==", + "version": "5.5.7-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.7-fb-charts2410.0.tgz", + "integrity": "sha512-sKsTBgCUWcIY0VXtFQ5e+6rVFy6ih5CK2JbfJrjnvmDVbNwjRaOqxXfq0pEJ64j6/RN/WKbvipcJPXrCPy512Q==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/experiment/package.json b/experiment/package.json index 1df899ab24e..96481a6f09a 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.5.2" + "@labkey/components": "5.5.7-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 610b65cf96a..a47bf207faa 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.2" + "@labkey/components": "5.5.7-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.2.tgz", - "integrity": "sha512-oEoa5r5dd9CgA8LnJczVFJetjlwT/9tlljNaJCKexjM+B67ALdB4U7/00MyxvrmSfBblMGQ66Uco2bVudSEysg==", + "version": "5.5.7-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.7-fb-charts2410.0.tgz", + "integrity": "sha512-sKsTBgCUWcIY0VXtFQ5e+6rVFy6ih5CK2JbfJrjnvmDVbNwjRaOqxXfq0pEJ64j6/RN/WKbvipcJPXrCPy512Q==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/pipeline/package.json b/pipeline/package.json index 316a3f6dcaa..d116bdb85a1 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.5.2" + "@labkey/components": "5.5.7-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", From b9558d748d2e450a4de57eea68dc5b4ec2f52a94 Mon Sep 17 00:00:00 2001 From: cnathe Date: Fri, 20 Sep 2024 13:40:22 -0500 Subject: [PATCH 02/17] Charting updates to default tickOverlapRotation and bottom margin for long/overlapping tick labels --- core/webapp/vis/src/internal/D3Renderer.js | 2 +- .../web/vis/genericChart/genericChartHelper.js | 15 +++++++++------ 2 files changed, 10 insertions(+), 7 deletions(-) diff --git a/core/webapp/vis/src/internal/D3Renderer.js b/core/webapp/vis/src/internal/D3Renderer.js index 8eb45520bb3..4201950e373 100644 --- a/core/webapp/vis/src/internal/D3Renderer.js +++ b/core/webapp/vis/src/internal/D3Renderer.js @@ -15,7 +15,7 @@ LABKEY.vis.internal.Axis = function() { tickRectCls, tickRectHeightOffset = 12, tickRectWidthOffset = 8, tickClick, axisSel, tickSel, textSel, gridLineSel, borderSel, grid, scalesList = [], gridLinesVisible = 'both', tickDigits, tickValues, tickMax, tickLabelMax, tickColor = '#000000', tickTextColor = '#000000', gridLineColor = '#DDDDDD', borderColor = '#000000', - tickPadding = 0, tickLength = 8, tickWidth = 1, tickOverlapRotation = 15, gridLineWidth = 1, borderWidth = 1, + tickPadding = 0, tickLength = 8, tickWidth = 1, tickOverlapRotation = 25, gridLineWidth = 1, borderWidth = 1, fontFamily = 'Roboto, arial, helvetica, sans-serif', fontSize = 11, adjustedStarts, adjustedEnds, xLogGutterBorder = 0, yLogGutterBorder = 0, yGutterXOffset = 0, xGutterYOffset = 0, addLogGutterLabel = false, xGridExtension = 0, yGridExtension = 0, logGutterSel; diff --git a/visualization/resources/web/vis/genericChart/genericChartHelper.js b/visualization/resources/web/vis/genericChart/genericChartHelper.js index dd9fa76fec6..a99e4573874 100644 --- a/visualization/resources/web/vis/genericChart/genericChartHelper.js +++ b/visualization/resources/web/vis/genericChart/genericChartHelper.js @@ -1026,7 +1026,8 @@ LABKEY.vis.GenericChartHelper = new function(){ renderTo: renderTo, rendererType: 'd3', width: chartConfig.width, - height: chartConfig.height + height: chartConfig.height, + gridLinesVisible: chartConfig.gridLinesVisible, }; if (renderType === 'pie_chart') { @@ -1181,7 +1182,7 @@ LABKEY.vis.GenericChartHelper = new function(){ var _willRotateXAxisTickText = function(scales, plotConfig, maxTickLength, data) { if (scales.x && scales.x.scaleType === 'discrete') { var tickCount = scales.x && scales.x.tickLabelMax ? Math.min(scales.x.tickLabelMax, data.length) : data.length; - return (tickCount * maxTickLength * 5) > (plotConfig.width - 150); + return (tickCount * maxTickLength * 4) > (plotConfig.width - 150); } return false; @@ -1195,15 +1196,17 @@ LABKEY.vis.GenericChartHelper = new function(){ var maxLen = 0; $.each(data, function(idx, d) { var val = LABKEY.Utils.isFunction(aes.x) ? aes.x(d) : d[aes.x]; - if (LABKEY.Utils.isString(val)) { + var subVal = LABKEY.Utils.isFunction(aes.xSub) ? aes.xSub(d) : d[aes.xSub]; + if (LABKEY.Utils.isString(subVal)) { + maxLen = Math.max(maxLen, subVal.length); + } else if (LABKEY.Utils.isString(val)) { maxLen = Math.max(maxLen, val.length); } }); if (_willRotateXAxisTickText(scales, plotConfig, maxLen, data)) { - // min bottom margin: 50, max bottom margin: 275 - var bottomMargin = Math.min(Math.max(50, maxLen*5), 275); - margins.bottom = bottomMargin; + // min bottom margin: 50, max bottom margin: 150 + margins.bottom = Math.min(Math.max(50, maxLen*5), 175); } } From dc116fa70eb084d46d0b9eafec8b2fda04b631b5 Mon Sep 17 00:00:00 2001 From: cnathe Date: Wed, 25 Sep 2024 10:47:37 -0500 Subject: [PATCH 03/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 8 ++++---- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 20 insertions(+), 20 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 227999d52c8..7c8b01d5911 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.10" + "@labkey/components": "5.5.10-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.10", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10.tgz", - "integrity": "sha512-lGIFtgYWpabYXAhPBxW5RN1Smyui0dEaXTyjVWycnSOuO8LCaBqtVFylQ8wVDMr107PDggJmRV8mbl+7T+07tg==", + "version": "5.5.10-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10-fb-charts2410.0.tgz", + "integrity": "sha512-9ObmNnVqZmOmbCEdMLuczlgqoAsFihQchQIRDDESnhs1DIs7QLqQt0EZKq6seZLiphWGRSotYdSTt78mHA2zSw==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/assay/package.json b/assay/package.json index 6c8d77a2720..e0209b73c5d 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.5.10" + "@labkey/components": "5.5.10-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 2c5b8edb5fd..cb6200524e0 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.10", + "@labkey/components": "5.5.10-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3488,9 +3488,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.10", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10.tgz", - "integrity": "sha512-lGIFtgYWpabYXAhPBxW5RN1Smyui0dEaXTyjVWycnSOuO8LCaBqtVFylQ8wVDMr107PDggJmRV8mbl+7T+07tg==", + "version": "5.5.10-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10-fb-charts2410.0.tgz", + "integrity": "sha512-9ObmNnVqZmOmbCEdMLuczlgqoAsFihQchQIRDDESnhs1DIs7QLqQt0EZKq6seZLiphWGRSotYdSTt78mHA2zSw==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/core/package.json b/core/package.json index 671f09805b4..0834de6baaf 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.5.10", + "@labkey/components": "5.5.10-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index 60b9ba61df9..6d4954c10a4 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.10" + "@labkey/components": "5.5.10-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.10", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10.tgz", - "integrity": "sha512-lGIFtgYWpabYXAhPBxW5RN1Smyui0dEaXTyjVWycnSOuO8LCaBqtVFylQ8wVDMr107PDggJmRV8mbl+7T+07tg==", + "version": "5.5.10-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10-fb-charts2410.0.tgz", + "integrity": "sha512-9ObmNnVqZmOmbCEdMLuczlgqoAsFihQchQIRDDESnhs1DIs7QLqQt0EZKq6seZLiphWGRSotYdSTt78mHA2zSw==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/experiment/package.json b/experiment/package.json index 8ffdcb5ccf8..f7cfb57edaa 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.5.10" + "@labkey/components": "5.5.10-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 32397ddcdc7..9dbd7da3aba 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.5.10" + "@labkey/components": "5.5.10-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.5.10", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10.tgz", - "integrity": "sha512-lGIFtgYWpabYXAhPBxW5RN1Smyui0dEaXTyjVWycnSOuO8LCaBqtVFylQ8wVDMr107PDggJmRV8mbl+7T+07tg==", + "version": "5.5.10-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.5.10-fb-charts2410.0.tgz", + "integrity": "sha512-9ObmNnVqZmOmbCEdMLuczlgqoAsFihQchQIRDDESnhs1DIs7QLqQt0EZKq6seZLiphWGRSotYdSTt78mHA2zSw==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/pipeline/package.json b/pipeline/package.json index 60531bcc789..f53678bbfdf 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.5.10" + "@labkey/components": "5.5.10-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", From 7fbf438f6b5eee0c45ee51d19c642ad7c1e663b1 Mon Sep 17 00:00:00 2001 From: cnathe Date: Thu, 26 Sep 2024 07:35:32 -0500 Subject: [PATCH 04/17] Issue 48860: Allow for Generic Charts to be inherited in child folders --- .../api/reports/report/ReportDescriptor.java | 7 +++- .../reports/report/view/ReportDesignBean.java | 4 +-- .../api/reports/report/view/ReportUtil.java | 2 +- .../query/reports/ReportServiceImpl.java | 2 +- .../web/vis/chartWizard/genericChartPanel.js | 2 ++ .../web/vis/chartWizard/saveOptionsPanel.js | 27 ++++++++++++---- .../VisualizationController.java | 32 +++++++++++++++++-- 7 files changed, 63 insertions(+), 13 deletions(-) diff --git a/api/src/org/labkey/api/reports/report/ReportDescriptor.java b/api/src/org/labkey/api/reports/report/ReportDescriptor.java index d2fa4d62722..b0e5050c611 100644 --- a/api/src/org/labkey/api/reports/report/ReportDescriptor.java +++ b/api/src/org/labkey/api/reports/report/ReportDescriptor.java @@ -288,6 +288,11 @@ public void setWasShared() _wasShared = true; } + public boolean isInheritable() + { + return (getFlags() & FLAG_INHERITABLE) != 0; + } + @Nullable public Integer getAuthor() { @@ -780,7 +785,7 @@ public boolean isInherited(Container c) // if the report has been configured to be shared to child folders or is in the shared folder then // flag it as inherited. // - if (((getFlags() & ReportDescriptor.FLAG_INHERITABLE) != 0) || (ContainerManager.getSharedContainer().equals(srcContainer))) + if (isInheritable() || (ContainerManager.getSharedContainer().equals(srcContainer))) { return !c.equals(srcContainer); } diff --git a/api/src/org/labkey/api/reports/report/view/ReportDesignBean.java b/api/src/org/labkey/api/reports/report/view/ReportDesignBean.java index 29ac5c50876..79e3f283336 100644 --- a/api/src/org/labkey/api/reports/report/view/ReportDesignBean.java +++ b/api/src/org/labkey/api/reports/report/view/ReportDesignBean.java @@ -292,8 +292,8 @@ void populateFromDescriptor(ReportDescriptor descriptor) setCached(BooleanUtils.toBoolean(descriptor.getProperty(ReportDescriptor.Prop.cached))); setReportAccess(descriptor.getAccess()); - setShareReport((descriptor.isShared())); - setInheritable((descriptor.getFlags() & ReportDescriptor.FLAG_INHERITABLE) != 0); + setShareReport(descriptor.isShared()); + setInheritable(descriptor.isInheritable()); setRedirectUrl(getViewContext().getActionURL().getParameter(ReportDescriptor.Prop.redirectUrl.name())); } } diff --git a/api/src/org/labkey/api/reports/report/view/ReportUtil.java b/api/src/org/labkey/api/reports/report/view/ReportUtil.java index f99c19b2de9..1328ff75ce1 100644 --- a/api/src/org/labkey/api/reports/report/view/ReportUtil.java +++ b/api/src/org/labkey/api/reports/report/view/ReportUtil.java @@ -325,7 +325,7 @@ public static boolean isReportInherited(Container c, Report report) return !ContainerManager.getSharedContainer().equals(c); } - if ((report.getDescriptor().getFlags() & ReportDescriptor.FLAG_INHERITABLE) != 0) + if (report.getDescriptor().isInheritable()) { Container reportContainer = ContainerManager.getForId(report.getDescriptor().getContainerId()); if (!c.equals(reportContainer)) diff --git a/query/src/org/labkey/query/reports/ReportServiceImpl.java b/query/src/org/labkey/query/reports/ReportServiceImpl.java index 337648f45b6..cac81b7c21b 100644 --- a/query/src/org/labkey/query/reports/ReportServiceImpl.java +++ b/query/src/org/labkey/query/reports/ReportServiceImpl.java @@ -606,7 +606,7 @@ public Report getReport(Container c, int rowId) if (null != report) { - if ((report.getDescriptor().getFlags() & ReportDescriptor.FLAG_INHERITABLE) != 0) + if (report.getDescriptor().isInheritable()) return report; else return null; diff --git a/visualization/resources/web/vis/chartWizard/genericChartPanel.js b/visualization/resources/web/vis/chartWizard/genericChartPanel.js index bb20885d61e..cdcbe6d4b40 100644 --- a/visualization/resources/web/vis/chartWizard/genericChartPanel.js +++ b/visualization/resources/web/vis/chartWizard/genericChartPanel.js @@ -1079,6 +1079,7 @@ Ext4.define('LABKEY.ext4.GenericChartPanel', { reportConfig.description = data.reportDescription; reportConfig["public"] = data.shared; + reportConfig.inheritable = data.inheritable; reportConfig.thumbnailType = data.thumbnailType; reportConfig.svg = this.chartSVG; @@ -1189,6 +1190,7 @@ Ext4.define('LABKEY.ext4.GenericChartPanel', { name: config.name, description: config.description, shared: config.shared, + inheritable: config.inheritable, reportProps: config.reportProps, thumbnailURL: config.thumbnailURL }); diff --git a/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js b/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js index 4e41116d043..952260593eb 100644 --- a/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js +++ b/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js @@ -33,7 +33,6 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { this.thumbnailType = this.isSavedReport() && this.reportInfo.reportProps && this.reportInfo.reportProps.thumbnailType ? this.reportInfo.reportProps.thumbnailType : 'AUTO'; // Note that Readers are allowed to save new charts (readers own new charts they're creating)- this is by design. - this.currentlyShared = (this.isSavedReport() && this.reportInfo.shared) || (!this.isSavedReport() && this.canSaveSharedCharts()); this.createdBy = this.isSavedReport() ? this.reportInfo.createdBy : LABKEY.Security.currentUser.id; // generate unique id for the thumbnail preview div @@ -61,6 +60,7 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { this.saveForm = Ext4.create('Ext.form.Panel', { region: 'center', cls: 'region-panel save-form-panel', + bodyStyle: 'padding: 10px;', border: false, items: [ Ext4.create('Ext.form.field.Text', { @@ -102,14 +102,23 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { value: Ext4.util.Format.htmlEncode(this.isSavedReport() ? this.reportInfo.description : null), anchor: '100%' }), + Ext4.create('Ext.form.Checkbox', { + itemId: 'reportInheritable', + name: 'reportInheritable', + fieldLabel: 'Inherit', + boxLabel: 'Make this report available in child folders', + labelWidth: 125, + width: 400, + // hidden: false, // TODO hide for guest and if no child folders + }), Ext4.create('Ext.form.RadioGroup', { itemId: 'reportShared', fieldLabel: 'Viewable By', labelWidth: 125, width: 350, items : [ - { itemId: 'allReaders', name: 'reportShared', boxLabel: 'All readers', inputValue: 'true', disabled: !this.canSaveSharedCharts(), checked: this.currentlyShared, width: 140 }, - { itemId: 'onlyMe', name: 'reportShared', boxLabel: 'Only me', inputValue: 'false', disabled: !this.canSaveSharedCharts(), checked: !this.currentlyShared, width: 140 } + { itemId: 'allReaders', name: 'reportShared', boxLabel: 'All readers', inputValue: 'true', disabled: !this.canSaveSharedCharts(), width: 140 }, + { itemId: 'onlyMe', name: 'reportShared', boxLabel: 'Only me', inputValue: 'false', disabled: !this.canSaveSharedCharts(), width: 140 } ] }), Ext4.create('Ext.form.RadioGroup', { @@ -197,11 +206,13 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { // the save button will not allow for replace if this is a new chart, // but will force replace if this is a change to a saved chart var shared = Ext4.isString(formVals.reportShared) ? 'true' == formVals.reportShared : (new Boolean(formVals.reportShared)).valueOf(); + var inheritable = Ext4.isString(formVals.reportInheritable) ? 'on' == formVals.reportInheritable : (new Boolean(formVals.reportInheritable)).valueOf(); this.fireEvent('saveChart', { isSaveAs: this.isSaveAs, replace: !this.isSaveAs ? this.isSavedReport() : false, reportName: formVals.reportName, reportDescription: formVals.reportDescription, + inheritable: inheritable, shared: shared, thumbnailType: formVals.reportThumbnailType, canSaveSharedCharts: this.canSaveSharedCharts(), @@ -209,10 +220,11 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { }); // store the update report properties - if(this.reportInfo){ + if (this.reportInfo){ this.reportInfo.name = formVals.reportName; this.reportInfo.description = formVals.reportDescription; - this.currentlyShared = shared; + this.reportInfo.shared = shared; + this.reportInfo.inheritable = inheritable; this.thumbnailType = formVals.reportThumbnailType; } @@ -280,6 +292,7 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { this.down('#onlyMe').setValue(true); else this.down('#allReaders').setValue(true); + this.down('#reportInheritable').setValue(false); this.down('#reportThumbnailType').setValue({reportThumbnailType: 'AUTO'}); this.down('#keepCustom').hide(); @@ -299,10 +312,11 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { this.down('#reportDescriptionDisplay').setVisible(!this.canSaveChanges()); this.down('#reportDescriptionDisplay').setValue(Ext4.util.Format.htmlEncode(this.isSavedReport() ? this.reportInfo.description : null)); - if (!this.currentlyShared) + if (this.isSavedReport() && !this.reportInfo.shared) this.down('#onlyMe').setValue(true); else this.down('#allReaders').setValue(true); + this.down('#reportInheritable').setValue(this.isSavedReport() ? this.reportInfo.inheritable : false); this.down('#reportThumbnailType').setValue({reportThumbnailType: this.thumbnailType}); // hide the Keep existing option if there hasn't been a custom thumbnail saved for this report @@ -368,6 +382,7 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { this.down('#reportDescription').setValue(config.description); this.down('#reportDescriptionDisplay').setValue(config.description); this.down('#reportShared').setValue(config.shared); + this.down('#reportInheritable').setValue(config.inheritable); if(config.reportProps && config.reportProps.thumbnailType){ this.thumbnailType = config.reportProps.thumbnailType; diff --git a/visualization/src/org/labkey/visualization/VisualizationController.java b/visualization/src/org/labkey/visualization/VisualizationController.java index d4c6de080d2..424ba68f409 100644 --- a/visualization/src/org/labkey/visualization/VisualizationController.java +++ b/visualization/src/org/labkey/visualization/VisualizationController.java @@ -984,12 +984,22 @@ public void validateForm(ChartWizardReportForm form, Errors errors) errors.reject(ERROR_MSG, "Visualization \"" + form.getName() + "\" does not exist in " + getContainer().getPath() + "."); } - if (report == null || report.getDescriptor().getContainerId() == null || !report.getDescriptor().getContainerId().equals(getContainer().getId())) + if (report == null) { - errors.reject(ERROR_MSG, "Visualization \"" + form.getName() + "\" does not exist in " + getContainer().getPath() + "."); + String reportId = form.getReportId() != null ? form.getReportId().toString() : form.getName(); + errors.reject(ERROR_MSG, "Visualization for \"" + reportId + "\" does not exist in " + getContainer().getPath() + "."); return; } + if (report.getDescriptor().getContainerId() == null || !report.getDescriptor().getContainerId().equals(getContainer().getId())) + { + if (!report.getDescriptor().isInherited(getContainer())) + { + errors.reject(ERROR_MSG, "Visualization \"" + report.getDescriptor().getReportName() + "\" does not exist in " + getContainer().getPath() + "."); + return; + } + } + if (!report.getDescriptor().isShared() && report.getDescriptor().getOwner() != getUser().getUserId()) { errors.reject(ERROR_MSG, "You do not have permissions to view this private report."); @@ -1019,6 +1029,7 @@ public ApiResponse execute(ChartWizardReportForm form, BindException errors) thr resp.put("viewName", vizDescriptor.getProperty(ReportDescriptor.Prop.viewName)); resp.put("type", vizDescriptor.getReportType()); resp.put("shared", vizDescriptor.isShared()); + resp.put("inheritable", vizDescriptor.isInheritable()); resp.put("ownerId", !vizDescriptor.isShared() ? vizDescriptor.getOwner() : null); resp.put("createdBy", vizDescriptor.getCreatedBy()); resp.put("reportProps", vizDescriptor.getReportProps()); @@ -1312,6 +1323,11 @@ private Report getGenericReport(ChartWizardReportForm form) descriptor.setOwner(getUser().getUserId()); else descriptor.setOwner(null); + + if (form.isInheritable()) + descriptor.setFlags(descriptor.getFlags() | ReportDescriptor.FLAG_INHERITABLE); + else + descriptor.setFlags(descriptor.getFlags() & ~ReportDescriptor.FLAG_INHERITABLE); } return report; } @@ -1444,6 +1460,7 @@ public static class ChartWizardReportForm extends ReportUtil.JsonReportForm private String _autoColumnXName; private String _svg; private String _thumbnailType; + private boolean _inheritable; private boolean _allowToggleMode = false; // view vs. edit mode public String getRenderType() @@ -1528,6 +1545,16 @@ public void setThumbnailType(String thumbnailType) _thumbnailType = thumbnailType; } + public boolean isInheritable() + { + return _inheritable; + } + + public void setInheritable(boolean inheritable) + { + _inheritable = inheritable; + } + public boolean allowToggleMode() { return _allowToggleMode; @@ -1547,6 +1574,7 @@ public void bindJson(JSONObject json) _dataRegionName = json.optString("dataRegionName", null); _svg = json.optString("svg", null); _thumbnailType = json.optString("thumbnailType", null); + _inheritable = json.optBoolean("inheritable", false); Object jsonData = json.opt("jsonData"); if (jsonData != null) From 9d6b5b82838450308e0377d5ab97008b788f917b Mon Sep 17 00:00:00 2001 From: cnathe Date: Mon, 30 Sep 2024 09:24:15 -0500 Subject: [PATCH 05/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 8 ++++---- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 20 insertions(+), 20 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 8b2ea3e4b34..47ab7d4ac3a 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0" + "@labkey/components": "5.7.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0.tgz", - "integrity": "sha512-pEIrN0ackB9R4b2jcxu8ozVPV1UGrOEkRZ1Q80VIXCUc/QmPcGlXCbpnW3dt+C7Y6SeUfGy+6JCoLihlo3V1xA==", + "version": "5.7.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", + "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/assay/package.json b/assay/package.json index 67ce9b2ffe0..83bed1507bc 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.7.0" + "@labkey/components": "5.7.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 39e1f475e41..6043c4794fe 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0", + "@labkey/components": "5.7.0-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3488,9 +3488,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0.tgz", - "integrity": "sha512-pEIrN0ackB9R4b2jcxu8ozVPV1UGrOEkRZ1Q80VIXCUc/QmPcGlXCbpnW3dt+C7Y6SeUfGy+6JCoLihlo3V1xA==", + "version": "5.7.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", + "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/core/package.json b/core/package.json index 27b399b18db..1d99d2f6d9a 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.7.0", + "@labkey/components": "5.7.0-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index ff7b6d534f4..8bab7fd4e1d 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0" + "@labkey/components": "5.7.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0.tgz", - "integrity": "sha512-pEIrN0ackB9R4b2jcxu8ozVPV1UGrOEkRZ1Q80VIXCUc/QmPcGlXCbpnW3dt+C7Y6SeUfGy+6JCoLihlo3V1xA==", + "version": "5.7.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", + "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/experiment/package.json b/experiment/package.json index 784985f973f..f10e00edde7 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.7.0" + "@labkey/components": "5.7.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 66329df73bc..d41d74a2b37 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0" + "@labkey/components": "5.7.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0.tgz", - "integrity": "sha512-pEIrN0ackB9R4b2jcxu8ozVPV1UGrOEkRZ1Q80VIXCUc/QmPcGlXCbpnW3dt+C7Y6SeUfGy+6JCoLihlo3V1xA==", + "version": "5.7.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", + "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/pipeline/package.json b/pipeline/package.json index cb12ac6bdf5..21a51033367 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.7.0" + "@labkey/components": "5.7.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", From 3ee6adf90f38efef04757c6e6313ec952e103564 Mon Sep 17 00:00:00 2001 From: cnathe Date: Mon, 30 Sep 2024 11:09:28 -0500 Subject: [PATCH 06/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 8 ++++---- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 20 insertions(+), 20 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 47ab7d4ac3a..c44d597ef2f 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0" + "@labkey/components": "5.7.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", - "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", + "version": "5.7.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", + "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/assay/package.json b/assay/package.json index 83bed1507bc..9a59ac015d8 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0" + "@labkey/components": "5.7.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 6043c4794fe..a26bd2f6831 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0", + "@labkey/components": "5.7.0-fb-charts2410.1", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3488,9 +3488,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", - "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", + "version": "5.7.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", + "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/core/package.json b/core/package.json index 1d99d2f6d9a..78a827e4cb6 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0", + "@labkey/components": "5.7.0-fb-charts2410.1", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index 8bab7fd4e1d..d1f34add212 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0" + "@labkey/components": "5.7.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", - "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", + "version": "5.7.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", + "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/experiment/package.json b/experiment/package.json index f10e00edde7..3260d787d7d 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0" + "@labkey/components": "5.7.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index d41d74a2b37..6b9c4d82ce0 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0" + "@labkey/components": "5.7.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.0.tgz", - "integrity": "sha512-Ur0OG4JsZNmHqpqFPGKDCQe5iSaZ5sOT+c/eGY0X6ltS/P6HX7Ti8+HZhByrkwFXbgj5NIhgXi4J1AtFRdzHsg==", + "version": "5.7.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", + "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1", diff --git a/pipeline/package.json b/pipeline/package.json index 21a51033367..b64bff6a743 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.0" + "@labkey/components": "5.7.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", From 18a9c1ee4b2764000c75a028e7c24ccbfd9425ba Mon Sep 17 00:00:00 2001 From: cnathe Date: Mon, 30 Sep 2024 11:10:34 -0500 Subject: [PATCH 07/17] Save chart inherit checkbox to be hidden/shown based on user perm (to match R Report inherit checkbox perm check) --- .../web/vis/chartWizard/baseChartWizardPanel.js | 3 +++ .../resources/web/vis/chartWizard/genericChartPanel.js | 1 + .../resources/web/vis/chartWizard/saveOptionsPanel.js | 9 ++++++++- .../src/org/labkey/visualization/views/chartWizard.jsp | 3 +++ 4 files changed, 15 insertions(+), 1 deletion(-) diff --git a/visualization/resources/web/vis/chartWizard/baseChartWizardPanel.js b/visualization/resources/web/vis/chartWizard/baseChartWizardPanel.js index fc44f629621..7296f2df8c4 100644 --- a/visualization/resources/web/vis/chartWizard/baseChartWizardPanel.js +++ b/visualization/resources/web/vis/chartWizard/baseChartWizardPanel.js @@ -16,6 +16,7 @@ Ext4.define('LABKEY.ext4.BaseChartWizardPanel', { canShare: false, isDeveloper: false, defaultNumberFormat: null, + allowInherit: false, allowEditMode: false, editModeURL: null, baseUrl: null, @@ -116,6 +117,7 @@ Ext4.define('LABKEY.ext4.BaseChartWizardPanel', { canShare: this.canShare, isDeveloper: this.isDeveloper, defaultNumberFormat: this.defaultNumberFormat, + allowInherit: this.allowInherit, allowEditMode: this.allowEditMode, editModeURL: this.editModeURL, @@ -151,6 +153,7 @@ Ext4.define('LABKEY.ext4.BaseChartWizardPanel', { canShare: this.canShare, isDeveloper: this.isDeveloper, defaultNumberFormat: this.defaultNumberFormat, + allowInherit: this.allowInherit, allowEditMode: this.allowEditMode, editModeURL: this.editModeURL, diff --git a/visualization/resources/web/vis/chartWizard/genericChartPanel.js b/visualization/resources/web/vis/chartWizard/genericChartPanel.js index cdcbe6d4b40..ada18dbc776 100644 --- a/visualization/resources/web/vis/chartWizard/genericChartPanel.js +++ b/visualization/resources/web/vis/chartWizard/genericChartPanel.js @@ -632,6 +632,7 @@ Ext4.define('LABKEY.ext4.GenericChartPanel', { if (!this.savePanel) { this.savePanel = Ext4.create('LABKEY.vis.SaveOptionsPanel', { + allowInherit: this.allowInherit, canEdit: this.canEdit, canShare: this.canShare, listeners: { diff --git a/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js b/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js index 952260593eb..96897a04d90 100644 --- a/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js +++ b/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js @@ -107,9 +107,9 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { name: 'reportInheritable', fieldLabel: 'Inherit', boxLabel: 'Make this report available in child folders', + hidden: this.shouldAllowInherit(), labelWidth: 125, width: 400, - // hidden: false, // TODO hide for guest and if no child folders }), Ext4.create('Ext.form.RadioGroup', { itemId: 'reportShared', @@ -262,6 +262,11 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { return Ext4.isObject(this.reportInfo); }, + shouldAllowInherit : function() + { + return this.allowInherit; + }, + canSaveChanges : function() { return this.canEdit; @@ -316,6 +321,8 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { this.down('#onlyMe').setValue(true); else this.down('#allReaders').setValue(true); + + this.down('#reportInheritable').setVisible(this.shouldAllowInherit()); this.down('#reportInheritable').setValue(this.isSavedReport() ? this.reportInfo.inheritable : false); this.down('#reportThumbnailType').setValue({reportThumbnailType: this.thumbnailType}); diff --git a/visualization/src/org/labkey/visualization/views/chartWizard.jsp b/visualization/src/org/labkey/visualization/views/chartWizard.jsp index a7509b01601..8e57c05d761 100644 --- a/visualization/src/org/labkey/visualization/views/chartWizard.jsp +++ b/visualization/src/org/labkey/visualization/views/chartWizard.jsp @@ -32,6 +32,7 @@ <%@ page import="org.labkey.api.view.ViewContext" %> <%@ page import="org.labkey.api.view.template.ClientDependencies" %> <%@ page import="org.labkey.visualization.VisualizationController" %> +<%@ page import="org.labkey.api.security.roles.ProjectAdminRole" %> <%@ page extends="org.labkey.api.jsp.JspBase" %> <%@ taglib prefix="labkey" uri="http://www.labkey.org/taglib" %> <%! @@ -58,6 +59,7 @@ boolean canShare = ctx.hasPermission(ShareReportPermission.class); boolean isDeveloper = user.isBrowserDev(); boolean allowEditMode = !user.isGuest() && form.allowToggleMode(); + boolean allowInherit = user.hasRootAdminPermission() || ReportUtil.isInRole(user, c, ProjectAdminRole.class); boolean canEdit = false; ActionURL editUrl = null; @@ -126,6 +128,7 @@ canShare: <%=canShare%>, isDeveloper: <%=isDeveloper%>, defaultNumberFormat: eval(<%=q(numberFormatFn)%>), + allowInherit: <%=allowInherit%>, allowEditMode: <%=allowEditMode%>, editModeURL: editUrl, From 6da4fef1a20d1624c57a27fb47e6fb76a4a6dcfc Mon Sep 17 00:00:00 2001 From: cnathe Date: Mon, 30 Sep 2024 13:12:05 -0500 Subject: [PATCH 08/17] update @labkey/components package version --- assay/package-lock.json | 16 ++++++++-------- assay/package.json | 2 +- core/package-lock.json | 16 ++++++++-------- core/package.json | 2 +- experiment/package-lock.json | 16 ++++++++-------- experiment/package.json | 2 +- pipeline/package-lock.json | 16 ++++++++-------- pipeline/package.json | 2 +- 8 files changed, 36 insertions(+), 36 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index c44d597ef2f..2321a5554ca 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1" + "@labkey/components": "5.7.0-fb-charts2410.2" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2528,9 +2528,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -2569,12 +2569,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", - "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", + "version": "5.7.0-fb-charts2410.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", + "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/assay/package.json b/assay/package.json index 9a59ac015d8..2744bd50b94 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1" + "@labkey/components": "5.7.0-fb-charts2410.2" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index a26bd2f6831..605b1416c73 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1", + "@labkey/components": "5.7.0-fb-charts2410.2", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3447,9 +3447,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3488,12 +3488,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", - "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", + "version": "5.7.0-fb-charts2410.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", + "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/core/package.json b/core/package.json index 78a827e4cb6..46a32447293 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1", + "@labkey/components": "5.7.0-fb-charts2410.2", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index d1f34add212..26d3efd3c82 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1" + "@labkey/components": "5.7.0-fb-charts2410.2" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3255,9 +3255,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3296,12 +3296,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", - "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", + "version": "5.7.0-fb-charts2410.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", + "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/experiment/package.json b/experiment/package.json index 3260d787d7d..92635e43d07 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1" + "@labkey/components": "5.7.0-fb-charts2410.2" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 6b9c4d82ce0..f5faafd0585 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1" + "@labkey/components": "5.7.0-fb-charts2410.2" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2701,9 +2701,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -2742,12 +2742,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.1.tgz", - "integrity": "sha512-0bayerGZudqWzWK0CvIVF6nRqDOByw0PJVJZRjnsBjeLJ3xrSVLxs1mLof5KaoXXXfrAQQBnQDAPSurzJ1NF9g==", + "version": "5.7.0-fb-charts2410.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", + "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/pipeline/package.json b/pipeline/package.json index b64bff6a743..2858912072d 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.1" + "@labkey/components": "5.7.0-fb-charts2410.2" }, "devDependencies": { "@labkey/build": "7.7.1", From 54d5760da6f86de73894ce49a2337dcf65c10a59 Mon Sep 17 00:00:00 2001 From: cnathe Date: Mon, 30 Sep 2024 15:41:17 -0500 Subject: [PATCH 09/17] Add ReportAuditProvider with events for insert, update and delete of reports and charts --- query/src/org/labkey/query/QueryModule.java | 2 + .../query/reports/ReportAuditProvider.java | 195 ++++++++++++++++++ .../query/reports/ReportServiceImpl.java | 15 +- 3 files changed, 208 insertions(+), 4 deletions(-) create mode 100644 query/src/org/labkey/query/reports/ReportAuditProvider.java diff --git a/query/src/org/labkey/query/QueryModule.java b/query/src/org/labkey/query/QueryModule.java index 204998781a4..c74886310c2 100644 --- a/query/src/org/labkey/query/QueryModule.java +++ b/query/src/org/labkey/query/QueryModule.java @@ -106,6 +106,7 @@ import org.labkey.query.reports.LinkReport; import org.labkey.query.reports.ModuleReportCache; import org.labkey.query.reports.ReportAndDatasetChangeDigestProviderImpl; +import org.labkey.query.reports.ReportAuditProvider; import org.labkey.query.reports.ReportImporter; import org.labkey.query.reports.ReportNotificationInfoProvider; import org.labkey.query.reports.ReportServiceImpl; @@ -288,6 +289,7 @@ public void doStartup(ModuleContext moduleContext) AuditLogService.get().registerAuditType(new QueryExportAuditProvider()); AuditLogService.get().registerAuditType(new QueryUpdateAuditProvider()); } + AuditLogService.get().registerAuditType(new ReportAuditProvider()); ReportAndDatasetChangeDigestProvider.get().addNotificationInfoProvider(new ReportNotificationInfoProvider()); DailyMessageDigest.getInstance().addProvider(ReportAndDatasetChangeDigestProvider.get()); diff --git a/query/src/org/labkey/query/reports/ReportAuditProvider.java b/query/src/org/labkey/query/reports/ReportAuditProvider.java new file mode 100644 index 00000000000..bf3e7a85885 --- /dev/null +++ b/query/src/org/labkey/query/reports/ReportAuditProvider.java @@ -0,0 +1,195 @@ +package org.labkey.query.reports; + + +import org.jetbrains.annotations.NotNull; +import org.labkey.api.audit.AbstractAuditTypeProvider; +import org.labkey.api.audit.AuditTypeEvent; +import org.labkey.api.audit.query.AbstractAuditDomainKind; +import org.labkey.api.data.Container; +import org.labkey.api.exp.PropertyDescriptor; +import org.labkey.api.exp.PropertyType; +import org.labkey.api.query.FieldKey; +import org.labkey.api.reports.report.ReportDB; +import org.labkey.api.reports.report.ReportDescriptor; + +import java.util.ArrayList; +import java.util.LinkedHashSet; +import java.util.List; +import java.util.Map; +import java.util.Set; + +public class ReportAuditProvider extends AbstractAuditTypeProvider +{ + private static final String EVENT_NAME = "ReportEvent"; + private static final String COLUMN_NAME_REPORT_ID = "ReportId"; + private static final String COLUMN_NAME_REPORT_NAME = "ReportName"; + private static final String COLUMN_NAME_REPORT_KEY = "ReportKey"; + private static final String COLUMN_NAME_REPORT_TYPE = "ReportType"; + + + static final List defaultVisibleColumns = new ArrayList<>(); + + static { + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_CREATED)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_CONTAINER)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_CREATED_BY)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_IMPERSONATED_BY)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_REPORT_ID)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_REPORT_NAME)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_REPORT_KEY)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_REPORT_TYPE)); + defaultVisibleColumns.add(FieldKey.fromParts(COLUMN_NAME_COMMENT)); + } + + @Override + protected AbstractAuditDomainKind getDomainKind() + { + return new ReportAuditDomainKind(); + } + + @Override + public String getEventName() + { + return EVENT_NAME; + } + + @Override + public String getLabel() + { + return "Report events"; + } + + @Override + public String getDescription() + { + return "Events related to the creation and modification of reports and charts."; + } + + @Override + public Class getEventClass() + { + return (Class) ReportAuditEvent.class; + } + + @Override + public List getDefaultVisibleColumns() + { + return defaultVisibleColumns; + } + + public static class ReportAuditEvent extends AuditTypeEvent + { + private String reportId; + private String reportName; + private String reportKey; + private String reportType; + + public ReportAuditEvent() + { + } + + public ReportAuditEvent(@NotNull ReportDB report, @NotNull ReportDescriptor descriptor, Container container, String comment) + { + this(report.getRowId(), descriptor, container, comment); + this.reportKey = report.getReportKey(); + } + + public ReportAuditEvent(int reportId, @NotNull ReportDescriptor descriptor, Container container, String comment) + { + super(EVENT_NAME, container, comment); + this.reportId = String.valueOf(reportId); + this.reportName = descriptor.getReportName(); + this.reportKey = descriptor.getReportKey(); + this.reportType = descriptor.getReportType(); + } + + public String getReportId() + { + return reportId; + } + + public void setReportId(@NotNull Integer reportId) + { + this.reportId = String.valueOf(reportId); + } + + public String getReportName() + { + return reportName; + } + + public void setReportName(String reportName) + { + this.reportName = reportName; + } + + public String getReportKey() + { + return reportKey; + } + + public void setReportKey(String reportKey) + { + this.reportKey = reportKey; + } + + public String getReportType() + { + return reportType; + } + + public void setReportType(String reportType) + { + this.reportType = reportType; + } + + @Override + public Map getAuditLogMessageElements() + { + Map elements = super.getAuditLogMessageElements(); + elements.put("reportId", reportId); + elements.put("reportName", reportName); + elements.put("reportKey", reportKey); + elements.put("reportType", reportType); + return elements; + } + + } + + public static class ReportAuditDomainKind extends AbstractAuditDomainKind + { + public static final String NAME = "ReportAuditDomain"; + public static String NAMESPACE_PREFIX = "Report-" + NAME; + + private final Set fields; + + public ReportAuditDomainKind() + { + super(EVENT_NAME); + + fields = new LinkedHashSet<>(); + fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_ID, PropertyType.STRING, "Report Id", null, true)); + fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_NAME, PropertyType.STRING, "Report Name", null, true)); + fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_KEY, PropertyType.STRING, "Report Key", null, false)); + fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_TYPE, PropertyType.STRING, "Report Type", null, true)); + } + + @Override + protected String getNamespacePrefix() + { + return NAMESPACE_PREFIX; + } + + @Override + public Set getProperties() + { + return fields; + } + + @Override + public String getKindName() + { + return NAME; + } + } +} diff --git a/query/src/org/labkey/query/reports/ReportServiceImpl.java b/query/src/org/labkey/query/reports/ReportServiceImpl.java index cac81b7c21b..d6bf0a9d754 100644 --- a/query/src/org/labkey/query/reports/ReportServiceImpl.java +++ b/query/src/org/labkey/query/reports/ReportServiceImpl.java @@ -29,6 +29,7 @@ import org.jetbrains.annotations.Nullable; import org.labkey.api.admin.FolderExportContext; import org.labkey.api.admin.FolderImportContext; +import org.labkey.api.audit.AuditLogService; import org.labkey.api.collections.MultiSetUtils; import org.labkey.api.data.Container; import org.labkey.api.data.ContainerManager; @@ -309,18 +310,21 @@ public void deleteReport(ContainerUser context, Report report) report.beforeDelete(context); final ReportDescriptor descriptor = report.getDescriptor(); - _deleteReport(context.getContainer(), reportId.getRowId()); + _deleteReport(context.getContainer(), context.getUser(), reportId.getRowId(), descriptor); SecurityPolicyManager.deletePolicy(descriptor); tx.commit(); } } - private void _deleteReport(Container c, int reportId) + private void _deleteReport(Container c, User u, int reportId, ReportDescriptor descriptor) { SimpleFilter filter = new SimpleFilter(FieldKey.fromParts("ContainerId"), c.getId()); filter.addCondition(FieldKey.fromParts("RowId"), reportId); Table.delete(getTable(), filter); DatabaseReportCache.uncache(c); + + ReportAuditProvider.ReportAuditEvent event = new ReportAuditProvider.ReportAuditEvent(reportId, descriptor, c, "Report deleted"); + AuditLogService.get().addEvent(u, event); } @Override @@ -432,14 +436,17 @@ private ReportDB _saveDbReport(User user, Container c, String key, ReportDescrip else throw new RuntimeException("Can't save a report that is not stored in the database!"); - - if (null != reportId && reportExists(reportId.getRowId())) + boolean reportExists = null != reportId && reportExists(reportId.getRowId()); + if (reportExists) reportDB = Table.update(user, getTable(), reportDB, reportId.getRowId()); else reportDB = Table.insert(user, getTable(), reportDB); DatabaseReportCache.uncache(c); + ReportAuditProvider.ReportAuditEvent event = new ReportAuditProvider.ReportAuditEvent(reportDB, descriptor, c, reportExists ? "Report updated" : "Report created"); + AuditLogService.get().addEvent(user, event); + return reportDB; } From d6f86036de76f1b1b1fa71e8e9f90a1b7ab1c70e Mon Sep 17 00:00:00 2001 From: cnathe Date: Mon, 30 Sep 2024 16:01:45 -0500 Subject: [PATCH 10/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 8 ++++---- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 20 insertions(+), 20 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 2321a5554ca..c5ec2275a46 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2" + "@labkey/components": "5.7.0-fb-charts2410.4" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", - "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", + "version": "5.7.0-fb-charts2410.4", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.4.tgz", + "integrity": "sha512-maPZySl24z1gWzCGgc9Q7/KjOVRvazPPOAp7UQYrije6YNE1+dECoGnO1e0S+tynnHHT9zvOsh5NYpozvLIntg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/assay/package.json b/assay/package.json index 2744bd50b94..3ae768f3cfd 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2" + "@labkey/components": "5.7.0-fb-charts2410.4" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 605b1416c73..d5cf43f97b7 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2", + "@labkey/components": "5.7.0-fb-charts2410.4", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3488,9 +3488,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", - "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", + "version": "5.7.0-fb-charts2410.4", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.4.tgz", + "integrity": "sha512-maPZySl24z1gWzCGgc9Q7/KjOVRvazPPOAp7UQYrije6YNE1+dECoGnO1e0S+tynnHHT9zvOsh5NYpozvLIntg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/core/package.json b/core/package.json index 46a32447293..914c1888b51 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2", + "@labkey/components": "5.7.0-fb-charts2410.4", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index 26d3efd3c82..e384496a2e3 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2" + "@labkey/components": "5.7.0-fb-charts2410.4" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", - "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", + "version": "5.7.0-fb-charts2410.4", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.4.tgz", + "integrity": "sha512-maPZySl24z1gWzCGgc9Q7/KjOVRvazPPOAp7UQYrije6YNE1+dECoGnO1e0S+tynnHHT9zvOsh5NYpozvLIntg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/experiment/package.json b/experiment/package.json index 92635e43d07..9543c4f07e4 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2" + "@labkey/components": "5.7.0-fb-charts2410.4" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index f5faafd0585..f891eab3556 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2" + "@labkey/components": "5.7.0-fb-charts2410.4" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.2.tgz", - "integrity": "sha512-r+UdJADxTvFj3lFhoSdWWZKhMkuaQIS/XZfvCC5tZUfnHPQpu5HTEwwAXdORuKB+tzTPTPU9PxVzE5E2GV6BZg==", + "version": "5.7.0-fb-charts2410.4", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.4.tgz", + "integrity": "sha512-maPZySl24z1gWzCGgc9Q7/KjOVRvazPPOAp7UQYrije6YNE1+dECoGnO1e0S+tynnHHT9zvOsh5NYpozvLIntg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/pipeline/package.json b/pipeline/package.json index 2858912072d..060c355b16b 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.2" + "@labkey/components": "5.7.0-fb-charts2410.4" }, "devDependencies": { "@labkey/build": "7.7.1", From 28a11d2ff37c7a2cdfdb8bf48a9764f099bcb729 Mon Sep 17 00:00:00 2001 From: cnathe Date: Tue, 1 Oct 2024 08:20:35 -0500 Subject: [PATCH 11/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 16 ++++++++-------- core/package.json | 2 +- experiment/package-lock.json | 16 ++++++++-------- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 28 insertions(+), 28 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index c5ec2275a46..9deb89d2794 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.4" + "@labkey/components": "5.8.2-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.4", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.4.tgz", - "integrity": "sha512-maPZySl24z1gWzCGgc9Q7/KjOVRvazPPOAp7UQYrije6YNE1+dECoGnO1e0S+tynnHHT9zvOsh5NYpozvLIntg==", + "version": "5.8.2-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", + "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/assay/package.json b/assay/package.json index 3ae768f3cfd..32c3b59e865 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.4" + "@labkey/components": "5.8.2-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 1bf42fc8ff5..5f67b374859 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.8.2", + "@labkey/components": "5.8.2-fb-charts2410.1", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3447,9 +3447,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3488,12 +3488,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.8.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2.tgz", - "integrity": "sha512-/DLBB/g0by5tVI0aycrM8gCg6sFHS3N/VayJrrSqqf+lMUfWtJK7n2seBLJL3bPxY2R7eatWzoHG/PS4zKYYXA==", + "version": "5.8.2-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", + "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/core/package.json b/core/package.json index bbb7ae3358d..79d44c5c52b 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.8.2", + "@labkey/components": "5.8.2-fb-charts2410.1", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index 6bd79ddbf76..bee0025da25 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.8.2" + "@labkey/components": "5.8.2-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3255,9 +3255,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3296,12 +3296,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.8.2", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2.tgz", - "integrity": "sha512-/DLBB/g0by5tVI0aycrM8gCg6sFHS3N/VayJrrSqqf+lMUfWtJK7n2seBLJL3bPxY2R7eatWzoHG/PS4zKYYXA==", + "version": "5.8.2-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", + "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/experiment/package.json b/experiment/package.json index ca42464d7e5..383316bb9a4 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.8.2" + "@labkey/components": "5.8.2-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index f891eab3556..1126e3106a9 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.4" + "@labkey/components": "5.8.2-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.7.0-fb-charts2410.4", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.7.0-fb-charts2410.4.tgz", - "integrity": "sha512-maPZySl24z1gWzCGgc9Q7/KjOVRvazPPOAp7UQYrije6YNE1+dECoGnO1e0S+tynnHHT9zvOsh5NYpozvLIntg==", + "version": "5.8.2-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", + "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/pipeline/package.json b/pipeline/package.json index 060c355b16b..e1f180160f8 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.7.0-fb-charts2410.4" + "@labkey/components": "5.8.2-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", From 090821555c8448b8ae58d0a06f5358a7eacca477 Mon Sep 17 00:00:00 2001 From: cnathe Date: Tue, 1 Oct 2024 08:52:00 -0500 Subject: [PATCH 12/17] change reportId prop to integer --- .../labkey/query/reports/ReportAuditProvider.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/query/src/org/labkey/query/reports/ReportAuditProvider.java b/query/src/org/labkey/query/reports/ReportAuditProvider.java index bf3e7a85885..09d45f617e5 100644 --- a/query/src/org/labkey/query/reports/ReportAuditProvider.java +++ b/query/src/org/labkey/query/reports/ReportAuditProvider.java @@ -79,7 +79,7 @@ public List getDefaultVisibleColumns() public static class ReportAuditEvent extends AuditTypeEvent { - private String reportId; + private int reportId; private String reportName; private String reportKey; private String reportType; @@ -97,20 +97,20 @@ public ReportAuditEvent(@NotNull ReportDB report, @NotNull ReportDescriptor desc public ReportAuditEvent(int reportId, @NotNull ReportDescriptor descriptor, Container container, String comment) { super(EVENT_NAME, container, comment); - this.reportId = String.valueOf(reportId); + this.reportId = reportId; this.reportName = descriptor.getReportName(); this.reportKey = descriptor.getReportKey(); this.reportType = descriptor.getReportType(); } - public String getReportId() + public int getReportId() { return reportId; } - public void setReportId(@NotNull Integer reportId) + public void setReportId(int reportId) { - this.reportId = String.valueOf(reportId); + this.reportId = reportId; } public String getReportName() @@ -168,7 +168,7 @@ public ReportAuditDomainKind() super(EVENT_NAME); fields = new LinkedHashSet<>(); - fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_ID, PropertyType.STRING, "Report Id", null, true)); + fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_ID, PropertyType.INTEGER, "Report Id", null, true)); fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_NAME, PropertyType.STRING, "Report Name", null, true)); fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_KEY, PropertyType.STRING, "Report Key", null, false)); fields.add(createPropertyDescriptor(COLUMN_NAME_REPORT_TYPE, PropertyType.STRING, "Report Type", null, true)); From ae27f0ed3ba9ff1869bc071eeb2ad5ea1ec130d8 Mon Sep 17 00:00:00 2001 From: cnathe Date: Tue, 1 Oct 2024 14:28:47 -0500 Subject: [PATCH 13/17] Selenium test for LKS chart "make available to child folders" --- visualization/resources/web/vis/chartWizard/saveOptionsPanel.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js b/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js index 96897a04d90..82b686422b5 100644 --- a/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js +++ b/visualization/resources/web/vis/chartWizard/saveOptionsPanel.js @@ -107,7 +107,7 @@ Ext4.define('LABKEY.vis.SaveOptionsPanel', { name: 'reportInheritable', fieldLabel: 'Inherit', boxLabel: 'Make this report available in child folders', - hidden: this.shouldAllowInherit(), + hidden: !this.shouldAllowInherit(), labelWidth: 125, width: 400, }), From 8a8316041d09fc05f729a614f69a79d50c0d29a1 Mon Sep 17 00:00:00 2001 From: cnathe Date: Wed, 2 Oct 2024 08:51:56 -0500 Subject: [PATCH 14/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 16 ++++++++-------- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 24 insertions(+), 24 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 9deb89d2794..4012436fa62 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.8.2-fb-charts2410.1" + "@labkey/components": "5.10.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.8.2-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", - "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", + "version": "5.10.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", + "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/assay/package.json b/assay/package.json index 32c3b59e865..bb2dd845eea 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.8.2-fb-charts2410.1" + "@labkey/components": "5.10.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index a69a4df76ae..d492dc7e61b 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0", + "@labkey/components": "5.10.0-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3447,9 +3447,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1.tgz", - "integrity": "sha512-oVThx06H9Hnz2TJ92Xg8RRJ6nAARA4IsjgTXFyJNDNWOYcy7Zpl9Th2iujiboyg4mppZ6WaWJl9cNVaGi7DFYg==" + "version": "1.35.1-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", + "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3488,12 +3488,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0.tgz", - "integrity": "sha512-PYbBzAKLOJtn9PlaUxRjBwTnE/irLd6w8SHj2IM8YdJv8dH5dIibceR7rtejMVMEp5MmfNQf0LRfq3DIFpvyPA==", + "version": "5.10.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", + "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1", + "@labkey/api": "1.35.1-fb-charts2410.0", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/core/package.json b/core/package.json index cce17d9077d..9ea391e4c8c 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.10.0", + "@labkey/components": "5.10.0-fb-charts2410.0", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index bee0025da25..9a19f390fce 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.8.2-fb-charts2410.1" + "@labkey/components": "5.10.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.8.2-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", - "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", + "version": "5.10.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", + "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/experiment/package.json b/experiment/package.json index 383316bb9a4..6e88d7982ee 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.8.2-fb-charts2410.1" + "@labkey/components": "5.10.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 1126e3106a9..67cdfe940c9 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.8.2-fb-charts2410.1" + "@labkey/components": "5.10.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.8.2-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.8.2-fb-charts2410.1.tgz", - "integrity": "sha512-Wn70Apyq3srWMniDNqxAI/1797WUuwq74s7YQokL72rt5gOQL+cttb96vYwOI0orEzeLiocNSgXPORKWP1WIOg==", + "version": "5.10.0-fb-charts2410.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", + "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/pipeline/package.json b/pipeline/package.json index e1f180160f8..2b9a56bbf38 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.8.2-fb-charts2410.1" + "@labkey/components": "5.10.0-fb-charts2410.0" }, "devDependencies": { "@labkey/build": "7.7.1", From a443c72d9968afe7e19c7bdfc3829f393f720240 Mon Sep 17 00:00:00 2001 From: cnathe Date: Fri, 4 Oct 2024 08:58:06 -0500 Subject: [PATCH 15/17] update @labkey/components package version --- assay/package-lock.json | 8 ++++---- assay/package.json | 2 +- core/package-lock.json | 8 ++++---- core/package.json | 2 +- experiment/package-lock.json | 8 ++++---- experiment/package.json | 2 +- pipeline/package-lock.json | 8 ++++---- pipeline/package.json | 2 +- 8 files changed, 20 insertions(+), 20 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 4012436fa62..0e1d2cbdab8 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0" + "@labkey/components": "5.10.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2569,9 +2569,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", - "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", + "version": "5.10.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", + "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/assay/package.json b/assay/package.json index bb2dd845eea..dd3fa79518c 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0" + "@labkey/components": "5.10.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index d492dc7e61b..674aa977db1 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0", + "@labkey/components": "5.10.0-fb-charts2410.1", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3488,9 +3488,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", - "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", + "version": "5.10.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", + "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/core/package.json b/core/package.json index 9ea391e4c8c..65f55d09985 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0", + "@labkey/components": "5.10.0-fb-charts2410.1", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index 9a19f390fce..e80acc3ba5d 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0" + "@labkey/components": "5.10.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3296,9 +3296,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", - "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", + "version": "5.10.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", + "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/experiment/package.json b/experiment/package.json index 6e88d7982ee..04caf82fcae 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0" + "@labkey/components": "5.10.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 67cdfe940c9..175fec91dce 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0" + "@labkey/components": "5.10.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2742,9 +2742,9 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.0.tgz", - "integrity": "sha512-6LDEnJss66fsiYxd2ETDx3JCNXtFq8qTf8bKKvqDIzvFLpQn3TXTDDlhZlO3dVw8X/DyTKy2reLzFGAgOW7MyQ==", + "version": "5.10.0-fb-charts2410.1", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", + "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", "dependencies": { "@hello-pangea/dnd": "16.6.0", "@labkey/api": "1.35.1-fb-charts2410.0", diff --git a/pipeline/package.json b/pipeline/package.json index 2b9a56bbf38..edb72c11a9f 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.0" + "@labkey/components": "5.10.0-fb-charts2410.1" }, "devDependencies": { "@labkey/build": "7.7.1", From df6c7ee638cbe8adde34a5f08d8510bd06206431 Mon Sep 17 00:00:00 2001 From: cnathe Date: Fri, 4 Oct 2024 09:27:16 -0500 Subject: [PATCH 16/17] Fix for LKS chart rendering get data query to include xSub measure --- .../resources/web/vis/chartWizard/genericChartPanel.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/visualization/resources/web/vis/chartWizard/genericChartPanel.js b/visualization/resources/web/vis/chartWizard/genericChartPanel.js index ada18dbc776..db9fb40c017 100644 --- a/visualization/resources/web/vis/chartWizard/genericChartPanel.js +++ b/visualization/resources/web/vis/chartWizard/genericChartPanel.js @@ -921,7 +921,7 @@ Ext4.define('LABKEY.ext4.GenericChartPanel', { columns.push(this.autoColumnName.toString()); } - Ext4.each(['color', 'shape', 'series'], function(name) { + Ext4.each(['ySub', 'xSub', 'color', 'shape', 'series'], function(name) { if (measures[name]) { this.addMeasureForColumnQuery(columns, measures[name]); } From d34a0331534b275d9ce349deff46bcc1ab449b18 Mon Sep 17 00:00:00 2001 From: cnathe Date: Fri, 4 Oct 2024 15:16:29 -0500 Subject: [PATCH 17/17] update @labkey/components package version --- assay/package-lock.json | 16 ++++++++-------- assay/package.json | 2 +- core/package-lock.json | 16 ++++++++-------- core/package.json | 2 +- experiment/package-lock.json | 16 ++++++++-------- experiment/package.json | 2 +- pipeline/package-lock.json | 16 ++++++++-------- pipeline/package.json | 2 +- 8 files changed, 36 insertions(+), 36 deletions(-) diff --git a/assay/package-lock.json b/assay/package-lock.json index 0e1d2cbdab8..95267b7e561 100644 --- a/assay/package-lock.json +++ b/assay/package-lock.json @@ -8,7 +8,7 @@ "name": "assay", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1" + "@labkey/components": "5.11.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2528,9 +2528,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", - "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" + "version": "1.35.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.2.tgz", + "integrity": "sha512-ksOQ9GpkWyuL04p3h0lr2JvKwRfYR9xw5AH6KP5ZsCiX3ScqaJA30hHM6Qy+RD9vm+HRQvWsa41U3van+qJtaA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -2569,12 +2569,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", - "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", + "version": "5.11.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.11.0.tgz", + "integrity": "sha512-t+J0owiGHlFxfVL6ACy49x3Qu5KAtpJ0x22KmV5tycguSIZlyYsqZzv4vsyMnnE6vzDQ9Nvukh7IPsd/jhtZEQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1-fb-charts2410.0", + "@labkey/api": "1.35.2", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/assay/package.json b/assay/package.json index dd3fa79518c..ea51c3a864c 100644 --- a/assay/package.json +++ b/assay/package.json @@ -12,7 +12,7 @@ "clean": "rimraf resources/web/assay/gen && rimraf resources/views/gen && rimraf resources/web/gen" }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1" + "@labkey/components": "5.11.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/core/package-lock.json b/core/package-lock.json index 674aa977db1..303398cdc74 100644 --- a/core/package-lock.json +++ b/core/package-lock.json @@ -8,7 +8,7 @@ "name": "labkey-core", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1", + "@labkey/components": "5.11.0", "@labkey/themes": "1.3.3" }, "devDependencies": { @@ -3447,9 +3447,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", - "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" + "version": "1.35.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.2.tgz", + "integrity": "sha512-ksOQ9GpkWyuL04p3h0lr2JvKwRfYR9xw5AH6KP5ZsCiX3ScqaJA30hHM6Qy+RD9vm+HRQvWsa41U3van+qJtaA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3488,12 +3488,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", - "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", + "version": "5.11.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.11.0.tgz", + "integrity": "sha512-t+J0owiGHlFxfVL6ACy49x3Qu5KAtpJ0x22KmV5tycguSIZlyYsqZzv4vsyMnnE6vzDQ9Nvukh7IPsd/jhtZEQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1-fb-charts2410.0", + "@labkey/api": "1.35.2", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/core/package.json b/core/package.json index 65f55d09985..429500c431f 100644 --- a/core/package.json +++ b/core/package.json @@ -54,7 +54,7 @@ } }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1", + "@labkey/components": "5.11.0", "@labkey/themes": "1.3.3" }, "devDependencies": { diff --git a/experiment/package-lock.json b/experiment/package-lock.json index e80acc3ba5d..c2f73dc1d4b 100644 --- a/experiment/package-lock.json +++ b/experiment/package-lock.json @@ -8,7 +8,7 @@ "name": "experiment", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1" + "@labkey/components": "5.11.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -3255,9 +3255,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", - "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" + "version": "1.35.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.2.tgz", + "integrity": "sha512-ksOQ9GpkWyuL04p3h0lr2JvKwRfYR9xw5AH6KP5ZsCiX3ScqaJA30hHM6Qy+RD9vm+HRQvWsa41U3van+qJtaA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -3296,12 +3296,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", - "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", + "version": "5.11.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.11.0.tgz", + "integrity": "sha512-t+J0owiGHlFxfVL6ACy49x3Qu5KAtpJ0x22KmV5tycguSIZlyYsqZzv4vsyMnnE6vzDQ9Nvukh7IPsd/jhtZEQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1-fb-charts2410.0", + "@labkey/api": "1.35.2", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/experiment/package.json b/experiment/package.json index 04caf82fcae..b15072dfe53 100644 --- a/experiment/package.json +++ b/experiment/package.json @@ -13,7 +13,7 @@ "test-integration": "cross-env NODE_ENV=test jest --ci --runInBand -c test/js/jest.config.integration.js" }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1" + "@labkey/components": "5.11.0" }, "devDependencies": { "@labkey/build": "7.7.1", diff --git a/pipeline/package-lock.json b/pipeline/package-lock.json index 175fec91dce..f3fdd92acef 100644 --- a/pipeline/package-lock.json +++ b/pipeline/package-lock.json @@ -8,7 +8,7 @@ "name": "pipeline", "version": "0.0.0", "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1" + "@labkey/components": "5.11.0" }, "devDependencies": { "@labkey/build": "7.7.1", @@ -2701,9 +2701,9 @@ } }, "node_modules/@labkey/api": { - "version": "1.35.1-fb-charts2410.0", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.1-fb-charts2410.0.tgz", - "integrity": "sha512-l0/gSH/mYM3zsmsLxeBdduIGxNuoh0gsxq1JXiP3XthSqdEAzbb+o7whXtGK6poT0Mae5Ysd4cLJCOZjPGjPOA==" + "version": "1.35.2", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/api/-/@labkey/api-1.35.2.tgz", + "integrity": "sha512-ksOQ9GpkWyuL04p3h0lr2JvKwRfYR9xw5AH6KP5ZsCiX3ScqaJA30hHM6Qy+RD9vm+HRQvWsa41U3van+qJtaA==" }, "node_modules/@labkey/build": { "version": "7.7.1", @@ -2742,12 +2742,12 @@ } }, "node_modules/@labkey/components": { - "version": "5.10.0-fb-charts2410.1", - "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.10.0-fb-charts2410.1.tgz", - "integrity": "sha512-jMGhSiT19UGm0sVOG4YDWVS7C74e8vwC+Vv5o74XbpuhaWsFiINou/Mov4Dvcj2FRx46iMeflKTr9sX2A1vmLA==", + "version": "5.11.0", + "resolved": "https://labkey.jfrog.io/artifactory/api/npm/libs-client/@labkey/components/-/@labkey/components-5.11.0.tgz", + "integrity": "sha512-t+J0owiGHlFxfVL6ACy49x3Qu5KAtpJ0x22KmV5tycguSIZlyYsqZzv4vsyMnnE6vzDQ9Nvukh7IPsd/jhtZEQ==", "dependencies": { "@hello-pangea/dnd": "16.6.0", - "@labkey/api": "1.35.1-fb-charts2410.0", + "@labkey/api": "1.35.2", "@testing-library/dom": "~10.4.0", "@testing-library/jest-dom": "~6.5.0", "@testing-library/react": "~16.0.1", diff --git a/pipeline/package.json b/pipeline/package.json index edb72c11a9f..4b5ce33fa7d 100644 --- a/pipeline/package.json +++ b/pipeline/package.json @@ -14,7 +14,7 @@ "build-prod": "npm run clean && cross-env NODE_ENV=production PROD_SOURCE_MAP=source-map webpack --config node_modules/@labkey/build/webpack/prod.config.js --color --progress --profile" }, "dependencies": { - "@labkey/components": "5.10.0-fb-charts2410.1" + "@labkey/components": "5.11.0" }, "devDependencies": { "@labkey/build": "7.7.1",