diff --git a/.github/workflows/docker_img.yml b/.github/workflows/docker_img.yml
index 6f4387e6e00d644e4d658ac911ffc12d3be4dab1..3366bae8a3fcb7db4160d5c103970f0ae56d39e2 100644
--- a/.github/workflows/docker_img.yml
+++ b/.github/workflows/docker_img.yml
@@ -55,22 +55,26 @@ jobs:
- name: 'Set version variable & expmt feature flag'
run: |
+ GIT_HASH=$(git rev-parse --short HEAD)
+ echo "Setting GIT_HASH: $GIT_HASH"
+ echo "GIT_HASH=$GIT_HASH" >> $GITHUB_ENV
+
+ VERSION=$(jq -r '.version' package.json)
+ echo "Setting VERSION: $VERSION"
+ echo "VERSION=$VERSION" >> $GITHUB_ENV
if [[ "$GITHUB_REF" == 'refs/heads/master' ]] || [[ "$GITHUB_REF" == 'refs/heads/staging' ]]
then
- echo "Either master or staging, using package.json"
- VERSION=$(jq -r '.version' package.json)
+ echo "prod/staging build, do not enable experimental features"
else
- echo "Using git hash"
- VERSION=$(git rev-parse --short HEAD)
+ echo "dev bulid, enable experimental features"
echo "EXPERIMENTAL_FEATURE_FLAG=true" >> $GITHUB_ENV
fi
- echo "Setting VERSION: $VERSION"
- echo "VERSION=$VERSION" >> $GITHUB_ENV
- name: 'Build docker image'
run: |
DOCKER_BUILT_TAG=${{ env.DOCKER_REGISTRY }}siibra-explorer:$BRANCH_NAME
echo "Building $DOCKER_BUILT_TAG"
docker build \
+ --build-arg GIT_HASH=$GIT_HASH \
--build-arg VERSION=$VERSION \
--build-arg MATOMO_URL=$MATOMO_URL \
--build-arg MATOMO_ID=$MATOMO_ID \
diff --git a/.github/workflows/repo_sync_ebrains.yml b/.github/workflows/repo_sync_ebrains.yml
new file mode 100644
index 0000000000000000000000000000000000000000..548b768b0cd23b694a323919a7e64c71ee512d92
--- /dev/null
+++ b/.github/workflows/repo_sync_ebrains.yml
@@ -0,0 +1,20 @@
+name: "[repo-sync] repo-sync gitlab.ebrains.eu"
+
+on:
+ push:
+ branches:
+ - master
+
+jobs:
+ sync:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v2
+ - uses: wei/git-sync@v3
+ with:
+ source_repo: ${GITHUB_REPOSITORY}
+ source_branch: ${GITHUB_REF_NAME}
+ destination_repo: ${{ secrets.GITLAB_MIRROR_EBRAINS_DEST }}
+ destination_branch: ${GITHUB_REF_NAME}
+ destination_ssh_private_key: ${{ secrets.GITLAB_MIRROR_EBRAINS_SSH }}
+
diff --git a/Dockerfile b/Dockerfile
index bc614aacaf8dcef73da19f4a59285e6a31e51daf..3b017144535e201d255259c3b28ec4833d965e0c 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -24,6 +24,12 @@ ENV MATOMO_ID=${MATOMO_ID}
ARG EXPERIMENTAL_FEATURE_FLAG
ENV EXPERIMENTAL_FEATURE_FLAG=${EXPERIMENTAL_FEATURE_FLAG:-false}
+ARG GIT_HASH
+ENV GIT_HASH=${GIT_HASH:-unknownhash}
+
+ARG VERSION
+ENV VERSION=${VERSION:-unknownversion}
+
COPY . /iv
WORKDIR /iv
@@ -33,8 +39,6 @@ RUN node ./src/environments/parseEnv.js
# When building in local, where node_module already exist, prebuilt binary may throw an error
RUN rm -rf ./node_modules
-ARG VERSION
-ENV VERSION=${VERSION}
RUN npm i
RUN npm run build-aot
diff --git a/angular.json b/angular.json
index 0cd29585665857860e19b5f64ad44a737dbc86c2..68b6001b6a2bd6b197221bcf374c11f378bff086 100644
--- a/angular.json
+++ b/angular.json
@@ -37,9 +37,43 @@
"src/overwrite.scss",
"src/extra_styles.css"
],
- "scripts": [
- "worker/worker.js"
- ]
+ "scripts": [{
+ "input": "worker/worker.js",
+ "inject": false,
+ "bundleName": "worker"
+ },{
+ "input": "worker/worker-plotly.js",
+ "inject": false,
+ "bundleName": "worker-plotly"
+ },{
+ "input": "worker/worker-nifti.js",
+ "inject": false,
+ "bundleName": "worker-nifti"
+ },
+
+ {
+ "input": "third_party/catchSyntaxError.js",
+ "inject": false,
+ "bundleName": "catchSyntaxError"
+ },{
+ "input": "third_party/syntaxError.js",
+ "inject": false,
+ "bundleName": "syntaxError"
+ },
+
+ {
+ "input": "third_party/vanilla_nehuba.js",
+ "inject": false,
+ "bundleName": "vanilla_nehuba"
+ },{
+ "input": "export-nehuba/dist/min/main.bundle.js",
+ "inject": false,
+ "bundleName": "main.bundle"
+ },{
+ "input": "export-nehuba/dist/min/chunk_worker.bundle.js",
+ "inject": false,
+ "bundleName": "chunk_worker.bundle"
+ }]
},
"configurations": {
"production": {
@@ -50,7 +84,14 @@
}
],
"buildOptimizer": true,
- "optimization": false,
+ "optimization": {
+ "scripts": true,
+ "fonts": true,
+ "styles": {
+ "inlineCritical": false,
+ "minify": true
+ }
+ },
"vendorChunk": true,
"extractLicenses": false,
"sourceMap": false,
diff --git a/common/util.js b/common/util.js
index 9bf92e25986ce6b3a1e5650b550b1045a79e189b..9c66cad1bdf0964447c9e646b4035857efa24d14 100644
--- a/common/util.js
+++ b/common/util.js
@@ -47,6 +47,14 @@
return true
}
+ exports.arrayOrderedEql = function arrayOrderedEql(arr1, arr2) {
+ if (arr1.length !== arr2.length) return false
+ for (const idx in arr1) {
+ if (arr1[idx] !== arr2[idx]) return false
+ }
+ return true
+ }
+
exports.strToRgb = str => {
if (typeof str !== 'string') throw new Error(`strToRgb input must be typeof string !`)
diff --git a/common/util.spec.js b/common/util.spec.js
index 2af63766c202999af0a9ca2ccf57a341d947f0ee..b817b6607331d06fdb9373c20d70e1d4eb90e08a 100644
--- a/common/util.spec.js
+++ b/common/util.spec.js
@@ -1,4 +1,4 @@
-import { getIdFromFullId, strToRgb, verifyPositionArg } from './util'
+import { getIdFromFullId, strToRgb, verifyPositionArg, arrayOrderedEql } from './util'
describe('common/util.js', () => {
describe('getIdFromFullId', () => {
@@ -117,4 +117,37 @@ describe('common/util.js', () => {
})
})
})
+
+ describe('> arrayOrderedEql', () => {
+ describe('> if array eql', () => {
+ it('> returns true', () => {
+ expect(
+ arrayOrderedEql(['foo', 3], ['foo', 3])
+ ).toBeTrue()
+ })
+ })
+ describe('> if array not eql', () => {
+ describe('> not ordered eql', () => {
+ it('> returns false', () => {
+ expect(
+ arrayOrderedEql(['foo', 'bar'], ['bar', 'foo'])
+ ).toBeFalse()
+ })
+ })
+ describe('> item not eql', () => {
+ it('> returns false', () => {
+ expect(
+ arrayOrderedEql(['foo', null], ['foo', undefined])
+ ).toBeFalse()
+ })
+ })
+ describe('> size not eql', () => {
+ it('> returns false', () => {
+ expect(
+ arrayOrderedEql([], [1])
+ ).toBeFalse()
+ })
+ })
+ })
+ })
})
diff --git a/deploy/auth/hbp-oidc-v2.js b/deploy/auth/hbp-oidc-v2.js
index 102aa25c236b6a61801fcae04541622af23ea676..d5427480817ab9ad0a58e9a2c2a103eafa6d5f3b 100644
--- a/deploy/auth/hbp-oidc-v2.js
+++ b/deploy/auth/hbp-oidc-v2.js
@@ -78,8 +78,4 @@ module.exports = {
console.error('oidcv2 auth error', e)
}
},
- getClient: async () => {
- await memoizedInit()
- return client
- }
}
diff --git a/deploy/auth/index.spec.js b/deploy/auth/index.spec.js
index 84691ddd6ccae9169f321858453c2eacf0dcb28c..8e33c30bd02054d71c31b862b22928d12f359928 100644
--- a/deploy/auth/index.spec.js
+++ b/deploy/auth/index.spec.js
@@ -8,7 +8,7 @@ const appGetStub = sinon.stub()
describe('auth/index.js', () => {
before(() => {
require.cache[require.resolve('./util')] = {
- exports: { initPassportJs: initPassportJsStub }
+ exports: { initPassportJs: initPassportJsStub, objStoreDb: new Map() }
}
require.cache[require.resolve('./hbp-oidc-v2')] = {
exports: {
diff --git a/deploy/csp/index.js b/deploy/csp/index.js
index c0a8f43d54dc83c5cabc49349b6f0acaf7b791bb..b3e439e8160c050734f23307341d71b782705dc2 100644
--- a/deploy/csp/index.js
+++ b/deploy/csp/index.js
@@ -116,7 +116,7 @@ module.exports = {
'unpkg.com/kg-dataset-previewer@1.2.0/', // preview component
'cdnjs.cloudflare.com/ajax/libs/mathjax/', // math jax
'https://unpkg.com/three-surfer@0.0.10/dist/bundle.js', // for threeSurfer (freesurfer support in browser)
- 'https://unpkg.com/ng-layer-tune@0.0.4/dist/ng-layer-tune/ng-layer-tune.esm.js', // needed for ng layer control
+ 'https://unpkg.com/ng-layer-tune@0.0.5/dist/ng-layer-tune/', // needed for ng layer control
(req, res) => res.locals.nonce ? `'nonce-${res.locals.nonce}'` : null,
...SCRIPT_SRC,
...WHITE_LIST_SRC,
diff --git a/deploy/package-lock.json b/deploy/package-lock.json
index 5d707a769ac807f9c1473f3940fe0464dba4657b..b8399147d5ab12a520ea0f2068cba33945db6f9b 100644
--- a/deploy/package-lock.json
+++ b/deploy/package-lock.json
@@ -379,11 +379,6 @@
}
}
},
- "bowser": {
- "version": "2.5.4",
- "resolved": "https://registry.npmjs.org/bowser/-/bowser-2.5.4.tgz",
- "integrity": "sha512-74GGwfc2nzYD19JCiA0RwCxdq7IY5jHeEaSrrgm/5kusEuK+7UK0qDG3gyzN47c4ViNyO4osaKtZE+aSV6nlpQ=="
- },
"brace-expansion": {
"version": "1.1.11",
"resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-1.1.11.tgz",
@@ -436,11 +431,6 @@
"resolved": "https://registry.npmjs.org/camelcase/-/camelcase-5.3.1.tgz",
"integrity": "sha512-L28STB170nwWS63UjtlEOE3dldQApaJXZkOI1uMFfzf3rRuPegHaHesyee+YxQ+W6SvRDQV6UrdOdRiR153wJg=="
},
- "camelize": {
- "version": "1.0.0",
- "resolved": "https://registry.npmjs.org/camelize/-/camelize-1.0.0.tgz",
- "integrity": "sha1-FkpUg+Yw+kMh5a8HAg5TGDGyYJs="
- },
"caseless": {
"version": "0.12.0",
"resolved": "https://registry.npmjs.org/caseless/-/caseless-0.12.0.tgz",
@@ -513,6 +503,11 @@
"wrap-ansi": "^2.0.0"
}
},
+ "clone": {
+ "version": "1.0.4",
+ "resolved": "https://registry.npmjs.org/clone/-/clone-1.0.4.tgz",
+ "integrity": "sha1-2jCcwmPfFZlMaIypAheco8fNfH4="
+ },
"clone-response": {
"version": "1.0.2",
"resolved": "https://registry.npmjs.org/clone-response/-/clone-response-1.0.2.tgz",
@@ -581,11 +576,6 @@
"resolved": "https://registry.npmjs.org/content-disposition/-/content-disposition-0.5.2.tgz",
"integrity": "sha1-DPaLud318r55YcOoUXjLhdunjLQ="
},
- "content-security-policy-builder": {
- "version": "2.1.0",
- "resolved": "https://registry.npmjs.org/content-security-policy-builder/-/content-security-policy-builder-2.1.0.tgz",
- "integrity": "sha512-/MtLWhJVvJNkA9dVLAp6fg9LxD2gfI6R2Fi1hPmfjYXSahJJzcfvoeDOxSyp4NvxMuwWv3WMssE9o31DoULHrQ=="
- },
"content-type": {
"version": "1.0.4",
"resolved": "https://registry.npmjs.org/content-type/-/content-type-1.0.4.tgz",
@@ -680,11 +670,6 @@
"assert-plus": "^1.0.0"
}
},
- "dasherize": {
- "version": "2.0.0",
- "resolved": "https://registry.npmjs.org/dasherize/-/dasherize-2.0.0.tgz",
- "integrity": "sha1-bYCcnNDPe7iVLYD8hPoT1H3bEwg="
- },
"debug": {
"version": "2.6.9",
"resolved": "https://registry.npmjs.org/debug/-/debug-2.6.9.tgz",
@@ -715,6 +700,14 @@
"type-detect": "^4.0.0"
}
},
+ "defaults": {
+ "version": "1.0.3",
+ "resolved": "https://registry.npmjs.org/defaults/-/defaults-1.0.3.tgz",
+ "integrity": "sha1-xlYFHpgX2f8I7YgUd/P+QBnz730=",
+ "requires": {
+ "clone": "^1.0.2"
+ }
+ },
"defer-to-connect": {
"version": "2.0.0",
"resolved": "https://registry.npmjs.org/defer-to-connect/-/defer-to-connect-2.0.0.tgz",
@@ -956,6 +949,11 @@
}
}
},
+ "express-rate-limit": {
+ "version": "5.5.1",
+ "resolved": "https://registry.npmjs.org/express-rate-limit/-/express-rate-limit-5.5.1.tgz",
+ "integrity": "sha512-MTjE2eIbHv5DyfuFz4zLYWxpqVhEhkTiwFGuB74Q9CSou2WHO52nlE5y3Zlg6SIsiYUIPj6ifFxnkPz6O3sIUg=="
+ },
"express-session": {
"version": "1.15.6",
"resolved": "https://registry.npmjs.org/express-session/-/express-session-1.15.6.tgz",
@@ -1195,15 +1193,9 @@
"dev": true
},
"helmet-csp": {
- "version": "2.9.1",
- "resolved": "https://registry.npmjs.org/helmet-csp/-/helmet-csp-2.9.1.tgz",
- "integrity": "sha512-HgdXSJ6AVyXiy5ohVGpK6L7DhjI9KVdKVB1xRoixxYKsFXFwoVqtLKgDnfe3u8FGGKf9Ml9k//C9rnncIIAmyA==",
- "requires": {
- "bowser": "2.5.4",
- "camelize": "1.0.0",
- "content-security-policy-builder": "2.1.0",
- "dasherize": "2.0.0"
- }
+ "version": "3.4.0",
+ "resolved": "https://registry.npmjs.org/helmet-csp/-/helmet-csp-3.4.0.tgz",
+ "integrity": "sha512-a+YgzWw6dajqhQfb6ktxil0FsQuWTKzrLSUfy55dxS8fuvl1jidTIMPZ2udN15mjjcpBPgTHNHGF5tyWKYyR8w=="
},
"http-cache-semantics": {
"version": "4.0.4",
@@ -2144,6 +2136,15 @@
"resolved": "https://registry.npmjs.org/range-parser/-/range-parser-1.2.0.tgz",
"integrity": "sha1-9JvmtIeJTdxA3MlKMi9hEJLgDV4="
},
+ "rate-limit-redis": {
+ "version": "2.1.0",
+ "resolved": "https://registry.npmjs.org/rate-limit-redis/-/rate-limit-redis-2.1.0.tgz",
+ "integrity": "sha512-6SAsTCzY0v6UCIKLOLLYqR2XzFmgdtF7jWXlSPq2FrNIZk8tZ7xwBvyGW7GFMCe5I4S9lYNdrSJ9E84rz3/CpA==",
+ "requires": {
+ "defaults": "^1.0.3",
+ "redis": "^3.0.2"
+ }
+ },
"raw-body": {
"version": "2.4.0",
"resolved": "https://registry.npmjs.org/raw-body/-/raw-body-2.4.0.tgz",
diff --git a/deploy/package.json b/deploy/package.json
index 8a165a87192525bf0b6bcd9ba86df86fe5ddb30c..04cdc908c505ae7a5d8663fbfae313f561b0be80 100644
--- a/deploy/package.json
+++ b/deploy/package.json
@@ -17,15 +17,17 @@
"connect-redis": "^5.0.0",
"cookie-parser": "^1.4.5",
"express": "^4.16.4",
+ "express-rate-limit": "^5.5.1",
"express-session": "^1.15.6",
"got": "^10.5.5",
"hbp-seafile": "^0.2.0",
- "helmet-csp": "^2.8.0",
+ "helmet-csp": "^3.4.0",
"lru-cache": "^5.1.1",
"memorystore": "^1.6.1",
"nomiseco": "0.0.2",
"openid-client": "^4.4.0",
"passport": "^0.4.0",
+ "rate-limit-redis": "^2.1.0",
"redis": "^3.1.2",
"request": "^2.88.0",
"showdown": "^1.9.1",
diff --git a/deploy/saneUrl/depcObjStore.js b/deploy/saneUrl/depcObjStore.js
index a46d6c845936fef10b77b4ac9bb9ed278c8a00e9..c67876686470650d08217cc72141a6936f703511 100644
--- a/deploy/saneUrl/depcObjStore.js
+++ b/deploy/saneUrl/depcObjStore.js
@@ -21,6 +21,10 @@ class Store {
})
}
+ async del(id){
+ // noop
+ }
+
async set(id, val){
throw new Error(`Object store is deprecated. Please use seafile storage instead`)
}
diff --git a/deploy/saneUrl/index.js b/deploy/saneUrl/index.js
index 06659fa88748f40db5d57459c43c896b03e7babb..72d0f9f9e2a8a13f98f96b7989c167ae61e2f47a 100644
--- a/deploy/saneUrl/index.js
+++ b/deploy/saneUrl/index.js
@@ -1,44 +1,33 @@
const express = require('express')
const router = express.Router()
-const { FallbackStore: Store, NotFoundError } = require('./store')
+const { GitlabSnippetStore: Store, NotFoundError } = require('./store')
const { Store: DepcStore } = require('./depcObjStore')
+const RateLimit = require('express-rate-limit')
+const RedisStore = require('rate-limit-redis')
+const { redisURL } = require('../lruStore')
+const { ProxyStore, NotExactlyPromiseAny } = require('./util')
const store = new Store()
const depStore = new DepcStore()
-const { HOSTNAME, HOST_PATHNAME } = process.env
+const proxyStore = new ProxyStore(store)
-const acceptHtmlProg = /text\/html/i
-
-const getFileFromStore = async (name, store) => {
- try {
- const value = await store.get(name)
- const json = JSON.parse(value)
- const { expiry } = json
- if ( expiry && ((Date.now() - expiry) > 0) ) {
- return null
- }
-
- return value
- } catch (e) {
- if (e instanceof NotFoundError) {
- return null
- }
- throw e
- }
-}
+const {
+ HOSTNAME,
+ HOST_PATHNAME,
+ DISABLE_LIMITER,
+} = process.env
-const getFile = async name => {
- const value = await getFileFromStore(name, depStore)
- || await getFileFromStore(name, store)
+const limiter = new RateLimit({
+ windowMs: 1e3 * 5,
+ max: 5,
+ ...( redisURL ? { store: new RedisStore({ redisURL }) } : {} )
+})
+const passthrough = (_, __, next) => next()
- return value
-}
+const acceptHtmlProg = /text\/html/i
-const hardCodedMap = new Map([
- ['whs4', '#/a:minds:core:parcellationatlas:v1.0.0:522b368e-49a3-49fa-88d3-0870a307974a/t:minds:core:referencespace:v1.0.0:d5717c4a-0fa1-46e6-918c-b8003069ade8/p:minds:core:parcellationatlas:v1.0.0:ebb923ba-b4d5-4b82-8088-fa9215c2e1fe-v4/@:0.0.0.-W000.._eCwg.2-FUe3._-s_W.2_evlu..kxV..0.0.0..8Yu'],
- ['mebrains', '#/a:juelich:iav:atlas:v1.0.0:monkey/t:minds:core:referencespace:v1.0.0:MEBRAINS_T1.masked/p:minds:core:parcellationatlas:v1.0.0:mebrains-tmp-id/@:0.0.0.-W000.._eCwg.2-FUe3._-s_W.2_evlu..7LIx..0.0.0..1LSm']
-])
+const REAL_HOSTNAME = `${HOSTNAME}${HOST_PATHNAME || ''}/`
router.get('/:name', async (req, res) => {
const { name } = req.params
@@ -47,31 +36,34 @@ router.get('/:name', async (req, res) => {
const redirectFlag = acceptHtmlProg.test(headers['accept'])
try {
- const REAL_HOSTNAME = `${HOSTNAME}${HOST_PATHNAME || ''}/`
- const hardcodedRedir = hardCodedMap.get(name)
- if (hardcodedRedir) {
- if (redirectFlag) res.redirect(`${REAL_HOSTNAME}${hardcodedRedir}`)
- else res.status(200).send(hardcodedRedir)
- return
- }
- const value = await getFile(name)
- if (!value) throw new NotFoundError()
- const json = JSON.parse(value)
- const { queryString } = json
+ const json = await NotExactlyPromiseAny([
+ ProxyStore.StaticGet(depStore, req, name),
+ proxyStore.get(req, name)
+ ])
+ const { queryString, hashPath, ...rest } = json
- if (redirectFlag) res.redirect(`${REAL_HOSTNAME}?${queryString}`)
- else res.status(200).send(value)
+ const xtraRoutes = []
+ for (const key in rest) {
+ if (/^x-/.test(key)) xtraRoutes.push(`${key}:${name}`)
+ }
+ if (redirectFlag) {
+ if (queryString) return res.redirect(`${REAL_HOSTNAME}?${queryString}`)
+ if (hashPath) return res.redirect(`${REAL_HOSTNAME}#${hashPath}/${xtraRoutes.join('/')}`)
+ } else {
+ return res.status(200).send(json)
+ }
} catch (e) {
+ const notFoundFlag = e instanceof NotFoundError
if (redirectFlag) {
const REAL_HOSTNAME = `${HOSTNAME}${HOST_PATHNAME || ''}/`
res.cookie(
'iav-error',
- e instanceof NotFoundError ? `${name}
+ notFoundFlag ? `${name}
not found` : `error while fetching ${name}.`,
{
@@ -82,13 +74,25 @@ router.get('/:name', async (req, res) => {
)
return res.redirect(REAL_HOSTNAME)
}
- if (e instanceof NotFoundError) return res.status(404).end()
+ if (notFoundFlag) return res.status(404).end()
else return res.status(500).send(e.toString())
}
})
router.post('/:name',
- (_req, res) => res.status(410).end()
+ DISABLE_LIMITER ? passthrough : limiter,
+ express.json(),
+ async (req, res) => {
+ if (req.headers['x-noop']) return res.status(200).end()
+ const { name } = req.params
+ try {
+ await proxyStore.set(req, name, req.body)
+ res.status(201).end()
+ } catch (e) {
+ console.log(e.body)
+ res.status(500).send(e.toString())
+ }
+ }
)
router.use((_, res) => {
diff --git a/deploy/saneUrl/store.js b/deploy/saneUrl/store.js
index 656a070f4fcd5805d816c54390449546fa2885a1..9ef35e8408fd089bb7778ed02d68bd3e18fede58 100644
--- a/deploy/saneUrl/store.js
+++ b/deploy/saneUrl/store.js
@@ -1,420 +1,112 @@
-const redis = require('redis')
-const { promisify } = require('util')
-const { getClient } = require('../auth/hbp-oidc-v2')
-const { jwtDecode } = require('../auth/oidc')
const request = require('request')
-const HBP_OIDC_V2_REFRESH_TOKEN_KEY = `HBP_OIDC_V2_REFRESH_TOKEN_KEY` // only insert valid refresh token. needs to be monitored to ensure always get new refresh token(s)
-const HBP_OIDC_V2_ACCESS_TOKEN_KEY = `HBP_OIDC_V2_ACCESS_TOKEN_KEY` // only insert valid access token. if expired, get new one via refresh token, then pop & push
-const HBP_OIDC_V2_UPDATE_CHAN = `HBP_OIDC_V2_UPDATE_CHAN` // stringified JSON key val of above three, with time stamp
-const HBP_OIDC_V2_UPDATE_KEY = `HBP_OIDC_V2_UPDATE_KEY`
-const HBP_DATAPROXY_READURL = `HBP_DATAPROXY_READURL`
-const HBP_DATAPROXY_WRITEURL = `HBP_DATAPROXY_WRITEURL`
+const apiPath = '/api/v4'
+const saneUrlVer = `0.0.1`
+const titlePrefix = `[saneUrl]`
const {
__DEBUG__,
-
- HBP_V2_REFRESH_TOKEN,
- HBP_V2_ACCESS_TOKEN,
-
- DATA_PROXY_URL,
- DATA_PROXY_BUCKETNAME,
-
+ GITLAB_ENDPOINT,
+ GITLAB_PROJECT_ID,
+ GITLAB_TOKEN
} = process.env
class NotFoundError extends Error{}
-// not part of class, since class is exported, and prototype of class can be easily changed
-// using stderr instead of console.error, as by default, logs are collected via fluentd
-// debug messages should not be kept as logs
-function log(message){
- if (__DEBUG__) {
- process.stderr.write(`__DEBUG__`)
- process.stdout.write(`\n`)
- if (typeof message === 'object') {
- process.stderr.write(JSON.stringify(message, null, 2))
- }
- if (typeof message === 'number' || typeof message === 'string') {
- process.stdout.write(message)
- }
- process.stdout.write(`\n`)
- }
-}
-
-
-function _checkValid(urlString){
- log({
- breakpoint: '_checkValid',
- payload: urlString
- })
- if (!urlString) return false
- const url = new URL(urlString)
- const expiry = url.searchParams.get('temp_url_expires')
- return (new Date() - new Date(expiry)) < 1e3 * 10
-}
-
-class FallbackStore {
- async get(){
- throw new Error(`saneurl is currently offline for maintainence.`)
- }
- async set(){
- throw new Error(`saneurl is currently offline for maintainence.`)
- }
- async healthCheck() {
- return false
- }
-}
+class NotImplemented extends Error{}
-class Store {
+class GitlabSnippetStore {
constructor(){
- throw new Error(`Not implemented`)
-
- this.healthFlag = false
- this.seafileHandle = null
- this.seafileRepoId = null
-
- /**
- * setup redis(or mock) client
- */
- this.redisClient = redisURL
- ? redis.createClient({
- url: redisURL
- })
- : ({
- onCbs: [],
- keys: {},
- get: async (key, cb) => {
- await Promise.resolve()
- cb(null, this.keys[key])
- },
- set: async (key, value, cb) => {
- await Promise.resolve()
- this.keys[key] = value
- cb(null)
- },
- on(eventName, cb) {
- if (eventName === 'message') {
- this.onCbs.push(cb)
- }
- },
- publish(channel, message){
- for (const cb of this.onCbs){
- cb(channel, message)
- }
- },
- quit(){}
- })
-
- this.redisUtil = {
- asyncGet: promisify(this.redisClient.get).bind(this.redisClient),
- asyncSet: promisify(this.redisClient.set).bind(this.redisClient),
- }
-
- this.pending = {}
- this.keys = {}
-
- this.redisClient.on('message', async (chan, mess) => {
- /**
- * only liten to HBP_OIDC_V2_UPDATE_CHAN update
- */
- if (chan === HBP_OIDC_V2_UPDATE_CHAN) {
- try {
- const { pending, update } = JSON.parse(mess)
- this.pending = pending
- for (const key in update) {
- try {
- this.keys[key] = await this.redisUtil.asyncGet(key)
- console.log('on message get key', key, this.keys[key])
- } catch (e) {
- console.error(`[saneUrl][store.js] get key ${key} error`)
- }
- }
- } catch (e) {
- console.error(`[saneUrl][store.js] parse message HBP_OIDC_V2_UPDATE_CHAN error`)
- }
- }
- })
- this.init()
-
- /**
- * check expiry
- */
- this.intervalRef = setInterval(() => {
- this.checkExpiry()
- }, 1000 * 60)
- }
-
- async init() {
- this.keys = {
- [HBP_OIDC_V2_REFRESH_TOKEN_KEY]: (await this.redisUtil.asyncGet(HBP_OIDC_V2_REFRESH_TOKEN_KEY)) || HBP_V2_REFRESH_TOKEN,
- [HBP_OIDC_V2_ACCESS_TOKEN_KEY]: (await this.redisUtil.asyncGet(HBP_OIDC_V2_ACCESS_TOKEN_KEY)) || HBP_V2_ACCESS_TOKEN,
- [HBP_DATAPROXY_READURL]: await this.redisUtil.asyncGet(HBP_DATAPROXY_READURL),
- [HBP_DATAPROXY_WRITEURL]: await this.redisUtil.asyncGet(HBP_DATAPROXY_WRITEURL),
- }
-
- this.healthFlag = true
- }
-
- async checkExpiry(){
-
- const {
- [HBP_OIDC_V2_REFRESH_TOKEN_KEY]: refreshToken,
- [HBP_OIDC_V2_ACCESS_TOKEN_KEY]: accessToken
- } = this.keys
-
- log({
- breakpoint: 'async checkExpiry',
- })
-
- /**
- * if access token is absent
- * try to refresh token, without needing to check exp
- */
- if (!accessToken) {
- await this.doRefreshTokens()
- return true
- }
- const { exp: refreshExp } = jwtDecode(refreshToken)
- const { exp: accessExp } = jwtDecode(accessToken)
-
- const now = new Date().getTime() / 1000
-
- if (now > refreshExp) {
- console.warn(`[saneUrl] refresh token expired... Need a new refresh token`)
- return false
- }
- if (now > accessExp) {
- console.log(`[saneUrl] access token expired. Refreshing access token...`)
- await this.doRefreshTokens()
- console.log(`[saneUrl] access token successfully refreshed...`)
- return true
+ if (
+ GITLAB_ENDPOINT
+ && GITLAB_PROJECT_ID
+ && GITLAB_TOKEN
+ ) {
+ this.url = `${GITLAB_ENDPOINT}${apiPath}/projects/${GITLAB_PROJECT_ID}/snippets`
+ this.token = GITLAB_TOKEN
+ return
}
-
- return true
+ throw new NotImplemented('Gitlab snippet key value store cannot be configured')
}
- async doRefreshTokens(){
-
- log({
- breakpoint: 'doing refresh tokens'
+ _promiseRequest(...arg) {
+ return new Promise((rs, rj) => {
+ request(...arg, (err, resp, body) => {
+ if (err) return rj(err)
+ if (resp.statusCode >= 400) return rj(resp)
+ rs(body)
+ })
})
-
- /**
- * first, check if another process/pod is currently updating
- * if they are, give them 1 minute to complete the process
- * (usually takes takes less than 5 seconds)
- */
- const pendingStart = this.pending[HBP_OIDC_V2_REFRESH_TOKEN_KEY] &&
- this.pending[HBP_OIDC_V2_REFRESH_TOKEN_KEY].start + 1000 * 60
- const now = new Date()
- if (pendingStart && pendingStart > now) {
- return
- }
-
- /**
- * When start refreshing the tokens, set pending attribute, and start timestamp
- */
- const payload = {
- pending: {
- ...this.pending,
- [HBP_OIDC_V2_REFRESH_TOKEN_KEY]: {
- start: Date.now()
- }
- }
- }
- await this.redisUtil.asyncSet(HBP_OIDC_V2_UPDATE_KEY, JSON.stringify(payload))
- this.redisClient.publish(HBP_OIDC_V2_UPDATE_CHAN, JSON.stringify(payload))
-
- const client = await getClient()
- const tokenset = await client.refresh(this.keys[HBP_OIDC_V2_REFRESH_TOKEN_KEY])
- const { access_token: accessToken, refresh_token: refreshToken } = tokenset
-
- const { exp: accessTokenExp } = jwtDecode(accessToken)
- const { exp: refreshTokenExp } = jwtDecode(refreshToken)
-
- if (refreshTokenExp - accessTokenExp < 60 * 60 ) {
- console.warn(`[saneUrl] refreshToken expires within 1 hour of access token! ${accessTokenExp} ${refreshTokenExp}`)
- }
-
- /**
- * once tokens have been refreshed, set them in the redis store first
- * Then publish the update message
- */
- await this.redisUtil.asyncSet(HBP_OIDC_V2_REFRESH_TOKEN_KEY, refreshToken)
- await this.redisUtil.asyncSet(HBP_OIDC_V2_ACCESS_TOKEN_KEY, accessToken)
- this.redisClient.publish(HBP_OIDC_V2_UPDATE_CHAN, JSON.stringify({
- keys: {
- [HBP_OIDC_V2_REFRESH_TOKEN_KEY]: refreshToken,
- [HBP_OIDC_V2_ACCESS_TOKEN_KEY]: accessToken,
- },
- pending: {
- ...this.pending,
- [HBP_OIDC_V2_REFRESH_TOKEN_KEY]: null
- }
- }))
- this.keys = {
- [HBP_OIDC_V2_REFRESH_TOKEN_KEY]: refreshToken,
- [HBP_OIDC_V2_ACCESS_TOKEN_KEY]: accessToken,
- }
}
- async _getTmpUrl(permission){
- if (permission !== 'read' && permission !== 'write') throw new Error(`permission need to be either read or write!`)
- const method = permission === 'read'
- ? 'GET'
- : 'PUT'
- log({
- breakpoint: 'access key',
- access: this.keys[HBP_OIDC_V2_ACCESS_TOKEN_KEY]
- })
- const { url } = await new Promise((rs, rj) => {
- const payload = {
+ _request({ addPath = '', method = 'GET', headers = {}, opt = {} } = {}) {
+ return new Promise((rs, rj) => {
+ request(`${this.url}${addPath}`, {
method,
- uri: `${DATA_PROXY_URL}/tempurl/${DATA_PROXY_BUCKETNAME}?lifetime=very_long`,
headers: {
- 'Authorization': `Bearer ${this.keys[HBP_OIDC_V2_ACCESS_TOKEN_KEY]}`
- }
- }
- log({
- breakpoint: '_getTmpUrl',
- payload
- })
- request(payload, (err, resp, body) => {
+ 'PRIVATE-TOKEN': this.token,
+ ...headers
+ },
+ ...opt
+ }, (err, resp, body) => {
if (err) return rj(err)
- if (resp.statusCode >= 400) return rj(new Error(`${resp.statusCode}: ${resp.statusMessage}`))
- if (resp.statusCode === 200) {
- rs(JSON.parse(body))
- return
- }
- return rj(new Error(`[saneurl] [get] unknown error`))
+ if (resp.statusCode >= 400) return rj(resp)
+ return rs(body)
})
})
-
- return url
- }
-
- _getTmplTag(key, _strings, _proxyUrl, bucketName, id){
- const defaultReturn = `${_strings[0]}${_proxyUrl}${_strings[1]}${bucketName}${_strings[2]}${id}`
- if (!key) return defaultReturn
- const {
- [key]: urlOfInterest
- } = this.keys
- if (!urlOfInterest) return defaultReturn
- const url = new URL(urlOfInterest)
- url.pathname += id
- return url
- }
-
- _getWriteTmplTag(...args) {
- return this._getTmplTag(HBP_DATAPROXY_WRITEURL, ...args)
- }
-
- _getReadTmplTag(...args){
- return this._getTmplTag(HBP_DATAPROXY_READURL, ...args)
}
async get(id) {
- log({
- breakpoint: 'async get',
- id
- })
- const {
- [HBP_DATAPROXY_READURL]: readUrl
- } = this.keys
-
- if (!_checkValid(readUrl)) {
- log({
- breakpoint: 'read url not valid, getting new url'
- })
- const url = await this._getTmpUrl('read')
- const payload = {
- keys: {
- [HBP_DATAPROXY_READURL]: url
- }
- }
- this.redisClient.publish(HBP_OIDC_V2_UPDATE_CHAN, JSON.stringify(payload))
- this.redisUtil.asyncSet(HBP_DATAPROXY_READURL, url)
- this.keys[HBP_DATAPROXY_READURL] = url
+ const list = JSON.parse(await this._request())
+ const found = list.find(item => item.title === `${titlePrefix}${id}`)
+ if (!found) throw new NotFoundError()
+ const payloadObj = found.files.find(f => f.path === 'payload')
+ if (!payloadObj) {
+ console.error(`id found, but payload not found... this is strange. Check id: ${id}`)
+ throw new NotFoundError()
}
-
- return await new Promise((rs, rj) => {
- request({
- method: 'GET',
- uri: this._getReadTmplTag`${DATA_PROXY_URL}/buckets/${DATA_PROXY_BUCKETNAME}/${encodeURIComponent(id)}`,
- }, (err, resp, body) => {
- if (err) return rj(err)
- if (resp.statusCode === 404) return rj(new NotFoundError())
- if (resp.statusCode >= 400) return rj(new Error(`${resp.statusCode}: ${resp.statusMessage}`))
- if (resp.statusCode === 200) return rs(body)
- return rj(new Error(`[saneurl] [get] unknown error`))
- })
- })
+ if (!payloadObj.raw_url) {
+ throw new Error(`payloadObj.raw_url not found!`)
+ }
+ return await this._promiseRequest(payloadObj.raw_url)
}
- async _set(id, value) {
- const {
- [HBP_DATAPROXY_WRITEURL]: writeUrl
- } = this.keys
-
- if (!_checkValid(writeUrl)) {
- log({
- breakpoint: '_set',
- message: 'write url not valid, getting new one'
- })
- const url = await this._getTmpUrl('write')
- const payload = {
- keys: {
- [HBP_DATAPROXY_WRITEURL]: url
+ async set(id, value) {
+ return await this._request({
+ method: 'POST',
+ headers: {
+ 'Content-Type': 'application/json'
+ },
+ opt: {
+ json: {
+ title: `${titlePrefix}${id}`,
+ description: `Created programmatically. v${saneUrlVer}`,
+ visibility: 'public',
+ files: [{
+ file_path: 'payload',
+ content: value
+ }]
}
}
- this.redisClient.publish(HBP_OIDC_V2_UPDATE_CHAN, JSON.stringify(payload))
- this.redisUtil.asyncSet(HBP_DATAPROXY_WRITEURL, url)
- this.keys[HBP_DATAPROXY_WRITEURL] = url
- log({
- breakpoint: '_set',
- message: 'got new write url'
- })
- }
-
- await new Promise((rs, rj) => {
- const payload = {
- method: 'PUT',
- uri: this._getWriteTmplTag`${DATA_PROXY_URL}/buckets/${DATA_PROXY_BUCKETNAME}/${encodeURIComponent(id)}`,
- headers: {
- 'content-type': 'text/plain; charset=utf-8'
- },
- body: value
- }
- log({
- breakpoint: 'pre set',
- payload
- })
- request(payload, (err, resp, body) => {
- if (err) return rj(err)
- if (resp.statusCode === 404) return rj(new NotFoundError())
- if (resp.statusCode >= 400) return rj(new Error(`${resp.statusCode}: ${resp.statusMessage}`))
- if (resp.statusCode >= 200 && resp.statusCode < 300) return rs(body)
- return rj(new Error(`[saneurl] [get] unknown error`))
- })
})
}
- async set(id, value) {
- const result = await this._set(id, value)
- return result
+ async del(id) {
+ return await this._request({
+ addPath: `/${id}`,
+ method: 'DELETE',
+ })
}
- dispose(){
- clearInterval(this.intervalRef)
- this.redisClient && this.redisClient.quit()
+ async dispose(){
+
}
async healthCheck(){
- return this.healthFlag
+ return true
}
}
-exports.FallbackStore = FallbackStore
-exports.Store = Store
+exports.GitlabSnippetStore = GitlabSnippetStore
exports.NotFoundError = NotFoundError
diff --git a/deploy/saneUrl/util.js b/deploy/saneUrl/util.js
new file mode 100644
index 0000000000000000000000000000000000000000..949025cdc76bbf6b8794222bc67e2995ae1da482
--- /dev/null
+++ b/deploy/saneUrl/util.js
@@ -0,0 +1,68 @@
+const { NotFoundError } = require('./store')
+
+class ProxyStore {
+ static async StaticGet(store, req, name) {
+ const payload = JSON.parse(await store.get(name))
+ const { expiry, value, ...rest } = payload
+ if (expiry && (Date.now() > expiry)) {
+ await store.del(name)
+ throw new NotFoundError('Expired')
+ }
+ // backwards compatibility for depcObjStore .
+ // when depcObjStore is fully removed, || rest can also be removed
+ return value || rest
+ }
+
+ constructor(store){
+ this.store = store
+ }
+ async get(req, name) {
+ return await ProxyStore.StaticGet(this.store, req, name)
+ }
+
+ async set(req, name, value) {
+ const supplementary = req.user
+ ? {
+ userId: req.user.id,
+ expiry: null
+ }
+ : {
+ userId: null,
+ expiry: Date.now() + 1000 * 60 * 60 * 72
+ }
+
+ const fullPayload = {
+ value,
+ ...supplementary
+ }
+ return await this.store.set(name, JSON.stringify(fullPayload))
+ }
+}
+
+const NotExactlyPromiseAny = async arg => {
+ const errs = []
+ let resolvedFlag = false
+ return await new Promise((rs, rj) => {
+ let totalCounter = 0
+ for (const pr of arg) {
+ totalCounter ++
+ pr.then(val => {
+ if (!resolvedFlag) {
+ resolvedFlag = true
+ rs(val)
+ }
+ }).catch(e => {
+ errs.push(e)
+ totalCounter --
+ if (totalCounter <= 0) {
+ rj(new NotFoundError(errs))
+ }
+ })
+ }
+ })
+}
+
+module.exports = {
+ ProxyStore,
+ NotExactlyPromiseAny
+}
diff --git a/deploy/saneUrl/util.spec.js b/deploy/saneUrl/util.spec.js
new file mode 100644
index 0000000000000000000000000000000000000000..f19d8cc92e23f79d42384389a6232c6e0a1d6443
--- /dev/null
+++ b/deploy/saneUrl/util.spec.js
@@ -0,0 +1,206 @@
+const { ProxyStore, NotExactlyPromiseAny } = require('./util')
+const { expect, assert } = require('chai')
+const sinon = require('sinon')
+const { NotFoundError } = require('./store')
+
+const _name = 'foo-bar'
+const _stringValue = 'hello world'
+const _objValue = {
+ 'foo': 'bar'
+}
+const _req = {}
+
+describe('> saneUrl/util.js', () => {
+ describe('> ProxyStore', () => {
+ let store = {
+ set: sinon.stub(),
+ get: sinon.stub(),
+ del: sinon.stub(),
+ }
+ beforeEach(() => {
+ store.set.returns(Promise.resolve())
+ store.get.returns(Promise.resolve('{}'))
+ store.del.returns(Promise.resolve())
+ })
+ afterEach(() => {
+ store.set.resetHistory()
+ store.get.resetHistory()
+ store.del.resetHistory()
+ })
+ describe('> StaticGet', () => {
+ it('> should call store.get', () => {
+ ProxyStore.StaticGet(store, _req, _name)
+ assert(store.get.called, 'called')
+ assert(store.get.calledWith(_name), 'called with right param')
+ })
+ describe('> if not hit', () => {
+ const err = new NotFoundError('not found')
+ beforeEach(() => {
+ store.get.rejects(err)
+ })
+ it('should throw same error', async () => {
+ try{
+ await ProxyStore.StaticGet(store, _req, _name)
+ assert(false, 'Should throw')
+ } catch (e) {
+ assert(e instanceof NotFoundError)
+ }
+ })
+ })
+ describe('> if hit', () => {
+ describe('> if expired', () => {
+ beforeEach(() => {
+ store.get.returns(
+ Promise.resolve(
+ JSON.stringify({
+ expiry: Date.now() - 1000
+ })
+ )
+ )
+ })
+ it('> should call store.del', async () => {
+ try {
+ await ProxyStore.StaticGet(store, _req, _name)
+ } catch (e) {
+
+ }
+ assert(store.del.called, 'store.del should be called')
+ })
+ it('> should throw NotFoundError', async () => {
+ try {
+ await ProxyStore.StaticGet(store, _req, _name)
+ assert(false, 'expect throw')
+ } catch (e) {
+ assert(e instanceof NotFoundError, 'throws NotFoundError')
+ }
+ })
+ })
+ describe('> if not expired', () => {
+ it('> should return .value, if exists', async () => {
+ store.get.returns(
+ Promise.resolve(
+ JSON.stringify({
+ value: _objValue,
+ others: _stringValue
+ })
+ )
+ )
+ const returnObj = await ProxyStore.StaticGet(store, _req, _name)
+ expect(returnObj).to.deep.equal(_objValue)
+ })
+ it('> should return ...rest if .value does not exist', async () => {
+ store.get.returns(
+ Promise.resolve(
+ JSON.stringify({
+ others: _stringValue
+ })
+ )
+ )
+ const returnObj = await ProxyStore.StaticGet(store, _req, _name)
+ expect(returnObj).to.deep.equal({
+ others: _stringValue
+ })
+ })
+ })
+ })
+
+ })
+
+ describe('> get', () => {
+ let staticGetStub
+ beforeEach(() => {
+ staticGetStub = sinon.stub(ProxyStore, 'StaticGet')
+ staticGetStub.returns(Promise.resolve())
+ })
+ afterEach(() => {
+ staticGetStub.restore()
+ })
+ it('> proxies calls to StaticGet', async () => {
+ const store = {}
+ const proxyStore = new ProxyStore(store)
+ await proxyStore.get(_req, _name)
+ assert(staticGetStub.called)
+ assert(staticGetStub.calledWith(store, _req, _name))
+ })
+ })
+
+ describe('> set', () => {
+ let proxyStore
+ beforeEach(() => {
+ proxyStore = new ProxyStore(store)
+ store.set.returns(Promise.resolve())
+ })
+
+ describe('> no user', () => {
+ it('> sets expiry some time in the future', async () => {
+ await proxyStore.set(_req, _name, _objValue)
+ assert(store.set.called)
+
+ const [ name, stringifiedJson ] = store.set.args[0]
+ assert(name === _name, 'name is correct')
+ const payload = JSON.parse(stringifiedJson)
+ expect(payload.value).to.deep.equal(_objValue, 'payload is correct')
+ assert(!!payload.expiry, 'expiry exists')
+ assert((payload.expiry - Date.now()) > 1000 * 60 * 60 * 24, 'expiry is at least 24 hrs in the future')
+
+ assert(!payload.userId, 'userId does not exist')
+ })
+ })
+ describe('> yes user', () => {
+ let __req = {
+ ..._req,
+ user: {
+ id: 'foo-bar-2'
+ }
+ }
+ it('> does not set expiry, but sets userId', async () => {
+ await proxyStore.set(__req, _name, _objValue)
+ assert(store.set.called)
+
+ const [ name, stringifiedJson ] = store.set.args[0]
+ assert(name === _name, 'name is correct')
+ const payload = JSON.parse(stringifiedJson)
+ expect(payload.value).to.deep.equal(_objValue, 'payload is correct')
+ assert(!payload.expiry, 'expiry does not exist')
+
+ assert(payload.userId, 'userId exists')
+ assert(payload.userId === 'foo-bar-2', 'user-id matches')
+ })
+ })
+ })
+ })
+
+ describe('> NotExactlyPromiseAny', () => {
+ describe('> nothing resolves', () => {
+ it('> throws not found error', async () => {
+ try {
+ await NotExactlyPromiseAny([
+ (async () => {
+ throw new Error(`not here`)
+ })(),
+ new Promise((rs, rj) => setTimeout(rj, 100)),
+ new Promise((rs, rj) => rj('uhoh'))
+ ])
+ assert(false, 'expected to throw')
+ } catch (e) {
+ assert(e instanceof NotFoundError, 'expect to throw not found error')
+ }
+ })
+ })
+ describe('> something resolves', () => {
+ it('> returns the first to resolve', async () => {
+ try {
+
+ const result = await NotExactlyPromiseAny([
+ new Promise((rs, rj) => rj('uhoh')),
+ new Promise(rs => setTimeout(() => rs('hello world'), 100)),
+ Promise.resolve('foo-bar')
+ ])
+ assert(result == 'foo-bar', 'expecting first to resolve')
+ } catch (e) {
+ assert(false, 'not expecting to throw')
+ }
+ })
+ })
+ })
+})
diff --git a/docs/releases/v2.5.5.md b/docs/releases/v2.5.5.md
new file mode 100644
index 0000000000000000000000000000000000000000..cc33173305d0da142c2bf34b94b9926e1e123be1
--- /dev/null
+++ b/docs/releases/v2.5.5.md
@@ -0,0 +1,10 @@
+# v2.5.5
+
+## Feature
+
+- Added pre-release warning
+
+## Under the hood
+
+- Temporarily use up-to-date endpoint for MEBRAINS
+- Add fsaverage to checklist
diff --git a/docs/releases/v2.5.6.md b/docs/releases/v2.5.6.md
new file mode 100644
index 0000000000000000000000000000000000000000..1b7ec38867f0a49b941d65996a8b82c6827367d8
--- /dev/null
+++ b/docs/releases/v2.5.6.md
@@ -0,0 +1,6 @@
+# v2.5.6
+
+## Bugfix
+
+- re-introduced explore in KG button for parcellation citations
+
diff --git a/docs/releases/v2.5.7.md b/docs/releases/v2.5.7.md
new file mode 100644
index 0000000000000000000000000000000000000000..f043d78016f86f7dc85fe95aea053beb348320e2
--- /dev/null
+++ b/docs/releases/v2.5.7.md
@@ -0,0 +1,20 @@
+# v2.5.7
+
+## Bugfix
+
+- fixed version in console
+- fixed UI bug: multiple doi overlap
+- fixed UI bug: sometimes missing doi are also rendered
+- fixed waxholm v4 preview image
+
+## Feature
+
+- Add menu to change perspective orientation by coronal/sagittal/axial views.
+- re-introduced saneUrl
+
+## Under the hood
+
+- added saneurl for bigbrain isocortex, allen CCFv3 2017
+- fixed csp issues
+- optimised build (assets should be smaller)
+- added git hash in console
diff --git a/docs/releases/v2.5.8.md b/docs/releases/v2.5.8.md
new file mode 100644
index 0000000000000000000000000000000000000000..c9b68931c30f63fcbd0a5ab32686719e3cc92b1c
--- /dev/null
+++ b/docs/releases/v2.5.8.md
@@ -0,0 +1,9 @@
+# v2.5.8
+
+## Bugfix
+
+- fixed user annotation with saneurl
+
+## Under the hood
+
+- remove unnecessary UI refreshes
diff --git a/e2e/checklist.md b/e2e/checklist.md
index a42223294dd298cae6f7cbe080466def3f807be7..3c0cbe0a538168ea49aeff84877386c05826459b 100644
--- a/e2e/checklist.md
+++ b/e2e/checklist.md
@@ -46,13 +46,19 @@
- [ ] the navigation should be preserved
- [ ] in big brain v2.9 (or latest)
- [ ] high res hoc1, hoc2, hoc3, lam1-6 are visible
+ - [ ] pli dataset [link](https://siibra-explorer.apps.hbp.eu/staging/?templateSelected=Big+Brain+%28Histology%29&parcellationSelected=Grey%2FWhite+matter&cNavigation=0.0.0.-W000.._eCwg.2-FUe3._-s_W.2_evlu..7LIx..1uaTK.Bq5o~.lKmo~..NBW&previewingDatasetFiles=%5B%7B%22datasetId%22%3A%22minds%2Fcore%2Fdataset%2Fv1.0.0%2Fb08a7dbc-7c75-4ce7-905b-690b2b1e8957%22%2C%22filename%22%3A%22Overlay+of+data+modalities%22%7D%5D)
+ - [ ] redirects fine
+ - [ ] shows fine
+ - [ ] fsaverage
+ - [ ] can be loaded & visible
- [ ] Waxholm
- [ ] v4 are visible
- [ ] on hover, show correct region name(s)
-
+ - [ ] whole mesh loads
## saneURL
- [ ] [saneUrl](https://siibra-explorer.apps.hbp.eu/staging/saneUrl/bigbrainGreyWhite) redirects to big brain
- [ ] [saneUrl](https://siibra-explorer.apps.hbp.eu/staging/saneUrl/julichbrain) redirects to julich brain (colin 27)
- [ ] [saneUrl](https://siibra-explorer.apps.hbp.eu/staging/saneUrl/whs4) redirects to waxholm v4
- [ ] [saneUrl](https://siibra-explorer.apps.hbp.eu/staging/saneUrl/allen2017) redirects to allen 2017
- [ ] [saneUrl](https://siibra-explorer.apps.hbp.eu/staging/saneUrl/mebrains) redirects to monkey
+
diff --git a/mkdocs.yml b/mkdocs.yml
index 9fab0d68ff6d30462e2cb384f1d68b188da7f267..95827211ee2faac6753b6dae9b4d023d6e716530 100644
--- a/mkdocs.yml
+++ b/mkdocs.yml
@@ -41,6 +41,10 @@ pages:
- Display non-atlas volumes: 'advanced/otherVolumes.md'
- Release notes:
- v2.6.0: 'releases/v2.6.0.md'
+ - v2.5.8: 'releases/v2.5.8.md'
+ - v2.5.7: 'releases/v2.5.7.md'
+ - v2.5.6: 'releases/v2.5.6.md'
+ - v2.5.5: 'releases/v2.5.5.md'
- v2.5.4: 'releases/v2.5.4.md'
- v2.5.3: 'releases/v2.5.3.md'
- v2.5.2: 'releases/v2.5.2.md'
diff --git a/package.json b/package.json
index 7bf07fa3c9d273e95eb25397e684856e01424490..c9e41e939e48afb2cd1edaa045ce9fc801eb10ad 100644
--- a/package.json
+++ b/package.json
@@ -3,7 +3,7 @@
"version": "2.6.0",
"description": "HBP interactive atlas viewer. Integrating KG query, dataset previews & more. Based on humanbrainproject/nehuba & google/neuroglancer. Built with angular",
"scripts": {
- "build-aot": "VERSION=`node -e 'console.log(require(\"./package.json\").version)'` ng build && node ./third_party/matomo/processMatomo.js",
+ "build-aot": "ng build && node ./third_party/matomo/processMatomo.js",
"dev-server-aot": "ng serve",
"e2e": "echo NYI && exit 1",
"wd": "webdriver-manager",
diff --git a/src/assets/images/atlas-selection/waxholm-v4.png b/src/assets/images/atlas-selection/waxholm-v4.png
new file mode 100644
index 0000000000000000000000000000000000000000..d0af0a5be2b2cf40fd814dce032fbd0801da30a8
Binary files /dev/null and b/src/assets/images/atlas-selection/waxholm-v4.png differ
diff --git a/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.component.ts b/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.component.ts
index d3fcb22161ad3e74d66002f083f32c724f5224f1..4b911265407de5f74d5615cfe2eae2e7a14b6b6b 100644
--- a/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.component.ts
+++ b/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.component.ts
@@ -56,6 +56,11 @@ export class GenericInfoCmp extends BsRegionInputBase implements OnChanges, Afte
doi: string
}[]
+ public doiUrls: {
+ cite: string
+ doi: string
+ }[]
+
template: TemplateRef<any>
viewref: ViewRef
@@ -72,6 +77,7 @@ export class GenericInfoCmp extends BsRegionInputBase implements OnChanges, Afte
this.description = description
this.name = name
this.urls = urls
+ this.doiUrls = this.urls.filter(d => !!d.doi)
this.useClassicUi = useClassicUi
if (dataType) this.dataType = dataType
if (typeof isGdprProtected !== 'undefined') this.isGdprProtected = isGdprProtected
diff --git a/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.template.html b/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.template.html
index ab89e43adb9a94f391b1a29a3f67fc893754ae3b..2bce44e578033cd6aa517f3dae798aafb6944bc4 100644
--- a/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.template.html
+++ b/src/atlasComponents/regionalFeatures/bsFeatures/genericInfo/genericInfoCmp/genericInfo.template.html
@@ -27,10 +27,9 @@
<!-- explore -->
<ng-container>
- <a *ngFor="let kgRef of (urls || [])"
+ <a *ngFor="let kgRef of (doiUrls || [])"
[href]="kgRef.doi | doiParserPipe"
class="color-inherit"
- mat-icon-button
[matTooltip]="ARIA_LABELS.EXPLORE_DATASET_IN_KG"
target="_blank">
<iav-dynamic-mat-button
diff --git a/src/atlasComponents/userAnnotations/annotationList/annotationList.component.ts b/src/atlasComponents/userAnnotations/annotationList/annotationList.component.ts
index 07eae088ce2154228ecc4df9f2f3779ed3390d5a..bad568f3822e640174da67c0162142b3a2188894 100644
--- a/src/atlasComponents/userAnnotations/annotationList/annotationList.component.ts
+++ b/src/atlasComponents/userAnnotations/annotationList/annotationList.component.ts
@@ -1,16 +1,18 @@
-import { Component, Optional, ViewChild } from "@angular/core";
+import { Component, Optional, TemplateRef, ViewChild } from "@angular/core";
import { ARIA_LABELS, CONST } from "common/constants";
import { ModularUserAnnotationToolService } from "../tools/service";
import { IAnnotationGeometry, TExportFormats } from "../tools/type";
import { ComponentStore } from "src/viewerModule/componentStore";
import { map, shareReplay, startWith } from "rxjs/operators";
-import { Observable, Subscription } from "rxjs";
+import { combineLatest, Observable, Subscription } from "rxjs";
import { TZipFileConfig } from "src/zipFilesOutput/type";
import { TFileInputEvent } from "src/getFileInput/type";
import { FileInputDirective } from "src/getFileInput/getFileInput.directive";
import { MatSnackBar } from "@angular/material/snack-bar";
import { unzip } from "src/zipFilesOutput/zipFilesOutput.directive";
import { DialogService } from "src/services/dialogService.service";
+import { MatDialog } from "@angular/material/dialog";
+import { userAnnotationRouteKey } from "../constants";
const README = `{id}.sands.json file contains the data of annotations. {id}.desc.json contains the metadata of annotations.`
@@ -25,6 +27,8 @@ const README = `{id}.sands.json file contains the data of annotations. {id}.desc
})
export class AnnotationList {
+ public userAnnRoute = {}
+
public ARIA_LABELS = ARIA_LABELS
@ViewChild(FileInputDirective)
@@ -66,6 +70,7 @@ export class AnnotationList {
constructor(
private annotSvc: ModularUserAnnotationToolService,
private snackbar: MatSnackBar,
+ private dialog: MatDialog,
cStore: ComponentStore<{ useFormat: TExportFormats }>,
@Optional() private dialogSvc: DialogService,
) {
@@ -74,7 +79,22 @@ export class AnnotationList {
})
this.subs.push(
- this.managedAnnotations$.subscribe(anns => this.managedAnnotations = anns)
+ this.managedAnnotations$.subscribe(anns => this.managedAnnotations = anns),
+ combineLatest([
+ this.managedAnnotations$.pipe(
+ startWith([])
+ ),
+ this.annotationInOtherSpaces$.pipe(
+ startWith([])
+ )
+ ]).subscribe(([ann, annOther]) => {
+ this.userAnnRoute = {
+ [userAnnotationRouteKey]: [
+ ...ann.map(a => a.toJSON()),
+ ...annOther.map(a => a.toJSON()),
+ ]
+ }
+ })
)
}
@@ -175,4 +195,8 @@ export class AnnotationList {
}
}
}
+
+ public openDialog(template: TemplateRef<any>) {
+ this.dialog.open(template)
+ }
}
diff --git a/src/atlasComponents/userAnnotations/annotationList/annotationList.template.html b/src/atlasComponents/userAnnotations/annotationList/annotationList.template.html
index 7543d5d598ce9d77956aa6036636a1ba7b8408c5..f0832efe03d55a7295ad5699d058d35d029304de 100644
--- a/src/atlasComponents/userAnnotations/annotationList/annotationList.template.html
+++ b/src/atlasComponents/userAnnotations/annotationList/annotationList.template.html
@@ -34,6 +34,15 @@
<i class="fas fa-download"></i>
</button>
+ <!-- share -->
+ <button mat-icon-button
+ [matTooltip]="ARIA_LABELS.SHARE_CUSTOM_URL"
+ (click)="openDialog(saneUrlTmpl)"
+ iav-state-aggregator
+ #stateAggregator="iavStateAggregator">
+ <i class="fas fa-share-square"></i>
+ </button>
+
<!-- delete all annotations -->
<button mat-icon-button
color="warn"
@@ -102,3 +111,23 @@
</ng-template>
</mat-card-content>
</mat-card>
+
+
+<!-- sane url template -->
+<ng-template #saneUrlTmpl>
+ <h2 mat-dialog-title>
+ Create custom URL
+ </h2>
+ <div mat-dialog-content>
+ <iav-sane-url [stateTobeSaved]="stateAggregator.jsonifiedState$ | async | mergeObj : userAnnRoute">
+ </iav-sane-url>
+ </div>
+
+ <div mat-dialog-actions
+ class="d-flex justify-content-center">
+ <button mat-button
+ mat-dialog-close>
+ close
+ </button>
+ </div>
+</ng-template>
diff --git a/src/atlasComponents/userAnnotations/annotationMode/annotationMode.component.ts b/src/atlasComponents/userAnnotations/annotationMode/annotationMode.component.ts
index b316ac2632dd0c6b31cf43bb30345434d7f37941..5eacf72e95c4baadee42cbb70d28a09ddb84dc62 100644
--- a/src/atlasComponents/userAnnotations/annotationMode/annotationMode.component.ts
+++ b/src/atlasComponents/userAnnotations/annotationMode/annotationMode.component.ts
@@ -1,7 +1,5 @@
import { Component, Inject, OnDestroy, Optional } from "@angular/core";
-import { Store } from "@ngrx/store";
import { ModularUserAnnotationToolService } from "../tools/service";
-import { viewerStateSetViewerMode } from "src/services/state/viewerState.store.helper";
import { ARIA_LABELS } from 'common/constants'
import { ClickInterceptor, CLICK_INTERCEPTOR_INJECTOR, CONTEXT_MENU_ITEM_INJECTOR, TContextMenu } from "src/util";
import { TContextArg } from "src/viewerModule/viewer.interface";
@@ -31,7 +29,6 @@ export class AnnotationMode implements OnDestroy{
private onDestroyCb: (() => void)[] = []
constructor(
- private store$: Store<any>,
private modularToolSvc: ModularUserAnnotationToolService,
snackbar: MatSnackBar,
@Optional() @Inject(CLICK_INTERCEPTOR_INJECTOR) clickInterceptor: ClickInterceptor,
@@ -59,13 +56,6 @@ export class AnnotationMode implements OnDestroy{
})
}
- exitAnnotationMode(){
- this.store$.dispatch(
- viewerStateSetViewerMode({
- payload: null
- })
- )
- }
deselectTools(){
this.modularToolSvc.deselectTools()
}
diff --git a/src/atlasComponents/userAnnotations/annotationMode/annotationMode.template.html b/src/atlasComponents/userAnnotations/annotationMode/annotationMode.template.html
index a21e682b08bc6b9e7adcab3737e1261fe63d9e5a..7542cdba29fd4102f5e03c801468cbc69fb77b05 100644
--- a/src/atlasComponents/userAnnotations/annotationMode/annotationMode.template.html
+++ b/src/atlasComponents/userAnnotations/annotationMode/annotationMode.template.html
@@ -16,7 +16,8 @@
<button
mat-button
- (click)="exitAnnotationMode()"
+ annotation-switch
+ annotation-switch-mode="off"
class="tab-toggle"
[matTooltip]="ARIA_LABELS.EXIT_ANNOTATION_MODE"
color="warn">
diff --git a/src/atlasComponents/userAnnotations/constants.ts b/src/atlasComponents/userAnnotations/constants.ts
new file mode 100644
index 0000000000000000000000000000000000000000..ddb14d680ab6240706d014ac1e71ebcf117d65cc
--- /dev/null
+++ b/src/atlasComponents/userAnnotations/constants.ts
@@ -0,0 +1 @@
+export const userAnnotationRouteKey = 'x-user-anntn'
diff --git a/src/atlasComponents/userAnnotations/directives/annotationSwitch.directive.ts b/src/atlasComponents/userAnnotations/directives/annotationSwitch.directive.ts
index 1109fbdbc342124d842fe82b59ee02c2a51d0de4..068f6313632fe3e79f8bbb47b2cf58be31666bac 100644
--- a/src/atlasComponents/userAnnotations/directives/annotationSwitch.directive.ts
+++ b/src/atlasComponents/userAnnotations/directives/annotationSwitch.directive.ts
@@ -1,13 +1,5 @@
-import { Directive, HostListener, Inject, Input, Optional } from "@angular/core";
-import { viewerStateSetViewerMode } from "src/services/state/viewerState/actions";
-import { ARIA_LABELS } from "common/constants";
-import { select, Store } from "@ngrx/store";
-import { TContextArg } from "src/viewerModule/viewer.interface";
-import { TContextMenuReg } from "src/contextMenuModule";
-import { CONTEXT_MENU_ITEM_INJECTOR, TContextMenu } from "src/util";
+import { Directive, HostListener, Input } from "@angular/core";
import { ModularUserAnnotationToolService } from "../tools/service";
-import { Subscription } from "rxjs";
-import { viewerStateViewerModeSelector } from "src/services/state/viewerState/selectors";
@Directive({
selector: '[annotation-switch]'
@@ -17,37 +9,13 @@ export class AnnotationSwitch {
@Input('annotation-switch-mode')
mode: 'toggle' | 'off' | 'on' = 'on'
- private currMode = null
- private subs: Subscription[] = []
constructor(
- private store$: Store<any>,
private svc: ModularUserAnnotationToolService,
- @Optional() @Inject(CONTEXT_MENU_ITEM_INJECTOR) ctxMenuInterceptor: TContextMenu<TContextMenuReg<TContextArg<'nehuba' | 'threeSurfer'>>>
) {
- this.subs.push(
- this.store$.pipe(
- select(viewerStateViewerModeSelector)
- ).subscribe(mode => {
- this.currMode = mode
- })
- )
}
@HostListener('click')
onClick() {
- let payload = null
- if (this.mode === 'on') payload = ARIA_LABELS.VIEWER_MODE_ANNOTATING
- if (this.mode === 'off') {
- if (this.currMode === ARIA_LABELS.VIEWER_MODE_ANNOTATING) payload = null
- else return
- }
- if (this.mode === 'toggle') {
- payload = this.currMode === ARIA_LABELS.VIEWER_MODE_ANNOTATING
- ? null
- : ARIA_LABELS.VIEWER_MODE_ANNOTATING
- }
- this.store$.dispatch(
- viewerStateSetViewerMode({ payload })
- )
+ this.svc.switchAnnotationMode(this.mode)
}
}
diff --git a/src/atlasComponents/userAnnotations/module.ts b/src/atlasComponents/userAnnotations/module.ts
index 39e7f420136db395d43f95380450eb23be2cf759..ec2586e5741ddc2d25ff4c45657154d882386946 100644
--- a/src/atlasComponents/userAnnotations/module.ts
+++ b/src/atlasComponents/userAnnotations/module.ts
@@ -1,4 +1,4 @@
-import { NgModule } from "@angular/core";
+import { APP_INITIALIZER, NgModule } from "@angular/core";
import { CommonModule } from "@angular/common";
import { AngularMaterialModule } from "src/sharedModules";
import { FormsModule, ReactiveFormsModule } from "@angular/forms";
@@ -14,6 +14,9 @@ import { FileInputModule } from "src/getFileInput/module";
import { ZipFilesOutputModule } from "src/zipFilesOutput/module";
import { FilterAnnotationsBySpace } from "./filterAnnotationBySpace.pipe";
import { AnnotationEventDirective } from "./directives/annotationEv.directive";
+import { ShareModule } from "src/share";
+import { StateModule } from "src/state";
+import { RoutedAnnotationService } from "./routedAnnotation.service";
@NgModule({
imports: [
@@ -26,6 +29,8 @@ import { AnnotationEventDirective } from "./directives/annotationEv.directive";
UtilModule,
FileInputModule,
ZipFilesOutputModule,
+ ShareModule,
+ StateModule,
],
declarations: [
AnnotationMode,
@@ -43,6 +48,18 @@ import { AnnotationEventDirective } from "./directives/annotationEv.directive";
AnnotationList,
AnnotationSwitch,
AnnotationEventDirective
+ ],
+ providers: [
+ // initialize routerannotation service, so it will parse route and load annotations ...
+ // ... in url correctly
+ {
+ provide: APP_INITIALIZER,
+ useFactory:(svc: RoutedAnnotationService) => {
+ return () => Promise.resolve()
+ },
+ deps: [ RoutedAnnotationService ],
+ multi: true
+ }
]
})
diff --git a/src/atlasComponents/userAnnotations/routedAnnotation.service.spec.ts b/src/atlasComponents/userAnnotations/routedAnnotation.service.spec.ts
new file mode 100644
index 0000000000000000000000000000000000000000..7aecdd3d4a043e42699d2a8d2279c568bf48191c
--- /dev/null
+++ b/src/atlasComponents/userAnnotations/routedAnnotation.service.spec.ts
@@ -0,0 +1,178 @@
+import { fakeAsync, TestBed, tick } from "@angular/core/testing"
+import { BehaviorSubject, of } from "rxjs"
+import { RouterService } from "src/routerModule/router.service"
+import { SaneUrlSvc } from "src/share/saneUrl/saneUrl.service"
+import { ModularUserAnnotationToolService } from "./tools/service"
+import { RoutedAnnotationService } from './routedAnnotation.service'
+import { userAnnotationRouteKey } from "./constants"
+
+describe('> routedannotation.service.ts', () => {
+ describe('> RoutedAnnotationService', () => {
+ const customRouteSub = new BehaviorSubject(null)
+ const spyRService = {
+ customRoute$: customRouteSub.asObservable()
+ }
+
+ const spyAnnSvc = {
+ switchAnnotationMode: jasmine.createSpy('switchAnnotationMode'),
+ parseAnnotationObject: jasmine.createSpy('parseAnnotationObject'),
+ importAnnotation: jasmine.createSpy('importAnnotation')
+ }
+ beforeEach(() => {
+ TestBed.configureTestingModule({
+ providers: [
+ RoutedAnnotationService,
+ {
+ provide: RouterService,
+ useValue: spyRService
+ }, {
+ provide: SaneUrlSvc,
+ useFactory: () => {
+ return {
+ getKeyVal: jasmine.createSpy('getKeyVal').and.returnValue(
+ of({})
+ )
+ }
+ }
+ }, {
+ provide: ModularUserAnnotationToolService,
+ useValue: spyAnnSvc
+ }
+ ],
+ })
+ })
+ afterEach(() => {
+ spyAnnSvc.switchAnnotationMode.calls.reset()
+ spyAnnSvc.parseAnnotationObject.calls.reset()
+ spyAnnSvc.importAnnotation.calls.reset()
+ })
+
+ it('> can be init', () => {
+ const svc = TestBed.inject(RoutedAnnotationService)
+ expect(svc instanceof RoutedAnnotationService).toBeTrue()
+ })
+
+ describe('> normal operation', () => {
+ const mockVal = 'foo-bar'
+ const getKeyValReturn = {
+ [userAnnotationRouteKey]: [{
+ foo: 'bar'
+ }, {
+ foo: 'hello world'
+ }]
+ }
+ const parseAnnObjReturn = [{
+ bar: 'baz'
+ }, {
+ hello: 'world'
+ }]
+
+ beforeEach(() => {
+ const spySaneUrlSvc = TestBed.inject(SaneUrlSvc) as any
+ spySaneUrlSvc.getKeyVal.and.returnValue(
+ of(getKeyValReturn)
+ )
+ spyAnnSvc.parseAnnotationObject.and.returnValues(...parseAnnObjReturn)
+ })
+
+ it('> getKeyVal called after at least 160ms', fakeAsync(() => {
+ customRouteSub.next({
+ [userAnnotationRouteKey]: mockVal
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+ tick(200)
+ const spySaneUrlSvc = TestBed.inject(SaneUrlSvc) as any
+ expect(spySaneUrlSvc.getKeyVal).toHaveBeenCalled()
+ expect(spySaneUrlSvc.getKeyVal).toHaveBeenCalledWith(mockVal)
+ }))
+
+ it('> switchannotation mode is called with "on"', fakeAsync(() => {
+ customRouteSub.next({
+ [userAnnotationRouteKey]: mockVal
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+ tick(200)
+ expect(spyAnnSvc.switchAnnotationMode).toHaveBeenCalledOnceWith('on')
+ }))
+
+ it('> parseAnnotationObject is called expected number of times', fakeAsync(() => {
+ customRouteSub.next({
+ [userAnnotationRouteKey]: mockVal
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+ tick(200)
+
+ const userAnn = getKeyValReturn[userAnnotationRouteKey]
+ expect(spyAnnSvc.parseAnnotationObject).toHaveBeenCalledTimes(userAnn.length)
+ for (const ann of userAnn) {
+ expect(spyAnnSvc.parseAnnotationObject).toHaveBeenCalledWith(ann)
+ }
+ }))
+
+ it('> importAnnotation is called expected number of times', fakeAsync(() => {
+ customRouteSub.next({
+ [userAnnotationRouteKey]: mockVal
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+ tick(200)
+ expect(spyAnnSvc.importAnnotation).toHaveBeenCalledTimes(parseAnnObjReturn.length)
+ for (const obj of parseAnnObjReturn) {
+ expect(spyAnnSvc.importAnnotation).toHaveBeenCalledWith(obj)
+ }
+ }))
+ })
+
+ describe('> abnormal operation', () => {
+ describe('> routerSvc.customRoute$ emits after 160 ms', () => {
+ const mockVal = 'foo-bar'
+
+ it('> getKeyVal should only be called once', fakeAsync(() => {
+ customRouteSub.next({
+ [userAnnotationRouteKey]: mockVal
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+ tick(200)
+ customRouteSub.next({
+ [userAnnotationRouteKey]: 'hello world'
+ })
+ tick(200)
+ const spySaneUrlSvc = TestBed.inject(SaneUrlSvc) as any
+ expect(spySaneUrlSvc.getKeyVal).toHaveBeenCalledOnceWith(mockVal)
+ }))
+ })
+
+ describe('> routerSvc.customRoute$ does not emit valid key', () => {
+ it('> does not call getKeyVal', fakeAsync(() => {
+ customRouteSub.next({
+ 'hello-world': 'foo-bar'
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+ tick(200)
+ const spySaneUrlSvc = TestBed.inject(SaneUrlSvc) as any
+ expect(spySaneUrlSvc.getKeyVal).not.toHaveBeenCalled()
+ }))
+ })
+
+ describe('> getKeyVal returns invalid key', () => {
+ it('> does not call switchAnnotationMode', fakeAsync(() => {
+ const spySaneUrlSvc = TestBed.inject(SaneUrlSvc) as any
+ spySaneUrlSvc.getKeyVal.and.returnValue(
+ of({
+ 'hello-world': [{
+ foo: 'bar',
+ fuzz: 'bizz'
+ }]
+ })
+ )
+ customRouteSub.next({
+ [userAnnotationRouteKey]: 'foo-bar'
+ })
+ const svc = TestBed.inject(RoutedAnnotationService)
+
+ tick(320)
+ expect(spyAnnSvc.switchAnnotationMode).not.toHaveBeenCalled()
+ }))
+ })
+ })
+ })
+})
diff --git a/src/atlasComponents/userAnnotations/routedAnnotation.service.ts b/src/atlasComponents/userAnnotations/routedAnnotation.service.ts
new file mode 100644
index 0000000000000000000000000000000000000000..11a28a8fe6548ede886ab3b0c0ca1f09125aa918
--- /dev/null
+++ b/src/atlasComponents/userAnnotations/routedAnnotation.service.ts
@@ -0,0 +1,41 @@
+import { Injectable } from "@angular/core";
+import { NEVER } from "rxjs";
+import { debounceTime, map, switchMap, take } from "rxjs/operators";
+import { RouterService } from "src/routerModule/router.service";
+import { SaneUrlSvc } from "src/share/saneUrl/saneUrl.service";
+import { userAnnotationRouteKey } from "./constants";
+import { ModularUserAnnotationToolService } from "./tools/service";
+
+@Injectable({
+ providedIn: 'root'
+})
+
+export class RoutedAnnotationService{
+ constructor(
+ routerSvc: RouterService,
+ saneUrlSvc: SaneUrlSvc,
+ annSvc: ModularUserAnnotationToolService,
+ ){
+
+ routerSvc.customRoute$.pipe(
+ debounceTime(160),
+ take(1),
+ map(obj => obj[userAnnotationRouteKey]),
+ switchMap(
+ saneUrlKey => {
+ return saneUrlKey
+ ? saneUrlSvc.getKeyVal(saneUrlKey)
+ : NEVER
+ }
+ )
+ ).subscribe(val => {
+ if (val[userAnnotationRouteKey]) {
+ annSvc.switchAnnotationMode('on')
+ for (const ann of val[userAnnotationRouteKey]){
+ const geom = annSvc.parseAnnotationObject(ann)
+ annSvc.importAnnotation(geom)
+ }
+ }
+ })
+ }
+}
diff --git a/src/atlasComponents/userAnnotations/singleAnnotationUnit/singleAnnotationUnit.component.ts b/src/atlasComponents/userAnnotations/singleAnnotationUnit/singleAnnotationUnit.component.ts
index 956f23c080c1929b07fc62159a42024c68334490..8c9a11566b9f7f27350a684cc9ffc3976525c32e 100644
--- a/src/atlasComponents/userAnnotations/singleAnnotationUnit/singleAnnotationUnit.component.ts
+++ b/src/atlasComponents/userAnnotations/singleAnnotationUnit/singleAnnotationUnit.component.ts
@@ -48,8 +48,8 @@ export class SingleAnnotationUnit implements OnDestroy, AfterViewInit{
this.chSubs.push(
this.formGrp.valueChanges.subscribe(value => {
const { name, desc, spaceId } = value
- this.managedAnnotation.setName(name)
- this.managedAnnotation.setDesc(desc)
+ this.managedAnnotation.name = name
+ this.managedAnnotation.desc = desc
})
)
diff --git a/src/atlasComponents/userAnnotations/tools/line.ts b/src/atlasComponents/userAnnotations/tools/line.ts
index b025f386b103f9db31c5b37a9d515e1ba5f0030e..3d01d57dc41678c9958d1c52b41e734fce28ae28 100644
--- a/src/atlasComponents/userAnnotations/tools/line.ts
+++ b/src/atlasComponents/userAnnotations/tools/line.ts
@@ -46,7 +46,7 @@ export class Line extends IAnnotationGeometry{
})
if (!this.points[0]) this.points[0] = point
else this.points[1] = point
- this.sendUpdateSignal()
+ this.changed()
return point
}
@@ -173,11 +173,7 @@ export class Line extends IAnnotationGeometry{
for (const p of this.points){
p.translate(x, y, z)
}
- this.sendUpdateSignal()
- }
-
- private sendUpdateSignal(){
- this.updateSignal$.next(this.toString())
+ this.changed()
}
}
@@ -245,7 +241,7 @@ export class ToolLine extends AbsToolClass<Line> implements IAnnotationTools, On
* it only has a single point, and should be removed
*/
if (this.selectedLine) {
- this.removeAnnotation(this.selectedLine.id)
+ this.removeAnnotation(this.selectedLine)
}
this.selectedLine = null
}),
@@ -312,14 +308,4 @@ export class ToolLine extends AbsToolClass<Line> implements IAnnotationTools, On
ngOnDestroy(){
this.subs.forEach(s => s.unsubscribe())
}
-
- removeAnnotation(id: string){
- const idx = this.managedAnnotations.findIndex(ann => ann.id === id)
- if (idx < 0) {
- return
- }
- this.managedAnnotations.splice(idx, 1)
- this.managedAnnotations$.next(this.managedAnnotations)
- }
-
}
diff --git a/src/atlasComponents/userAnnotations/tools/point.ts b/src/atlasComponents/userAnnotations/tools/point.ts
index 90000057a5be5281776ab771c84889cd168ee164..f69baf0e4045e8300de93c42a086b57ceee7292e 100644
--- a/src/atlasComponents/userAnnotations/tools/point.ts
+++ b/src/atlasComponents/userAnnotations/tools/point.ts
@@ -96,7 +96,7 @@ export class Point extends IAnnotationGeometry {
this.x += x
this.y += y
this.z += z
- this.updateSignal$.next(this.toString())
+ this.changed()
}
}
@@ -178,20 +178,6 @@ export class ToolPoint extends AbsToolClass<Point> implements IAnnotationTools,
)
}
- /**
- * @description remove managed annotation via id
- * @param id id of annotation
- */
- removeAnnotation(id: string) {
- const idx = this.managedAnnotations.findIndex(ann => id === ann.id)
- if (idx < 0){
- throw new Error(`cannot find point idx ${idx}`)
- return
- }
- this.managedAnnotations.splice(idx, 1)
- this.managedAnnotations$.next(this.managedAnnotations)
- }
-
onMouseMoveRenderPreview(pos: [number, number, number]) {
return [{
id: `${ToolPoint.PREVIEW_ID}_0`,
diff --git a/src/atlasComponents/userAnnotations/tools/poly.ts b/src/atlasComponents/userAnnotations/tools/poly.ts
index 6769d3c226bc6afe8310e0868db367bbe47f8b7f..47dd27c7dc56924a1be4911045523577e7e9d1ad 100644
--- a/src/atlasComponents/userAnnotations/tools/poly.ts
+++ b/src/atlasComponents/userAnnotations/tools/poly.ts
@@ -38,7 +38,7 @@ export class Polygon extends IAnnotationGeometry{
this.edges = this.edges.filter(([ idx1, idx2 ]) => idx1 !== ptIdx && idx2 !== ptIdx)
this.points.splice(ptIdx, 1)
- this.sendUpdateSignal()
+ this.changed()
}
public addPoint(p: Point | {x: number, y: number, z: number}, linkTo?: Point): Point {
@@ -57,7 +57,7 @@ export class Polygon extends IAnnotationGeometry{
if (!this.hasPoint(pointToBeAdded)) {
this.points.push(pointToBeAdded)
const sub = pointToBeAdded.updateSignal$.subscribe(
- () => this.sendUpdateSignal()
+ () => this.changed()
)
this.ptWkMp.set(pointToBeAdded, {
onremove: () => {
@@ -72,7 +72,7 @@ export class Polygon extends IAnnotationGeometry{
] as [number, number]
this.edges.push(newEdge)
}
- this.sendUpdateSignal()
+ this.changed()
return pointToBeAdded
}
@@ -210,15 +210,11 @@ export class Polygon extends IAnnotationGeometry{
this.edges = edges
}
- private sendUpdateSignal(){
- this.updateSignal$.next(this.toString())
- }
-
public translate(x: number, y: number, z: number) {
for (const p of this.points){
p.translate(x, y, z)
}
- this.sendUpdateSignal()
+ this.changed()
}
}
@@ -308,7 +304,7 @@ export class ToolPolygon extends AbsToolClass<Polygon> implements IAnnotationToo
* if edges < 3, discard poly
*/
if (edges.length < 3) {
- this.removeAnnotation(this.selectedPoly.id)
+ this.removeAnnotation(this.selectedPoly)
}
}
@@ -405,15 +401,6 @@ export class ToolPolygon extends AbsToolClass<Polygon> implements IAnnotationToo
)
}
- removeAnnotation(id: string) {
- const idx = this.managedAnnotations.findIndex(ann => ann.id === id)
- if (idx < 0) {
- return
- }
- this.managedAnnotations.splice(idx, 1)
- this.managedAnnotations$.next(this.managedAnnotations)
- }
-
ngOnDestroy(){
if (this.subs.length > 0) this.subs.pop().unsubscribe()
}
diff --git a/src/atlasComponents/userAnnotations/tools/service.ts b/src/atlasComponents/userAnnotations/tools/service.ts
index 1a5cbf2d74d60997a5de42f615e4f04a26f8fe25..e533cdcc17aba6d94b662b9c97193d2a357a7d00 100644
--- a/src/atlasComponents/userAnnotations/tools/service.ts
+++ b/src/atlasComponents/userAnnotations/tools/service.ts
@@ -15,6 +15,7 @@ import { Point } from "./point";
import { FilterAnnotationsBySpace } from "../filterAnnotationBySpace.pipe";
import { retry } from 'common/util'
import { MatSnackBar } from "@angular/material/snack-bar";
+import { viewerStateSetViewerMode } from "src/services/state/viewerState.store.helper";
const LOCAL_STORAGE_KEY = 'userAnnotationKey'
@@ -502,6 +503,7 @@ export class ModularUserAnnotationToolService implements OnDestroy{
store.pipe(
select(viewerStateViewerModeSelector)
).subscribe(viewerMode => {
+ this.currMode = viewerMode
if (viewerMode === ModularUserAnnotationToolService.VIEWER_MODE) {
if (this.ngAnnotationLayer) this.ngAnnotationLayer.setVisible(true)
else {
@@ -709,8 +711,8 @@ export class ModularUserAnnotationToolService implements OnDestroy{
// potentially overwriting existing name and desc...
// maybe should show warning?
- existingAnn.setName(json.name)
- existingAnn.setDesc(json.desc)
+ existingAnn.name = json.name
+ existingAnn.desc = json.desc
return existingAnn
} else {
const { id, name, desc } = json
@@ -720,8 +722,8 @@ export class ModularUserAnnotationToolService implements OnDestroy{
} else {
const metadata = this.metadataMap.get(returnObj.id)
if (returnObj && metadata) {
- returnObj.setName(metadata?.name || null)
- returnObj.setDesc(metadata?.desc || null)
+ returnObj.name = metadata?.name || null
+ returnObj.desc = metadata?.desc || null
this.metadataMap.delete(returnObj.id)
}
}
@@ -742,6 +744,25 @@ export class ModularUserAnnotationToolService implements OnDestroy{
ngOnDestroy(){
while(this.subscription.length > 0) this.subscription.pop().unsubscribe()
}
+
+ private currMode: string
+ switchAnnotationMode(mode: 'on' | 'off' | 'toggle' = 'toggle') {
+
+ let payload = null
+ if (mode === 'on') payload = ARIA_LABELS.VIEWER_MODE_ANNOTATING
+ if (mode === 'off') {
+ if (this.currMode === ARIA_LABELS.VIEWER_MODE_ANNOTATING) payload = null
+ else return
+ }
+ if (mode === 'toggle') {
+ payload = this.currMode === ARIA_LABELS.VIEWER_MODE_ANNOTATING
+ ? null
+ : ARIA_LABELS.VIEWER_MODE_ANNOTATING
+ }
+ this.store.dispatch(
+ viewerStateSetViewerMode({ payload })
+ )
+ }
}
export function parseNgAnnotation(ann: INgAnnotationTypes[keyof INgAnnotationTypes]){
diff --git a/src/atlasComponents/userAnnotations/tools/type.spec.ts b/src/atlasComponents/userAnnotations/tools/type.spec.ts
new file mode 100644
index 0000000000000000000000000000000000000000..f4b28738913465c7eda8ee898f28fdc724dbf016
--- /dev/null
+++ b/src/atlasComponents/userAnnotations/tools/type.spec.ts
@@ -0,0 +1,166 @@
+import { Subject, Subscription } from "rxjs"
+import { AbsToolClass, IAnnotationEvents, IAnnotationGeometry, TAnnotationEvent } from "./type"
+
+class TmpCls extends IAnnotationGeometry{
+ getNgAnnotationIds(){
+ return []
+ }
+ toNgAnnotation() {
+ return []
+ }
+ toJSON() {
+ return {}
+ }
+ toString() {
+ return ''
+ }
+ toSands(){
+ return {} as any
+ }
+}
+
+class TmpToolCls extends AbsToolClass<TmpCls> {
+ iconClass = ''
+ name: 'tmplClsTool'
+ subs = []
+ onMouseMoveRenderPreview() {
+ return []
+ }
+
+ protected managedAnnotations: TmpCls[] = []
+ public managedAnnotations$ = new Subject<TmpCls[]>()
+}
+
+describe('> types.ts', () => {
+ describe('> AbsToolClass', () => {
+ let tool: TmpToolCls
+ let ann: TmpCls
+ let managedAnn: jasmine.Spy
+ const subs: Subscription[] = []
+
+ beforeEach(() => {
+ const ev$ = new Subject<TAnnotationEvent<keyof IAnnotationEvents>>()
+ tool = new TmpToolCls(ev$, () => {})
+ ann = new TmpCls()
+ managedAnn = jasmine.createSpy('managedannspy')
+ subs.push(
+ tool.managedAnnotations$.subscribe(managedAnn)
+ )
+ })
+
+ afterEach(() => {
+ managedAnn.calls.reset()
+ while(subs.length) subs.pop().unsubscribe()
+ })
+
+ it('> shuld init just fine', () => {
+ expect(true).toEqual(true)
+ })
+ describe('> managedAnnotations$', () => {
+ describe('> on point add', () => {
+ it('> should emit new managedannotations', () => {
+ tool.addAnnotation(ann)
+ expect(managedAnn).toHaveBeenCalled()
+ expect(managedAnn).toHaveBeenCalledTimes(1)
+ const firstCallArgs = managedAnn.calls.allArgs()[0]
+ const firstArg = firstCallArgs[0]
+ expect(firstArg).toEqual([ann])
+ })
+ })
+ describe('> on point update', () => {
+ it('> should emit new managedannotations', () => {
+ tool.addAnnotation(ann)
+ ann.name = 'blabla'
+ expect(managedAnn).toHaveBeenCalledTimes(2)
+ const firstCallArgs = managedAnn.calls.allArgs()[0]
+ const secondCallArgs = managedAnn.calls.allArgs()[1]
+ expect(firstCallArgs).toEqual(secondCallArgs)
+ })
+ })
+ describe('> on point rm', () => {
+ it('> managed annotation === 0', () => {
+ tool.addAnnotation(ann)
+
+ const firstCallArgs = managedAnn.calls.allArgs()[0]
+ const subManagedAnn0 = firstCallArgs[0]
+ expect(subManagedAnn0.length).toEqual(1)
+
+ ann.remove()
+ expect(managedAnn).toHaveBeenCalledTimes(2)
+
+ const secondCallArgs = managedAnn.calls.allArgs()[1]
+ const subManagedAnn1 = secondCallArgs[0]
+ expect(subManagedAnn1.length).toEqual(0)
+ })
+ it('> does not trigger after rm', () => {
+ tool.addAnnotation(ann)
+ ann.remove()
+ ann.name = 'blabla'
+ expect(managedAnn).toHaveBeenCalledTimes(2)
+ })
+ })
+ })
+ })
+
+ describe('> IAnnotationGeometry', () => {
+ it('> can be init fine', () => {
+ new TmpCls()
+ expect(true).toBe(true)
+ })
+
+ describe('> updateSignal$', () => {
+ class TmpCls extends IAnnotationGeometry{
+ getNgAnnotationIds(){
+ return []
+ }
+ toNgAnnotation() {
+ return []
+ }
+ toJSON() {
+ return {}
+ }
+ toString() {
+ return `${this.name || ''}:${this.desc || ''}`
+ }
+ toSands(){
+ return {} as any
+ }
+ }
+
+ let tmp: TmpCls
+ let subs: Subscription[] = []
+ let updateStub: jasmine.Spy
+ beforeEach(() => {
+ tmp = new TmpCls()
+ updateStub = jasmine.createSpy('updateSpy')
+ subs.push(
+ tmp.updateSignal$.subscribe(
+ val => updateStub(val)
+ )
+ )
+ })
+ afterEach(() => {
+ while(subs.length) subs.pop().unsubscribe()
+ updateStub.calls.reset()
+ })
+ it('> is fired on setting name', () => {
+ tmp.name = 'test'
+ expect(updateStub).toHaveBeenCalled()
+ expect(updateStub).toHaveBeenCalledTimes(1)
+ })
+
+ it('> is fired on setting desc', () => {
+ tmp.desc = 'testdesc'
+ expect(updateStub).toHaveBeenCalled()
+ expect(updateStub).toHaveBeenCalledTimes(1)
+ })
+
+ it('> should be fired even if same string', () => {
+
+ tmp.name = null
+ tmp.name = undefined
+ expect(updateStub).toHaveBeenCalledTimes(2)
+ })
+ })
+ })
+})
diff --git a/src/atlasComponents/userAnnotations/tools/type.ts b/src/atlasComponents/userAnnotations/tools/type.ts
index eecdec68228f44d367f024af93ebddb6fd6555f9..8e33221b9bdf575fd3e2a196711236dce78b6d3b 100644
--- a/src/atlasComponents/userAnnotations/tools/type.ts
+++ b/src/atlasComponents/userAnnotations/tools/type.ts
@@ -18,9 +18,8 @@ export abstract class AbsToolClass<T extends IAnnotationGeometry> {
public abstract name: string
public abstract iconClass: string
- public abstract removeAnnotation(id: string): void
public abstract managedAnnotations$: Subject<T[]>
- protected abstract managedAnnotations: T[]
+ protected managedAnnotations: T[] = []
abstract subs: Subscription[]
protected space: TBaseAnnotationGeomtrySpec['space']
@@ -158,10 +157,26 @@ export abstract class AbsToolClass<T extends IAnnotationGeometry> {
public addAnnotation(geom: T) {
const found = this.managedAnnotations.find(ann => ann.id === geom.id)
if (found) found.remove()
- geom.remove = () => this.removeAnnotation(geom.id)
+ const sub = geom.updateSignal$.subscribe(() => {
+ this.managedAnnotations$.next(this.managedAnnotations)
+ })
+ geom.remove = () => {
+ this.removeAnnotation(geom)
+ sub.unsubscribe()
+ }
this.managedAnnotations.push(geom)
this.managedAnnotations$.next(this.managedAnnotations)
}
+
+ public removeAnnotation(geom: T) {
+ const idx = this.managedAnnotations.findIndex(ann => ann.id === geom.id)
+ if (idx < 0) {
+ console.warn(`removeAnnotation error: cannot annotation with id: ${geom.id}`)
+ return
+ }
+ this.managedAnnotations.splice(idx, 1)
+ this.managedAnnotations$.next(this.managedAnnotations)
+ }
}
export type TToolType = 'selecting' | 'drawing' | 'deletion'
@@ -276,8 +291,25 @@ export abstract class Highlightable {
export abstract class IAnnotationGeometry extends Highlightable {
public id: string
- public name: string
- public desc: string
+ private _name: string
+ set name(val: string) {
+ if (val === this._name) return
+ this._name = val
+ this.changed()
+ }
+ get name(): string {
+ return this._name
+ }
+
+ private _desc: string
+ set desc(val: string) {
+ if (val === this._desc) return
+ this._desc = val
+ this.changed()
+ }
+ get desc(): string {
+ return this._desc
+ }
public space: TBaseAnnotationGeomtrySpec['space']
@@ -291,7 +323,13 @@ export abstract class IAnnotationGeometry extends Highlightable {
public remove() {
throw new Error(`The remove method needs to be overwritten by the tool manager`)
}
- public updateSignal$ = new Subject()
+
+ private _updateSignal$ = new Subject()
+
+ public updateSignal$ = this._updateSignal$.asObservable()
+ protected changed(){
+ this._updateSignal$.next(Date.now())
+ }
constructor(spec?: TBaseAnnotationGeomtrySpec){
super()
@@ -300,15 +338,6 @@ export abstract class IAnnotationGeometry extends Highlightable {
this.name = spec?.name
this.desc = spec?.desc
}
-
- setName(name: string) {
- this.name = name
- this.updateSignal$.next(this.toString())
- }
- setDesc(desc: string) {
- this.desc = desc
- this.updateSignal$.next(this.toString())
- }
}
export interface IAnnotationTools {
diff --git a/src/atlasViewer/atlasViewer.workerService.service.ts b/src/atlasViewer/atlasViewer.workerService.service.ts
index 62395463088ed3939c419ec61329c18e68865993..653764d97a7670d16ca8805c1033f09775de0cb0 100644
--- a/src/atlasViewer/atlasViewer.workerService.service.ts
+++ b/src/atlasViewer/atlasViewer.workerService.service.ts
@@ -3,15 +3,7 @@ import { fromEvent } from "rxjs";
import { filter, take } from "rxjs/operators";
import { getUuid } from "src/util/fn";
-/* telling webpack to pack the worker file */
-
-import '!!file-loader?name=worker.js!worker/worker.js'
-import '!!file-loader?name=worker-plotly.js!worker/worker-plotly.js'
-import '!!file-loader?name=worker-nifti.js!worker/worker-nifti.js'
-
-/**
- * export the worker, so that services that does not require dependency injection can import the worker
- */
+// worker is now exported in angular.json file
export const worker = new Worker('worker.js')
interface IWorkerMessage {
diff --git a/src/components/confirmDialog/confirmDialog.component.ts b/src/components/confirmDialog/confirmDialog.component.ts
index ec2e20c9fb6469c9ec26f88c77b73f50a77c57d3..8361ee409762ebccccfb6ae9c85d064207587dfc 100644
--- a/src/components/confirmDialog/confirmDialog.component.ts
+++ b/src/components/confirmDialog/confirmDialog.component.ts
@@ -25,15 +25,19 @@ export class ConfirmDialogComponent {
@Input()
public markdown: string
+ @Input()
+ public confirmOnly: boolean = false
+
public hideActionBar = false
constructor(@Inject(MAT_DIALOG_DATA) data: any) {
- const { title = null, message = null, markdown, okBtnText, cancelBtnText, hideActionBar} = data || {}
+ const { title = null, message = null, markdown, okBtnText, cancelBtnText, hideActionBar, confirmOnly} = data || {}
if (title) this.title = title
if (message) this.message = message
if (markdown) this.markdown = markdown
if (okBtnText) this.okBtnText = okBtnText
if (cancelBtnText) this.cancelBtnText = cancelBtnText
if (hideActionBar) this.hideActionBar = hideActionBar
+ if (confirmOnly) this.confirmOnly = confirmOnly
}
}
diff --git a/src/components/confirmDialog/confirmDialog.template.html b/src/components/confirmDialog/confirmDialog.template.html
index f5f0549460e034215176f4393e3ea2507c83533d..8b8a0224a96105f7b475cfc587d2a25d11932b23 100644
--- a/src/components/confirmDialog/confirmDialog.template.html
+++ b/src/components/confirmDialog/confirmDialog.template.html
@@ -18,6 +18,6 @@
<mat-divider></mat-divider>
<mat-dialog-actions *ngIf="!hideActionBar" class="justify-content-start flex-row-reverse">
- <button [mat-dialog-close]="true" mat-raised-button color="primary">{{ okBtnText }}</button>
- <button [mat-dialog-close]="false" mat-button>{{ cancelBtnText }}</button>
+ <button [mat-dialog-close]="true" mat-raised-button color="primary">{{ okBtnText }}</button>
+ <button *ngIf="!confirmOnly" [mat-dialog-close]="false" mat-button>{{ cancelBtnText }}</button>
</mat-dialog-actions>
diff --git a/src/environments/environment.common.ts b/src/environments/environment.common.ts
index 52bdf0c70442a366f643e161bca3502346b756c4..85a5e68ae63408c740790bd514fb6ad3fdbe8bb2 100644
--- a/src/environments/environment.common.ts
+++ b/src/environments/environment.common.ts
@@ -1,6 +1,7 @@
export const environment = {
- VERSION: 'unspecificied hash',
+ GIT_HASH: 'unknown hash',
+ VERSION: 'unknown version',
PRODUCTION: true,
BACKEND_URL: null,
DATASET_PREVIEW_URL: 'https://hbp-kg-dataset-previewer.apps.hbp.eu/v2',
diff --git a/src/environments/parseEnv.js b/src/environments/parseEnv.js
index 2b3052d932dd2be8ca0f738ec2f4e26c91ef1801..84c95658b5b7466938de4d8c8f675950662b1162 100644
--- a/src/environments/parseEnv.js
+++ b/src/environments/parseEnv.js
@@ -13,17 +13,21 @@ const main = async () => {
MATOMO_ID,
BS_REST_URL,
VERSION,
- GIT_HASH,
+ GIT_HASH = 'unknown hash',
EXPERIMENTAL_FEATURE_FLAG
} = process.env
const version = JSON.stringify(
- VERSION || GIT_HASH || 'unspecificied hash'
- )
+ VERSION || 'unknown version'
+ )
+ const gitHash = JSON.stringify(
+ GIT_HASH || 'unknown hash'
+ )
const outputTxt = `
import { environment as commonEnv } from './environment.common'
export const environment = {
...commonEnv,
+ GIT_HASH: ${gitHash},
VERSION: ${version},
BS_REST_URL: ${JSON.stringify(BS_REST_URL)},
BACKEND_URL: ${JSON.stringify(BACKEND_URL)},
diff --git a/src/main-common.ts b/src/main-common.ts
index 3afe8d0ad8e5dd329f4392e4e7de3aff1228fe00..c3f3f5ee2d1f07260b1932bbe9152d65c7e81dc7 100644
--- a/src/main-common.ts
+++ b/src/main-common.ts
@@ -1,17 +1,19 @@
// Included to include a copy of vanilla nehuba
import '!!file-loader?context=third_party&name=vanilla.html!third_party/vanilla.html'
import '!!file-loader?context=third_party&name=vanilla_styles.css!third_party/styles.css'
-import '!!file-loader?context=third_party&name=vanilla_nehuba.js!third_party/vanilla_nehuba.js'
import '!!file-loader?context=third_party&name=preinit_vanilla.html!third_party/preinit_vanilla.html'
/**
* Catching Safari 10 bug:
*
* https://bugs.webkit.org/show_bug.cgi?id=171041
-*
+*
+* moved to angular.json
+* look for
+* - third_party/catchSyntaxError.js
+* - third_party/syntaxError.js
*/
-import '!!file-loader?context=third_party&name=catchSyntaxError.js!third_party/catchSyntaxError.js'
-import '!!file-loader?context=third_party&name=syntaxError.js!third_party/syntaxError.js'
+
import '!!file-loader?context=src/res&name=icons/iav-icons.css!src/res/icons/iav-icons.css'
import '!!file-loader?context=src/res&name=icons/iav-icons.ttf!src/res/icons/iav-icons.ttf'
@@ -36,9 +38,9 @@ import { platformBrowserDynamic } from '@angular/platform-browser-dynamic'
import { MainModule } from './main.module';
import { environment } from 'src/environments/environment'
-const { PRODUCTION, VERSION } = environment
+const { PRODUCTION, VERSION, GIT_HASH } = environment
if (PRODUCTION) enableProdMode()
-if (PRODUCTION) { console.log(`Interactive Atlas Viewer: ${VERSION}`) }
+if (PRODUCTION) { console.log(`Siibra Explorer: ${VERSION}::${GIT_HASH}`) }
platformBrowserDynamic().bootstrapModule(MainModule)
diff --git a/src/routerModule/router.service.spec.ts b/src/routerModule/router.service.spec.ts
index 5507f4a5ba5d4bb05c86853bf91768b4d7eaf213..2b933c6f1a5711b8503776d082203e684299f563 100644
--- a/src/routerModule/router.service.spec.ts
+++ b/src/routerModule/router.service.spec.ts
@@ -3,6 +3,7 @@ import { discardPeriodicTasks, fakeAsync, TestBed, tick } from "@angular/core/te
import { Router } from "@angular/router"
import { RouterTestingModule } from '@angular/router/testing'
import { MockStore, provideMockStore } from "@ngrx/store/testing"
+import { cold } from "jasmine-marbles"
import { of } from "rxjs"
import { PureContantService } from "src/util"
import { RouterService } from "./router.service"
@@ -290,5 +291,114 @@ describe('> router.service.ts', () => {
})
})
})
+
+ describe('> customRoute$', () => {
+ let decodeCustomStateSpy: jasmine.Spy
+ beforeEach(() => {
+ decodeCustomStateSpy = jasmine.createSpy('decodeCustomState')
+ spyOnProperty(util, 'decodeCustomState').and.returnValue(decodeCustomStateSpy)
+
+ router = TestBed.inject(Router)
+ })
+
+ afterEach(() => {
+ decodeCustomStateSpy.calls.reset()
+ })
+
+ it('> emits return record from decodeCustomState', fakeAsync(() => {
+ const value = {
+ 'x-foo': 'bar'
+ }
+ decodeCustomStateSpy.and.returnValue(value)
+ const rService = TestBed.inject(RouterService)
+ router.navigate(['foo'])
+ tick(320)
+
+ expect(rService.customRoute$).toBeObservable(
+ cold('a', {
+ a: {
+ 'x-foo': 'bar'
+ }
+ })
+ )
+ discardPeriodicTasks()
+ }))
+ it('> merges observable from _customRoutes$', fakeAsync(() => {
+ decodeCustomStateSpy.and.returnValue({})
+ const rService = TestBed.inject(RouterService)
+ rService.setCustomRoute('x-fizz', 'buzz')
+ tick(320)
+
+ expect(rService.customRoute$).toBeObservable(
+ cold('(ba)', {
+ a: {
+ 'x-fizz': 'buzz'
+ },
+ b: {}
+ })
+ )
+ discardPeriodicTasks()
+ }))
+
+ it('> merges from both sources', fakeAsync(() => {
+ const value = {
+ 'x-foo': 'bar'
+ }
+ decodeCustomStateSpy.and.returnValue(value)
+ const rService = TestBed.inject(RouterService)
+ rService.setCustomRoute('x-fizz', 'buzz')
+ tick(320)
+
+ expect(rService.customRoute$).toBeObservable(
+ cold('(ba)', {
+ a: {
+ 'x-fizz': 'buzz',
+ 'x-foo': 'bar'
+ },
+ b: {
+ 'x-foo': 'bar'
+ }
+ })
+ )
+ discardPeriodicTasks()
+ }))
+
+ it('> subsequent emits overwrites', fakeAsync(() => {
+ decodeCustomStateSpy.and.returnValue({})
+ const rService = TestBed.inject(RouterService)
+ spyOn(router, 'navigateByUrl').and.callFake((() => {
+ console.log('navigate by url')
+ }) as any)
+
+ const customRouteSpy = jasmine.createSpy('customRouteSpy')
+ rService.customRoute$.subscribe(customRouteSpy)
+
+ rService.setCustomRoute('x-fizz', 'buzz')
+ tick(20)
+ rService.setCustomRoute('x-foo', 'bar')
+ tick(20)
+ rService.setCustomRoute('x-foo', null)
+
+ tick(320)
+
+ const expectedCalls = {
+ z: {
+ 'x-fizz': 'buzz',
+ 'x-foo': null
+ },
+ a: {
+ 'x-fizz': 'buzz',
+ 'x-foo': 'bar'
+ },
+ b: {
+ 'x-fizz': 'buzz'
+ }
+ }
+ for (const c in expectedCalls) {
+ expect(customRouteSpy).toHaveBeenCalledWith(expectedCalls[c])
+ }
+ discardPeriodicTasks()
+ }))
+ })
})
})
diff --git a/src/routerModule/router.service.ts b/src/routerModule/router.service.ts
index 0a9ce582a948aae79538f17c67401d8c7b079257..ee0246f79302d60fae4c9caa8477c65120929d52 100644
--- a/src/routerModule/router.service.ts
+++ b/src/routerModule/router.service.ts
@@ -3,10 +3,12 @@ import { APP_BASE_HREF } from "@angular/common";
import { Inject } from "@angular/core";
import { NavigationEnd, Router } from '@angular/router'
import { Store } from "@ngrx/store";
-import { debounceTime, filter, map, shareReplay, switchMapTo, take, withLatestFrom } from "rxjs/operators";
+import { debounceTime, distinctUntilChanged, filter, map, shareReplay, startWith, switchMapTo, take, tap, withLatestFrom } from "rxjs/operators";
import { generalApplyState } from "src/services/stateStore.helper";
import { PureContantService } from "src/util";
-import { cvtStateToHashedRoutes, cvtFullRouteToState } from "./util";
+import { cvtStateToHashedRoutes, cvtFullRouteToState, encodeCustomState, decodeCustomState, verifyCustomState } from "./util";
+import { BehaviorSubject, combineLatest, merge, Observable } from 'rxjs'
+import { scan } from 'rxjs/operators'
@Injectable({
providedIn: 'root'
@@ -18,6 +20,21 @@ export class RouterService {
console.log(...e)
}
+ private _customRoute$ = new BehaviorSubject<{
+ [key: string]: string
+ }>({})
+
+ public customRoute$: Observable<Record<string, any>>
+
+ setCustomRoute(key: string, state: string){
+ if (!verifyCustomState(key)) {
+ throw new Error(`custom state key must start with x- `)
+ }
+ this._customRoute$.next({
+ [key]: state
+ })
+ }
+
constructor(
router: Router,
pureConstantService: PureContantService,
@@ -40,6 +57,40 @@ export class RouterService {
shareReplay(1),
)
+ this.customRoute$ = ready$.pipe(
+ switchMapTo(
+ merge(
+ navEnd$.pipe(
+ map((ev: NavigationEnd) => {
+ const fullPath = ev.urlAfterRedirects
+ const customState = decodeCustomState(
+ router.parseUrl(fullPath)
+ )
+ return customState || {}
+ }),
+ ),
+ this._customRoute$
+ ).pipe(
+ scan<Record<string, string>>((acc, curr) => {
+ return {
+ ...acc,
+ ...curr
+ }
+ }, {}),
+ // TODO add obj eql distinctuntilchanged check
+ distinctUntilChanged((o, n) => {
+ if (Object.keys(o).length !== Object.keys(n).length) {
+ return false
+ }
+ for (const key in o) {
+ if (o[key] !== n[key]) return false
+ }
+ return true
+ }),
+ )
+ ),
+ )
+
ready$.pipe(
switchMapTo(
navEnd$.pipe(
@@ -70,15 +121,28 @@ export class RouterService {
// which may or many not be
ready$.pipe(
switchMapTo(
- store$.pipe(
- debounceTime(160),
- map(state => {
- try {
- return cvtStateToHashedRoutes(state)
- } catch (e) {
- this.logError(e)
- return ``
+ combineLatest([
+ store$.pipe(
+ debounceTime(160),
+ map(state => {
+ try {
+ return cvtStateToHashedRoutes(state)
+ } catch (e) {
+ this.logError(e)
+ return ``
+ }
+ })
+ ),
+ this.customRoute$,
+ ]).pipe(
+ map(([ routePath, customPath ]) => {
+ let returnPath = routePath
+ for (const key in customPath) {
+ const customStatePath = encodeCustomState(key, customPath[key])
+ if (!customStatePath) continue
+ returnPath += `/${customStatePath}`
}
+ return returnPath
})
)
)
diff --git a/src/routerModule/util.spec.ts b/src/routerModule/util.spec.ts
index b2ed55ea82db7f09f581114c2157a40ad2731bb6..35e0baefc23aa62bd6531b3ea32a2659744c63e7 100644
--- a/src/routerModule/util.spec.ts
+++ b/src/routerModule/util.spec.ts
@@ -2,7 +2,7 @@ import { TestBed } from '@angular/core/testing'
import { MockStore, provideMockStore } from '@ngrx/store/testing'
import { uiStatePreviewingDatasetFilesSelector } from 'src/services/state/uiState/selectors'
import { viewerStateGetSelectedAtlas, viewerStateSelectedParcellationSelector, viewerStateSelectedRegionsSelector, viewerStateSelectedTemplateSelector, viewerStateSelectorNavigation, viewerStateSelectorStandaloneVolumes } from 'src/services/state/viewerState/selectors'
-import { cvtFullRouteToState, cvtStateToHashedRoutes, DummyCmp, routes } from './util'
+import { cvtFullRouteToState, cvtStateToHashedRoutes, DummyCmp, encodeCustomState, routes, verifyCustomState } from './util'
import { encodeNumber } from './cipher'
import { Router } from '@angular/router'
import { RouterTestingModule } from '@angular/router/testing'
@@ -158,4 +158,35 @@ describe('> util.ts', () => {
})
})
})
+
+ describe('> verifyCustomState', () => {
+ it('> should return false on bad custom state', () => {
+ expect(verifyCustomState('hello')).toBeFalse()
+ })
+ it('> should return true on valid custom state', () => {
+ expect(verifyCustomState('x-test')).toBeTrue()
+ })
+ })
+
+ describe('> encodeCustomState', () => {
+ describe('> malformed values', () => {
+ describe('> bad key', () => {
+ it('> throws', () => {
+ expect(() => {
+ encodeCustomState('hello', 'world')
+ }).toThrow()
+ })
+ })
+ describe('> falsy value', () => {
+ it('> returns falsy value', () => {
+ expect(encodeCustomState('x-test', null)).toBeFalsy()
+ })
+ })
+ })
+ describe('> correct values', () => {
+ it('> encodes correctly', () => {
+ expect(encodeCustomState('x-test', 'foo-bar')).toEqual('x-test:foo-bar')
+ })
+ })
+ })
})
diff --git a/src/routerModule/util.ts b/src/routerModule/util.ts
index bbf8791d3e53851ffb05a9e0acff9646003ff356..39ec477aebc60ba1bd201971923bc5c3a6f539c5 100644
--- a/src/routerModule/util.ts
+++ b/src/routerModule/util.ts
@@ -306,6 +306,34 @@ export const cvtStateToHashedRoutes = (state): string => {
: `${routesArr.join('/')}?${searchParam.toString()}`
}
+export const verifyCustomState = (key: string) => {
+ return /^x-/.test(key)
+}
+
+export const decodeCustomState = (fullPath: UrlTree) => {
+ const returnObj: Record<string, string> = {}
+
+ const pathFragments: UrlSegment[] = fullPath.root.hasChildren()
+ ? fullPath.root.children['primary'].segments
+ : []
+
+ for (const f of pathFragments) {
+ if (!verifyCustomState(f.path)) continue
+ const { key, val } = decodePath(f.path) || {}
+ if (!key || !val) continue
+ returnObj[key] = val[0]
+ }
+ return returnObj
+}
+
+export const encodeCustomState = (key: string, value: string) => {
+ if (!verifyCustomState(key)) {
+ throw new Error(`custom state must start with x-`)
+ }
+ if (!value) return null
+ return endcodePath(key, value)
+}
+
@Component({
template: ''
})
diff --git a/src/services/dialogService.service.ts b/src/services/dialogService.service.ts
index eb0844dd7f3102b921fbe30de7a21a86280fe577..933b862275ca6838deb4ebabec3768ff604e4769 100644
--- a/src/services/dialogService.service.ts
+++ b/src/services/dialogService.service.ts
@@ -82,4 +82,5 @@ export interface DialogConfig {
message: string
markdown?: string
iconClass: string
+ confirmOnly: boolean
}
diff --git a/src/services/state/ngViewerState.store.ts b/src/services/state/ngViewerState.store.ts
index 4b30e79372125b271161308125f9835400ec7886..5b1758d60906076840930cafbf878ab834e7e88d 100644
--- a/src/services/state/ngViewerState.store.ts
+++ b/src/services/state/ngViewerState.store.ts
@@ -191,29 +191,6 @@ export class NgViewerUseEffect implements OnDestroy {
private http: HttpClient,
){
- // TODO either split backend user to be more granular, or combine the user config into a single subscription
- this.subscriptions.push(
- this.store$.pipe(
- select('ngViewerState'),
- debounceTime(200),
- skip(1),
- // Max frequency save once every second
-
- // properties to be saved
- map(({ panelMode, panelOrder }) => {
- return { panelMode, panelOrder }
- }),
- distinctUntilChanged(),
- throttleTime(1000)
- ).subscribe(ngViewerState => {
- this.http.post(`${this.pureConstantService.backendUrl}user/config`, JSON.stringify({ ngViewerState }), {
- headers: {
- 'Content-type': 'application/json'
- }
- })
- })
- )
-
this.applySavedUserConfig$ = this.http.get<TUserConfigResp>(`${this.pureConstantService.backendUrl}user/config`).pipe(
map(json => {
if (json.error) {
diff --git a/src/share/saneUrl/saneUrl.component.spec.ts b/src/share/saneUrl/saneUrl.component.spec.ts
index 4b830bb1aac5339f3a27cf9963328be5d9cd45c1..5ab173950c92148144b7b3025f63cddf95205a30 100644
--- a/src/share/saneUrl/saneUrl.component.spec.ts
+++ b/src/share/saneUrl/saneUrl.component.spec.ts
@@ -1,26 +1,49 @@
-import { TestBed, async, fakeAsync, tick, flush } from '@angular/core/testing'
+import { TestBed, fakeAsync, tick, flush } from '@angular/core/testing'
import { ShareModule } from '../share.module'
import { SaneUrl } from './saneUrl.component'
import { HttpClientTestingModule, HttpTestingController } from '@angular/common/http/testing'
import { By } from '@angular/platform-browser'
import { BACKENDURL } from 'src/util/constants'
import { NoopAnimationsModule } from '@angular/platform-browser/animations'
+import { SaneUrlSvc } from './saneUrl.service'
+import { AngularMaterialModule } from 'src/sharedModules'
+import { CUSTOM_ELEMENTS_SCHEMA, Directive } from '@angular/core'
+import { of } from 'rxjs'
const inputCss = `input[aria-label="Custom link"]`
const submitCss = `button[aria-label="Create custom link"]`
const copyBtnCss = `button[aria-label="Copy created custom URL to clipboard"]`
+@Directive({
+ selector: '[iav-auth-auth-state]',
+ exportAs: 'iavAuthAuthState'
+})
+
+class AuthStateDummy {
+ user$ = of(null)
+}
+
describe('> saneUrl.component.ts', () => {
describe('> SaneUrl', () => {
- beforeEach(async(() => {
- TestBed.configureTestingModule({
+ beforeEach(async () => {
+ await TestBed.configureTestingModule({
imports: [
- ShareModule,
HttpClientTestingModule,
NoopAnimationsModule,
+ AngularMaterialModule,
+ ],
+ providers: [
+ SaneUrlSvc,
+ ],
+ declarations: [
+ SaneUrl,
+ AuthStateDummy
+ ],
+ schemas: [
+ CUSTOM_ELEMENTS_SCHEMA
]
}).compileComponents()
- }))
+ })
afterEach(() => {
const ctrl = TestBed.inject(HttpTestingController)
@@ -65,8 +88,6 @@ describe('> saneUrl.component.ts', () => {
fixture.detectChanges()
const input = fixture.debugElement.query( By.css( inputCss ) )
- const invalid = input.attributes['aria-invalid']
- expect(invalid.toString()).toEqual('true')
}))
@@ -89,8 +110,6 @@ describe('> saneUrl.component.ts', () => {
fixture.detectChanges()
const input = fixture.debugElement.query( By.css( inputCss ) )
- const invalid = input.attributes['aria-invalid']
- expect(invalid.toString()).toEqual('false')
})
it('> on entering string in input, makes debounced GET request', fakeAsync(() => {
@@ -139,8 +158,6 @@ describe('> saneUrl.component.ts', () => {
fixture.detectChanges()
const input = fixture.debugElement.query( By.css( inputCss ) )
- const invalid = input.attributes['aria-invalid']
- expect(invalid.toString()).toEqual('true')
const submit = fixture.debugElement.query( By.css( submitCss ) )
const disabled = !!submit.attributes['disabled']
@@ -173,8 +190,6 @@ describe('> saneUrl.component.ts', () => {
fixture.detectChanges()
const input = fixture.debugElement.query( By.css( inputCss ) )
- const invalid = input.attributes['aria-invalid']
- expect(invalid.toString()).toEqual('false')
const submit = fixture.debugElement.query( By.css( submitCss ) )
const disabled = !!submit.attributes['disabled']
@@ -207,8 +222,6 @@ describe('> saneUrl.component.ts', () => {
fixture.detectChanges()
const input = fixture.debugElement.query( By.css( inputCss ) )
- const invalid = input.attributes['aria-invalid']
- expect(invalid.toString()).toEqual('true')
const submit = fixture.debugElement.query( By.css( submitCss ) )
const disabled = !!submit.attributes['disabled']
@@ -307,8 +320,6 @@ describe('> saneUrl.component.ts', () => {
fixture.detectChanges()
const input = fixture.debugElement.query( By.css( inputCss ) )
- const invalid = input.attributes['aria-invalid']
- expect(invalid.toString()).toEqual('true')
}))
})
diff --git a/src/share/saneUrl/saneUrl.component.ts b/src/share/saneUrl/saneUrl.component.ts
index c397478f5b62b0a7574d3b4b00a5d95fc8e7253a..dbcbb722efd73c5d9f8a15dc4e6c282901ff7cfa 100644
--- a/src/share/saneUrl/saneUrl.component.ts
+++ b/src/share/saneUrl/saneUrl.component.ts
@@ -1,12 +1,12 @@
import { Component, OnDestroy, Input } from "@angular/core";
-import { HttpClient } from '@angular/common/http'
-import { BACKENDURL } from 'src/util/constants'
import { Observable, merge, of, Subscription, BehaviorSubject, combineLatest } from "rxjs";
import { startWith, mapTo, map, debounceTime, switchMap, catchError, shareReplay, filter, tap, takeUntil, distinctUntilChanged } from "rxjs/operators";
import { FormControl } from "@angular/forms";
import { ErrorStateMatcher } from "@angular/material/core";
import { Clipboard } from "@angular/cdk/clipboard";
import { MatSnackBar } from "@angular/material/snack-bar";
+import { SaneUrlSvc } from "./saneUrl.service";
+import { NotFoundError } from "../type";
export class SaneUrlErrorStateMatcher implements ErrorStateMatcher{
isErrorState(ctrl: FormControl | null){
@@ -43,6 +43,8 @@ enum ESavingStatus {
export class SaneUrl implements OnDestroy{
+ public saneUrlRoot = this.svc.saneUrlRoot
+
@Input() stateTobeSaved: any
private subscriptions: Subscription[] = []
@@ -62,9 +64,9 @@ export class SaneUrl implements OnDestroy{
public saved$: Observable<boolean>
constructor(
- private http: HttpClient,
private clipboard: Clipboard,
private snackbar: MatSnackBar,
+ private svc: SaneUrlSvc,
){
const validatedValueInput$ = this.customUrl.valueChanges.pipe(
@@ -84,11 +86,10 @@ export class SaneUrl implements OnDestroy{
debounceTime(500),
switchMap(val => val === ''
? of(false)
- : this.http.get(`${this.saneUrlRoot}${val}`).pipe(
+ : this.svc.getKeyVal(val).pipe(
mapTo(false),
catchError((err, obs) => {
- const { status } = err
- if (status === 404) return of(true)
+ if (err instanceof NotFoundError) return of(true)
return of(false)
})
)
@@ -105,10 +106,10 @@ export class SaneUrl implements OnDestroy{
)
)
- this.btnHintTxt$ = combineLatest(
+ this.btnHintTxt$ = combineLatest([
this.savingStatus$,
this.savingProgress$,
- ).pipe(
+ ]).pipe(
map(([savingStatus, savingProgress]) => {
if (savingProgress === ESavingProgress.DONE) return EBtnTxt.CREATED
if (savingProgress === ESavingProgress.INPROGRESS) return EBtnTxt.CREATING
@@ -123,10 +124,10 @@ export class SaneUrl implements OnDestroy{
startWith(true)
)
- this.iconClass$ = combineLatest(
+ this.iconClass$ = combineLatest([
this.savingStatus$,
this.savingProgress$,
- ).pipe(
+ ]).pipe(
map(([savingStatus, savingProgress]) => {
if (savingProgress === ESavingProgress.DONE) return `fas fa-check`
if (savingProgress === ESavingProgress.INPROGRESS) return `fas fa-spinner fa-spin`
@@ -159,8 +160,8 @@ export class SaneUrl implements OnDestroy{
saveLink(){
this.savingProgress$.next(ESavingProgress.INPROGRESS)
this.customUrl.disable()
- this.http.post(
- `${this.saneUrlRoot}${this.customUrl.value}`,
+ this.svc.setKeyVal(
+ this.customUrl.value,
this.stateTobeSaved
).subscribe(
resp => {
@@ -184,6 +185,4 @@ export class SaneUrl implements OnDestroy{
{ duration: 1000 }
)
}
-
- public saneUrlRoot = `${BACKENDURL}saneUrl/`
}
diff --git a/src/share/saneUrl/saneUrl.service.ts b/src/share/saneUrl/saneUrl.service.ts
new file mode 100644
index 0000000000000000000000000000000000000000..8f3af1f3d55fcb9baec57d44ddbbda76ac24c2df
--- /dev/null
+++ b/src/share/saneUrl/saneUrl.service.ts
@@ -0,0 +1,43 @@
+import { HttpClient } from "@angular/common/http";
+import { Injectable } from "@angular/core";
+import { throwError } from "rxjs";
+import { catchError, mapTo } from "rxjs/operators";
+import { BACKENDURL } from 'src/util/constants'
+import { IKeyValStore, NotFoundError } from '../type'
+
+@Injectable({
+ providedIn: 'root'
+})
+
+export class SaneUrlSvc implements IKeyValStore{
+ public saneUrlRoot = `${BACKENDURL}saneUrl/`
+ constructor(
+ private http: HttpClient
+ ){
+
+ }
+
+ getKeyVal(key: string) {
+ return this.http.get<Record<string, any>>(
+ `${this.saneUrlRoot}${key}`,
+ { responseType: 'json' }
+ ).pipe(
+ catchError((err, obs) => {
+ const { status } = err
+ if (status === 404) {
+ return throwError(new NotFoundError('Not found'))
+ }
+ return throwError(err)
+ })
+ )
+ }
+
+ setKeyVal(key: string, value: any) {
+ return this.http.post(
+ `${this.saneUrlRoot}${key}`,
+ value
+ ).pipe(
+ mapTo(`${this.saneUrlRoot}${key}`)
+ )
+ }
+}
diff --git a/src/share/saneUrl/saneUrl.template.html b/src/share/saneUrl/saneUrl.template.html
index a673dcef78ef91587d47cf9406af51af7de8b151..cd191f957b3e6e59a687bd37593d5058238cfed0 100644
--- a/src/share/saneUrl/saneUrl.template.html
+++ b/src/share/saneUrl/saneUrl.template.html
@@ -1,3 +1,36 @@
+<div iav-auth-auth-state
+ #authState="iavAuthAuthState">
+
+<!-- Logged in. Explain that links will not expire, offer to logout -->
+<ng-container *ngIf="authState.user$ | async as user; else otherTmpl">
+ <span>
+ Logged in as {{ user.name }}
+ </span>
+ <button mat-button
+ color="warn"
+ tabindex="-1">
+ <i class="fas fa-sign-in-alt"></i>
+ <span>
+ Logout
+ </span>
+ </button>
+</ng-container>
+
+<!-- Not logged in. Offer to login -->
+<ng-template #otherTmpl>
+ <span>
+ Not logged in
+ </span>
+ <signin-modal></signin-modal>
+</ng-template>
+</div>
+
+<!-- explain links expiration -->
+<div class="text-muted mat-small">
+{{ (authState.user$ | async) ? 'Links you generate will not expire' : 'Links you generate will expire after 72 hours' }}
+</div>
+
+
<mat-form-field class="mr-2">
<span matPrefix class="text-muted">
{{ saneUrlRoot }}
diff --git a/src/share/share.module.ts b/src/share/share.module.ts
index 28e16704f18648e8a4a554d1ad63d0f50e14a1c6..9634e12279dd09f165c6a6d4fd34653cbf52550f 100644
--- a/src/share/share.module.ts
+++ b/src/share/share.module.ts
@@ -5,6 +5,7 @@ import { HttpClientModule } from "@angular/common/http";
import { SaneUrl } from "./saneUrl/saneUrl.component";
import { CommonModule } from "@angular/common";
import { ReactiveFormsModule, FormsModule } from "@angular/forms";
+import { AuthModule } from "src/auth";
@NgModule({
imports: [
@@ -13,6 +14,7 @@ import { ReactiveFormsModule, FormsModule } from "@angular/forms";
CommonModule,
FormsModule,
ReactiveFormsModule,
+ AuthModule,
],
declarations: [
ClipboardCopy,
diff --git a/src/share/type.ts b/src/share/type.ts
new file mode 100644
index 0000000000000000000000000000000000000000..ead6dfdbdb8c409f6051532bfec218cda0625d46
--- /dev/null
+++ b/src/share/type.ts
@@ -0,0 +1,8 @@
+import { Observable } from "rxjs";
+
+export class NotFoundError extends Error{}
+
+export interface IKeyValStore {
+ getKeyVal(key: string): Observable<unknown>
+ setKeyVal(key: string, val: unknown): Observable<void|string>
+}
diff --git a/src/state/stateAggregator.directive.ts b/src/state/stateAggregator.directive.ts
index 62d66666227e91a18bd1aac9ba2309d7a5c43a8d..28df9bf19c5000333fbe228743b1495932f117fd 100644
--- a/src/state/stateAggregator.directive.ts
+++ b/src/state/stateAggregator.directive.ts
@@ -1,14 +1,14 @@
-import { Directive } from "@angular/core";
+import { Directive, OnDestroy } from "@angular/core";
import { NavigationEnd, Router } from "@angular/router";
-import { Observable } from "rxjs";
-import { filter, map } from "rxjs/operators";
+import { Observable, Subscription } from "rxjs";
+import { filter, map, startWith } from "rxjs/operators";
const jsonVersion = '1.0.0'
// ver 0.0.1 === query param
interface IJsonifiedState {
ver: string
- queryString: any
+ hashPath: string
}
@Directive({
@@ -16,20 +16,30 @@ interface IJsonifiedState {
exportAs: 'iavStateAggregator'
})
-export class StateAggregator{
+export class StateAggregator implements OnDestroy{
- public jsonifiedSstate$: Observable<IJsonifiedState>
+ public jsonifiedState: IJsonifiedState
+ public jsonifiedState$: Observable<IJsonifiedState> = this.router.events.pipe(
+ filter(ev => ev instanceof NavigationEnd),
+ map((ev: NavigationEnd) => ev.urlAfterRedirects),
+ startWith(this.router.url),
+ map((path: string) => {
+ return {
+ ver: jsonVersion,
+ hashPath: path
+ }
+ }),
+ )
constructor(
- router: Router
+ private router: Router
){
- this.jsonifiedSstate$ = router.events.pipe(
- filter(ev => ev instanceof NavigationEnd),
- map((ev: NavigationEnd) => {
- return {
- ver: jsonVersion,
- queryString: ev.urlAfterRedirects
- }
- })
+ this.subscriptions.push(
+ this.jsonifiedState$.subscribe(val => this.jsonifiedState = val)
)
}
+
+ private subscriptions: Subscription[] = []
+ ngOnDestroy(){
+ while (this.subscriptions.length) this.subscriptions.pop().unsubscribe()
+ }
}
diff --git a/src/ui/logoContainer/logoContainer.component.ts b/src/ui/logoContainer/logoContainer.component.ts
index 3ae73afe9c647971c928716477b7b69723719f2e..105e0726251a75522bc2d747767c0fe7b3eb50c2 100644
--- a/src/ui/logoContainer/logoContainer.component.ts
+++ b/src/ui/logoContainer/logoContainer.component.ts
@@ -1,6 +1,7 @@
import { Component } from "@angular/core";
import { PureContantService } from "src/util";
import { Subscription } from "rxjs";
+import { distinctUntilChanged } from "rxjs/operators";
const imageDark = 'assets/logo/ebrains-logo-dark.svg'
const imageLight = 'assets/logo/ebrains-logo-light.svg'
@@ -26,7 +27,9 @@ export class LogoContainer {
private pureConstantService: PureContantService
){
this.subscriptions.push(
- pureConstantService.darktheme$.subscribe(flag => {
+ pureConstantService.darktheme$.pipe(
+ distinctUntilChanged()
+ ).subscribe(flag => {
this.containerStyle = {
backgroundImage: `url('${flag ? imageLight : imageDark}')`
}
diff --git a/src/util/mergeObj.pipe.ts b/src/util/mergeObj.pipe.ts
new file mode 100644
index 0000000000000000000000000000000000000000..104592d6732bcadff54c8a5a92a32405e40f6025
--- /dev/null
+++ b/src/util/mergeObj.pipe.ts
@@ -0,0 +1,17 @@
+import { Pipe, PipeTransform } from "@angular/core";
+
+type TObj = Record<string, any>
+
+@Pipe({
+ name: 'mergeObj',
+ pure: true
+})
+
+export class MergeObjPipe implements PipeTransform{
+ public transform(o1: TObj, o2: TObj){
+ return {
+ ...o1,
+ ...o2
+ }
+ }
+}
diff --git a/src/util/pureConstant.service.spec.ts b/src/util/pureConstant.service.spec.ts
index 28c60d696e3648daafefe1e70a48fd85aa0f853f..fa9ccf741bd443baf767d929d5786a08ecd190b7 100644
--- a/src/util/pureConstant.service.spec.ts
+++ b/src/util/pureConstant.service.spec.ts
@@ -5,7 +5,7 @@ import { MockStore, provideMockStore } from "@ngrx/store/testing"
import { BS_ENDPOINT } from "src/atlasComponents/regionalFeatures/bsFeatures"
import { AtlasWorkerService } from "src/atlasViewer/atlasViewer.workerService.service"
import { viewerStateFetchedAtlasesSelector, viewerStateFetchedTemplatesSelector } from "src/services/state/viewerState/selectors"
-import { PureContantService, SIIBRA_API_VERSION_HEADER_KEY } from "./pureConstant.service"
+import { PureContantService, SIIBRA_API_VERSION_HEADER_KEY, SIIBRA_API_VERSION } from "./pureConstant.service"
import { TAtlas } from "./siibraApiConstants/types"
const MOCK_BS_ENDPOINT = `http://localhost:1234`
@@ -68,13 +68,14 @@ describe('> pureConstant.service.ts', () => {
const exp = httpController.expectOne(`${MOCK_BS_ENDPOINT}/atlases`)
exp.flush([mockAtlas], {
headers: {
- [SIIBRA_API_VERSION_HEADER_KEY]: '0.1.5'
+ [SIIBRA_API_VERSION_HEADER_KEY]: SIIBRA_API_VERSION
}
})
service.allFetchingReady$.subscribe()
const expT1 = httpController.expectOne(`${MOCK_BS_ENDPOINT}/atlases/${encodeURIComponent(mockAtlas.id)}/spaces`)
expT1.flush([])
+
const expP1 = httpController.expectOne(`${MOCK_BS_ENDPOINT}/atlases/${encodeURIComponent(mockAtlas.id)}/parcellations`)
expP1.flush([])
})
diff --git a/src/util/pureConstant.service.ts b/src/util/pureConstant.service.ts
index cc8e857a378d6b1c2a000a041f8055e8be66f0f8..5d69b140153b700a2d76e35070602b6fba98dc23 100644
--- a/src/util/pureConstant.service.ts
+++ b/src/util/pureConstant.service.ts
@@ -17,7 +17,7 @@ import { MatSnackBar } from "@angular/material/snack-bar";
import { TTemplateImage } from "./interfaces";
export const SIIBRA_API_VERSION_HEADER_KEY='x-siibra-api-version'
-export const SIIBRA_API_VERSION = '0.1.5'
+export const SIIBRA_API_VERSION = '0.1.8'
const validVolumeType = new Set([
'neuroglancer/precomputed',
@@ -456,7 +456,6 @@ Raise/track issues at github repo: <a target = "_blank" href = "${this.repoUrl}"
const { EXPERIMENTAL_FEATURE_FLAG } = environment
if (EXPERIMENTAL_FEATURE_FLAG) return arr
return arr
- // return arr.filter(atlas => !/pre.?release/i.test(atlas.name))
}),
shareReplay(1),
)
@@ -510,7 +509,7 @@ Raise/track issues at github repo: <a target = "_blank" href = "${this.repoUrl}"
// }]
}
}),
- originDatainfos: (parc._dataset_specs || []).filter(spec => spec["@type"] === 'fzj/tmp/simpleOriginInfo/v0.0.1')
+ originDatainfos: [...(parc.infos || []), ...(parc._dataset_specs || []).filter(spec => spec["@type"] === 'fzj/tmp/simpleOriginInfo/v0.0.1')]
}
})
}
@@ -599,25 +598,6 @@ Raise/track issues at github repo: <a target = "_blank" href = "${this.repoUrl}"
? 'right hemisphere'
: 'whole brain'
- /**
- * TODO fix in siibra-api
- */
- if (
- tmpl.id !== 'minds/core/referencespace/v1.0.0/a1655b99-82f1-420f-a3c2-fe80fd4c8588'
- && parc.id === 'minds/core/parcellationatlas/v1.0.0/94c1125b-b87e-45e4-901c-00daee7f2579-290'
- && hemisphereKey === 'whole brain'
- ) {
- region.labelIndex = null
- return
- }
-
- if (
- tmpl.id === 'minds/core/referencespace/v1.0.0/a1655b99-82f1-420f-a3c2-fe80fd4c8588'
- && parc.id === 'minds/core/parcellationatlas/v1.0.0/94c1125b-b87e-45e4-901c-00daee7f2579-290'
- && hemisphereKey === 'whole brain'
- ) {
- region.children = []
- }
if (!region['ngId']) {
const hemispheredNgId = getNgId(atlas['@id'], tmpl.id, parc.id, hemisphereKey)
region['ngId'] = hemispheredNgId
@@ -776,9 +756,12 @@ Raise/track issues at github repo: <a target = "_blank" href = "${this.repoUrl}"
const templateImages: TTemplateImage[] = []
for (const precomputedItem of precomputedArr) {
const ngIdKey = MultiDimMap.GetKey(precomputedItem["@id"])
+ const precomputedUrl = 'https://neuroglancer.humanbrainproject.eu/precomputed/data-repo-ng-bot/20211001-mebrain/precomputed/images/MEBRAINS_T1.masked' === precomputedItem.url
+ ? 'https://neuroglancer.humanbrainproject.eu/precomputed/data-repo-ng-bot/20211018-mebrains-masked-templates/precomputed/images/MEBRAINS_T1_masked'
+ : precomputedItem.url
initialLayers[ngIdKey] = {
type: "image",
- source: `precomputed://${precomputedItem.url}`,
+ source: `precomputed://${precomputedUrl}`,
transform: precomputedItem.detail['neuroglancer/precomputed'].transform,
visible
}
@@ -844,7 +827,7 @@ Raise/track issues at github repo: <a target = "_blank" href = "${this.repoUrl}"
'@id': parc.id,
name: parc.name,
regions,
- originDatainfos: (fullParcInfo?._dataset_specs || []).filter(spec => spec["@type"] === 'fzj/tmp/simpleOriginInfo/v0.0.1')
+ originDatainfos: [...fullParcInfo.infos, ...(fullParcInfo?._dataset_specs || []).filter(spec => spec["@type"] === 'fzj/tmp/simpleOriginInfo/v0.0.1')]
}
}),
...threeSurferConfig
diff --git a/src/util/siibraApiConstants/types.ts b/src/util/siibraApiConstants/types.ts
index 40feae057c4f86091709af44a3346ce8fdb3b2bb..0fdd05528b261722876b883b7fd00d30e74fdaf7 100644
--- a/src/util/siibraApiConstants/types.ts
+++ b/src/util/siibraApiConstants/types.ts
@@ -147,9 +147,10 @@ export type TParc = {
}[]
links: {
self: THref
+ regions: THref
+ features: THref
}
- regions: THref
- features: THref
+ infos: TDatainfosDetail[]
modality: TParcModality
version: TVersion
_dataset_specs: TDatasetSpec[]
diff --git a/src/util/util.module.ts b/src/util/util.module.ts
index 649508a287e6b1e7a7985dce66232ce274c3f6ba..f128cf3fd266a1851d2951bf21c78ffffd4cedfd 100644
--- a/src/util/util.module.ts
+++ b/src/util/util.module.ts
@@ -21,6 +21,7 @@ import { FilterArrayPipe } from "./pipes/filterArray.pipe";
import { DoiParserPipe } from "./pipes/doiPipe.pipe";
import { GetFilenamePipe } from "./pipes/getFilename.pipe";
import { CombineFnPipe } from "./pipes/combineFn.pipe";
+import { MergeObjPipe } from "./mergeObj.pipe";
@NgModule({
imports:[
@@ -47,6 +48,7 @@ import { CombineFnPipe } from "./pipes/combineFn.pipe";
DoiParserPipe,
GetFilenamePipe,
CombineFnPipe,
+ MergeObjPipe,
],
exports: [
FilterRowsByVisbilityPipe,
@@ -69,6 +71,7 @@ import { CombineFnPipe } from "./pipes/combineFn.pipe";
DoiParserPipe,
GetFilenamePipe,
CombineFnPipe,
+ MergeObjPipe,
]
})
diff --git a/src/viewerModule/nehuba/nehubaViewer/nehubaViewer.component.ts b/src/viewerModule/nehuba/nehubaViewer/nehubaViewer.component.ts
index 8582aa6519ef689c6f8d09c5e721c10d65d6f798..8281c6565f4f6c717e8586877285781f631a1069 100644
--- a/src/viewerModule/nehuba/nehubaViewer/nehubaViewer.component.ts
+++ b/src/viewerModule/nehuba/nehubaViewer/nehubaViewer.component.ts
@@ -1,17 +1,18 @@
import { Component, ElementRef, EventEmitter, OnDestroy, OnInit, Output, Inject, Optional, ChangeDetectionStrategy } from "@angular/core";
import { fromEvent, Subscription, ReplaySubject, BehaviorSubject, Observable, race, timer, Subject } from 'rxjs'
-import { debounceTime, filter, map, scan, startWith, mapTo, switchMap, take, skip } from "rxjs/operators";
+import { debounceTime, filter, map, scan, startWith, mapTo, switchMap, take, skip, tap, distinctUntilChanged } from "rxjs/operators";
import { AtlasWorkerService } from "src/atlasViewer/atlasViewer.workerService.service";
import { StateInterface as ViewerConfiguration } from "src/services/state/viewerConfig.store";
import { LoggingService } from "src/logging";
import { bufferUntil, getExportNehuba, getViewer, setNehubaViewer, switchMapWaitFor } from "src/util/fn";
import { NEHUBA_INSTANCE_INJTKN, scanSliceViewRenderFn } from "../util";
-import { deserialiseParcRegionId } from 'common/util'
+import { deserialiseParcRegionId, arrayOrderedEql } from 'common/util'
import { IMeshesToLoad, SET_MESHES_TO_LOAD } from "../constants";
import { IColorMap, SET_COLORMAP_OBS, SET_LAYER_VISIBILITY } from "../layerCtrl.service";
-import '!!file-loader?context=third_party&name=main.bundle.js!export-nehuba/dist/min/main.bundle.js'
-import '!!file-loader?context=third_party&name=chunk_worker.bundle.js!export-nehuba/dist/min/chunk_worker.bundle.js'
+/**
+ * import of nehuba js files moved to angular.json
+ */
import { INgLayerCtrl, NG_LAYER_CONTROL, SET_SEGMENT_VISIBILITY, TNgLayerCtrl } from "../layerCtrl.service/layerCtrl.util";
const NG_LANDMARK_LAYER_NAME = 'spatial landmark layer'
@@ -311,6 +312,7 @@ export class NehubaViewerUnit implements OnInit, OnDestroy {
this.ondestroySubscriptions.push(
this.layerVis$.pipe(
switchMap(switchMapWaitFor({ condition: () => !!(this.nehubaViewer?.ngviewer) })),
+ distinctUntilChanged(arrayOrderedEql),
debounceTime(160),
).subscribe((layerNames: string[]) => {
/**
diff --git a/src/viewerModule/nehuba/statusCard/statusCard.component.ts b/src/viewerModule/nehuba/statusCard/statusCard.component.ts
index 739e7178255e892bcbfa54d759323eb4a436fe32..6ac175a1d93dba044ed00e3a7d5e09bc6857dd31 100644
--- a/src/viewerModule/nehuba/statusCard/statusCard.component.ts
+++ b/src/viewerModule/nehuba/statusCard/statusCard.component.ts
@@ -56,8 +56,6 @@ export class StatusCardComponent implements OnInit, OnChanges{
order: 6,
}
- public saneUrlDeprecated = `Custom URL is going away. New custom URLs can no longer be created. Custom URLs you generated in the past will continue to work.`
-
public SHARE_BTN_ARIA_LABEL = ARIA_LABELS.SHARE_BTN
public COPY_URL_TO_CLIPBOARD_ARIA_LABEL = ARIA_LABELS.SHARE_COPY_URL_CLIPBOARD
public SHARE_CUSTOM_URL_ARIA_LABEL = ARIA_LABELS.SHARE_CUSTOM_URL
diff --git a/src/viewerModule/nehuba/statusCard/statusCard.template.html b/src/viewerModule/nehuba/statusCard/statusCard.template.html
index 10542f7c52238767a0c069f9ca916751154f1f55..d3086513b1c12eb6207f9b142a66777cb59cde88 100644
--- a/src/viewerModule/nehuba/statusCard/statusCard.template.html
+++ b/src/viewerModule/nehuba/statusCard/statusCard.template.html
@@ -146,8 +146,9 @@
<mat-list-item
[attr.aria-label]="SHARE_CUSTOM_URL_ARIA_LABEL"
[attr.tab-index]="10"
- [matTooltip]="saneUrlDeprecated"
- class="text-muted">
+ (click)="openDialog(shareSaneUrl, { ariaLabel: SHARE_CUSTOM_URL_ARIA_LABEL })"
+ [matTooltip]="SHARE_CUSTOM_URL_ARIA_LABEL"
+ >
<mat-icon
class="mr-4"
fontSet="fas"
@@ -168,40 +169,8 @@
</h2>
<div mat-dialog-content>
- <div iav-auth-auth-state
- #authState="iavAuthAuthState">
-
- <!-- Logged in. Explain that links will not expire, offer to logout -->
- <ng-container *ngIf="authState.user$ | async as user; else otherTmpl">
- <span>
- Logged in as {{ user.name }}
- </span>
- <button mat-button
- color="warn"
- tabindex="-1">
- <i class="fas fa-sign-in-alt"></i>
- <span>
- Logout
- </span>
- </button>
- </ng-container>
-
- <!-- Not logged in. Offer to login -->
- <ng-template #otherTmpl>
- <span>
- Not logged in
- </span>
- <signin-modal></signin-modal>
- </ng-template>
- </div>
-
- <!-- explain links expiration -->
- <div class="text-muted mat-small">
- {{ (authState.user$ | async) ? 'Links you generate will not expire' : 'Links you generate will expire after 72 hours' }}
- </div>
-
<iav-sane-url iav-state-aggregator
- [stateTobeSaved]="stateAggregator.jsonifiedSstate$ | async"
+ [stateTobeSaved]="stateAggregator.jsonifiedState$ | async"
#stateAggregator="iavStateAggregator">
</iav-sane-url>
</div>
diff --git a/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.html b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.html
new file mode 100644
index 0000000000000000000000000000000000000000..ad8aca3171ea86ce9172a856a9ea3bd192f4995f
--- /dev/null
+++ b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.html
@@ -0,0 +1,40 @@
+<button mat-menu-item [mat-menu-trigger-for]="perspectiveOrientationMenu">
+ Change orientation to
+</button>
+
+<mat-menu #perspectiveOrientationMenu="matMenu">
+
+ <div class="d-flex align-items-center text-light">
+ <button mat-button color="basic" class="flex-grow-1 text-left"
+ (click)="set3DViewPoint('coronal', 'first')">
+ Coronal view
+ </button>
+ <button class="flex-grow-0" mat-button
+ (click)="set3DViewPoint('coronal', 'second')">
+ <i class="fas fa-arrows-alt-h"></i>
+ </button>
+ </div>
+
+ <div class="d-flex align-items-center text-light"> <!--mat-menu-item-->
+ <button mat-button color="basic" class="flex-grow-1 text-left"
+ (click)="set3DViewPoint('sagittal', 'first')">
+ Sagittal view
+ </button>
+ <button class="flex-grow-0" mat-button
+ (click)="set3DViewPoint('sagittal', 'second')">
+ <i class="fas fa-arrows-alt-h"></i>
+ </button>
+ </div>
+
+ <div class="d-flex align-items-center text-light"> <!--mat-menu-item-->
+ <button mat-button color="basic" class="flex-grow-1 text-left"
+ (click)="set3DViewPoint('axial', 'first')">
+ Axial view
+ </button>
+ <button class="flex-grow-0" mat-button
+ (click)="set3DViewPoint('axial', 'second')">
+ <i class="fas fa-arrows-alt-h"></i>
+ </button>
+ </div>
+
+</mat-menu>
diff --git a/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.sass b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.sass
new file mode 100644
index 0000000000000000000000000000000000000000..e69de29bb2d1d6434b8b29ae775ad8c2e48c5391
diff --git a/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.spec.ts b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.spec.ts
new file mode 100644
index 0000000000000000000000000000000000000000..48c4d290030e11728232a182bf7aa200131b8876
--- /dev/null
+++ b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.spec.ts
@@ -0,0 +1,34 @@
+import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { ChangePerspectiveOrientationComponent } from './changePerspectiveOrientation.component';
+import { MockStore, provideMockStore } from "@ngrx/store/testing"
+import { AngularMaterialModule } from 'src/sharedModules';
+import { CommonModule } from '@angular/common';
+
+describe('ChangePerspectiveOrientationComponent', () => {
+ let component: ChangePerspectiveOrientationComponent;
+ let fixture: ComponentFixture<ChangePerspectiveOrientationComponent>;
+ let mockStore: MockStore;
+
+ beforeEach(async () => {
+ await TestBed.configureTestingModule({
+ imports: [
+ AngularMaterialModule,
+ CommonModule
+ ],
+ declarations: [ ChangePerspectiveOrientationComponent ],
+ providers: [provideMockStore()],
+
+ })
+ .compileComponents();
+ });
+
+ beforeEach(() => {
+ fixture = TestBed.createComponent(ChangePerspectiveOrientationComponent);
+ component = fixture.componentInstance;
+ fixture.detectChanges();
+ });
+
+ it('should create', () => {
+ expect(component).toBeTruthy();
+ });
+});
diff --git a/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.ts b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.ts
new file mode 100644
index 0000000000000000000000000000000000000000..0c4bdb1300fb0e7bc094a81d2dcac2d39f67c0d1
--- /dev/null
+++ b/src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component.ts
@@ -0,0 +1,33 @@
+import { Component } from '@angular/core';
+import {viewerStateChangeNavigation} from "src/services/state/viewerState/actions";
+import {Store} from "@ngrx/store";
+
+@Component({
+ selector: 'app-change-perspective-orientation',
+ templateUrl: './changePerspectiveOrientation.component.html',
+ styleUrls: ['./changePerspectiveOrientation.component.sass']
+})
+export class ChangePerspectiveOrientationComponent {
+
+ private viewOrientations = {
+ coronal: [[0,-1,1,0], [-1,0,0,1]],
+ sagittal: [[-1,-1,1,1], [-1,1,-1,1]],
+ axial: [[0,0,1,0], [1,0,0,0]]
+ }
+
+ constructor(private store$: Store<any>,) { }
+
+ public set3DViewPoint(plane: 'coronal' | 'sagittal' | 'axial', view: 'first' | 'second') {
+
+ const orientation = this.viewOrientations[plane][view === 'first'? 0 : 1]
+
+ this.store$.dispatch(
+ viewerStateChangeNavigation({
+ navigation: {
+ perspectiveOrientation: orientation,
+ }
+ })
+ )
+ }
+
+}
diff --git a/src/viewerModule/nehuba/viewerCtrl/module.ts b/src/viewerModule/nehuba/viewerCtrl/module.ts
index dbf9ed121ca150593b2c3ac08bd0f5e91bcb750e..7be1186474bddac9d4c33772d28fb682bfa61794 100644
--- a/src/viewerModule/nehuba/viewerCtrl/module.ts
+++ b/src/viewerModule/nehuba/viewerCtrl/module.ts
@@ -5,6 +5,7 @@ import { ComponentsModule } from "src/components";
import { AngularMaterialModule } from "src/sharedModules";
import { UtilModule } from "src/util";
import { ViewerCtrlCmp } from "./viewerCtrlCmp/viewerCtrlCmp.component";
+import {ChangePerspectiveOrientationComponent} from "src/viewerModule/nehuba/viewerCtrl/change-perspective-orientation/changePerspectiveOrientation.component";
@NgModule({
imports: [
@@ -17,6 +18,7 @@ import { ViewerCtrlCmp } from "./viewerCtrlCmp/viewerCtrlCmp.component";
],
declarations: [
ViewerCtrlCmp,
+ ChangePerspectiveOrientationComponent
],
exports: [
ViewerCtrlCmp
diff --git a/src/viewerModule/nehuba/viewerCtrl/viewerCtrlCmp/viewerCtrlCmp.template.html b/src/viewerModule/nehuba/viewerCtrl/viewerCtrlCmp/viewerCtrlCmp.template.html
index 783d582fd907bba374635025e2f6f8afe8cf1550..76aa348a8f659039b6c0ece363db4a79921be756 100644
--- a/src/viewerModule/nehuba/viewerCtrl/viewerCtrlCmp/viewerCtrlCmp.template.html
+++ b/src/viewerModule/nehuba/viewerCtrl/viewerCtrlCmp/viewerCtrlCmp.template.html
@@ -41,3 +41,4 @@
</form>
</ng-container>
+<app-change-perspective-orientation></app-change-perspective-orientation>
diff --git a/src/viewerModule/viewerCmp/viewerCmp.component.ts b/src/viewerModule/viewerCmp/viewerCmp.component.ts
index e494f52444163704feddb62690bf2692bfe48c09..f930dff417eaff53fb90ff95b6394696bb04e774 100644
--- a/src/viewerModule/viewerCmp/viewerCmp.component.ts
+++ b/src/viewerModule/viewerCmp/viewerCmp.component.ts
@@ -1,10 +1,11 @@
import { ChangeDetectionStrategy, Component, ComponentFactory, ComponentFactoryResolver, Inject, Injector, Input, OnDestroy, Optional, TemplateRef, ViewChild, ViewContainerRef } from "@angular/core";
import { select, Store } from "@ngrx/store";
import { combineLatest, merge, NEVER, Observable, of, Subscription } from "rxjs";
-import {catchError, distinctUntilChanged, map, shareReplay, startWith, switchMap } from "rxjs/operators";
+import {catchError, debounceTime, distinctUntilChanged, map, shareReplay, startWith, switchMap } from "rxjs/operators";
import { viewerStateSetSelectedRegions } from "src/services/state/viewerState/actions";
import {
viewerStateContextedSelectedRegionsSelector,
+ viewerStateGetSelectedAtlas,
viewerStateSelectedParcellationSelector,
viewerStateSelectedTemplateSelector,
viewerStateStandAloneVolumes,
@@ -24,6 +25,7 @@ import { GenericInfoCmp } from "src/atlasComponents/regionalFeatures/bsFeatures/
import { _PLI_VOLUME_INJ_TOKEN, _TPLIVal } from "src/glue";
import { uiActionSetPreviewingDatasetFiles } from "src/services/state/uiState.store.helper";
import { viewerStateSetViewerMode } from "src/services/state/viewerState.store.helper";
+import { DialogService } from "src/services/dialogService.service";
type TCStoreViewerCmp = {
overlaySideNav: any
@@ -111,7 +113,8 @@ export function ROIFactory(store: Store<any>, svc: PureContantService){
},
deps: [ ComponentStore ]
},
- ComponentStore
+ ComponentStore,
+ DialogService
],
changeDetection: ChangeDetectionStrategy.OnPush
})
@@ -227,6 +230,7 @@ export class ViewerCmp implements OnDestroy {
private viewerModuleSvc: ContextMenuService<TContextArg<'threeSurfer' | 'nehuba'>>,
private cStore: ComponentStore<TCStoreViewerCmp>,
cfr: ComponentFactoryResolver,
+ private dialogSvc: DialogService,
@Optional() @Inject(_PLI_VOLUME_INJ_TOKEN) private _pliVol$: Observable<_TPLIVal[]>,
@Optional() @Inject(REGION_OF_INTEREST) public regionOfInterest$: Observable<any>
){
@@ -249,7 +253,45 @@ export class ViewerCmp implements OnDestroy {
? getGetRegionFromLabelIndexId({ parcellation: p })
: null
}
- )
+ ),
+ combineLatest([
+ this.templateSelected$,
+ this.parcellationSelected$,
+ this.store$.pipe(
+ select(viewerStateGetSelectedAtlas)
+ )
+ ]).pipe(
+ debounceTime(160)
+ ).subscribe(async ([tmpl, parc, atlas]) => {
+ const regex = /pre.?release/i
+ const checkPrerelease = (obj: any) => {
+ if (obj?.name) return regex.test(obj.name)
+ return false
+ }
+ const message: string[] = []
+ if (checkPrerelease(atlas)) {
+ message.push(`- _${atlas.name}_`)
+ }
+ if (checkPrerelease(tmpl)) {
+ message.push(`- _${tmpl.name}_`)
+ }
+ if (checkPrerelease(parc)) {
+ message.push(`- _${parc.name}_`)
+ }
+ if (message.length > 0) {
+ message.unshift(`The following have been tagged pre-release, and may be updated frequently:`)
+ try {
+ await this.dialogSvc.getUserConfirm({
+ title: `Pre-release warning`,
+ markdown: message.join('\n\n'),
+ confirmOnly: true
+ })
+ // eslint-disable-next-line no-empty
+ } catch (e) {
+
+ }
+ }
+ })
)
}
diff --git a/src/viewerModule/viewerStateBreadCrumb/breadcrumb/breadcrumb.component.ts b/src/viewerModule/viewerStateBreadCrumb/breadcrumb/breadcrumb.component.ts
index 8bf705e68d933cdc60faa91903cb9506025c2895..5a4bea443845687db10084cd253a18ec1c44bb17 100644
--- a/src/viewerModule/viewerStateBreadCrumb/breadcrumb/breadcrumb.component.ts
+++ b/src/viewerModule/viewerStateBreadCrumb/breadcrumb/breadcrumb.component.ts
@@ -7,7 +7,7 @@ import { distinctUntilChanged, map } from "rxjs/operators";
import { viewerStateHelperSelectParcellationWithId, viewerStateRemoveAdditionalLayer, viewerStateSetSelectedRegions } from "src/services/state/viewerState.store.helper";
import { ngViewerActionClearView, ngViewerSelectorClearViewEntries } from "src/services/state/ngViewerState.store.helper";
import { OVERWRITE_SHOW_DATASET_DIALOG_TOKEN } from "src/util/interfaces";
-import { TDatainfosDetail } from "src/util/siibraApiConstants/types";
+import { TDatainfosDetail, TSimpleInfo } from "src/util/siibraApiConstants/types";
@Component({
selector: 'viewer-state-breadcrumb',
@@ -120,8 +120,12 @@ export class ViewerStateBreadCrumb {
})
export class OriginalDatainfoPipe implements PipeTransform{
- public transform(arr: TDatainfosDetail[]): TDatainfosDetail[]{
- if (arr.length > 0) {
+ public transform(arr: (TSimpleInfo | TDatainfosDetail)[]): TDatainfosDetail[]{
+ const detailedInfos = arr.filter(item => item['@type'] === 'minds/core/dataset/v1.0.0') as TDatainfosDetail[]
+ const simpleInfos = arr.filter(item => item['@type'] === 'fzj/tmp/simpleOriginInfo/v0.0.1') as TSimpleInfo[]
+
+ if (detailedInfos.length > 0) return detailedInfos
+ if (simpleInfos.length > 0) {
return arr.map(d => {
return {
'@type': 'minds/core/dataset/v1.0.0',