Fix: consolidate and extend unicode NFC normalization for filenames and matching

Consolidates the scattered unicodedata.normalize("NFC", ...) calls introduced
by my earlier filename/path normalization work into a single
documents.utils.normalize_unicode() helper, and closes several gaps where
NFD-normalized filenames could still slip through unnormalized:

- Document.get_public_filename() now normalizes, fixing exported filenames
  built from an NFD title/correspondent name (the default, non-format export
  path was not covered by the earlier fix).
- DocumentViewSet.update_version() now normalizes the uploaded filename,
  matching PostDocumentView's existing behavior.
- ConsumerPlugin normalizes self.filename once at consumption time, covering
  the title fallback and Document.original_filename for every document
  source (consume folder, mail, API, barcode splits).
- Workflow trigger and mail rule filename/path matching (documents/matching.py,
  paperless_mail/mail.py) now normalize the document/attachment side before
  comparing, so an NFD filename matches an NFC-typed filter pattern instead of
  silently failing to match.
- WorkflowTriggerSerializer and MailRuleSerializer normalize filter_filename/
  filter_path and the attachment include/exclude patterns once at write time,
  so the read side isn't re-normalizing an already-canonical value on every
  match.

.github/dependabot.yml

@@ -81,6 +81,7 @@ updates:
       # Data, NLP, and Search
       data-nlp-search:
         patterns:
+          - "nltk"
           - "scikit-learn"
           - "langdetect"
           - "rapidfuzz"

.github/workflows/ci-backend.yml

@@ -12,9 +12,7 @@ concurrency:
   cancel-in-progress: true
 env:
   DEFAULT_UV_VERSION: "0.12.x"
-  # Match the Docker image: nltk refuses to read hardlinked data files, such as
+  NLTK_DATA: "/usr/share/nltk_data"
-  # the copy bundled with llama-index when uv links packages from its cache
-  UV_LINK_MODE: copy
 permissions: {}
 jobs:
   changes:
@@ -102,7 +100,7 @@ jobs:
         with:
           python-version: "${{ matrix.python-version }}"
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: ${{ env.DEFAULT_UV_VERSION }}
           enable-cache: true
@@ -127,8 +125,12 @@ jobs:
       - name: List installed Python dependencies
         run: |
           uv pip list
+      - name: Install NLTK data
+        run: |
+          uv run python -m nltk.downloader punkt punkt_tab snowball_data stopwords -d "${NLTK_DATA}"
       - name: Run tests
         env:
+          NLTK_DATA: ${{ env.NLTK_DATA }}
           PAPERLESS_CI_TEST: 1
           PYTHON_VERSION: ${{ steps.setup-python.outputs.python-version }}
         run: |
@@ -176,7 +178,7 @@ jobs:
         with:
           python-version: "${{ env.DEFAULT_PYTHON }}"
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: ${{ env.DEFAULT_UV_VERSION }}
           enable-cache: true

.github/workflows/ci-docs.yml

@@ -78,7 +78,7 @@ jobs:
         with:
           python-version: ${{ env.DEFAULT_PYTHON_VERSION }}
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: ${{ env.DEFAULT_UV_VERSION }}
           enable-cache: true
@@ -111,7 +111,7 @@ jobs:
       url: ${{ steps.deployment.outputs.page_url }}
     steps:
       - name: Deploy GitHub Pages
-        uses: actions/deploy-pages@368f82528645a54fb793d4d04e342629a3f51346 # v5.0.1
+        uses: actions/deploy-pages@cd2ce8fcbc39b97be8ca5fce6e763baed58fa128 # v5.0.0
         id: deployment
         with:
           artifact_name: github-pages-${{ github.run_id }}-${{ github.run_attempt }}

.github/workflows/ci-frontend.yml

@@ -81,7 +81,7 @@ jobs:
         with:
           persist-credentials: false
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24
@@ -113,7 +113,7 @@ jobs:
         with:
           persist-credentials: false
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24
@@ -152,7 +152,7 @@ jobs:
         with:
           persist-credentials: false
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24
@@ -201,7 +201,7 @@ jobs:
         with:
           persist-credentials: false
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24
@@ -216,7 +216,7 @@ jobs:
         with:
           python-version: '3.12'
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: '0.12.x'
           enable-cache: false
@@ -255,7 +255,7 @@ jobs:
           fetch-depth: 2
           persist-credentials: false
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24

.github/workflows/ci-release.yml

@@ -40,7 +40,7 @@ jobs:
           persist-credentials: false
       # ---- Frontend Build ----
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24
@@ -59,7 +59,7 @@ jobs:
         with:
           python-version: ${{ env.DEFAULT_PYTHON_VERSION }}
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: ${{ env.DEFAULT_UV_VERSION }}
           enable-cache: false
@@ -212,7 +212,7 @@ jobs:
         with:
           python-version: ${{ env.DEFAULT_PYTHON_VERSION }}
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: ${{ env.DEFAULT_UV_VERSION }}
           enable-cache: false

.github/workflows/ci-static-analysis.yml

@@ -26,7 +26,7 @@ jobs:
         with:
           persist-credentials: false
       - name: Run zizmor
-        uses: zizmorcore/zizmor-action@cc914d7f3750a2d13d75c7f184a1060aa0e9d482 # v0.6.4
+        uses: zizmorcore/zizmor-action@3dc1ecc9bcb9e94e9b2c709687979e1298497054 # v0.6.2
   semgrep:
     name: Semgrep CE
     runs-on: ubuntu-24.04
@@ -44,7 +44,7 @@ jobs:
       - name: Run Semgrep
         run: semgrep scan --config auto --sarif-output results.sarif
       - name: Upload results to GitHub code scanning
-        uses: github/codeql-action/upload-sarif@b96794f015dfd88f77b49b1c93e0fa7110f94c63 # v4.38.0
+        uses: github/codeql-action/upload-sarif@cdf488f595d80d6e07e03d4674febd5ab45fa938 # v4.37.9
         if: always()
         with:
           sarif_file: results.sarif

.github/workflows/codeql-analysis.yml

@@ -39,7 +39,7 @@ jobs:
           persist-credentials: false
       # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
-        uses: github/codeql-action/init@b96794f015dfd88f77b49b1c93e0fa7110f94c63 # v4.38.0
+        uses: github/codeql-action/init@cdf488f595d80d6e07e03d4674febd5ab45fa938 # v4.37.9
         with:
           languages: ${{ matrix.language }}
           # If you wish to specify custom queries, you can do so here or in a config file.
@@ -47,4 +47,4 @@ jobs:
           # Prefix the list here with "+" to use these queries and those in the config file.
           # queries: ./path/to/local/query, your-org/your-repo/queries@main
       - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@b96794f015dfd88f77b49b1c93e0fa7110f94c63 # v4.38.0
+        uses: github/codeql-action/analyze@cdf488f595d80d6e07e03d4674febd5ab45fa938 # v4.37.9

.github/workflows/crowdin.yml

@@ -22,7 +22,7 @@ jobs:
           token: ${{ secrets.PNGX_BOT_PAT }}
           persist-credentials: false
       - name: crowdin action
-        uses: crowdin/github-action@0d5670f539973aea2f01abce61a8989934df0025 # v3.0.2
+        uses: crowdin/github-action@e4a6c1338b4063c77d46a81875265f9e8bd76f95 # v3.0.0
         with:
           upload_translations: false
           download_translations: true

.github/workflows/translate-strings.yml

@@ -29,7 +29,7 @@ jobs:
           sudo apt-get update -qq
           sudo apt-get install -qq --no-install-recommends gettext
       - name: Install uv
-        uses: astral-sh/setup-uv@bec219d24cd3e171d82865faccec33120bb574f4 # v10.1.0
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1
         with:
           version: ${{ env.DEFAULT_UV_VERSION }}
           enable-cache: true
@@ -43,7 +43,7 @@ jobs:
           PAPERLESS_SECRET_KEY: "ci-translate-not-a-real-secret"
         run: cd src/ && uv run manage.py makemessages -l en_US -i "samples*"
       - name: Install pnpm
-        uses: pnpm/action-setup@ea17c68df8912ef543352723c149a84f56e3d413 # v6.1.0
+        uses: pnpm/action-setup@0977fd99725f1db4007ccb2928dbb4e90d06cc86 # v6.0.10
         with:
           package_json_file: src-ui/package.json
       - name: Use Node.js 24

Dockerfile

@@ -30,7 +30,7 @@ RUN set -eux \
 # Purpose: Installs s6-overlay and rootfs
 # Comments:
 #  - Don't leave anything extra in here either
-FROM ghcr.io/astral-sh/uv:0.12.16-python3.14-trixie-slim AS s6-overlay-base
+FROM ghcr.io/astral-sh/uv:0.12.9-python3.14-trixie-slim AS s6-overlay-base
 
 WORKDIR /usr/src/s6
 
@@ -199,6 +199,10 @@ RUN set -eux \
       --index https://download.pytorch.org/whl/cpu \
       --index-strategy unsafe-best-match \
       --requirements requirements.txt \
+  && echo "Installing NLTK data" \
+    && python3 -W ignore::RuntimeWarning -m nltk.downloader -d "/usr/share/nltk_data" snowball_data \
+    && python3 -W ignore::RuntimeWarning -m nltk.downloader -d "/usr/share/nltk_data" stopwords \
+    && python3 -W ignore::RuntimeWarning -m nltk.downloader -d "/usr/share/nltk_data" punkt_tab \
   && echo "Cleaning up image" \
     && apt-get --yes purge ${BUILD_PACKAGES} \
     && apt-get --yes autoremove --purge \

docker/compose/docker-compose.ci-test.yml

@@ -4,7 +4,7 @@
 # correct networking for the tests
 services:
   gotenberg:
-    image: docker.io/gotenberg/gotenberg:8.37
+    image: docker.io/gotenberg/gotenberg:8.36
     hostname: gotenberg
     container_name: gotenberg
     network_mode: host
@@ -35,7 +35,7 @@ services:
       - "3143:3143" # IMAP
     restart: unless-stopped
   nginx:
-    image: docker.io/nginx:1.31.6-alpine
+    image: docker.io/nginx:1.31.5-alpine
     hostname: nginx
     container_name: nginx
     ports:

docker/compose/docker-compose.mariadb-tika.yml

@@ -72,7 +72,7 @@ services:
       PAPERLESS_TIKA_GOTENBERG_ENDPOINT: http://gotenberg:3000
       PAPERLESS_TIKA_ENDPOINT: http://tika:9998
   gotenberg:
-    image: docker.io/gotenberg/gotenberg:8.37
+    image: docker.io/gotenberg/gotenberg:8.36
     restart: unless-stopped
     # The gotenberg chromium route is used to convert .eml files. We do not
     # want to allow external content like tracking pixels or even javascript.

docker/compose/docker-compose.postgres-tika.yml

@@ -67,7 +67,7 @@ services:
       PAPERLESS_TIKA_GOTENBERG_ENDPOINT: http://gotenberg:3000
       PAPERLESS_TIKA_ENDPOINT: http://tika:9998
   gotenberg:
-    image: docker.io/gotenberg/gotenberg:8.37
+    image: docker.io/gotenberg/gotenberg:8.36
     restart: unless-stopped
     # The gotenberg chromium route is used to convert .eml files. We do not
     # want to allow external content like tracking pixels or even javascript.

docker/compose/docker-compose.sqlite-tika.yml

@@ -56,7 +56,7 @@ services:
       PAPERLESS_TIKA_GOTENBERG_ENDPOINT: http://gotenberg:3000
       PAPERLESS_TIKA_ENDPOINT: http://tika:9998
   gotenberg:
-    image: docker.io/gotenberg/gotenberg:8.37
+    image: docker.io/gotenberg/gotenberg:8.36
     restart: unless-stopped
     # The gotenberg chromium route is used to convert .eml files. We do not
     # want to allow external content like tracking pixels or even javascript.

docker/rootfs/etc/s6-overlay/s6-rc.d/init-compile-bytecode/run

@@ -1,58 +0,0 @@
-#!/command/with-contenv /usr/bin/bash
-# shellcheck shell=bash
-declare -r log_prefix="[init-compile-bytecode]"
-
-# PYTHONDONTWRITEBYTECODE=1 is set for the whole container. This unit compiles a
-# scoped set of libraries anyway, to speed up startup without bloating image size.
-
-# Handle the people using a read only file system
-if [[ "${S6_READ_ONLY_ROOT}" == "1" ]]; then
-	echo "${log_prefix} S6_READ_ONLY_ROOT=1, skipping (nothing to write bytecode to)"
-	exit 0
-fi
-
-# When running as a non-root user, site-packages is still root-owned and unwritable,
-# so this step would just fail loudly on every container start. Skip it.
-if [[ -n "${USER_IS_NON_ROOT}" ]]; then
-	echo "${log_prefix} USER_IS_NON_ROOT is set, skipping (site-packages is not writable)"
-	exit 0
-fi
-
-declare -r site_packages="$(python3 -c 'import site; print(site.getsitepackages()[0])')"
-
-# Deliberately scoped to packages that paperless.settings/paperless/__init__.py import
-# unconditionally on every manage.py invocation (Django itself, the always-loaded
-# INSTALLED_APPS, and celery). This is NOT "compile everything" - the optional AI stack
-# (torch, llama-index, sentence-transformers, ...) is intentionally excluded since it is
-# lazy-imported and large.
-declare -a scope=(
-	"${PAPERLESS_SRC_DIR}"
-	"${site_packages}/django"
-	"${site_packages}/celery"
-	"${site_packages}/kombu"
-	"${site_packages}/rest_framework"
-	"${site_packages}/django_filters"
-	"${site_packages}/whitenoise"
-	"${site_packages}/corsheaders"
-	"${site_packages}/django_extensions"
-	"${site_packages}/guardian"
-	"${site_packages}/allauth"
-	"${site_packages}/drf_spectacular"
-	"${site_packages}/drf_spectacular_sidecar"
-	"${site_packages}/treenode"
-	"${site_packages}/compression_middleware"
-)
-
-declare -a existing_scope=()
-for path in "${scope[@]}"; do
-	[[ -d "${path}" ]] && existing_scope+=("${path}")
-done
-
-echo "${log_prefix} Compiling bytecode for: ${existing_scope[*]}"
-declare -r start_seconds=${SECONDS}
-
-if ! PYTHONDONTWRITEBYTECODE= python3 -m compileall -q "${existing_scope[@]}"; then
-	echo "${log_prefix} WARNING: compileall reported errors (read-only filesystem or unwritable site-packages?); continuing without a bytecode cache"
-fi
-
-echo "${log_prefix} Done in $((SECONDS - start_seconds))s"

docker/rootfs/etc/s6-overlay/s6-rc.d/init-compile-bytecode/type

@@ -1 +0,0 @@
-oneshot

docker/rootfs/etc/s6-overlay/s6-rc.d/init-compile-bytecode/up

@@ -1 +0,0 @@
-/etc/s6-overlay/s6-rc.d/init-compile-bytecode/run

docs/administration.md

@@ -521,8 +521,7 @@ Pass `--recreate` to wipe the existing index before rebuilding. Use this when th
 index is corrupted or you want a fully clean rebuild.
 
 Pass `--if-needed` to skip the rebuild if the index is already up to date (schema
-version, schema fingerprint and search language all match). Safe to run on every
+version and search language match). Safe to run on every startup or upgrade.
-startup or upgrade.
 
 Specify `optimize` to optimize the index. This command is regularly invoked by the
 task scheduler.

docs/changelog.md

@@ -1,185 +1,5 @@
 # Changelog
 
-## paperless-ngx 3.2.0
-
-### Features / Enhancements
-
-- Enhancement (QoL): support deselecting single items from "select all" [@shamoon](https://github.com/shamoon) ([#14117](https://github.com/paperless-ngx/paperless-ngx/pull/14117))
-- Enhancement: Match fuzzy terms in place inside the parsed query [@stumpylog](https://github.com/stumpylog) ([#14157](https://github.com/paperless-ngx/paperless-ngx/pull/14157))
-- Enhancement: Match CJK terms through their bigram fields in place [@stumpylog](https://github.com/stumpylog) ([#14156](https://github.com/paperless-ngx/paperless-ngx/pull/14156))
-- Enhancement: centralized management of share links + bundles [@shamoon](https://github.com/shamoon) ([#14115](https://github.com/paperless-ngx/paperless-ngx/pull/14115))
-- Enhancement: parse advanced search with whoosh-compat and delete the handwritten translation [@stumpylog](https://github.com/stumpylog) ([#14072](https://github.com/paperless-ngx/paperless-ngx/pull/14072))
-- Enhancement: allow regex timeout configuration [@shamoon](https://github.com/shamoon) ([#14085](https://github.com/paperless-ngx/paperless-ngx/pull/14085))
-- Enhancement: hide-able sidebar items [@shamoon](https://github.com/shamoon) ([#14052](https://github.com/paperless-ngx/paperless-ngx/pull/14052))
-- Enhancement: Improve matching for correspondents, storage path and labels by removing bias + adding minimum match threshold [@dewey](https://github.com/dewey) ([#12164](https://github.com/paperless-ngx/paperless-ngx/pull/12164))
-- Enhancement: add Tantivy full-text fallback adapter for taxonomy candidates [@stumpylog](https://github.com/stumpylog) ([#13820](https://github.com/paperless-ngx/paperless-ngx/pull/13820))
-- Enhancement (QoL): surface externally-set options in Config UI [@shamoon](https://github.com/shamoon) ([#13989](https://github.com/paperless-ngx/paperless-ngx/pull/13989))
-- Enhancement: allow disabling auto-suggestions for inbox documents [@shamoon](https://github.com/shamoon) ([#13946](https://github.com/paperless-ngx/paperless-ngx/pull/13946))
-- Change: skip documents with empty content in apply AI suggestions WF [@shamoon](https://github.com/shamoon) ([#13985](https://github.com/paperless-ngx/paperless-ngx/pull/13985))
-- Enhancement: duplicates filter [@shamoon](https://github.com/shamoon) ([#13994](https://github.com/paperless-ngx/paperless-ngx/pull/13994))
-- Tweak: note that apply AI suggestions runs async in WF editor [@shamoon](https://github.com/shamoon) ([#14004](https://github.com/paperless-ngx/paperless-ngx/pull/14004))
-- Enhancement (QoL): attempt to localize firstDayOfWeek for date picker [@shamoon](https://github.com/shamoon) ([#13999](https://github.com/paperless-ngx/paperless-ngx/pull/13999))
-
-### Bug Fixes
-
-- Fix: don't redirect to signup on first install when regular login is disabled [@cyberb](https://github.com/cyberb) ([#14165](https://github.com/paperless-ngx/paperless-ngx/pull/14165))
-- Fix: better catch email workflow placeholder parsing errors [@shamoon](https://github.com/shamoon) ([#14129](https://github.com/paperless-ngx/paperless-ngx/pull/14129))
-- Fix: validate legacy bulk edit owner, rotate and split parameters [@stumpylog](https://github.com/stumpylog) ([#14120](https://github.com/paperless-ngx/paperless-ngx/pull/14120))
-- Fix: validate set\_permissions with a nested serializer [@stumpylog](https://github.com/stumpylog) ([#14119](https://github.com/paperless-ngx/paperless-ngx/pull/14119))
-- Fix: Use prefetching to reduce query counts during classifier training [@stumpylog](https://github.com/stumpylog) ([#14122](https://github.com/paperless-ngx/paperless-ngx/pull/14122))
-- Fix: reject non-dict user\_args/barcode\_tag\_mapping in config API [@stumpylog](https://github.com/stumpylog) ([#14118](https://github.com/paperless-ngx/paperless-ngx/pull/14118))
-- Fix: type edit\_pdf operations via a nested serializer [@stumpylog](https://github.com/stumpylog) ([#14116](https://github.com/paperless-ngx/paperless-ngx/pull/14116))
-- Fix: ensure django setup is run for management comments under 3.14 [@shamoon](https://github.com/shamoon) ([#14100](https://github.com/paperless-ngx/paperless-ngx/pull/14100))
-- Fix: validate PDF output doc indexes in bulk edit [@shamoon](https://github.com/shamoon) ([#14083](https://github.com/paperless-ngx/paperless-ngx/pull/14083))
-- Fix: avoid IntegrityError when a retried task republishes with the same ID [@stumpylog](https://github.com/stumpylog) ([#14096](https://github.com/paperless-ngx/paperless-ngx/pull/14096))
-- Fix: update some api global perms inconsistencies [@shamoon](https://github.com/shamoon) ([#14086](https://github.com/paperless-ngx/paperless-ngx/pull/14086))
-- Fix: ignore nested action IDs on WF create [@shamoon](https://github.com/shamoon) ([#14084](https://github.com/paperless-ngx/paperless-ngx/pull/14084))
-- Fix: correct text/stream compression workaround [@shamoon](https://github.com/shamoon) ([#14064](https://github.com/paperless-ngx/paperless-ngx/pull/14064))
-- Fix: ui version content switching inconsistencies [@shamoon](https://github.com/shamoon) ([#14066](https://github.com/paperless-ngx/paperless-ngx/pull/14066))
-- Fix: prevent saving changes to stale cached document object [@shamoon](https://github.com/shamoon) ([#14065](https://github.com/paperless-ngx/paperless-ngx/pull/14065))
-- Fix: ensure remove inbox tag children on remove\_inbox\_tags [@shamoon](https://github.com/shamoon) ([#14050](https://github.com/paperless-ngx/paperless-ngx/pull/14050))
-- Fix: connect add\_to\_index handler after document\_added [@shamoon](https://github.com/shamoon) ([#14058](https://github.com/paperless-ngx/paperless-ngx/pull/14058))
-- Fix: Drop empty files from tracking after the stability window has passed [@stumpylog](https://github.com/stumpylog) ([#14047](https://github.com/paperless-ngx/paperless-ngx/pull/14047))
-- Fixhancement: better LLM errors [@shamoon](https://github.com/shamoon) ([#14031](https://github.com/paperless-ngx/paperless-ngx/pull/14031))
-- Fix: prevent orphaned versions from bulk delete [@shamoon](https://github.com/shamoon) ([#14030](https://github.com/paperless-ngx/paperless-ngx/pull/14030))
-- Fixhancement: prevent overlapping mail-account processing runs [@stumpylog](https://github.com/stumpylog) ([#14046](https://github.com/paperless-ngx/paperless-ngx/pull/14046))
-- Fix: Use PAPERLESS\_REDIS\_PREFIX for Celery result backend keys [@bdd](https://github.com/bdd) ([#14015](https://github.com/paperless-ngx/paperless-ngx/pull/14015))
-- Fix: correct setting ai\_enabled to false via UI [@shamoon](https://github.com/shamoon) ([#13987](https://github.com/paperless-ngx/paperless-ngx/pull/13987))
-- Fix: catch some frontend failed object retrievals [@shamoon](https://github.com/shamoon) ([#14023](https://github.com/paperless-ngx/paperless-ngx/pull/14023))
-- Fix: more v3 icons cleanup [@shamoon](https://github.com/shamoon) ([#14017](https://github.com/paperless-ngx/paperless-ngx/pull/14017))
-- Fix: correct add version actor parity [@shamoon](https://github.com/shamoon) ([#14016](https://github.com/paperless-ngx/paperless-ngx/pull/14016))
-- Fix: fix v3 favicon file [@shamoon](https://github.com/shamoon) ([#14014](https://github.com/paperless-ngx/paperless-ngx/pull/14014))
-- Fix: change share link bundle dialog button to close after create, don't toast on copied [@shamoon](https://github.com/shamoon) ([#14002](https://github.com/paperless-ngx/paperless-ngx/pull/14002))
-- Fix: truncate mail subjects to field max length [@shamoon](https://github.com/shamoon) ([#13991](https://github.com/paperless-ngx/paperless-ngx/pull/13991))
-- Fix: enforce the overflow hidden rule on pdf editor thumbnails [@shamoon](https://github.com/shamoon) ([#13976](https://github.com/paperless-ngx/paperless-ngx/pull/13976))
-- Fix: also correct unbroken long names on small cards [@shamoon](https://github.com/shamoon) ([#13974](https://github.com/paperless-ngx/paperless-ngx/pull/13974))
-- Fix: ensure parent + child tags change together in bulk editor [@shamoon](https://github.com/shamoon) ([#13972](https://github.com/paperless-ngx/paperless-ngx/pull/13972))
-
-### Dependencies
-
-<details>
-<summary>29 changes</summary>
-
-- Chore(deps): Bump the utilities-patch group across 1 directory with 15 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14167](https://github.com/paperless-ngx/paperless-ngx/pull/14167))
-- docker(deps): Bump astral-sh/uv from 0.12.9-python3.14-trixie-slim to 0.12.16-python3.14-trixie-slim @[dependabot[bot]](https://github.com/apps/dependabot) ([#14134](https://github.com/paperless-ngx/paperless-ngx/pull/14134))
-- Chore(deps): Bump the utilities-minor group across 1 directory with 10 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14149](https://github.com/paperless-ngx/paperless-ngx/pull/14149))
-- Chore(deps): Bump the actions group across 1 directory with 8 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14162](https://github.com/paperless-ngx/paperless-ngx/pull/14162))
-- Chore(deps): Bump the frontend-angular-dependencies group across 1 directory with 19 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14141](https://github.com/paperless-ngx/paperless-ngx/pull/14141))
-- docker-compose(deps): bump gotenberg/gotenberg from 8.36 to 8.37 in /docker/compose @[dependabot[bot]](https://github.com/apps/dependabot) ([#14137](https://github.com/paperless-ngx/paperless-ngx/pull/14137))
-- docker-compose(deps): Bump nginx from 1.31.5-alpine to 1.31.6-alpine in /docker/compose @[dependabot[bot]](https://github.com/apps/dependabot) ([#14138](https://github.com/paperless-ngx/paperless-ngx/pull/14138))
-- Chore(deps): Bump pdfjs-dist from 6.2.108 to 6.3.289 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#14144](https://github.com/paperless-ngx/paperless-ngx/pull/14144))
-- Chore(deps-dev): Bump the frontend-eslint-dependencies group across 1 directory with 4 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14143](https://github.com/paperless-ngx/paperless-ngx/pull/14143))
-- Chore(deps-dev): Bump @types/node from 26.4.0 to 26.5.0 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#14146](https://github.com/paperless-ngx/paperless-ngx/pull/14146))
-- Chore(deps-dev): Bump the frontend-jest-dependencies group across 1 directory with 2 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14142](https://github.com/paperless-ngx/paperless-ngx/pull/14142))
-- Chore(deps): Bump the utilities-minor group across 1 directory with 11 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13988](https://github.com/paperless-ngx/paperless-ngx/pull/13988))
-- Chore(deps): Bump sentence-transformers from 5.6.1 to 6.0.0 @[dependabot[bot]](https://github.com/apps/dependabot) ([#13983](https://github.com/paperless-ngx/paperless-ngx/pull/13983))
-- Chore(deps-dev): Bump types-markdown from 3.10.2.20260518 to 3.10.2.20260712 @[dependabot[bot]](https://github.com/apps/dependabot) ([#13982](https://github.com/paperless-ngx/paperless-ngx/pull/13982))
-- Chore(deps): Bump the utilities-patch group across 1 directory with 6 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13980](https://github.com/paperless-ngx/paperless-ngx/pull/13980))
-- Chore(deps): Update granian[uvloop] requirement from ~=2.7.0 to >=2.7,\<2.9 @[dependabot[bot]](https://github.com/apps/dependabot) ([#13984](https://github.com/paperless-ngx/paperless-ngx/pull/13984))
-- Chore: Updates our direct Redis pin [@stumpylog](https://github.com/stumpylog) ([#13986](https://github.com/paperless-ngx/paperless-ngx/pull/13986))
-- Chore(deps-dev): Bump the frontend-eslint-dependencies group across 1 directory with 4 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13918](https://github.com/paperless-ngx/paperless-ngx/pull/13918))
-- Chore(deps): Bump uuid from 14.0.1 to 14.0.2 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#13921](https://github.com/paperless-ngx/paperless-ngx/pull/13921))
-- Chore(deps-dev): Bump @types/node from 26.2.0 to 26.4.0 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#13919](https://github.com/paperless-ngx/paperless-ngx/pull/13919))
-- Chore: update ng-select to v24, handle breaking changes [@shamoon](https://github.com/shamoon) ([#13951](https://github.com/paperless-ngx/paperless-ngx/pull/13951))
-- Chore(deps): Bump djangorestframework from 3.17.2 to 3.18.0 in the django-ecosystem group across 1 directory @[dependabot[bot]](https://github.com/apps/dependabot) ([#13912](https://github.com/paperless-ngx/paperless-ngx/pull/13912))
-- Chore(deps): Bump the pre-commit-dependencies group across 1 directory with 3 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13922](https://github.com/paperless-ngx/paperless-ngx/pull/13922))
-- Chore(deps): Bump the document-processing group across 1 directory with 4 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13916](https://github.com/paperless-ngx/paperless-ngx/pull/13916))
-- Chore(deps): Bump flower from 2.0.1 to 2.1.0 in the async-tasks group across 1 directory @[dependabot[bot]](https://github.com/apps/dependabot) ([#13913](https://github.com/paperless-ngx/paperless-ngx/pull/13913))
-- Chore(deps): Bump the actions group across 1 directory with 15 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13920](https://github.com/paperless-ngx/paperless-ngx/pull/13920))
-- docker-compose(deps): Bump gotenberg/gotenberg from 8.34 to 8.36 in /docker/compose @[dependabot[bot]](https://github.com/apps/dependabot) ([#13910](https://github.com/paperless-ngx/paperless-ngx/pull/13910))
-- Chore(deps-dev): Bump the development group across 1 directory with 2 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13911](https://github.com/paperless-ngx/paperless-ngx/pull/13911))
-- docker(deps): Bump astral-sh/uv from 0.12.5-python3.14-trixie-slim to 0.12.9-python3.14-trixie-slim @[dependabot[bot]](https://github.com/apps/dependabot) ([#13914](https://github.com/paperless-ngx/paperless-ngx/pull/13914))
-
-</details>
-
-### All App Changes
-
-<details>
-<summary>80 changes</summary>
-
-- Chore(deps): Bump the utilities-patch group across 1 directory with 15 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14167](https://github.com/paperless-ngx/paperless-ngx/pull/14167))
-- Fix: don't redirect to signup on first install when regular login is disabled [@cyberb](https://github.com/cyberb) ([#14165](https://github.com/paperless-ngx/paperless-ngx/pull/14165))
-- Chore(deps): Bump the utilities-minor group across 1 directory with 10 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14149](https://github.com/paperless-ngx/paperless-ngx/pull/14149))
-- Enhancement (QoL): support deselecting single items from "select all" [@shamoon](https://github.com/shamoon) ([#14117](https://github.com/paperless-ngx/paperless-ngx/pull/14117))
-- Chore(deps): Bump the frontend-angular-dependencies group across 1 directory with 19 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14141](https://github.com/paperless-ngx/paperless-ngx/pull/14141))
-- Enhancement: Match fuzzy terms in place inside the parsed query [@stumpylog](https://github.com/stumpylog) ([#14157](https://github.com/paperless-ngx/paperless-ngx/pull/14157))
-- Enhancement: Match CJK terms through their bigram fields in place [@stumpylog](https://github.com/stumpylog) ([#14156](https://github.com/paperless-ngx/paperless-ngx/pull/14156))
-- Chore(deps): Bump pdfjs-dist from 6.2.108 to 6.3.289 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#14144](https://github.com/paperless-ngx/paperless-ngx/pull/14144))
-- Chore(deps-dev): Bump the frontend-eslint-dependencies group across 1 directory with 4 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14143](https://github.com/paperless-ngx/paperless-ngx/pull/14143))
-- Chore(deps-dev): Bump @types/node from 26.4.0 to 26.5.0 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#14146](https://github.com/paperless-ngx/paperless-ngx/pull/14146))
-- Chore(deps-dev): Bump the frontend-jest-dependencies group across 1 directory with 2 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#14142](https://github.com/paperless-ngx/paperless-ngx/pull/14142))
-- Enhancement: centralized management of share links + bundles [@shamoon](https://github.com/shamoon) ([#14115](https://github.com/paperless-ngx/paperless-ngx/pull/14115))
-- Performance: Preprocess classifier text with Tantivy instead of NLTK [@stumpylog](https://github.com/stumpylog) ([#14127](https://github.com/paperless-ngx/paperless-ngx/pull/14127))
-- Performance: Drops fields from the classifier before pickling [@stumpylog](https://github.com/stumpylog) ([#14114](https://github.com/paperless-ngx/paperless-ngx/pull/14114))
-- Fix: better catch email workflow placeholder parsing errors [@shamoon](https://github.com/shamoon) ([#14129](https://github.com/paperless-ngx/paperless-ngx/pull/14129))
-- Performance: Improves the memory efficiency of classifier training [@stumpylog](https://github.com/stumpylog) ([#14124](https://github.com/paperless-ngx/paperless-ngx/pull/14124))
-- Performance: Streams the classifier pickle file during save as well [@stumpylog](https://github.com/stumpylog) ([#14121](https://github.com/paperless-ngx/paperless-ngx/pull/14121))
-- Fix: validate legacy bulk edit owner, rotate and split parameters [@stumpylog](https://github.com/stumpylog) ([#14120](https://github.com/paperless-ngx/paperless-ngx/pull/14120))
-- Fix: validate set\_permissions with a nested serializer [@stumpylog](https://github.com/stumpylog) ([#14119](https://github.com/paperless-ngx/paperless-ngx/pull/14119))
-- Fix: Use prefetching to reduce query counts during classifier training [@stumpylog](https://github.com/stumpylog) ([#14122](https://github.com/paperless-ngx/paperless-ngx/pull/14122))
-- Fix: reject non-dict user\_args/barcode\_tag\_mapping in config API [@stumpylog](https://github.com/stumpylog) ([#14118](https://github.com/paperless-ngx/paperless-ngx/pull/14118))
-- Fix: type edit\_pdf operations via a nested serializer [@stumpylog](https://github.com/stumpylog) ([#14116](https://github.com/paperless-ngx/paperless-ngx/pull/14116))
-- Performance: Loads the classifier through a memory view to reduce memory usage [@stumpylog](https://github.com/stumpylog) ([#14113](https://github.com/paperless-ngx/paperless-ngx/pull/14113))
-- Feature: parse advanced search with whoosh-compat and delete the handwritten translation [@stumpylog](https://github.com/stumpylog) ([#14072](https://github.com/paperless-ngx/paperless-ngx/pull/14072))
-- Fix: ensure django setup is run for management comments under 3.14 [@shamoon](https://github.com/shamoon) ([#14100](https://github.com/paperless-ngx/paperless-ngx/pull/14100))
-- Fix: validate PDF output doc indexes in bulk edit [@shamoon](https://github.com/shamoon) ([#14083](https://github.com/paperless-ngx/paperless-ngx/pull/14083))
-- Enhancement: allow regex timeout configuration [@shamoon](https://github.com/shamoon) ([#14085](https://github.com/paperless-ngx/paperless-ngx/pull/14085))
-- Fix: avoid IntegrityError when a retried task republishes with the same ID [@stumpylog](https://github.com/stumpylog) ([#14096](https://github.com/paperless-ngx/paperless-ngx/pull/14096))
-- Chore: include Apply AI Suggestions in the tasks UI filter dropdown [@shamoon](https://github.com/shamoon) ([#14093](https://github.com/paperless-ngx/paperless-ngx/pull/14093))
-- Fix: update some api global perms inconsistencies [@shamoon](https://github.com/shamoon) ([#14086](https://github.com/paperless-ngx/paperless-ngx/pull/14086))
-- Fix: ignore nested action IDs on WF create [@shamoon](https://github.com/shamoon) ([#14084](https://github.com/paperless-ngx/paperless-ngx/pull/14084))
-- Fix: correct text/stream compression workaround [@shamoon](https://github.com/shamoon) ([#14064](https://github.com/paperless-ngx/paperless-ngx/pull/14064))
-- Fix: ui version content switching inconsistencies [@shamoon](https://github.com/shamoon) ([#14066](https://github.com/paperless-ngx/paperless-ngx/pull/14066))
-- Fix: prevent saving changes to stale cached document object [@shamoon](https://github.com/shamoon) ([#14065](https://github.com/paperless-ngx/paperless-ngx/pull/14065))
-- Performance: batch permission assignment in bulk `set_permissions` [@stumpylog](https://github.com/stumpylog) ([#13806](https://github.com/paperless-ngx/paperless-ngx/pull/13806))
-- Performance: skip effective\_content annotation on document list unless required [@stumpylog](https://github.com/stumpylog) ([#13789](https://github.com/paperless-ngx/paperless-ngx/pull/13789))
-- Fix: ensure remove inbox tag children on remove\_inbox\_tags [@shamoon](https://github.com/shamoon) ([#14050](https://github.com/paperless-ngx/paperless-ngx/pull/14050))
-- Fix: connect add\_to\_index handler after document\_added [@shamoon](https://github.com/shamoon) ([#14058](https://github.com/paperless-ngx/paperless-ngx/pull/14058))
-- Enhancement: hide-able sidebar items [@shamoon](https://github.com/shamoon) ([#14052](https://github.com/paperless-ngx/paperless-ngx/pull/14052))
-- Performance: cut redundant per-document lookups in bulk `modify_custom_fields` [@stumpylog](https://github.com/stumpylog) ([#13807](https://github.com/paperless-ngx/paperless-ngx/pull/13807))
-- Enhancement: Improve matching for correspondents, storage path and labels by removing bias + adding minimum match threshold [@dewey](https://github.com/dewey) ([#12164](https://github.com/paperless-ngx/paperless-ngx/pull/12164))
-- Fix: Drop empty files from tracking after the stability window has passed [@stumpylog](https://github.com/stumpylog) ([#14047](https://github.com/paperless-ngx/paperless-ngx/pull/14047))
-- Fixhancement: better LLM errors [@shamoon](https://github.com/shamoon) ([#14031](https://github.com/paperless-ngx/paperless-ngx/pull/14031))
-- Fix: prevent orphaned versions from bulk delete [@shamoon](https://github.com/shamoon) ([#14030](https://github.com/paperless-ngx/paperless-ngx/pull/14030))
-- Enhancement: add Tantivy full-text fallback adapter for taxonomy candidates [@stumpylog](https://github.com/stumpylog) ([#13820](https://github.com/paperless-ngx/paperless-ngx/pull/13820))
-- Fixhancement: prevent overlapping mail-account processing runs [@stumpylog](https://github.com/stumpylog) ([#14046](https://github.com/paperless-ngx/paperless-ngx/pull/14046))
-- Performance: ensure version-aware content filters on querysets [@shamoon](https://github.com/shamoon) ([#13792](https://github.com/paperless-ngx/paperless-ngx/pull/13792))
-- Performance: skip nested TagSerializer construction when a tag has no children [@stumpylog](https://github.com/stumpylog) ([#14039](https://github.com/paperless-ngx/paperless-ngx/pull/14039))
-- Enhancement (QoL): surface externally-set options in Config UI [@shamoon](https://github.com/shamoon) ([#13989](https://github.com/paperless-ngx/paperless-ngx/pull/13989))
-- Enhancement: allow disabling auto-suggestions for inbox documents [@shamoon](https://github.com/shamoon) ([#13946](https://github.com/paperless-ngx/paperless-ngx/pull/13946))
-- Change: skip documents with empty content in apply AI suggestions WF [@shamoon](https://github.com/shamoon) ([#13985](https://github.com/paperless-ngx/paperless-ngx/pull/13985))
-- Performance: resolve index-write permissions and effective content in bulk [@stumpylog](https://github.com/stumpylog) ([#13869](https://github.com/paperless-ngx/paperless-ngx/pull/13869))
-- Fix: Use PAPERLESS\_REDIS\_PREFIX for Celery result backend keys [@bdd](https://github.com/bdd) ([#14015](https://github.com/paperless-ngx/paperless-ngx/pull/14015))
-- Enhancement: duplicates filter [@shamoon](https://github.com/shamoon) ([#13994](https://github.com/paperless-ngx/paperless-ngx/pull/13994))
-- Fix: correct setting ai\_enabled to false via UI [@shamoon](https://github.com/shamoon) ([#13987](https://github.com/paperless-ngx/paperless-ngx/pull/13987))
-- Chore(deps): Bump the utilities-minor group across 1 directory with 11 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13988](https://github.com/paperless-ngx/paperless-ngx/pull/13988))
-- Fix: catch some frontend failed object retrievals [@shamoon](https://github.com/shamoon) ([#14023](https://github.com/paperless-ngx/paperless-ngx/pull/14023))
-- Fix: more v3 icons cleanup [@shamoon](https://github.com/shamoon) ([#14017](https://github.com/paperless-ngx/paperless-ngx/pull/14017))
-- Fix: correct add version actor parity [@shamoon](https://github.com/shamoon) ([#14016](https://github.com/paperless-ngx/paperless-ngx/pull/14016))
-- Fix: fix v3 favicon file [@shamoon](https://github.com/shamoon) ([#14014](https://github.com/paperless-ngx/paperless-ngx/pull/14014))
-- Tweak: note that apply AI suggestions runs async in WF editor [@shamoon](https://github.com/shamoon) ([#14004](https://github.com/paperless-ngx/paperless-ngx/pull/14004))
-- Fix: change share link bundle dialog button to close after create, don't toast on copied [@shamoon](https://github.com/shamoon) ([#14002](https://github.com/paperless-ngx/paperless-ngx/pull/14002))
-- Enhancement (QoL): attempt to localize firstDayOfWeek for date picker [@shamoon](https://github.com/shamoon) ([#13999](https://github.com/paperless-ngx/paperless-ngx/pull/13999))
-- Fix: truncate mail subjects to field max length [@shamoon](https://github.com/shamoon) ([#13991](https://github.com/paperless-ngx/paperless-ngx/pull/13991))
-- Chore(deps): Bump sentence-transformers from 5.6.1 to 6.0.0 @[dependabot[bot]](https://github.com/apps/dependabot) ([#13983](https://github.com/paperless-ngx/paperless-ngx/pull/13983))
-- Chore(deps-dev): Bump types-markdown from 3.10.2.20260518 to 3.10.2.20260712 @[dependabot[bot]](https://github.com/apps/dependabot) ([#13982](https://github.com/paperless-ngx/paperless-ngx/pull/13982))
-- Chore(deps): Bump the utilities-patch group across 1 directory with 6 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13980](https://github.com/paperless-ngx/paperless-ngx/pull/13980))
-- Chore(deps): Update granian[uvloop] requirement from ~=2.7.0 to >=2.7,\<2.9 @[dependabot[bot]](https://github.com/apps/dependabot) ([#13984](https://github.com/paperless-ngx/paperless-ngx/pull/13984))
-- Chore: Updates our direct Redis pin [@stumpylog](https://github.com/stumpylog) ([#13986](https://github.com/paperless-ngx/paperless-ngx/pull/13986))
-- Fix: enforce the overflow hidden rule on pdf editor thumbnails [@shamoon](https://github.com/shamoon) ([#13976](https://github.com/paperless-ngx/paperless-ngx/pull/13976))
-- Fix: also correct unbroken long names on small cards [@shamoon](https://github.com/shamoon) ([#13974](https://github.com/paperless-ngx/paperless-ngx/pull/13974))
-- Chore(deps-dev): Bump the frontend-eslint-dependencies group across 1 directory with 4 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13918](https://github.com/paperless-ngx/paperless-ngx/pull/13918))
-- Chore(deps): Bump uuid from 14.0.1 to 14.0.2 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#13921](https://github.com/paperless-ngx/paperless-ngx/pull/13921))
-- Chore(deps-dev): Bump @types/node from 26.2.0 to 26.4.0 in /src-ui @[dependabot[bot]](https://github.com/apps/dependabot) ([#13919](https://github.com/paperless-ngx/paperless-ngx/pull/13919))
-- Chore: update ng-select to v24, handle breaking changes [@shamoon](https://github.com/shamoon) ([#13951](https://github.com/paperless-ngx/paperless-ngx/pull/13951))
-- Chore(deps): Bump djangorestframework from 3.17.2 to 3.18.0 in the django-ecosystem group across 1 directory @[dependabot[bot]](https://github.com/apps/dependabot) ([#13912](https://github.com/paperless-ngx/paperless-ngx/pull/13912))
-- Chore(deps): Bump the document-processing group across 1 directory with 4 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13916](https://github.com/paperless-ngx/paperless-ngx/pull/13916))
-- Chore(deps): Bump flower from 2.0.1 to 2.1.0 in the async-tasks group across 1 directory @[dependabot[bot]](https://github.com/apps/dependabot) ([#13913](https://github.com/paperless-ngx/paperless-ngx/pull/13913))
-- Chore(deps-dev): Bump the development group across 1 directory with 2 updates @[dependabot[bot]](https://github.com/apps/dependabot) ([#13911](https://github.com/paperless-ngx/paperless-ngx/pull/13911))
-- Fix: ensure parent + child tags change together in bulk editor [@shamoon](https://github.com/shamoon) ([#13972](https://github.com/paperless-ngx/paperless-ngx/pull/13972))
-
-</details>
-
 ## paperless-ngx 3.1.3
 
 ### Bug Fixes

docs/configuration.md

@@ -413,12 +413,18 @@ details.
 
     Defaults to `PAPERLESS_DATA_DIR/log/`.
 
-#### ~~[`PAPERLESS_NLTK_DIR`](#PAPERLESS_NLTK_DIR)~~ {#PAPERLESS_NLTK_DIR}
+#### [`PAPERLESS_NLTK_DIR=<path>`](#PAPERLESS_NLTK_DIR) {#PAPERLESS_NLTK_DIR}
 
-!!! failure "Removed in v3.2"
+: This is where paperless will search for the data required for NLTK
+processing, if you are using it. If you are using the Docker image,
+this should not be changed, as the data is included in the image
+already.
 
-    Removed and ignored. Any previously downloaded NLTK data folder can be
+Previously, the location defaulted to `PAPERLESS_DATA_DIR/nltk`.
-    deleted.
+Unless you are using this in a bare metal install or other setup,
+this folder is no longer needed and can be removed manually.
+
+Defaults to `/usr/share/nltk_data`
 
 #### [`PAPERLESS_MODEL_FILE=<path>`](#PAPERLESS_MODEL_FILE) {#PAPERLESS_MODEL_FILE}
 
@@ -1184,31 +1190,15 @@ for details on how to set it.
 
     Defaults to UTC.
 
-#### ~~[`PAPERLESS_ENABLE_NLTK`](#PAPERLESS_ENABLE_NLTK)~~ {#PAPERLESS_ENABLE_NLTK}
+#### [`PAPERLESS_ENABLE_NLTK=<bool>`](#PAPERLESS_ENABLE_NLTK) {#PAPERLESS_ENABLE_NLTK}
 
-!!! failure "Removed in v3.2"
+: Enables or disables the advanced natural language processing
+used during automatic classification. If disabled, paperless will
+still perform some basic text pre-processing before matching.
 
-    Removed and ignored. Automatic classification always removes stop words
+: See also `PAPERLESS_NLTK_DIR`.
-    and stems words when the primary OCR language is Danish, Dutch, English,
-    Finnish, French, German, Italian, Norwegian, Portuguese, Russian, Spanish
-    or Swedish. Other languages are only lowercased and split into words.
 
-#### [`PAPERLESS_CLASSIFIER_MATCH_THRESHOLD=<float>`](#PAPERLESS_CLASSIFIER_MATCH_THRESHOLD) {#PAPERLESS_CLASSIFIER_MATCH_THRESHOLD}
+    Defaults to true, enabling the feature.
-
-: Sets the minimum confidence score (0.0-1.0) required for the automatic
-classifier to assign a correspondent, document type, or storage path to a
-document. Predictions below this threshold are discarded and the field is
-left unassigned, preventing low-confidence guesses from being applied.
-
-    Defaults to 0.3.
-
-#### [`PAPERLESS_MATCH_REGEX_TIMEOUT_SECONDS=<float>`](#PAPERLESS_MATCH_REGEX_TIMEOUT_SECONDS) {#PAPERLESS_MATCH_REGEX_TIMEOUT_SECONDS}
-
-: Sets the timeout, in seconds, for regular expression matching. Increase this
-value if date parsing or user-defined matching rules time out when processing
-long documents, especially on slower hardware.
-
-    Defaults to 0.1 seconds.
 
 #### [`PAPERLESS_DATE_PARSER_LANGUAGES=<lang>`](#PAPERLESS_DATE_PARSER_LANGUAGES) {#PAPERLESS_DATE_PARSER_LANGUAGES}
 
@@ -1279,8 +1269,6 @@ Tantivy stemmer equivalent, stemming is disabled.
 matching. Fuzzy results rank below exact matches. A value of `0.5` is a reasonable
 starting point. Leave unset to disable fuzzy matching entirely.
 
-    Words of a single character are not fuzzy-matched, since a single-character approximate match would match nearly every term in the index.
-
     Defaults to unset (disabled).
 
 #### [`PAPERLESS_SANITY_TASK_CRON=<cron expression>`](#PAPERLESS_SANITY_TASK_CRON) {#PAPERLESS_SANITY_TASK_CRON}

docs/setup.md

@@ -430,6 +430,11 @@ to a positive number to enable polling and disable native filesystem notificatio
 This will reduce the size of generated PDF documents. You'll most likely need to compile this yourself, because this
 software has been patented until around 2017 and binary packages are not available for most distributions.
 
+**Optional: download the NLTK data**
+If using the NLTK machine-learning processing (see [`PAPERLESS_ENABLE_NLTK`](configuration.md#PAPERLESS_ENABLE_NLTK) for details),
+download the NLTK data for the Snowball Stemmer, Stopwords and Punkt tokenizer to `/usr/share/nltk_data`. Refer to the [NLTK
+instructions](https://www.nltk.org/data.html) for details on how to download the data.
+
 #### After installation
 
 Your Paperless-ngx instance should now be accessible at `http://localhost:8000` (or similar, depending on your configuration).
@@ -645,6 +650,9 @@ hardware, but a few settings can improve performance:
   `PAPERLESS_OCR_CLEAN=none`. This will speed up OCR times and use
   less memory at the expense of slightly worse OCR results.
 - If using Docker, consider setting [`PAPERLESS_WEBSERVER_WORKERS`](configuration.md#PAPERLESS_WEBSERVER_WORKERS) to 1. This will save some memory.
+- Consider setting [`PAPERLESS_ENABLE_NLTK`](configuration.md#PAPERLESS_ENABLE_NLTK) to false, to disable the
+  more advanced language processing, which can take more memory and
+  processing time.
 
 For details, refer to [configuration](configuration.md).
 

docs/usage.md

@@ -927,105 +927,52 @@ typed in the search bar. A few things to know about how matching works:
 
 Paperless also offers advanced search syntax if you want to drill down further.
 
-#### Combining terms
+Matching documents with logical expressions:
 
 ```
 shopname AND (product1 OR product2)
-invoice NOT draft
-"quick brown fox"
 ```
 
-- `AND`, `OR` and `NOT` must be written in capitals. Parentheses group terms.
+Matching specific tags, correspondents or types:
-- Terms with no operator between them are combined with `AND`.
-- Quotes match an exact phrase, with the words in that order.
-
-!!! warning
-
-    A leading `-` does **not** exclude a term. `invoice -secret` finds documents containing both words. Use `invoice NOT secret` instead.
-
-#### Searching by field
-
-Put a field name and a colon in front of a value to search only that field:
 
 ```
 type:invoice tag:unpaid
-correspondent:"acme corp"
+correspondent:university certificate
-tag:bills,unpaid
-asn:[50 to 150]
-checksum:9f86d081*
 ```
 
-| Field                     | Searches                                 |
+Matching dates:
-| ------------------------- | ---------------------------------------- |
-| `title`                   | Title                                    |
-| `content`                 | Text content                             |
-| `correspondent`           | Correspondent                            |
-| `document_type` or `type` | Document type                            |
-| `storage_path` or `path`  | Storage path                             |
-| `tag`                     | Tags                                     |
-| `original_filename`       | File name the document was consumed with |
-| `asn`                     | Archive serial number                    |
-| `page_count`              | Number of pages                          |
-| `num_notes`               | Number of notes                          |
-| `checksum`                | Checksum of the original file            |
-| `created`                 | Created date                             |
-| `added`                   | When the document was added to paperless |
-| `modified`                | When the document was last modified      |
-
-- A field applies only to the word right after it. Quote multi-word values: `correspondent:"acme corp"`.
-- A comma-separated `tag` list requires every listed tag, so `tag:bills,unpaid` only matches documents tagged with both.
-- `asn`, `page_count` and `num_notes` are numbers. They accept ranges like `asn:[50 to 150]`, but not wildcards.
-- `checksum` only matches the complete checksum, in lowercase. To search by its first few characters, add a wildcard: `checksum:9f86d081*`.
-- `created`, `added` and `modified` take the values described in [Searching by date](#searching-by-date).
-- Custom fields and notes have their own syntax, described [below](#searching-custom-fields).
-
-#### Wildcards
 
 ```
-invoice*
-title:Invoice*
-20[12]?
-20[!0]?
-```
-
-- `*` matches any number of characters, and `?` matches exactly one.
-- `[...]` matches one character from a set or range, and `[!...]` matches one character not in it. `20[12]?` matches 2010 to 2029.
-- Brackets only act as a wildcard when the value also contains a `*` or `?`. Otherwise they are searched as ordinary text. The exception is a single-character range such as `title:200[1-9]`, which is rejected with an error. Add a wildcard to use it as a pattern: `title:200[1-9]*`.
-
-!!! note
-
-    When a [stemmer is available](configuration.md#PAPERLESS_SEARCH_LANGUAGE) for your search language, words are indexed by their stem, so `copy*` also finds "copies". A prefix that runs past the stem can find nothing: `universit*` misses "university", which is stored as `univers`. If a wildcard finds nothing, try a shorter prefix, such as `univers*`.
-
-#### Searching by date
-
-```
-added:yesterday
-modified:"previous month"
 created:[2005 to 2009]
-added:[-1 week to now]
+added:yesterday
+modified:today
 ```
 
-These keywords each cover a whole period, and work with or without quotes: `today`, `yesterday`, `tomorrow`, `previous week`, `this month`, `previous month`, `previous quarter`, `this year`, `previous year`.
+Matching inexact words:
 
-Other supported forms:
+```
+produ*name
+```
 
-| Example                                                 | Matches                        | Quotes   |
+Matching natural date keywords:
-| ------------------------------------------------------- | ------------------------------ | -------- |
-| `created:2005`, `created:2005-01`, `created:2005-03-04` | That year, month or day        | Optional |
-| `added:january`                                         | That month in the current year | Optional |
-| `added:"next monday"`, `added:"last monday"`            | That day                       | Required |
-| `added:"12 december 2019"`                              | That day                       | Required |
-| `added:"2005-01-01T00:00:00Z"`                          | That exact time                | Required |
 
-Ranges take two bounds in square brackets, for example `created:[2005 to 2009]`. A bound can be any of the forms above, or a relative time like `-1 week`, `now-7d` or `now`. Bounds don't need quotes. If you do quote one, use single quotes (`added:['-1 week' to now]`), because double quotes are rejected.
+```
+added:today
+modified:yesterday
+created:"previous week"
+added:"previous month"
+modified:"this year"
+```
 
-!!! warning
+Supported date keywords: `today`, `yesterday`, `previous week`,
-
+`this month`, `previous month`, `this year`, `previous year`,
-    `now`, `noon`, `midnight` and relative times like `-1 week` only work as range bounds. On their own they mean a single instant, so `added:"-1 week"` finds nothing. Use `added:[-1 week to now]` instead. A bare weekday (`monday`) and spellings like `3 days ago` or `this week` are not supported at all.
+`previous quarter`.
 
 #### Searching custom fields
 
-Custom field names and values are included in the full-text index, but a plain search without a field name does not look at them. Use the advanced search syntax to search by field name or value:
+Custom field names and values are included in the full-text index, but they
+are not searched by a plain, unqualified query. Use the advanced search syntax
+to search by field name or value:
 
 ```
 custom_fields.value:policy
@@ -1036,9 +983,10 @@ custom_fields.name:Insurance custom_fields.value:policy
 - `custom_fields.value` matches against the value of any custom field.
 - `custom_fields.name` matches the name of the field (use quotes for multi-word names).
 - Combine both to find documents where a specific named field contains a specific value.
-- The bare `custom_fields:` prefix is shorthand for `custom_fields.value:`.
 
-Because separators are stripped during indexing, each part of a formatted code can be searched on its own. A value stored as `A-1312/99.50` is indexed as `a`, `1312`, `99` and `50`:
+Because separators are stripped during indexing, individual parts of formatted
+codes are searchable on their own. A value stored as `A-1312/99.50` produces the
+tokens `a`, `1312`, `99`, `50` — each searchable independently:
 
 ```
 custom_fields.value:1312
@@ -1047,11 +995,14 @@ custom_fields.name:"Contract Number" custom_fields.value:1312
 
 !!! note
 
-    Custom date fields do not support relative date syntax such as `[now to 2 weeks]`. For date ranges on custom date fields, use the document list filters in the web UI.
+    Custom date fields do not support relative date syntax (e.g. `[now to 2 weeks]`).
+    For date ranges on custom date fields, use the document list filters in the web UI.
 
 #### Searching notes
 
-Notes are included in the full-text index, but a plain search without a field name does not look at them. Use the advanced search syntax to search by note author or content:
+Notes are included in the full-text index, but they are not searched by a
+plain, unqualified query. Use the advanced search syntax to search by note
+author or content:
 
 ```
 notes.user:alice
@@ -1059,13 +1010,15 @@ notes.note:reminder
 notes.user:alice notes.note:insurance
 ```
 
-The bare `notes:` prefix is shorthand for `notes.note:`.
+All of these constructs can be combined as you see fit. If you want to
-
+learn more about the query language used by paperless, see the
-All of these can be combined. Syntax not described here may not work as expected, and an unknown field name is searched as ordinary text.
+[Tantivy query language documentation](https://docs.rs/tantivy/latest/tantivy/query/struct.QueryParser.html).
 
 !!! note
 
-    Fuzzy (approximate) matching can be enabled by setting [`PAPERLESS_ADVANCED_FUZZY_SEARCH_THRESHOLD`](configuration.md#PAPERLESS_ADVANCED_FUZZY_SEARCH_THRESHOLD). When enabled, paperless also includes near-miss results, ranked below exact matches.
+    Fuzzy (approximate) matching can be enabled by setting
+    [`PAPERLESS_ADVANCED_FUZZY_SEARCH_THRESHOLD`](configuration.md#PAPERLESS_ADVANCED_FUZZY_SEARCH_THRESHOLD).
+    When enabled, paperless will include near-miss results ranked below exact matches.
 
 ## Keyboard shortcuts / hotkeys
 

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "paperless-ngx"
-version = "3.2.0"
+version = "3.1.3"
 description = "A community-supported supercharged document management system: scan, index and archive all your physical documents"
 readme = "README.md"
 requires-python = ">=3.11"
@@ -39,7 +39,7 @@ dependencies = [
   "django-treenode>=0.24",
   "djangorestframework~=3.16",
   "drf-spectacular~=0.30",
-  "drf-spectacular-sidecar>=2026.7.1,<2026.10",
+  "drf-spectacular-sidecar>=2026.7.1,<2026.9",
   "drf-writable-nested~=0.7.1",
   "filelock~=3.32.0",
   "flower>=2.0.1,<2.2",
@@ -55,7 +55,8 @@ dependencies = [
   "llama-index-embeddings-openai-like>=0.2.2",
   "llama-index-llms-ollama>=0.9.1",
   "llama-index-llms-openai-like>=0.7.1",
-  "ocrmypdf[heic]>=17.12,<17.13",
+  "nltk~=3.10.0",
+  "ocrmypdf>=17.7,<17.12",
   "openai>=2.48",
   "pathvalidate~=3.3.1",
   "pdf2image~=1.17.0",
@@ -73,10 +74,9 @@ dependencies = [
   "sqlite-vec==0.1.9",
   "tantivy~=0.26.0",
   "tika-client[httpx]~=1.0",
-  "torch>=2.13,<2.15",
+  "torch~=2.13.0",
   "watchfiles>=1.2",
   "whitenoise~=6.11",
-  "whoosh-compat[tantivy]==0.3",
   "zxing-cpp~=3.1.0",
 ]
 [project.optional-dependencies]
@@ -109,7 +109,7 @@ lint = [
 testing = [
   "daphne",
   "factory-boy~=3.3.1",
-  "faker>=40.36,<40.39",
+  "faker>=40.36,<40.38",
   "imagehash",
   "pytest~=9.1.1",
   "pytest-cov~=7.1.0",
@@ -247,7 +247,7 @@ per-file-ignores."src/documents/models.py" = [
 isort.force-single-line = true
 
 [tool.codespell]
-ignore-words-list = "criterias,afterall,valeu,ureue,equest,ure,assertIn,Oktober,commitish,NIN,nin,reprot"
+ignore-words-list = "criterias,afterall,valeu,ureue,equest,ure,assertIn,Oktober,commitish,NIN,nin"
 skip = """\
   src-ui/src/locale/*,src-ui/pnpm-lock.yaml,src-ui/e2e/*,src/paperless_mail/tests/samples/*,src/paperless/tests/samples\
   /mail/*,src/documents/tests/samples/*,*.po,*.json\

src-ui/package.json

@@ -1,6 +1,6 @@
 {
   "name": "paperless-ngx-ui",
-  "version": "3.2.0",
+  "version": "3.1.3",
   "scripts": {
     "preinstall": "npx only-allow pnpm",
     "ng": "ng",
@@ -15,16 +15,16 @@
   },
   "private": true,
   "dependencies": {
-    "@angular/cdk": "^22.1.6",
+    "@angular/cdk": "^22.1.4",
-    "@angular/common": "~22.1.6",
+    "@angular/common": "~22.1.3",
-    "@angular/compiler": "~22.1.6",
+    "@angular/compiler": "~22.1.3",
-    "@angular/core": "~22.1.6",
+    "@angular/core": "~22.1.3",
-    "@angular/forms": "~22.1.6",
+    "@angular/forms": "~22.1.3",
-    "@angular/localize": "~22.1.6",
+    "@angular/localize": "~22.1.3",
-    "@angular/platform-browser": "~22.1.6",
+    "@angular/platform-browser": "~22.1.3",
-    "@angular/router": "~22.1.6",
+    "@angular/router": "~22.1.3",
     "@ng-bootstrap/ng-bootstrap": "^21.0.0",
-    "@ng-select/ng-select": "~24.1.1",
+    "@ng-select/ng-select": "~24.0.2",
     "@ngneat/dirty-check-forms": "^3.0.3",
     "@popperjs/core": "^2.11.8",
     "bootstrap": "^5.3.8",
@@ -37,7 +37,7 @@
     "ngx-device-detector": "^12.0.0",
     "ngx-ui-tour-ng-bootstrap": "^19.0.0",
     "normalize-diacritics": "^5.0.0",
-    "pdfjs-dist": "^6.3.289",
+    "pdfjs-dist": "^6.2.108",
     "rxjs": "^7.8.2",
     "tslib": "^2.8.1",
     "utif": "^3.1.0",
@@ -45,25 +45,25 @@
   },
   "devDependencies": {
     "@angular-builders/jest": "^22.0.1",
-    "@angular-devkit/core": "^22.1.8",
+    "@angular-devkit/core": "^22.1.6",
-    "@angular-devkit/schematics": "^22.1.8",
+    "@angular-devkit/schematics": "^22.1.6",
-    "@angular-eslint/builder": "22.5.0",
+    "@angular-eslint/builder": "22.1.0",
-    "@angular-eslint/eslint-plugin": "22.5.0",
+    "@angular-eslint/eslint-plugin": "22.1.0",
-    "@angular-eslint/eslint-plugin-template": "22.5.0",
+    "@angular-eslint/eslint-plugin-template": "22.1.0",
-    "@angular-eslint/schematics": "22.5.0",
+    "@angular-eslint/schematics": "22.1.0",
-    "@angular-eslint/template-parser": "22.5.0",
+    "@angular-eslint/template-parser": "22.1.0",
-    "@angular/build": "22.1.8",
+    "@angular/build": "22.1.6",
-    "@angular/cli": "22.1.8",
+    "@angular/cli": "22.1.6",
-    "@angular/compiler-cli": "~22.1.6",
+    "@angular/compiler-cli": "~22.1.3",
     "@playwright/test": "^1.62.1",
     "@types/jest": "^30.0.0",
-    "@types/node": "^26.5.0",
+    "@types/node": "^26.4.0",
-    "@typescript-eslint/eslint-plugin": "^8.70.0",
+    "@typescript-eslint/eslint-plugin": "^8.68.0",
-    "@typescript-eslint/parser": "^8.70.0",
+    "@typescript-eslint/parser": "^8.68.0",
-    "@typescript-eslint/utils": "^8.70.0",
+    "@typescript-eslint/utils": "^8.68.0",
-    "eslint": "^10.10.0",
+    "eslint": "^10.9.1",
-    "jest": "30.5.1",
+    "jest": "30.4.2",
-    "jest-environment-jsdom": "^30.5.1",
+    "jest-environment-jsdom": "^30.4.1",
     "jest-junit": "^17.0.0",
     "jest-preset-angular": "^17.0.0",
     "jest-websocket-mock": "^2.5.0",

src-ui/src/app/app-routing.module.ts

@@ -14,7 +14,6 @@ import { DocumentListComponent } from './components/document-list/document-list.
 import { DocumentAttributesComponent } from './components/manage/document-attributes/document-attributes.component'
 import { MailComponent } from './components/manage/mail/mail.component'
 import { SavedViewsComponent } from './components/manage/saved-views/saved-views.component'
-import { ShareLinksComponent } from './components/manage/share-links/share-links.component'
 import { WorkflowsComponent } from './components/manage/workflows/workflows.component'
 import { NotFoundComponent } from './components/not-found/not-found.component'
 import { DirtyDocGuard } from './guards/dirty-doc.guard'
@@ -311,24 +310,6 @@ export const routes: Routes = [
           componentName: 'SavedViewsComponent',
         },
       },
-      {
-        path: 'share-links',
-        component: ShareLinksComponent,
-        canActivate: [PermissionsGuard],
-        data: {
-          requiredPermissionAny: [
-            {
-              action: PermissionAction.View,
-              type: PermissionType.ShareLink,
-            },
-            {
-              action: PermissionAction.View,
-              type: PermissionType.ShareLinkBundle,
-            },
-          ],
-          componentName: 'ShareLinksComponent',
-        },
-      },
     ],
   },
 

src-ui/src/app/components/admin/settings/settings.component.html

@@ -112,22 +112,6 @@
 
                 <pngx-input-check i18n-title title="Use 'slim' sidebar (icons only)" formControlName="slimSidebarEnabled"></pngx-input-check>
 
-                <p class="mb-2 mt-3" i18n>Sidebar items to show:</p>
-                @for (option of sidebarItemOptions; track option.id) {
-                  <div class="form-check">
-                    <input
-                      class="form-check-input"
-                      type="checkbox"
-                      [id]="'sidebar-item-setting-' + option.id"
-                      [checked]="isSidebarItemShown(option.id)"
-                      (change)="toggleSidebarItem(option.id, $event.target.checked)"
-                    />
-                    <label class="form-check-label" [for]="'sidebar-item-setting-' + option.id">
-                      {{ option.label }}
-                    </label>
-                  </div>
-                }
-
               </div>
             </div>
 

src-ui/src/app/components/admin/settings/settings.component.spec.ts

@@ -24,7 +24,7 @@ import {
   SystemStatus,
   SystemStatusItemStatus,
 } from 'src/app/data/system-status'
-import { HideableSidebarItemID, SETTINGS_KEYS } from 'src/app/data/ui-settings'
+import { SETTINGS_KEYS } from 'src/app/data/ui-settings'
 import { IfOwnerDirective } from 'src/app/directives/if-owner.directive'
 import { IfPermissionsDirective } from 'src/app/directives/if-permissions.directive'
 import { PermissionsGuard } from 'src/app/guards/permissions.guard'
@@ -209,45 +209,6 @@ describe('SettingsComponent', () => {
     fixture.detectChanges()
   }
 
-  it('supports configuring sidebar items and canceling changes', () => {
-    completeSetup()
-
-    component.toggleSidebarItem(HideableSidebarItemID.Workflows, false)
-    fixture.detectChanges()
-
-    expect(component.settingsForm.value.sidebarHiddenItems).toContain(
-      HideableSidebarItemID.Workflows
-    )
-
-    settingsService.updateSidebarItemVisibility(
-      HideableSidebarItemID.Mail,
-      false
-    )
-
-    expect(component.settingsForm.value.sidebarHiddenItems).toContain(
-      HideableSidebarItemID.Mail
-    )
-
-    component.reset()
-
-    expect(component.settingsForm.value.sidebarHiddenItems).not.toContain(
-      HideableSidebarItemID.Workflows
-    )
-    expect(component.settingsForm.value.sidebarHiddenItems).not.toContain(
-      HideableSidebarItemID.Mail
-    )
-  })
-
-  it('enables sidebar item controls on general settings until destroyed', () => {
-    completeSetup()
-
-    expect(settingsService.organizingSidebarItems()).toBe(true)
-
-    component.ngOnDestroy()
-
-    expect(settingsService.organizingSidebarItems()).toBe(false)
-  })
-
   it('should support tabbed settings & change URL, prevent navigation if dirty confirmation rejected', async () => {
     completeSetup()
     const navigateSpy = jest.spyOn(router, 'navigate')
@@ -288,7 +249,6 @@ describe('SettingsComponent', () => {
 
   it('should support save local settings updating appearance settings and calling API, show error', () => {
     completeSetup()
-    component.toggleSidebarItem(HideableSidebarItemID.Workflows, false)
     const toastErrorSpy = jest.spyOn(toastService, 'showError')
     const toastSpy = jest.spyOn(toastService, 'show')
     const storeSpy = jest.spyOn(settingsService, 'storeSettings')
@@ -307,10 +267,7 @@ describe('SettingsComponent', () => {
     expect(toastErrorSpy).toHaveBeenCalled()
     expect(storeSpy).toHaveBeenCalled()
     expect(appearanceSettingsSpy).not.toHaveBeenCalled()
-    expect(setSpy).toHaveBeenCalledTimes(34)
+    expect(setSpy).toHaveBeenCalledTimes(33)
-    expect(setSpy).toHaveBeenCalledWith(SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS, [
-      HideableSidebarItemID.Workflows,
-    ])
 
     // succeed
     storeSpy.mockReturnValueOnce(of(true))

src-ui/src/app/components/admin/settings/settings.component.ts

@@ -39,12 +39,7 @@ import {
   SystemStatus,
   SystemStatusItemStatus,
 } from 'src/app/data/system-status'
-import {
+import { GlobalSearchType, SETTINGS_KEYS } from 'src/app/data/ui-settings'
-  GlobalSearchType,
-  HIDEABLE_SIDEBAR_ITEM_IDS,
-  HideableSidebarItemID,
-  SETTINGS_KEYS,
-} from 'src/app/data/ui-settings'
 import { User } from 'src/app/data/user'
 import { IfPermissionsDirective } from 'src/app/directives/if-permissions.directive'
 import { CustomDatePipe } from 'src/app/pipes/custom-date.pipe'
@@ -107,15 +102,6 @@ const documentDetailFieldOptions = [
   { id: DocumentDetailFieldID.Tags, label: $localize`Tags` },
 ]
 
-const sidebarItemLabels: Record<HideableSidebarItemID, string> = {
-  [HideableSidebarItemID.Dashboard]: $localize`Dashboard`,
-  [HideableSidebarItemID.SavedViews]: $localize`Saved Views`,
-  [HideableSidebarItemID.ShareLinks]: $localize`Share Links`,
-  [HideableSidebarItemID.Workflows]: $localize`Workflows`,
-  [HideableSidebarItemID.Mail]: $localize`Mail`,
-  [HideableSidebarItemID.Documentation]: $localize`Documentation`,
-}
-
 @Component({
   selector: 'pngx-settings',
   templateUrl: './settings.component.html',
@@ -163,7 +149,6 @@ export class SettingsComponent
     bulkEditApplyOnClose: new FormControl(null),
     documentListItemPerPage: new FormControl(null),
     slimSidebarEnabled: new FormControl(null),
-    sidebarHiddenItems: new FormControl<HideableSidebarItemID[]>([]),
     darkModeUseSystem: new FormControl(null),
     darkModeEnabled: new FormControl(null),
     darkModeInvertThumbs: new FormControl(null),
@@ -201,7 +186,6 @@ export class SettingsComponent
 
   store: BehaviorSubject<any>
   storeSub: Subscription
-  sidebarItemsSub: Subscription
   isDirty$: Observable<boolean>
   isDirty: boolean = false
   unsubscribeNotifier: Subject<any> = new Subject()
@@ -219,10 +203,6 @@ export class SettingsComponent
   public readonly PdfEditorEditMode = PdfEditorEditMode
 
   public readonly documentDetailFieldOptions = documentDetailFieldOptions
-  public readonly sidebarItemOptions = HIDEABLE_SIDEBAR_ITEM_IDS.map((id) => ({
-    id,
-    label: sidebarItemLabels[id],
-  }))
 
   get systemStatusHasErrors(): boolean {
     const status = this.systemStatus()
@@ -250,10 +230,6 @@ export class SettingsComponent
 
   constructor() {
     super()
-    this.sidebarItemsSub =
-      this.settings.sidebarHiddenItemsEditingChanged.subscribe((hiddenItems) =>
-        this.settingsForm.controls.sidebarHiddenItems.setValue(hiddenItems)
-      )
     this.settings.settingsSaved.subscribe(() => {
       if (!this.savePending) this.initialize()
       this.savedViewsService.maybeRefreshDocumentCounts()
@@ -303,21 +279,14 @@ export class SettingsComponent
 
     this.activatedRoute.paramMap.subscribe((paramMap) => {
       const section = paramMap.get('section')
-      let navID = SettingsNavIDs.General
       if (section) {
         const navIDKey: string = Object.keys(SettingsNavIDs).find(
           (navID) => navID.toLowerCase() == section
         )
         if (navIDKey) {
-          navID = SettingsNavIDs[navIDKey]
+          this.activeNavID.set(SettingsNavIDs[navIDKey])
         }
       }
-      this.activeNavID.set(navID)
-      this.settings.sidebarHiddenItemsEditing.set(
-        navID === SettingsNavIDs.General
-          ? [...this.settingsForm.controls.sidebarHiddenItems.value]
-          : null
-      )
     })
   }
 
@@ -341,7 +310,6 @@ export class SettingsComponent
         SETTINGS_KEYS.DOCUMENT_LIST_SIZE
       ),
       slimSidebarEnabled: this.settings.get(SETTINGS_KEYS.SLIM_SIDEBAR),
-      sidebarHiddenItems: this.settings.get(SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS),
       darkModeUseSystem: this.settings.get(SETTINGS_KEYS.DARK_MODE_USE_SYSTEM),
       darkModeEnabled: this.settings.get(SETTINGS_KEYS.DARK_MODE_ENABLED),
       darkModeInvertThumbs: this.settings.get(
@@ -468,12 +436,6 @@ export class SettingsComponent
       this.settingsForm.patchValue(currentFormValue)
     }
 
-    if (this.settings.organizingSidebarItems()) {
-      this.settings.sidebarHiddenItemsEditing.set([
-        ...this.settingsForm.controls.sidebarHiddenItems.value,
-      ])
-    }
-
     if (this.canViewSystemStatus) {
       this.systemStatusService.get().subscribe((status) => {
         this.systemStatus.set(status)
@@ -482,18 +444,8 @@ export class SettingsComponent
   }
 
   ngOnDestroy() {
-    this.settings.sidebarHiddenItemsEditing.set(null)
     if (this.isDirty) this.settings.updateAppearanceSettings() // in case user changed appearance but didn't save
     this.storeSub && this.storeSub.unsubscribe()
-    this.sidebarItemsSub.unsubscribe()
-  }
-
-  isSidebarItemShown(item: HideableSidebarItemID): boolean {
-    return !(this.settingsForm.value.sidebarHiddenItems || []).includes(item)
-  }
-
-  toggleSidebarItem(item: HideableSidebarItemID, checked: boolean): void {
-    this.settings.updateSidebarItemVisibility(item, checked)
   }
 
   public saveSettings() {
@@ -521,10 +473,6 @@ export class SettingsComponent
       SETTINGS_KEYS.SLIM_SIDEBAR,
       this.settingsForm.value.slimSidebarEnabled
     )
-    this.settings.set(
-      SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS,
-      this.settingsForm.value.sidebarHiddenItems
-    )
     this.settings.set(
       SETTINGS_KEYS.DARK_MODE_USE_SYSTEM,
       this.settingsForm.value.darkModeUseSystem
@@ -684,11 +632,6 @@ export class SettingsComponent
 
   reset() {
     this.settingsForm.patchValue(this.store.getValue())
-    if (this.settings.organizingSidebarItems()) {
-      this.settings.sidebarHiddenItemsEditing.set([
-        ...this.settingsForm.controls.sidebarHiddenItems.value,
-      ])
-    }
   }
 
   clearThemeColor() {

src-ui/src/app/components/admin/tasks/tasks.component.ts

@@ -99,10 +99,6 @@ const TASK_TYPE_OPTIONS: Array<{
     value: PaperlessTaskType.BulkDelete,
     label: $localize`Bulk Delete`,
   },
-  {
-    value: PaperlessTaskType.ApplyAiSuggestions,
-    label: $localize`Apply AI Suggestions`,
-  },
 ]
 
 const TRIGGER_SOURCE_OPTIONS: Array<{

src-ui/src/app/components/app-frame/app-frame.component.html

@@ -86,15 +86,12 @@
       }
       <div class="sidebar-sticky pt-3 pb-1 d-flex flex-column justify-space-around">
         <ul class="nav flex-column">
-          <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Dashboard) && !settingsService.organizingSidebarItems()">
+          <li class="nav-item app-link">
-            <a class="nav-link" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Dashboard)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()" routerLink="dashboard" routerLinkActive="active" (click)="closeMenu()"
+            <a class="nav-link" routerLink="dashboard" routerLinkActive="active" (click)="closeMenu()"
               ngbPopover="Dashboard" i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end"
               container="body" triggers="mouseenter:mouseleave" popoverClass="popover-slim">
               <i-bs class="me-2" name="house"></i-bs><span class="nav-link-label"><ng-container i18n>Dashboard</ng-container></span>
             </a>
-            @if (settingsService.organizingSidebarItems()) {
-              <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Dashboard" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.Dashboard)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.Dashboard, $event)"></pngx-input-switch>
-            }
           </li>
           <li class="nav-item app-link" *pngxIfPermissions="{ action: PermissionAction.View, type: PermissionType.Document }">
             <a class="nav-link" routerLink="documents" routerLinkActive="active"
@@ -240,50 +237,29 @@
                 </div>
               </li>
             }
-            <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.SavedViews) && !settingsService.organizingSidebarItems()" *pngxIfPermissions="{ action: PermissionAction.View, type: PermissionType.SavedView }">
+            <li class="nav-item app-link" *pngxIfPermissions="{ action: PermissionAction.View, type: PermissionType.SavedView }">
-              <a class="nav-link" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.SavedViews)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()" routerLink="savedviews" routerLinkActive="active" (click)="closeMenu()"
+              <a class="nav-link" routerLink="savedviews" routerLinkActive="active" (click)="closeMenu()"
                 ngbPopover="Saved Views" i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end"
                 container="body" triggers="mouseenter:mouseleave" popoverClass="popover-slim">
                 <i-bs class="me-2" name="window-stack"></i-bs><span class="nav-link-label"><ng-container i18n>Saved Views</ng-container></span>
               </a>
-              @if (settingsService.organizingSidebarItems()) {
-                <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Saved Views" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.SavedViews)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.SavedViews, $event)"></pngx-input-switch>
-              }
             </li>
-            @if (canManageShareLinks) {
+            <li class="nav-item app-link"
-              <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.ShareLinks) && !settingsService.organizingSidebarItems()">
-                <a class="nav-link" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.ShareLinks)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()" routerLink="share-links" routerLinkActive="active" (click)="closeMenu()"
-                  ngbPopover="Share links" i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end"
-                  container="body" triggers="mouseenter:mouseleave" popoverClass="popover-slim">
-                  <i-bs class="me-2" name="link"></i-bs><span class="nav-link-label"><ng-container i18n>Share links</ng-container></span>
-                </a>
-                @if (settingsService.organizingSidebarItems()) {
-                  <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Share Links" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.ShareLinks)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.ShareLinks, $event)"></pngx-input-switch>
-                }
-              </li>
-            }
-            <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows) && !settingsService.organizingSidebarItems()"
               *pngxIfPermissions="{ action: PermissionAction.View, type: PermissionType.Workflow }"
               tourAnchor="tour.workflows">
-              <a class="nav-link" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()" routerLink="workflows" routerLinkActive="active" (click)="closeMenu()"
+              <a class="nav-link" routerLink="workflows" routerLinkActive="active" (click)="closeMenu()"
                 ngbPopover="Workflows" i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end"
                 container="body" triggers="mouseenter:mouseleave" popoverClass="popover-slim">
                 <i-bs class="me-2" name="boxes"></i-bs><span class="nav-link-label"><ng-container i18n>Workflows</ng-container></span>
               </a>
-              @if (settingsService.organizingSidebarItems()) {
-                <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Workflows" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.Workflows, $event)"></pngx-input-switch>
-              }
             </li>
-            <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Mail) && !settingsService.organizingSidebarItems()" *pngxIfPermissions="{ action: PermissionAction.View, type: PermissionType.MailAccount }"
+            <li class="nav-item app-link" *pngxIfPermissions="{ action: PermissionAction.View, type: PermissionType.MailAccount }"
               tourAnchor="tour.mail">
-              <a class="nav-link" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Mail)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()" routerLink="mail" routerLinkActive="active" (click)="closeMenu()" ngbPopover="Mail"
+              <a class="nav-link" routerLink="mail" routerLinkActive="active" (click)="closeMenu()" ngbPopover="Mail"
                 i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end" container="body"
                 triggers="mouseenter:mouseleave" popoverClass="popover-slim">
                 <i-bs class="me-2" name="envelope"></i-bs><span class="nav-link-label"><ng-container i18n>Mail</ng-container></span>
               </a>
-              @if (settingsService.organizingSidebarItems()) {
-                <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Mail" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.Mail)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.Mail, $event)"></pngx-input-switch>
-              }
             </li>
             <li class="nav-item app-link" *pngxIfPermissions="{ action: PermissionAction.Delete, type: PermissionType.Document }">
               <a class="nav-link" routerLink="trash" routerLinkActive="active" (click)="closeMenu()" ngbPopover="Trash"
@@ -346,16 +322,13 @@
                 </a>
               </li>
             }
-            <li class="nav-item mt-2 position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Documentation) && !settingsService.organizingSidebarItems()" tourAnchor="tour.outro">
+            <li class="nav-item mt-2" tourAnchor="tour.outro">
-              <a class="text-muted small d-flex align-items-center flex-wrap text-decoration-none nav-anchor" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Documentation)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()"
+              <a class="text-muted small d-flex align-items-center flex-wrap text-decoration-none nav-anchor"
                 target="_blank" rel="noopener noreferrer" href="https://docs.paperless-ngx.com" ngbPopover="Documentation"
                 i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end" container="body"
                 triggers="mouseenter:mouseleave" popoverClass="popover-slim">
                 <i-bs class="d-flex me-2" name="question-circle"></i-bs><span><ng-container i18n>Documentation</ng-container></span>
               </a>
-              @if (settingsService.organizingSidebarItems()) {
-                <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Documentation" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.Documentation)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.Documentation, $event)"></pngx-input-switch>
-              }
             </li>
             <li class="nav-item" [class.visually-hidden]="slimSidebarEnabled">
               <div class="text-muted small d-flex align-items-center flex-wrap nav-label">

src-ui/src/app/components/app-frame/app-frame.component.spec.ts

@@ -15,7 +15,7 @@ import { provideUiTour } from 'ngx-ui-tour-ng-bootstrap'
 import { of, throwError } from 'rxjs'
 import { routes } from 'src/app/app-routing.module'
 import { SavedView } from 'src/app/data/saved-view'
-import { HideableSidebarItemID, SETTINGS_KEYS } from 'src/app/data/ui-settings'
+import { SETTINGS_KEYS } from 'src/app/data/ui-settings'
 import { IfPermissionsDirective } from 'src/app/directives/if-permissions.directive'
 import { PermissionsGuard } from 'src/app/guards/permissions.guard'
 import {
@@ -287,87 +287,6 @@ describe('AppFrameComponent', () => {
     jest.useRealTimers()
   })
 
-  it('should hide configured sidebar items', () => {
-    settingsService.set(SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS, [
-      HideableSidebarItemID.Dashboard,
-      HideableSidebarItemID.Workflows,
-      HideableSidebarItemID.ShareLinks,
-    ])
-    fixture.detectChanges()
-
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="dashboard"]')
-        .parentElement.classList
-    ).toContain('d-none')
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="workflows"]')
-        .parentElement.classList
-    ).toContain('d-none')
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="share-links"]')
-        .parentElement.classList
-    ).toContain('d-none')
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="mail"]').parentElement
-        .classList
-    ).not.toContain('d-none')
-  })
-
-  it('should show hidden items and visibility switches while customizing', () => {
-    settingsService.set(SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS, [
-      HideableSidebarItemID.Dashboard,
-    ])
-    settingsService.sidebarHiddenItemsEditing.set([
-      HideableSidebarItemID.Dashboard,
-    ])
-    fixture.detectChanges()
-
-    expect(
-      fixture.nativeElement.querySelectorAll('pngx-input-switch').length
-    ).toBe(6)
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="dashboard"]')
-        .parentElement.classList
-    ).not.toContain('d-none')
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="dashboard"]').classList
-    ).toContain('opacity-50')
-
-    settingsService.set(SETTINGS_KEYS.SLIM_SIDEBAR, true)
-    fixture.detectChanges()
-
-    expect(
-      Array.from(
-        fixture.nativeElement.querySelectorAll('pngx-input-switch')
-      ).every((toggle: HTMLElement) => toggle.classList.contains('d-none'))
-    ).toBe(true)
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="dashboard"]').classList
-    ).not.toContain('pe-5')
-
-    settingsService.set(SETTINGS_KEYS.SLIM_SIDEBAR, false)
-    component.slimSidebarAnimating.set(true)
-    fixture.detectChanges()
-
-    expect(
-      Array.from(
-        fixture.nativeElement.querySelectorAll('pngx-input-switch')
-      ).every((toggle: HTMLElement) => toggle.classList.contains('d-none'))
-    ).toBe(true)
-
-    component.slimSidebarAnimating.set(false)
-    fixture.detectChanges()
-
-    expect(
-      Array.from(
-        fixture.nativeElement.querySelectorAll('pngx-input-switch')
-      ).every((toggle: HTMLElement) => !toggle.classList.contains('d-none'))
-    ).toBe(true)
-    expect(
-      fixture.nativeElement.querySelector('[routerLink="dashboard"]').classList
-    ).toContain('pe-5')
-  })
-
   it('should show error on toggle slim sidebar if store settings fails', () => {
     jest.spyOn(console, 'warn').mockImplementation(() => {})
     const toastSpy = jest.spyOn(toastService, 'showError')

src-ui/src/app/components/app-frame/app-frame.component.ts

@@ -7,7 +7,6 @@ import {
 } from '@angular/cdk/drag-drop'
 import { NgClass } from '@angular/common'
 import { Component, HostListener, inject, OnInit, signal } from '@angular/core'
-import { FormsModule } from '@angular/forms'
 import { ActivatedRoute, Router, RouterModule } from '@angular/router'
 import {
   NgbCollapseModule,
@@ -22,11 +21,7 @@ import { Observable } from 'rxjs'
 import { first } from 'rxjs/operators'
 import { Document } from 'src/app/data/document'
 import { SavedView } from 'src/app/data/saved-view'
-import {
+import { CollapsibleSection, SETTINGS_KEYS } from 'src/app/data/ui-settings'
-  CollapsibleSection,
-  HideableSidebarItemID,
-  SETTINGS_KEYS,
-} from 'src/app/data/ui-settings'
 import { IfPermissionsDirective } from 'src/app/directives/if-permissions.directive'
 import { ComponentCanDeactivate } from 'src/app/guards/dirty-doc.guard'
 import { DocumentTitlePipe } from 'src/app/pipes/document-title.pipe'
@@ -53,7 +48,6 @@ import { ChatComponent } from '../chat/chat/chat.component'
 import { BrandMarkComponent } from '../common/logo/brand-mark/brand-mark.component'
 import { LogoComponent } from '../common/logo/logo.component'
 import { ProfileEditDialogComponent } from '../common/profile-edit-dialog/profile-edit-dialog.component'
-import { SwitchComponent } from '../common/input/switch/switch.component'
 import { DocumentDetailComponent } from '../document-detail/document-detail.component'
 import { ComponentWithPermissions } from '../with-permissions/with-permissions.component'
 import { GlobalSearchComponent } from './global-search/global-search.component'
@@ -82,8 +76,6 @@ const SCROLL_THRESHOLD = 16
     NgxBootstrapIconsModule,
     DragDropModule,
     TourNgBootstrap,
-    FormsModule,
-    SwitchComponent,
   ],
 })
 export class AppFrameComponent
@@ -106,7 +98,6 @@ export class AppFrameComponent
   readonly isMenuCollapsed = signal(true)
   readonly slimSidebarAnimating = signal(false)
   readonly mobileSearchHidden = signal(false)
-  readonly HideableSidebarItemID = HideableSidebarItemID
   private readonly versionSetting = this.settingsService.getSignal<string>(
     SETTINGS_KEYS.VERSION
   )
@@ -204,10 +195,6 @@ export class AppFrameComponent
     }, 200) // slightly longer than css animation for slim sidebar
   }
 
-  toggleSidebarItem(item: HideableSidebarItemID, visible: boolean): void {
-    this.settingsService.updateSidebarItemVisibility(item, visible)
-  }
-
   toggleAttributesSections(event?: Event): void {
     event?.preventDefault()
     event?.stopPropagation()
@@ -234,19 +221,6 @@ export class AppFrameComponent
     return this.appTitleSetting() || environment.appTitle
   }
 
-  get canManageShareLinks(): boolean {
-    return (
-      this.permissionsService.currentUserCan(
-        PermissionAction.View,
-        PermissionType.ShareLink
-      ) ||
-      this.permissionsService.currentUserCan(
-        PermissionAction.View,
-        PermissionType.ShareLinkBundle
-      )
-    )
-  }
-
   get customAppTitle(): string {
     return this.appTitleSetting()
   }

src-ui/src/app/components/common/advanced-search-dialog/advanced-search-dialog.component.html

@@ -1,347 +0,0 @@
-<div class="modal-header">
-  <h4 class="modal-title" id="advanced-search-dialog-title" i18n>
-    Advanced search
-  </h4>
-  <button
-    type="button"
-    class="btn-close"
-    aria-label="Close"
-    i18n-aria-label
-    (click)="cancel()"
-  ></button>
-</div>
-<div class="modal-body">
-  @if (unreadable) {
-    <div class="alert alert-warning d-flex flex-wrap gap-2 align-items-center">
-      <div class="flex-grow-1">
-        <span i18n
-          >The current query uses syntax this editor can't show. It still works
-          as typed.</span
-        >
-      </div>
-      <button
-        type="button"
-        class="btn btn-sm btn-outline-secondary"
-        (click)="startOver()"
-        i18n
-      >
-        Start a new query
-      </button>
-    </div>
-  }
-
-  <ng-container
-    *ngTemplateOutlet="
-      groupTemplate;
-      context: { group: root, parent: null, depth: 0 }
-    "
-  ></ng-container>
-
-  <div class="mt-4">
-    <label
-      class="form-label small text-muted"
-      for="advanced-search-preview"
-      i18n
-      >Query</label
-    >
-    <pre
-      id="advanced-search-preview"
-      class="query-preview mb-0 p-2 rounded border"
-    >@if (generatedQuery) {{{ generatedQuery }}} @else {<span class="text-muted fst-italic" i18n>Nothing to search for yet</span>}</pre>
-  </div>
-</div>
-<div class="modal-footer">
-  <button
-    type="button"
-    class="btn btn-outline-secondary"
-    (click)="cancel()"
-    i18n
-  >
-    Cancel
-  </button>
-  <button
-    type="button"
-    class="btn btn-primary"
-    (click)="apply()"
-    [disabled]="!generatedQuery"
-    i18n
-  >
-    Apply
-  </button>
-</div>
-
-<ng-template
-  #groupTemplate
-  let-group="group"
-  let-parent="parent"
-  let-depth="depth"
->
-  <div class="d-flex w-100 gap-2">
-    <div class="d-flex flex-grow-1 flex-column">
-      <div class="d-flex align-items-center flex-wrap">
-        <div
-          class="btn-group btn-group-xs"
-          role="group"
-          aria-label="Match"
-          i18n-aria-label
-        >
-          <input
-            type="radio"
-            class="btn-check"
-            [(ngModel)]="group.operator"
-            [ngModelOptions]="{ standalone: true }"
-            [value]="LogicalOperator.Or"
-            id="advancedSearchAny_{{ idFor(group) }}"
-            name="advancedSearchAny_{{ idFor(group) }}"
-          />
-          <label
-            class="btn btn-outline-primary"
-            for="advancedSearchAny_{{ idFor(group) }}"
-            i18n
-            >Any</label
-          >
-          <input
-            type="radio"
-            class="btn-check"
-            [(ngModel)]="group.operator"
-            [ngModelOptions]="{ standalone: true }"
-            [value]="LogicalOperator.And"
-            id="advancedSearchAll_{{ idFor(group) }}"
-            name="advancedSearchAll_{{ idFor(group) }}"
-          />
-          <label
-            class="btn btn-outline-primary"
-            for="advancedSearchAll_{{ idFor(group) }}"
-            i18n
-            >All</label
-          >
-          <input
-            type="radio"
-            class="btn-check"
-            [(ngModel)]="group.operator"
-            [ngModelOptions]="{ standalone: true }"
-            [value]="LogicalOperator.Not"
-            id="advancedSearchNot_{{ idFor(group) }}"
-            name="advancedSearchNot_{{ idFor(group) }}"
-          />
-          <label
-            class="btn btn-outline-secondary"
-            for="advancedSearchNot_{{ idFor(group) }}"
-            i18n
-            >Not</label
-          >
-        </div>
-        <span class="small text-muted ms-2">
-          @switch (group.operator) {
-            @case (LogicalOperator.And) {
-              <ng-container i18n>match all of these</ng-container>
-            }
-            @case (LogicalOperator.Or) {
-              <ng-container i18n>match any of these</ng-container>
-            }
-            @case (LogicalOperator.Not) {
-              <ng-container i18n>match none of these</ng-container>
-            }
-          }
-        </span>
-      </div>
-      <div class="list-group list-group-flush">
-        @for (element of group.children; track element) {
-          <div class="list-group-item px-0 d-flex flex-nowrap">
-            @if (element.type === ElementType.Group) {
-              <ng-container
-                *ngTemplateOutlet="
-                  groupTemplate;
-                  context: { group: element, parent: group, depth: depth + 1 }
-                "
-              ></ng-container>
-            } @else {
-              <ng-container
-                *ngTemplateOutlet="
-                  atomTemplate;
-                  context: { atom: element, parent: group }
-                "
-              ></ng-container>
-            }
-          </div>
-        }
-      </div>
-    </div>
-    <div
-      class="btn-group-vertical align-self-start ms-2 ps-2 border-start"
-      role="group"
-      aria-label="Group actions"
-      i18n-aria-label
-    >
-      <button
-        type="button"
-        class="btn btn-sm btn-outline-secondary text-primary"
-        title="Add condition"
-        i18n-title
-        (click)="addAtom(group)"
-        [disabled]="group.children.length >= maxAtoms"
-      >
-        <i-bs name="node-plus"></i-bs>
-      </button>
-      <button
-        type="button"
-        class="btn btn-sm btn-outline-secondary text-primary"
-        title="Add group"
-        i18n-title
-        (click)="addGroup(group)"
-        [disabled]="depth >= maxDepth"
-      >
-        <i-bs name="braces"></i-bs>
-      </button>
-      @if (parent) {
-        <button
-          type="button"
-          class="btn btn-sm btn-outline-secondary text-danger"
-          aria-label="Remove group"
-          i18n-aria-label
-          (click)="remove(parent, group)"
-        >
-          <i-bs name="x-circle"></i-bs>
-        </button>
-      }
-    </div>
-  </div>
-</ng-template>
-
-<ng-template #atomTemplate let-atom="atom" let-parent="parent">
-  <div class="d-flex align-items-center gap-1 w-100">
-    <div class="input-group input-group-sm flex-wrap">
-      <select
-        class="form-select flex-grow-0 w-auto"
-        [(ngModel)]="atom.field"
-        [ngModelOptions]="{ standalone: true }"
-        (ngModelChange)="onFieldChange(atom)"
-        aria-label="Field"
-        i18n-aria-label
-      >
-        @for (fieldGroup of fieldGroups; track fieldGroup.label) {
-          <optgroup [label]="fieldGroup.label">
-            @for (field of fieldGroup.fields; track field) {
-              <option [ngValue]="field">{{ fieldLabels[field] }}</option>
-            }
-          </optgroup>
-        }
-      </select>
-      <select
-        class="form-select flex-grow-0 w-auto"
-        [(ngModel)]="atom.operator"
-        [ngModelOptions]="{ standalone: true }"
-        (ngModelChange)="onOperatorChange(atom)"
-        aria-label="Condition"
-        i18n-aria-label
-      >
-        @for (operator of operatorsFor(atom); track operator) {
-          <option [ngValue]="operator">
-            {{ operatorLabel(atom, operator) }}
-          </option>
-        }
-      </select>
-
-      @switch (atom.operator) {
-        @case (Operator.DateKeyword) {
-          <select
-            class="form-select"
-            [(ngModel)]="atom.value"
-            [ngModelOptions]="{ standalone: true }"
-            aria-label="Period"
-            i18n-aria-label
-          >
-            <option [ngValue]="''" disabled i18n>Choose a period</option>
-            @for (keyword of dateKeywords; track keyword) {
-              <option [ngValue]="keyword">
-                {{ dateKeywordLabels[keyword] }}
-              </option>
-            }
-          </select>
-        }
-        @case (Operator.WithinLast) {
-          <input
-            class="form-control amount"
-            type="number"
-            min="1"
-            [(ngModel)]="atom.value"
-            [ngModelOptions]="{ standalone: true }"
-            aria-label="Amount"
-            i18n-aria-label
-          />
-          <select
-            class="form-select"
-            [(ngModel)]="atom.unit"
-            [ngModelOptions]="{ standalone: true }"
-            aria-label="Unit"
-            i18n-aria-label
-          >
-            @for (unit of dateUnits; track unit) {
-              <option [ngValue]="unit">{{ dateUnitLabels[unit] }}</option>
-            }
-          </select>
-        }
-        @case (Operator.Between) {
-          <input
-            class="form-control"
-            [type]="kindOf(atom) === FieldKind.Date ? 'date' : 'number'"
-            [(ngModel)]="atom.value"
-            [ngModelOptions]="{ standalone: true }"
-            aria-label="From"
-            i18n-aria-label
-          />
-          <span class="input-group-text" i18n>and</span>
-          <input
-            class="form-control"
-            [type]="kindOf(atom) === FieldKind.Date ? 'date' : 'number'"
-            [(ngModel)]="atom.valueTo"
-            [ngModelOptions]="{ standalone: true }"
-            aria-label="To"
-            i18n-aria-label
-          />
-        }
-        @default {
-          @if (kindOf(atom) === FieldKind.Date) {
-            <input
-              class="form-control"
-              type="date"
-              [(ngModel)]="atom.value"
-              [ngModelOptions]="{ standalone: true }"
-              aria-label="Date"
-              i18n-aria-label
-            />
-          } @else if (kindOf(atom) === FieldKind.Number) {
-            <input
-              class="form-control"
-              type="number"
-              min="0"
-              [(ngModel)]="atom.value"
-              [ngModelOptions]="{ standalone: true }"
-              aria-label="Number"
-              i18n-aria-label
-            />
-          } @else {
-            <input
-              class="form-control"
-              type="text"
-              [(ngModel)]="atom.value"
-              [ngModelOptions]="{ standalone: true }"
-              [placeholder]="placeholderFor(atom)"
-              aria-label="Value"
-              i18n-aria-label
-            />
-          }
-        }
-      }
-    </div>
-    <button
-      class="btn btn-link btn-sm text-danger px-1"
-      type="button"
-      (click)="remove(parent, atom)"
-      aria-label="Remove condition"
-      i18n-aria-label
-    >
-      <i-bs name="x-circle"></i-bs>
-    </button>
-  </div>
-</ng-template>

src-ui/src/app/components/common/advanced-search-dialog/advanced-search-dialog.component.scss

@@ -1,11 +0,0 @@
-.query-preview {
-  font-size: 0.8125rem;
-  white-space: pre-wrap;
-  word-break: break-word;
-  background-color: var(--pngx-bg-darker);
-  border-color: var(--bs-border-color) !important;
-}
-
-.input-group .amount {
-  flex: 0 1 5rem;
-}

src-ui/src/app/components/common/advanced-search-dialog/advanced-search-dialog.component.spec.ts

@@ -1,186 +0,0 @@
-import { ComponentFixture, TestBed } from '@angular/core/testing'
-
-import { NgbActiveModal } from '@ng-bootstrap/ng-bootstrap'
-import { allIcons, NgxBootstrapIconsModule } from 'ngx-bootstrap-icons'
-import {
-  AdvancedSearchDateUnit,
-  AdvancedSearchField,
-  AdvancedSearchLogicalOperator,
-  AdvancedSearchOperator,
-  AdvancedSearchQueryAtom,
-  AdvancedSearchQueryElementType,
-  AdvancedSearchQueryGroup,
-} from 'src/app/data/advanced-search-query'
-import { AdvancedSearchDialogComponent } from './advanced-search-dialog.component'
-
-describe('AdvancedSearchDialogComponent', () => {
-  let component: AdvancedSearchDialogComponent
-  let fixture: ComponentFixture<AdvancedSearchDialogComponent>
-  let activeModal: NgbActiveModal
-
-  const firstAtom = (): AdvancedSearchQueryAtom =>
-    component.root.children[0] as AdvancedSearchQueryAtom
-
-  beforeEach(async () => {
-    await TestBed.configureTestingModule({
-      imports: [
-        AdvancedSearchDialogComponent,
-        NgxBootstrapIconsModule.pick(allIcons),
-      ],
-      providers: [NgbActiveModal],
-    }).compileComponents()
-
-    fixture = TestBed.createComponent(AdvancedSearchDialogComponent)
-    activeModal = TestBed.inject(NgbActiveModal)
-    component = fixture.componentInstance
-    fixture.detectChanges()
-  })
-
-  it('should start with one empty condition and nothing to search for', () => {
-    expect(component.root.children).toHaveLength(1)
-    expect(component.generatedQuery).toBe('')
-    expect(component.unreadable).toBeFalsy()
-  })
-
-  it('should show an existing query as conditions', () => {
-    component.query = 'title:invoice AND NOT tag:paid'
-    expect(component.unreadable).toBeFalsy()
-    expect(component.root.children).toHaveLength(2)
-    expect(component.generatedQuery).toBe('title:invoice AND NOT tag:paid')
-  })
-
-  it('should flag a query it cannot show and start empty', () => {
-    component.query = 'title:invoice^2'
-    expect(component.unreadable).toBeTruthy()
-    expect(component.generatedQuery).toBe('')
-  })
-
-  it('should clear the warning when starting a new query', () => {
-    component.query = 'title:invoice^2'
-    component.startOver()
-    expect(component.unreadable).toBeFalsy()
-    expect(component.root.children).toHaveLength(1)
-  })
-
-  it('should treat an empty query as a fresh start', () => {
-    component.query = '   '
-    expect(component.unreadable).toBeFalsy()
-    expect(component.root.children).toHaveLength(1)
-  })
-
-  it('should write the query as conditions are filled in', () => {
-    const atom = firstAtom()
-    atom.field = AdvancedSearchField.Title
-    atom.value = 'unpaid invoice'
-    expect(component.generatedQuery).toBe('title:unpaid AND title:invoice')
-  })
-
-  it('should offer the conditions of the chosen field', () => {
-    const atom = firstAtom()
-    atom.field = AdvancedSearchField.Added
-    component.onFieldChange(atom)
-    expect(component.operatorsFor(atom)).toContain(
-      AdvancedSearchOperator.WithinLast
-    )
-    expect(component.operatorsFor(atom)).not.toContain(
-      AdvancedSearchOperator.Phrase
-    )
-  })
-
-  it('should keep a condition the new field still offers', () => {
-    const atom = firstAtom()
-    atom.operator = AdvancedSearchOperator.Phrase
-    atom.field = AdvancedSearchField.Correspondent
-    component.onFieldChange(atom)
-    expect(atom.operator).toBe(AdvancedSearchOperator.Phrase)
-  })
-
-  it('should replace a condition the new field does not offer, and clear the value', () => {
-    const atom = firstAtom()
-    atom.operator = AdvancedSearchOperator.Phrase
-    atom.value = 'invoice'
-    atom.field = AdvancedSearchField.ASN
-    component.onFieldChange(atom)
-    expect(atom.operator).toBe(AdvancedSearchOperator.Equals)
-    expect(atom.value).toBe('')
-  })
-
-  it('should give a within-the-last condition a unit to start from', () => {
-    const atom = firstAtom()
-    atom.field = AdvancedSearchField.Added
-    atom.operator = AdvancedSearchOperator.WithinLast
-    component.onOperatorChange(atom)
-    expect(atom.unit).toBe(AdvancedSearchDateUnit.Day)
-    atom.value = '3'
-    expect(component.generatedQuery).toBe('added:[-3 days to now]')
-  })
-
-  it('should label date comparisons as dates', () => {
-    const atom = firstAtom()
-    atom.field = AdvancedSearchField.Created
-    expect(
-      component.operatorLabel(atom, AdvancedSearchOperator.AtLeast)
-    ).toEqual('is on or after')
-    atom.field = AdvancedSearchField.ASN
-    expect(
-      component.operatorLabel(atom, AdvancedSearchOperator.AtLeast)
-    ).toEqual('is at least')
-  })
-
-  it('should add and remove conditions', () => {
-    component.addAtom(component.root)
-    expect(component.root.children).toHaveLength(2)
-    component.remove(component.root, component.root.children[1])
-    expect(component.root.children).toHaveLength(1)
-  })
-
-  it('should add a group, which starts as Any', () => {
-    component.addGroup(component.root)
-    const group = component.root.children[1] as AdvancedSearchQueryGroup
-    expect(group.type).toBe(AdvancedSearchQueryElementType.Group)
-    expect(group.operator).toBe(AdvancedSearchLogicalOperator.Or)
-    expect(group.children).toHaveLength(1)
-  })
-
-  it('should give every group its own id, once', () => {
-    component.addGroup(component.root)
-    const group = component.root.children[1] as AdvancedSearchQueryGroup
-    expect(component.idFor(component.root)).not.toEqual(component.idFor(group))
-    expect(component.idFor(group)).toEqual(component.idFor(group))
-  })
-
-  it('should apply the query and close', () => {
-    const emitSpy = jest.spyOn(component.queryApplied, 'emit')
-    const closeSpy = jest.spyOn(activeModal, 'close')
-    const atom = firstAtom()
-    atom.value = 'invoice'
-    component.apply()
-    expect(emitSpy).toHaveBeenCalledWith('content:invoice')
-    expect(closeSpy).toHaveBeenCalled()
-  })
-
-  it('should close without applying on cancel', () => {
-    const emitSpy = jest.spyOn(component.queryApplied, 'emit')
-    const closeSpy = jest.spyOn(activeModal, 'close')
-    component.cancel()
-    expect(emitSpy).not.toHaveBeenCalled()
-    expect(closeSpy).toHaveBeenCalled()
-  })
-
-  it('should show the query it will apply', () => {
-    component.query = 'content:invoice OR content:receipt'
-    fixture.detectChanges()
-    const preview = fixture.nativeElement.querySelector(
-      '#advanced-search-preview'
-    )
-    expect(preview.textContent).toContain('content:invoice OR content:receipt')
-  })
-
-  it('should not offer to apply an empty query', () => {
-    fixture.detectChanges()
-    const apply = Array.from(
-      fixture.nativeElement.querySelectorAll('.modal-footer button')
-    ).pop() as HTMLButtonElement
-    expect(apply.disabled).toBeTruthy()
-  })
-})

src-ui/src/app/components/common/advanced-search-dialog/advanced-search-dialog.component.ts

@@ -1,197 +0,0 @@
-import { NgTemplateOutlet } from '@angular/common'
-import { Component, EventEmitter, inject, Input, Output } from '@angular/core'
-import { FormsModule } from '@angular/forms'
-import { NgbActiveModal } from '@ng-bootstrap/ng-bootstrap'
-import { NgxBootstrapIconsModule } from 'ngx-bootstrap-icons'
-import {
-  ADVANCED_SEARCH_DATE_KEYWORD_LABELS,
-  ADVANCED_SEARCH_DATE_KEYWORDS,
-  ADVANCED_SEARCH_DATE_OPERATOR_LABELS,
-  ADVANCED_SEARCH_DATE_UNIT_LABELS,
-  ADVANCED_SEARCH_FIELD_GROUPS,
-  ADVANCED_SEARCH_FIELD_KINDS,
-  ADVANCED_SEARCH_FIELD_LABELS,
-  ADVANCED_SEARCH_MAX_ATOMS,
-  ADVANCED_SEARCH_MAX_DEPTH,
-  ADVANCED_SEARCH_OPERATOR_LABELS,
-  ADVANCED_SEARCH_OPERATORS_BY_KIND,
-  AdvancedSearchDateUnit,
-  AdvancedSearchField,
-  AdvancedSearchFieldKind,
-  AdvancedSearchLogicalOperator,
-  AdvancedSearchOperator,
-  AdvancedSearchQueryAtom,
-  AdvancedSearchQueryElement,
-  AdvancedSearchQueryElementType,
-  AdvancedSearchQueryGroup,
-} from 'src/app/data/advanced-search-query'
-import {
-  parseAdvancedSearchQuery,
-  serializeAdvancedSearchQuery,
-} from 'src/app/utils/advanced-search-query'
-import { LoadingComponentWithPermissions } from '../../loading-component/loading.component'
-
-@Component({
-  selector: 'pngx-advanced-search-dialog',
-  templateUrl: './advanced-search-dialog.component.html',
-  styleUrl: './advanced-search-dialog.component.scss',
-  imports: [FormsModule, NgTemplateOutlet, NgxBootstrapIconsModule],
-})
-export class AdvancedSearchDialogComponent extends LoadingComponentWithPermissions {
-  private activeModal = inject(NgbActiveModal)
-
-  public readonly ElementType = AdvancedSearchQueryElementType
-  public readonly LogicalOperator = AdvancedSearchLogicalOperator
-  public readonly Operator = AdvancedSearchOperator
-  public readonly FieldKind = AdvancedSearchFieldKind
-  public readonly fieldGroups = ADVANCED_SEARCH_FIELD_GROUPS
-  public readonly fieldLabels = ADVANCED_SEARCH_FIELD_LABELS
-  public readonly dateKeywords = ADVANCED_SEARCH_DATE_KEYWORDS
-  public readonly dateKeywordLabels = ADVANCED_SEARCH_DATE_KEYWORD_LABELS
-  public readonly dateUnits = Object.values(AdvancedSearchDateUnit)
-  public readonly dateUnitLabels = ADVANCED_SEARCH_DATE_UNIT_LABELS
-  public readonly maxDepth = ADVANCED_SEARCH_MAX_DEPTH
-  public readonly maxAtoms = ADVANCED_SEARCH_MAX_ATOMS
-
-  @Output()
-  public queryApplied = new EventEmitter<string>()
-
-  public root: AdvancedSearchQueryGroup = this.emptyRoot()
-
-  // True when the query in the search box uses syntax the editor cannot show
-  public unreadable: boolean = false
-
-  private _query: string = ''
-
-  @Input()
-  set query(query: string) {
-    this._query = query ?? ''
-    const parsed = parseAdvancedSearchQuery(this._query)
-    this.unreadable = !!this._query.trim() && !parsed
-    this.root = parsed ?? this.emptyRoot()
-  }
-
-  get query(): string {
-    return this._query
-  }
-
-  constructor() {
-    super()
-    this.loading.set(false)
-  }
-
-  // Stable ids for the radio groups, without putting them in the query model
-  private ids = new WeakMap<object, number>()
-  private nextId = 0
-
-  public idFor(element: AdvancedSearchQueryElement): number {
-    if (!this.ids.has(element)) {
-      this.ids.set(element, this.nextId++)
-    }
-    return this.ids.get(element)
-  }
-
-  private emptyRoot(): AdvancedSearchQueryGroup {
-    return {
-      type: AdvancedSearchQueryElementType.Group,
-      operator: AdvancedSearchLogicalOperator.And,
-      children: [this.newAtom()],
-    }
-  }
-
-  private newAtom(): AdvancedSearchQueryAtom {
-    return {
-      type: AdvancedSearchQueryElementType.Atom,
-      field: AdvancedSearchField.Content,
-      operator: AdvancedSearchOperator.AllWords,
-      value: '',
-    }
-  }
-
-  public get generatedQuery(): string {
-    return serializeAdvancedSearchQuery(this.root)
-  }
-
-  public kindOf(atom: AdvancedSearchQueryAtom): AdvancedSearchFieldKind {
-    return ADVANCED_SEARCH_FIELD_KINDS[atom.field]
-  }
-
-  public operatorsFor(atom: AdvancedSearchQueryAtom): AdvancedSearchOperator[] {
-    return ADVANCED_SEARCH_OPERATORS_BY_KIND[this.kindOf(atom)]
-  }
-
-  public operatorLabel(
-    atom: AdvancedSearchQueryAtom,
-    operator: AdvancedSearchOperator
-  ): string {
-    return this.kindOf(atom) === AdvancedSearchFieldKind.Date
-      ? (ADVANCED_SEARCH_DATE_OPERATOR_LABELS[operator] ??
-          ADVANCED_SEARCH_OPERATOR_LABELS[operator])
-      : ADVANCED_SEARCH_OPERATOR_LABELS[operator]
-  }
-
-  public placeholderFor(atom: AdvancedSearchQueryAtom): string {
-    switch (atom.operator) {
-      case AdvancedSearchOperator.Phrase:
-        return $localize`phrase`
-      case AdvancedSearchOperator.StartsWith:
-        return $localize`beginning of a word`
-      default:
-        return $localize`words`
-    }
-  }
-
-  public onFieldChange(atom: AdvancedSearchQueryAtom) {
-    // Keep the condition only if the new field still offers it
-    if (!this.operatorsFor(atom).includes(atom.operator)) {
-      atom.operator = this.operatorsFor(atom)[0]
-    }
-    this.clearValues(atom)
-  }
-
-  public onOperatorChange(atom: AdvancedSearchQueryAtom) {
-    this.clearValues(atom)
-  }
-
-  private clearValues(atom: AdvancedSearchQueryAtom) {
-    atom.value = ''
-    atom.valueTo = undefined
-    atom.unit =
-      atom.operator === AdvancedSearchOperator.WithinLast
-        ? AdvancedSearchDateUnit.Day
-        : undefined
-  }
-
-  public addAtom(group: AdvancedSearchQueryGroup) {
-    group.children.push(this.newAtom())
-  }
-
-  public addGroup(group: AdvancedSearchQueryGroup) {
-    group.children.push({
-      type: AdvancedSearchQueryElementType.Group,
-      operator: AdvancedSearchLogicalOperator.Or,
-      children: [this.newAtom()],
-    })
-  }
-
-  public remove(
-    parent: AdvancedSearchQueryGroup,
-    element: AdvancedSearchQueryElement
-  ) {
-    parent.children = parent.children.filter((child) => child !== element)
-  }
-
-  public startOver() {
-    this.unreadable = false
-    this.root = this.emptyRoot()
-  }
-
-  public apply() {
-    this.queryApplied.emit(this.generatedQuery)
-    this.activeModal.close()
-  }
-
-  public cancel() {
-    this.activeModal.close()
-  }
-}

src-ui/src/app/components/common/input/switch/switch.component.html

@@ -1,6 +1,6 @@
-<div [class.mb-3]="!compact">
+<div class="mb-3">
-  <div [class.row]="!compact">
+  <div class="row">
-    @if (!horizontal && !compact) {
+    @if (!horizontal) {
       <div class="d-flex align-items-center position-relative hidden-button-container col-md-3">
         <label class="form-label" [for]="inputId" [ngbTooltip]="showUnsetNote && isUnset ? tipContent: null" placement="end">
           {{title}}
@@ -17,8 +17,8 @@
     }
     <div [ngClass]="{'align-items-center': horizontal, 'd-flex': horizontal}">
       <div class="form-check form-switch">
-        <input #inputField type="checkbox" class="form-check-input" [id]="inputId" [(ngModel)]="value" [ngModelOptions]="{standalone: true}" (change)="onChange(value)" (blur)="onTouched()" [disabled]="disabled" [attr.aria-label]="compact ? title : null">
+        <input #inputField type="checkbox" class="form-check-input" [id]="inputId" [(ngModel)]="value" [ngModelOptions]="{standalone: true}" (change)="onChange(value)" (blur)="onTouched()" [disabled]="disabled">
-        @if (horizontal && !compact) {
+        @if (horizontal) {
           <label class="form-check-label" [class.text-muted]="showUnsetNote && isUnset" [for]="inputId" [ngbTooltip]="showUnsetNote && isUnset ? tipContent: null" placement="end">
             {{title}}
             @if (showUnsetNote && isUnset) {

src-ui/src/app/components/common/input/switch/switch.component.spec.ts

@@ -48,14 +48,4 @@ describe('SwitchComponent', () => {
     component.value = undefined
     expect(component.isUnset).toBeTruthy()
   })
-
-  it('should support a compact layout', () => {
-    component.compact = true
-    component.title = 'Test switch'
-    fixture.detectChanges()
-
-    expect(fixture.nativeElement.querySelector('.mb-3')).toBeNull()
-    expect(fixture.nativeElement.querySelector('.row')).toBeNull()
-    expect(input.getAttribute('aria-label')).toEqual('Test switch')
-  })
 })

src-ui/src/app/components/common/input/switch/switch.component.ts

@@ -25,9 +25,6 @@ export class SwitchComponent extends AbstractInputComponent<boolean> {
   @Input()
   showUnsetNote: boolean = false
 
-  @Input()
-  compact: boolean = false
-
   constructor() {
     super()
   }

src-ui/src/app/components/common/share-link-bundle-manage-dialog/share-link-bundle-manage-dialog.component.html

@@ -1,22 +1,38 @@
-<div class="border border-top-0 rounded-bottom p-3">
+<div class="modal-header">
+  <h4 class="modal-title">{{ title }}</h4>
+  <button type="button" class="btn-close" aria-label="Close" (click)="close()"></button>
+</div>
+
+<div class="modal-body">
+  @if (loading()) {
+    <div class="d-flex align-items-center gap-2">
+      <div class="spinner-border spinner-border-sm" role="status"></div>
+      <span i18n>Loading share link bundles…</span>
+    </div>
+  }
   @if (!loading() && error()) {
     <div class="alert alert-danger mb-0" role="alert">
       {{ error() }}
     </div>
   }
   @if (!loading() && !error()) {
+    <div class="d-flex justify-content-between align-items-center mb-2">
+      <p class="mb-0 text-muted small">
+        <ng-container i18n>Status updates every few seconds while bundles are being prepared.</ng-container>
+      </p>
+    </div>
     @if (bundles().length === 0) {
       <p class="mb-0 text-muted fst-italic" i18n>No share link bundles currently exist.</p>
     }
     @if (bundles().length > 0) {
       <div class="table-responsive">
-        <table class="table table-sm align-middle mb-0 bg-body">
+        <table class="table table-sm align-middle mb-0">
           <thead>
             <tr>
-              <th scope="col" class="fw-normal" pngxSortable="created" [currentSortField]="sortField()" [currentSortReverse]="sortReverse()" (sort)="onSort($event)" i18n>Created</th>
+              <th scope="col" i18n>Created</th>
-              <th scope="col" class="fw-normal" pngxSortable="status" [currentSortField]="sortField()" [currentSortReverse]="sortReverse()" (sort)="onSort($event)" i18n>Status</th>
+              <th scope="col" i18n>Status</th>
               <th scope="col" i18n>Size</th>
-              <th scope="col" class="fw-normal" pngxSortable="expiration" [currentSortField]="sortField()" [currentSortReverse]="sortReverse()" (sort)="onSort($event)" i18n>Expires</th>
+              <th scope="col" i18n>Expires</th>
               <th scope="col" i18n>Documents</th>
               <th scope="col" i18n>File version</th>
               <th scope="col" class="text-end" i18n>Actions</th>
@@ -80,9 +96,6 @@
                 <td>
                   @if (bundle.expiration) {
                     {{ bundle.expiration | date: 'short' }}
-                    @if (isExpired(bundle.expiration)) {
-                      <span class="badge text-bg-danger ms-2" i18n>Expired</span>
-                    }
                   }
                   @if (!bundle.expiration) {
                     <span i18n>Never</span>
@@ -91,49 +104,42 @@
                 <td>{{ bundle.document_count }}</td>
                 <td>{{ fileVersionLabel(bundle.file_version) }}</td>
                 <td class="text-end">
-                  <div class="d-inline-block position-relative">
+                  <div class="btn-group btn-group-sm">
-                    <span
+                    <button
-                      class="badge bg-primary small fade position-absolute top-50 end-100 translate-middle-y me-2 pe-none z-3 text-nowrap"
+                      type="button"
-                      [class.show]="copiedSlug() === bundle.slug"
+                      class="btn btn-outline-primary"
-                      i18n
+                      [disabled]="bundle.status !== statuses.Ready"
-                    >Copied!</span>
+                      (click)="copy(bundle)"
-                    <div class="btn-group btn-group-sm">
+                      title="Copy share link"
+                      i18n-title
+                    >
+                      @if (copiedSlug() === bundle.slug) {
+                        <i-bs name="clipboard-check"></i-bs>
+                      }
+                      @if (copiedSlug() !== bundle.slug) {
+                        <i-bs name="clipboard"></i-bs>
+                      }
+                      <span class="visually-hidden" i18n>Copy share link</span>
+                    </button>
+                    @if (bundle.status === statuses.Failed) {
                       <button
                         type="button"
-                        class="btn btn-outline-primary"
+                        class="btn btn-outline-warning"
-                        [disabled]="bundle.status !== statuses.Ready"
-                        (click)="copy(bundle)"
-                        title="Copy share link"
-                        i18n-title
-                      >
-                        @if (copiedSlug() === bundle.slug) {
-                          <i-bs name="clipboard-check"></i-bs>
-                        }
-                        @if (copiedSlug() !== bundle.slug) {
-                          <i-bs name="clipboard"></i-bs>
-                        }
-                        <span class="visually-hidden" i18n>Copy share link</span>
-                      </button>
-                      @if (bundle.status === statuses.Failed) {
-                        <button
-                          type="button"
-                          class="btn btn-outline-warning"
-                          [disabled]="loading()"
-                          (click)="retry(bundle)"
-                        >
-                          <i-bs name="arrow-clockwise"></i-bs>
-                          <span class="visually-hidden" i18n>Retry</span>
-                        </button>
-                      }
-                      <pngx-confirm-button
-                        buttonClasses="btn btn-sm btn-outline-danger"
                         [disabled]="loading()"
-                        (confirm)="delete(bundle)"
+                        (click)="retry(bundle)"
-                        iconName="trash"
                       >
-                        <span class="visually-hidden" i18n>Delete share link bundle</span>
+                        <i-bs name="arrow-clockwise"></i-bs>
-                      </pngx-confirm-button>
+                        <span class="visually-hidden" i18n>Retry</span>
-                    </div>
+                      </button>
+                    }
+                    <pngx-confirm-button
+                      buttonClasses="btn btn-sm btn-outline-danger"
+                      [disabled]="loading()"
+                      (confirm)="delete(bundle)"
+                      iconName="trash"
+                    >
+                      <span class="visually-hidden" i18n>Delete share link bundle</span>
+                    </pngx-confirm-button>
                   </div>
                 </td>
               </tr>
@@ -141,32 +147,10 @@
           </tbody>
         </table>
       </div>
-      <div class="d-flex flex-wrap justify-content-end align-items-center gap-3 mt-3 ms-auto">
-        <div class="d-flex flex-wrap justify-content-end align-items-center gap-3">
-          <div class="d-flex align-items-center">
-            <label class="small text-muted me-2" for="shareLinkBundlePageSize" i18n>Show:</label>
-            <select id="shareLinkBundlePageSize" class="form-select form-select-sm w-auto" [(ngModel)]="pageSize">
-              <option [ngValue]="25">25</option>
-              <option [ngValue]="50">50</option>
-              <option [ngValue]="100">100</option>
-            </select>
-            <span class="small text-muted ms-2 d-none d-md-inline" i18n>per page</span>
-          </div>
-          @if (total() > pageSize) {
-            <ngb-pagination
-              class="mb-0"
-              [pageSize]="pageSize"
-              [collectionSize]="total()"
-              [page]="page()"
-              [maxSize]="5"
-              (pageChange)="setPage($event)"
-              size="sm"
-              aria-label="Share link bundles pagination"
-              i18n-aria-label
-            ></ngb-pagination>
-          }
-        </div>
-      </div>
     }
   }
 </div>
+
+<div class="modal-footer">
+  <button type="button" class="btn btn-outline-secondary btn-sm" (click)="close()" i18n>Close</button>
+</div>

src-ui/src/app/components/common/share-link-bundle-manage-dialog/share-link-bundle-manage-dialog.component.spec.ts

@@ -1,5 +1,6 @@
 import { Clipboard } from '@angular/cdk/clipboard'
 import { ComponentFixture, TestBed } from '@angular/core/testing'
+import { NgbActiveModal } from '@ng-bootstrap/ng-bootstrap'
 import { NgxBootstrapIconsModule, allIcons } from 'ngx-bootstrap-icons'
 import { of, throwError } from 'rxjs'
 import { FileVersion } from 'src/app/data/share-link'
@@ -7,15 +8,13 @@ import {
   ShareLinkBundleStatus,
   ShareLinkBundleSummary,
 } from 'src/app/data/share-link-bundle'
-import { SETTINGS_KEYS } from 'src/app/data/ui-settings'
 import { ShareLinkBundleService } from 'src/app/services/rest/share-link-bundle.service'
-import { SettingsService } from 'src/app/services/settings.service'
 import { ToastService } from 'src/app/services/toast.service'
 import { environment } from 'src/environments/environment'
-import { ShareLinkBundleListComponent } from './share-link-bundle-list.component'
+import { ShareLinkBundleManageDialogComponent } from './share-link-bundle-manage-dialog.component'
 
 class MockShareLinkBundleService {
-  list = jest.fn()
+  listAllBundles = jest.fn()
   delete = jest.fn()
   rebuildBundle = jest.fn()
 }
@@ -25,12 +24,13 @@ class MockToastService {
   showError = jest.fn()
 }
 
-describe('ShareLinkBundleListComponent', () => {
+describe('ShareLinkBundleManageDialogComponent', () => {
-  let component: ShareLinkBundleListComponent
+  let component: ShareLinkBundleManageDialogComponent
-  let fixture: ComponentFixture<ShareLinkBundleListComponent>
+  let fixture: ComponentFixture<ShareLinkBundleManageDialogComponent>
   let service: MockShareLinkBundleService
   let toastService: MockToastService
   let clipboard: Clipboard
+  let activeModal: NgbActiveModal
   let originalApiBaseUrl: string
 
   beforeEach(() => {
@@ -38,24 +38,26 @@ describe('ShareLinkBundleListComponent', () => {
     toastService = new MockToastService()
     originalApiBaseUrl = environment.apiBaseUrl
 
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     service.delete.mockReturnValue(of(true))
     service.rebuildBundle.mockReturnValue(of(sampleBundle()))
 
     TestBed.configureTestingModule({
       imports: [
-        ShareLinkBundleListComponent,
+        ShareLinkBundleManageDialogComponent,
         NgxBootstrapIconsModule.pick(allIcons),
       ],
       providers: [
+        NgbActiveModal,
         { provide: ShareLinkBundleService, useValue: service },
         { provide: ToastService, useValue: toastService },
       ],
     })
 
-    fixture = TestBed.createComponent(ShareLinkBundleListComponent)
+    fixture = TestBed.createComponent(ShareLinkBundleManageDialogComponent)
     component = fixture.componentInstance
     clipboard = TestBed.inject(Clipboard)
+    activeModal = TestBed.inject(NgbActiveModal)
   })
 
   afterEach(() => {
@@ -82,28 +84,28 @@ describe('ShareLinkBundleListComponent', () => {
   it('loads bundles on init and polls periodically', () => {
     jest.useFakeTimers()
     const bundles = [sampleBundle({ status: ShareLinkBundleStatus.Ready })]
-    service.list.mockReset()
+    service.listAllBundles.mockReset()
-    service.list
+    service.listAllBundles
-      .mockReturnValueOnce(of({ count: bundles.length, results: bundles }))
+      .mockReturnValueOnce(of(bundles))
-      .mockReturnValue(of({ count: bundles.length, results: bundles }))
+      .mockReturnValue(of(bundles))
 
     fixture.detectChanges()
 
-    expect(service.list).toHaveBeenCalledWith(1, 25, 'created', true)
+    expect(service.listAllBundles).toHaveBeenCalledTimes(1)
     expect(component.bundles()).toEqual(bundles)
     expect(component.loading()).toBe(false)
     expect(component.error()).toBeNull()
 
     jest.advanceTimersByTime(5000)
-    expect(service.list).toHaveBeenCalledTimes(2)
+    expect(service.listAllBundles).toHaveBeenCalledTimes(2)
   })
 
   it('handles errors when loading bundles', () => {
     jest.useFakeTimers()
-    service.list.mockReset()
+    service.listAllBundles.mockReset()
-    service.list
+    service.listAllBundles
       .mockReturnValueOnce(throwError(() => new Error('load fail')))
-      .mockReturnValue(of({ count: 0, results: [] }))
+      .mockReturnValue(of([]))
 
     fixture.detectChanges()
 
@@ -112,57 +114,7 @@ describe('ShareLinkBundleListComponent', () => {
     expect(component.loading()).toBe(false)
 
     jest.advanceTimersByTime(5000)
-    expect(service.list).toHaveBeenCalledTimes(2)
+    expect(service.listAllBundles).toHaveBeenCalledTimes(2)
-  })
-
-  it('loads another page', () => {
-    fixture.detectChanges()
-
-    component.setPage(2)
-
-    expect(service.list).toHaveBeenLastCalledWith(2, 25, 'created', true)
-  })
-
-  it('sorts bundles and returns to the first page', () => {
-    fixture.detectChanges()
-    component.page.set(2)
-
-    component.onSort({ column: 'status', reverse: false })
-
-    expect(component.page()).toBe(1)
-    expect(service.list).toHaveBeenLastCalledWith(1, 25, 'status', false)
-  })
-
-  it('marks expired share link bundles', () => {
-    service.list.mockReturnValue(
-      of({
-        count: 1,
-        results: [sampleBundle({ expiration: '2000-01-01T00:00:00.000Z' })],
-      })
-    )
-
-    fixture.detectChanges()
-
-    expect(fixture.nativeElement.textContent).toContain('Expired')
-  })
-
-  it('stores a changed page size and reloads from the first page', () => {
-    fixture.detectChanges()
-    const settingsService = TestBed.inject(SettingsService)
-    jest
-      .spyOn(settingsService, 'get')
-      .mockReturnValueOnce({ share_link_bundles: 25 })
-    const setSpy = jest.spyOn(settingsService, 'set')
-    jest.spyOn(settingsService, 'storeSettings').mockReturnValue(of({}))
-    component.page.set(2)
-
-    component.pageSize = 100
-
-    expect(setSpy).toHaveBeenCalledWith(SETTINGS_KEYS.OBJECT_LIST_SIZES, {
-      share_link_bundles: 100,
-    })
-    expect(component.page()).toBe(1)
-    expect(service.list).toHaveBeenLastCalledWith(1, 100, 'created', true)
   })
 
   it('copies bundle links when ready', () => {
@@ -174,24 +126,16 @@ describe('ShareLinkBundleListComponent', () => {
       slug: 'ready-slug',
       status: ShareLinkBundleStatus.Ready,
     })
-    component.bundles.set([readyBundle])
-    fixture.detectChanges()
     component.copy(readyBundle)
 
     expect(clipboard.copy).toHaveBeenCalledWith(
       component.getShareUrl(readyBundle)
     )
     expect(component.copiedSlug()).toBe('ready-slug')
-    expect(toastService.showInfo).not.toHaveBeenCalled()
+    expect(toastService.showInfo).toHaveBeenCalled()
-    fixture.detectChanges()
-    expect(
-      fixture.nativeElement.querySelector('.badge.show').textContent
-    ).toContain('Copied!')
 
     jest.advanceTimersByTime(3000)
     expect(component.copiedSlug()).toBeNull()
-    fixture.detectChanges()
-    expect(fixture.nativeElement.querySelector('.badge.show')).toBeNull()
   })
 
   it('ignores copy requests for non-ready bundles', () => {
@@ -202,7 +146,7 @@ describe('ShareLinkBundleListComponent', () => {
   })
 
   it('deletes bundles and refreshes list', () => {
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     service.delete.mockReturnValue(of(true))
 
     fixture.detectChanges()
@@ -213,12 +157,12 @@ describe('ShareLinkBundleListComponent', () => {
     expect(toastService.showInfo).toHaveBeenCalledWith(
       expect.stringContaining('deleted.')
     )
-    expect(service.list).toHaveBeenCalledTimes(2)
+    expect(service.listAllBundles).toHaveBeenCalledTimes(2)
     expect(component.loading()).toBe(false)
   })
 
   it('handles delete errors gracefully', () => {
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     service.delete.mockReturnValue(throwError(() => new Error('delete fail')))
 
     fixture.detectChanges()
@@ -230,7 +174,7 @@ describe('ShareLinkBundleListComponent', () => {
   })
 
   it('retries bundle build and replaces existing entry', () => {
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     const updated = sampleBundle({ status: ShareLinkBundleStatus.Ready })
     service.rebuildBundle.mockReturnValue(of(updated))
 
@@ -245,7 +189,7 @@ describe('ShareLinkBundleListComponent', () => {
   })
 
   it('adds new bundle when retry returns unknown entry', () => {
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     service.rebuildBundle.mockReturnValue(
       of(sampleBundle({ id: 99, slug: 'new-slug' }))
     )
@@ -259,7 +203,7 @@ describe('ShareLinkBundleListComponent', () => {
   })
 
   it('handles retry errors', () => {
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     service.rebuildBundle.mockReturnValue(throwError(() => new Error('fail')))
 
     fixture.detectChanges()
@@ -269,8 +213,8 @@ describe('ShareLinkBundleListComponent', () => {
     expect(toastService.showError).toHaveBeenCalled()
   })
 
-  it('maps status and file version helpers', () => {
+  it('maps helpers and closes dialog', () => {
-    service.list.mockReturnValue(of({ count: 0, results: [] }))
+    service.listAllBundles.mockReturnValue(of([]))
     fixture.detectChanges()
 
     expect(component.statusLabel(ShareLinkBundleStatus.Processing)).toContain(
@@ -283,5 +227,9 @@ describe('ShareLinkBundleListComponent', () => {
     environment.apiBaseUrl = 'https://example.com/api/'
     const url = component.getShareUrl(sampleBundle({ slug: 'sluggy' }))
     expect(url).toBe('https://example.com/share/sluggy')
+
+    const closeSpy = jest.spyOn(activeModal, 'close')
+    component.close()
+    expect(closeSpy).toHaveBeenCalled()
   })
 })

src-ui/src/app/components/common/share-link-bundle-manage-dialog/share-link-bundle-manage-dialog.component.ts

@@ -1,11 +1,7 @@
 import { Clipboard } from '@angular/cdk/clipboard'
 import { CommonModule } from '@angular/common'
 import { Component, OnDestroy, OnInit, inject, signal } from '@angular/core'
-import { FormsModule } from '@angular/forms'
+import { NgbActiveModal, NgbPopoverModule } from '@ng-bootstrap/ng-bootstrap'
-import {
-  NgbPaginationModule,
-  NgbPopoverModule,
-} from '@ng-bootstrap/ng-bootstrap'
 import { NgxBootstrapIconsModule } from 'ngx-bootstrap-icons'
 import { Subject, catchError, of, switchMap, takeUntil, timer } from 'rxjs'
 import { FileVersion } from 'src/app/data/share-link'
@@ -15,77 +11,42 @@ import {
   ShareLinkBundleStatus,
   ShareLinkBundleSummary,
 } from 'src/app/data/share-link-bundle'
-import { SETTINGS_KEYS } from 'src/app/data/ui-settings'
-import {
-  SortEvent,
-  SortableDirective,
-} from 'src/app/directives/sortable.directive'
 import { FileSizePipe } from 'src/app/pipes/file-size.pipe'
 import { ShareLinkBundleService } from 'src/app/services/rest/share-link-bundle.service'
-import { SettingsService } from 'src/app/services/settings.service'
 import { ToastService } from 'src/app/services/toast.service'
 import { environment } from 'src/environments/environment'
-import { ConfirmButtonComponent } from 'src/app/components/common/confirm-button/confirm-button.component'
+import { LoadingComponentWithPermissions } from '../../loading-component/loading.component'
-import { LoadingComponentWithPermissions } from 'src/app/components/loading-component/loading.component'
+import { ConfirmButtonComponent } from '../confirm-button/confirm-button.component'
 
 @Component({
-  selector: 'pngx-share-link-bundle-list',
+  selector: 'pngx-share-link-bundle-manage-dialog',
-  templateUrl: './share-link-bundle-list.component.html',
+  templateUrl: './share-link-bundle-manage-dialog.component.html',
-  styleUrls: ['./share-link-bundle-list.component.scss'],
+  styleUrls: ['./share-link-bundle-manage-dialog.component.scss'],
   imports: [
     ConfirmButtonComponent,
     CommonModule,
-    FormsModule,
-    NgbPaginationModule,
     NgbPopoverModule,
     NgxBootstrapIconsModule,
-    SortableDirective,
     FileSizePipe,
   ],
 })
-export class ShareLinkBundleListComponent
+export class ShareLinkBundleManageDialogComponent
   extends LoadingComponentWithPermissions
   implements OnInit, OnDestroy
 {
+  private readonly activeModal = inject(NgbActiveModal)
   private readonly shareLinkBundleService = inject(ShareLinkBundleService)
-  private readonly settingsService = inject(SettingsService)
   private readonly toastService = inject(ToastService)
   private readonly clipboard = inject(Clipboard)
 
+  title = $localize`Share link bundles`
   readonly bundles = signal<ShareLinkBundleSummary[]>([])
   readonly error = signal<string | null>(null)
   readonly copiedSlug = signal<string | null>(null)
-  readonly total = signal(0)
-  readonly page = signal(1)
-  readonly sortField = signal('created')
-  readonly sortReverse = signal(true)
 
   readonly statuses = ShareLinkBundleStatus
   readonly fileVersions = FileVersion
 
-  get pageSize(): number {
-    return (
-      this.settingsService.get(SETTINGS_KEYS.OBJECT_LIST_SIZES)
-        ?.share_link_bundles || 25
-    )
-  }
-
-  set pageSize(pageSize: number) {
-    this.settingsService.set(SETTINGS_KEYS.OBJECT_LIST_SIZES, {
-      ...this.settingsService.get(SETTINGS_KEYS.OBJECT_LIST_SIZES),
-      share_link_bundles: pageSize,
-    })
-    this.settingsService.storeSettings().subscribe({
-      next: () => {
-        this.page.set(1)
-        this.triggerRefresh(false)
-      },
-      error: (error) => {
-        this.toastService.showError($localize`Error saving settings`, error)
-      },
-    })
-  }
-
   private readonly refresh$ = new Subject<boolean>()
 
   ngOnInit(): void {
@@ -96,33 +57,25 @@ export class ShareLinkBundleListComponent
             this.loading.set(true)
           }
           this.error.set(null)
-          return this.shareLinkBundleService
+          return this.shareLinkBundleService.listAllBundles().pipe(
-            .list(
+            catchError((error) => {
-              this.page(),
+              if (!silent) {
-              this.pageSize,
+                this.loading.set(false)
-              this.sortField(),
+              }
-              this.sortReverse()
+              this.error.set($localize`Failed to load share link bundles.`)
-            )
+              this.toastService.showError(
-            .pipe(
+                $localize`Error retrieving share link bundles.`,
-              catchError((error) => {
+                error
-                if (!silent) {
+              )
-                  this.loading.set(false)
+              return of(null)
-                }
+            })
-                this.error.set($localize`Failed to load share link bundles.`)
+          )
-                this.toastService.showError(
-                  $localize`Error retrieving share link bundles.`,
-                  error
-                )
-                return of(null)
-              })
-            )
         }),
         takeUntil(this.unsubscribeNotifier)
       )
       .subscribe((results) => {
         if (results) {
-          this.bundles.set(results.results)
+          this.bundles.set(results)
-          this.total.set(results.count)
           this.copiedSlug.set(null)
         }
         this.loading.set(false)
@@ -145,18 +98,6 @@ export class ShareLinkBundleListComponent
     }`
   }
 
-  setPage(page: number): void {
-    this.page.set(page)
-    this.triggerRefresh(false)
-  }
-
-  onSort(event: SortEvent): void {
-    this.sortField.set(event.column || 'created')
-    this.sortReverse.set(event.column ? event.reverse : true)
-    this.page.set(1)
-    this.triggerRefresh(false)
-  }
-
   copy(bundle: ShareLinkBundleSummary): void {
     if (bundle.status !== ShareLinkBundleStatus.Ready) {
       return
@@ -167,6 +108,7 @@ export class ShareLinkBundleListComponent
       setTimeout(() => {
         this.copiedSlug.set(null)
       }, 3000)
+      this.toastService.showInfo($localize`Share link copied to clipboard.`)
     }
   }
 
@@ -175,9 +117,6 @@ export class ShareLinkBundleListComponent
     this.loading.set(true)
     this.shareLinkBundleService.delete(bundle).subscribe({
       next: () => {
-        if (this.bundles().length === 1 && this.page() > 1) {
-          this.page.update((page) => page - 1)
-        }
         this.toastService.showInfo($localize`Share link bundle deleted.`)
         this.triggerRefresh(false)
       },
@@ -214,8 +153,8 @@ export class ShareLinkBundleListComponent
     return SHARE_LINK_BUNDLE_FILE_VERSION_LABELS[version] ?? version
   }
 
-  isExpired(expiration?: string): boolean {
+  close(): void {
-    return !!expiration && Date.parse(expiration) <= Date.now()
+    this.activeModal.close()
   }
 
   private replaceBundle(updated: ShareLinkBundleSummary): void {

src-ui/src/app/components/document-detail/document-detail.component.spec.ts

@@ -28,9 +28,8 @@ import { Subject, of, throwError } from 'rxjs'
 import { routes } from 'src/app/app-routing.module'
 import { Correspondent } from 'src/app/data/correspondent'
 import { CustomFieldDataType } from 'src/app/data/custom-field'
-import { CustomFieldInstance } from 'src/app/data/custom-field-instance'
 import { DataType } from 'src/app/data/datatype'
-import { Document, DocumentVersionInfo } from 'src/app/data/document'
+import { Document } from 'src/app/data/document'
 import { DocumentType } from 'src/app/data/document-type'
 import {
   FILTER_CORRESPONDENT,
@@ -101,18 +100,13 @@ const doc: Document = {
   custom_fields: [
     {
       field: 0,
+      document: 3,
+      created: new Date(),
       value: 'custom foo bar',
     },
-  ] as CustomFieldInstance[],
+  ],
 }
 
-// Newest first, as the API returns them: 12 is the latest, 3 is the root
-const docVersions: DocumentVersionInfo[] = [
-  { id: 12, is_root: false },
-  { id: 10, is_root: false },
-  { id: doc.id, is_root: true },
-]
-
 const customFields = [
   {
     id: 0,
@@ -2051,208 +2045,6 @@ describe('DocumentDetailComponent', () => {
     expect(saveSpy).toHaveBeenCalled()
   })
 
-  it('selectVersion should use the version content as the baseline and ignore stale responses', () => {
-    initNormally()
-    const version10Content = new Subject<Document>()
-    jest
-      .spyOn(documentService, 'get')
-      .mockReturnValueOnce(version10Content)
-      .mockReturnValueOnce(of({ content: 'version 12 content' } as Document))
-    const version10Metadata = new Subject<any>()
-    jest
-      .spyOn(documentService, 'getMetadata')
-      .mockReturnValueOnce(version10Metadata)
-      .mockReturnValueOnce(of({ lang: 'de' }))
-
-    component.selectVersion(10)
-    component.selectVersion(12)
-    version10Content.next({ content: 'version 10 content' } as Document)
-    version10Metadata.next({ lang: 'en' })
-
-    expect(component.documentForm.get('content').value).toEqual(
-      'version 12 content'
-    )
-    expect(component.store.value.content).toEqual('version 12 content')
-    expect(component.metadata().lang).toEqual('de')
-    expect(
-      httpTestingController.expectOne(component.previewUrl()).cancelled
-    ).toBeFalsy()
-    expect(
-      httpTestingController.match((req) => req.url.includes('version=10'))[0]
-        ?.cancelled
-    ).toBeTruthy()
-  })
-
-  it('should confirm before discarding unsaved content edits when switching versions', () => {
-    initNormally()
-    component.document().versions = docVersions
-    jest
-      .spyOn(documentService, 'get')
-      .mockImplementation((id, versionID) =>
-        of({ content: `version ${versionID} content` } as Document)
-      )
-    let openModal: NgbModalRef
-    modalService.activeInstances.subscribe((modals) => (openModal = modals[0]))
-    const modalSpy = jest.spyOn(modalService, 'open')
-
-    // shared fields carry over between versions, so no confirmation
-    component.documentForm.get('title').setValue('Edited title')
-    component.documentForm.get('title').markAsDirty()
-    component.documentForm.get('content').markAsDirty()
-    component.onVersionSelected(12)
-    expect(modalSpy).not.toHaveBeenCalled()
-    expect(component.selectedVersionId()).toEqual(12)
-
-    component.documentForm.get('content').setValue('edited content')
-    component.documentForm.get('content').markAsDirty()
-    component.onVersionSelected(12) // already selected, nothing to do
-    expect(modalSpy).not.toHaveBeenCalled()
-    component.onVersionSelected(10)
-    expect(modalSpy).toHaveBeenCalledWith(
-      ConfirmDialogComponent,
-      expect.anything()
-    )
-    openModal.componentInstance.cancel()
-    expect(component.selectedVersionId()).toEqual(12)
-    expect(component.documentForm.get('content').value).toEqual(
-      'edited content'
-    )
-
-    component.onVersionSelected(10)
-    openModal.componentInstance.confirmClicked.emit()
-    expect(component.selectedVersionId()).toEqual(10)
-    expect(component.documentForm.get('content').value).toEqual(
-      'version 10 content'
-    )
-    expect(component.documentForm.get('content').dirty).toBeFalsy()
-    expect(component.documentForm.get('title').value).toEqual('Edited title')
-  })
-
-  it('should save unsaved content edits to the current version before switching, and stay if that fails', () => {
-    initNormally()
-    component.document().versions = docVersions
-    component.selectedVersionId.set(12)
-    jest
-      .spyOn(documentService, 'get')
-      .mockReturnValue(of({ content: 'version 10 content' } as Document))
-    const savedDoc = new Subject<Document>()
-    const patchSpy = jest
-      .spyOn(documentService, 'patch')
-      .mockReturnValueOnce(throwError(() => new Error('failed to save')))
-      .mockReturnValueOnce(savedDoc)
-    const modalSpy = jest.spyOn(modalService, 'open')
-    component.documentForm.get('content').setValue('edited content')
-    component.documentForm.get('content').markAsDirty()
-
-    component.onVersionSelected(10)
-    let modal: NgbModalRef = modalSpy.mock.results[0].value
-    const closeSpy = jest.spyOn(modal, 'close')
-    modal.componentInstance.alternativeClicked.emit()
-    expect(closeSpy).toHaveBeenCalled()
-    expect(component.selectedVersionId()).toEqual(12)
-    expect(component.documentForm.get('content').value).toEqual(
-      'edited content'
-    )
-
-    component.onVersionSelected(10)
-    modal = modalSpy.mock.results[1].value
-    modal.componentInstance.alternativeClicked.emit()
-    expect(patchSpy).toHaveBeenLastCalledWith(
-      expect.objectContaining({ content: 'edited content' }),
-      12
-    )
-    component.onVersionSelected(doc.id) // ignored while saving
-    expect(modalSpy).toHaveBeenCalledTimes(2)
-    savedDoc.next(doc)
-    expect(component.selectedVersionId()).toEqual(10)
-    expect(component.documentForm.get('content').value).toEqual(
-      'version 10 content'
-    )
-  })
-
-  it('should switch without confirmation when the selected version was deleted, even while saving', () => {
-    initNormally()
-    component.document().versions = docVersions
-    component.selectedVersionId.set(10)
-    jest
-      .spyOn(documentService, 'get')
-      .mockReturnValue(of({ content: 'version 12 content' } as Document))
-    const modalSpy = jest.spyOn(modalService, 'open')
-    component.documentForm.get('content').setValue('edited content')
-    component.documentForm.get('content').markAsDirty()
-    component.networkActive.set(true)
-
-    // the version dropdown emits this after deleting the selected version
-    component.onVersionsUpdated(docVersions.filter((v) => v.id !== 10))
-    component.onVersionSelected(12)
-
-    expect(modalSpy).not.toHaveBeenCalled()
-    expect(component.selectedVersionId()).toEqual(12)
-    expect(component.documentForm.get('content').value).toEqual(
-      'version 12 content'
-    )
-  })
-
-  it('should restore the selected version and its unsaved content when returning to a document', () => {
-    initNormally()
-    const openDoc = component.document()
-    openDoc.versions = docVersions
-    jest.spyOn(openDocumentsService, 'getOpenDocument').mockReturnValue(openDoc)
-    jest
-      .spyOn(documentService, 'get')
-      .mockImplementation((id, versionID) =>
-        of(
-          (versionID
-            ? { content: `version ${versionID} content` }
-            : { ...doc, versions: docVersions }) as Document
-        )
-      )
-    component.selectVersion(10)
-    // an edit that happens to match the latest version's content
-    component.documentForm.get('content').setValue(doc.content)
-    openDoc.__changedFields = ['content']
-
-    component['loadDocument'](doc.id)
-
-    expect(component.selectedVersionId()).toEqual(10)
-    expect(component.documentForm.get('content').value).toEqual(doc.content)
-    expect(openDocumentsService.isDirty(openDoc)).toBeTruthy()
-    const patchSpy = jest
-      .spyOn(documentService, 'patch')
-      .mockReturnValue(of(doc))
-    component.save()
-    expect(patchSpy).toHaveBeenCalledWith(
-      expect.objectContaining({ content: doc.content }),
-      10
-    )
-  })
-
-  it('should fall back to the latest version when the remembered version no longer exists', () => {
-    initNormally()
-    const openDoc = component.document()
-    openDoc.versions = docVersions
-    jest.spyOn(openDocumentsService, 'getOpenDocument').mockReturnValue(openDoc)
-    jest.spyOn(documentService, 'get').mockImplementation((id, versionID) =>
-      of(
-        (versionID
-          ? { content: `version ${versionID} content` }
-          : {
-              ...doc,
-              content: 'version 12 content',
-              versions: docVersions.filter((v) => v.id !== 10),
-            }) as Document
-      )
-    )
-    component.selectVersion(10)
-
-    component['loadDocument'](doc.id)
-
-    expect(component.selectedVersionId()).toEqual(12)
-    expect(component.documentForm.get('content').value).toEqual(
-      'version 12 content'
-    )
-  })
-
   it('createDisabled should return true if the user does not have permission to add the specified data type', () => {
     currentUserCan = false
     expect(component.createDisabled(DataType.Correspondent)).toBeTruthy()

src-ui/src/app/components/document-detail/document-detail.component.ts

@@ -98,8 +98,8 @@ import { ISODateAdapter } from 'src/app/utils/ngb-iso-date-adapter'
 import * as UTIF from 'utif'
 import { DocumentDetailFieldID } from '../admin/settings/settings.component'
 import { ConfirmDialogComponent } from '../common/confirm-dialog/confirm-dialog.component'
-import { PasswordRemovalConfirmDialogComponent } from '../common/confirm-dialog/password-removal-confirm-dialog/password-removal-confirm-dialog.component'
 import { ReprocessConfirmDialogComponent } from '../common/confirm-dialog/reprocess-confirm-dialog/reprocess-confirm-dialog.component'
+import { PasswordRemovalConfirmDialogComponent } from '../common/confirm-dialog/password-removal-confirm-dialog/password-removal-confirm-dialog.component'
 import { CustomFieldsDropdownComponent } from '../common/custom-fields-dropdown/custom-fields-dropdown.component'
 import { CorrespondentEditDialogComponent } from '../common/edit-dialog/correspondent-edit-dialog/correspondent-edit-dialog.component'
 import { DocumentTypeEditDialogComponent } from '../common/edit-dialog/document-type-edit-dialog/document-type-edit-dialog.component'
@@ -304,7 +304,6 @@ export class DocumentDetailComponent
   isDirty$: Observable<boolean>
   unsubscribeNotifier: Subject<any> = new Subject()
   docChangeNotifier: Subject<any> = new Subject()
-  versionChangeNotifier: Subject<void> = new Subject()
   private incomingUpdateModal: NgbModalRef
   private pendingIncomingUpdate: IncomingDocumentUpdate
   private lastLocalSaveModified: string | null = null
@@ -418,8 +417,7 @@ export class DocumentDetailComponent
       .pipe(
         first(),
         takeUntil(this.unsubscribeNotifier),
-        takeUntil(this.docChangeNotifier),
+        takeUntil(this.docChangeNotifier)
-        takeUntil(this.versionChangeNotifier)
       )
       .subscribe({
         next: (result) => {
@@ -535,8 +533,7 @@ export class DocumentDetailComponent
       .pipe(
         first(),
         takeUntil(this.unsubscribeNotifier),
-        takeUntil(this.docChangeNotifier),
+        takeUntil(this.docChangeNotifier)
-        takeUntil(this.versionChangeNotifier)
       )
       .subscribe({
         next: (res) => this.previewText.set(res.toString()),
@@ -598,13 +595,6 @@ export class DocumentDetailComponent
             openDocument.duplicate_documents = doc.duplicate_documents
             this.openDocumentService.save()
           }
-          // use server versions
-          if (openDocument) {
-            openDocument.versions = doc.versions
-            if (!openDocument.__changedFields?.includes('content')) {
-              openDocument.content = doc.content
-            }
-          }
           let useDoc = openDocument || doc
           if (openDocument && forceRemote) {
             Object.assign(openDocument, doc)
@@ -652,14 +642,7 @@ export class DocumentDetailComponent
               this.documentForm.patchValue({ title: titleValue })
               this.documentForm.get('title').markAsDirty()
             })
-          const keepContentEdits =
-            useDoc.__selectedVersionId === this.selectedVersionId() &&
-            !!useDoc.__changedFields?.includes('content')
           this.setupDirtyTracking(useDoc, doc)
-          // Maybe load the stored version
-          if (useDoc.__selectedVersionId) {
-            this.selectVersion(this.selectedVersionId(), keepContentEdits)
-          }
         },
       })
   }
@@ -920,11 +903,9 @@ export class DocumentDetailComponent
 
   updateComponent(doc: Document) {
     this.document.set(doc)
-    // Load the selected version, or default to API first (newest)
+    // Default selected version is the newest version, which the API returns first
     const versions = doc.versions ?? []
-    const selectedVersion =
+    this.selectedVersionId.set(versions.length ? versions[0].id : doc.id)
-      versions.find((v) => v.id === doc.__selectedVersionId) ?? versions[0]
-    this.selectedVersionId.set(selectedVersion?.id ?? doc.id)
     this.previewLoaded.set(false)
     this.requiresPassword = false
     this.updateFormForCustomFields()
@@ -959,12 +940,8 @@ export class DocumentDetailComponent
   }
 
   // Update file preview and download target to a specific version (by document id)
-  selectVersion(versionId: number, keepContentEdits: boolean = false) {
+  selectVersion(versionId: number) {
-    this.versionChangeNotifier.next()
     this.selectedVersionId.set(versionId)
-    // remember so the version can be restored when returning to the document
-    this.document().__selectedVersionId = versionId
-    this.openDocumentService.save()
     this.previewLoaded.set(false)
     this.previewUrl.set(
       this.documentsService.getPreviewUrl(
@@ -986,20 +963,20 @@ export class DocumentDetailComponent
       .pipe(
         first(),
         takeUntil(this.unsubscribeNotifier),
-        takeUntil(this.docChangeNotifier),
+        takeUntil(this.docChangeNotifier)
-        takeUntil(this.versionChangeNotifier)
       )
       .subscribe({
         next: (doc) => {
           const content = doc?.content ?? ''
-          if (keepContentEdits) {
+          this.document().content = content
-            this.store.next({ ...this.store.value, content })
+          this.documentForm.patchValue(
-          } else {
+            {
-            // Update in-place and avoid the debounce wait
+              content,
-            this.store.value.content = content
+            },
-            this.documentForm.patchValue({ content })
+            {
-            this.documentForm.get('content').markAsPristine()
+              emitEvent: false,
-          }
+            }
+          )
         },
         error: (error) => {
           this.toastService.showError(
@@ -1014,8 +991,7 @@ export class DocumentDetailComponent
       .pipe(
         first(),
         takeUntil(this.unsubscribeNotifier),
-        takeUntil(this.docChangeNotifier),
+        takeUntil(this.docChangeNotifier)
-        takeUntil(this.versionChangeNotifier)
       )
       .subscribe({
         next: (res) => this.previewText.set(res.toString()),
@@ -1029,39 +1005,7 @@ export class DocumentDetailComponent
   }
 
   onVersionSelected(versionId: number) {
-    if (versionId === this.selectedVersionId()) return
+    this.selectVersion(versionId)
-    // Bail if the selected version was just deleted.
-    const selectedVersionExists = this.document()?.versions?.some(
-      (v) => v.id === this.selectedVersionId()
-    )
-    if (this.networkActive() && selectedVersionExists) return
-    if (
-      !selectedVersionExists ||
-      this.documentForm.get('content').value === this.store.value.content
-    ) {
-      this.selectVersion(versionId)
-      return
-    }
-
-    // Confirm any unsaved content changes
-    const modal = this.modalService.open(ConfirmDialogComponent, {
-      backdrop: 'static',
-    })
-    modal.componentInstance.title = $localize`Unsaved Changes`
-    modal.componentInstance.messageBold = $localize`You have unsaved changes to the content of this version.`
-    modal.componentInstance.message = $localize`Switching versions will discard them.`
-    modal.componentInstance.btnClass = 'btn-secondary'
-    modal.componentInstance.btnCaption = $localize`Discard and switch`
-    modal.componentInstance.alternativeBtnClass = 'btn-primary'
-    modal.componentInstance.alternativeBtnCaption = $localize`Save and switch`
-    modal.componentInstance.confirmClicked.pipe(first()).subscribe(() => {
-      modal.close()
-      this.selectVersion(versionId)
-    })
-    modal.componentInstance.alternativeClicked.pipe(first()).subscribe(() => {
-      modal.close()
-      this.save(false, () => this.selectVersion(versionId))
-    })
   }
 
   onVersionsUpdated(versions: DocumentVersionInfo[]) {
@@ -1289,7 +1233,7 @@ export class DocumentDetailComponent
     return changes
   }
 
-  save(close: boolean = false, savedCallback: () => void = null) {
+  save(close: boolean = false) {
     this.networkActive.set(true)
     ;(document.activeElement as HTMLElement)?.dispatchEvent(new Event('change'))
     this.documentsService
@@ -1322,7 +1266,6 @@ export class DocumentDetailComponent
             this.flushPendingIncomingUpdate()
           }
           this.savedViewService.maybeRefreshDocumentCounts()
-          savedCallback?.()
         },
         error: (error) => {
           this.networkActive.set(false)

src-ui/src/app/components/document-list/bulk-editor/bulk-editor.component.spec.ts

@@ -7,9 +7,8 @@ import {
 import { EventEmitter, signal } from '@angular/core'
 import { ComponentFixture, TestBed } from '@angular/core/testing'
 import { By } from '@angular/platform-browser'
-import { Router } from '@angular/router'
 import { NgbModal, NgbModalRef } from '@ng-bootstrap/ng-bootstrap'
-import { allIcons, NgxBootstrapIconsModule } from 'ngx-bootstrap-icons'
+import { NgxBootstrapIconsModule, allIcons } from 'ngx-bootstrap-icons'
 import { of, throwError } from 'rxjs'
 import { Correspondent } from 'src/app/data/correspondent'
 import { CustomField, CustomFieldDataType } from 'src/app/data/custom-field'
@@ -47,6 +46,7 @@ import { StoragePathEditDialogComponent } from '../../common/edit-dialog/storage
 import { TagEditDialogComponent } from '../../common/edit-dialog/tag-edit-dialog/tag-edit-dialog.component'
 import { FilterableDropdownComponent } from '../../common/filterable-dropdown/filterable-dropdown.component'
 import { ShareLinkBundleDialogComponent } from '../../common/share-link-bundle-dialog/share-link-bundle-dialog.component'
+import { ShareLinkBundleManageDialogComponent } from '../../common/share-link-bundle-manage-dialog/share-link-bundle-manage-dialog.component'
 import { BulkEditorComponent } from './bulk-editor.component'
 
 const selectionData: SelectionData = {
@@ -82,7 +82,6 @@ describe('BulkEditorComponent', () => {
   let customFieldsService: CustomFieldsService
   let httpTestingController: HttpTestingController
   let shareLinkBundleService: ShareLinkBundleService
-  let router: Router
 
   beforeEach(async () => {
     TestBed.configureTestingModule({
@@ -168,14 +167,11 @@ describe('BulkEditorComponent', () => {
           provide: ShareLinkBundleService,
           useValue: {
             createBundle: jest.fn(),
+            listAllBundles: jest.fn(),
             rebuildBundle: jest.fn(),
             delete: jest.fn(),
           },
         },
-        {
-          provide: Router,
-          useValue: { navigate: jest.fn().mockResolvedValue(true) },
-        },
         provideHttpClient(withInterceptorsFromDi()),
         provideHttpClientTesting(),
       ],
@@ -193,7 +189,6 @@ describe('BulkEditorComponent', () => {
     customFieldsService = TestBed.inject(CustomFieldsService)
     httpTestingController = TestBed.inject(HttpTestingController)
     shareLinkBundleService = TestBed.inject(ShareLinkBundleService)
-    router = TestBed.inject(Router)
 
     fixture = TestBed.createComponent(BulkEditorComponent)
     component = fixture.componentInstance
@@ -392,42 +387,6 @@ describe('BulkEditorComponent', () => {
     expect(component.tagSelectionModel.selectionSize()).toEqual(1)
   })
 
-  it('should request selection data for tags when documents are excluded from an all-filtered selection', () => {
-    jest.spyOn(permissionsService, 'currentUserCan').mockReturnValue(true)
-    fixture.detectChanges()
-    jest
-      .spyOn(documentListViewService, 'allSelected', 'get')
-      .mockReturnValue(true)
-    jest
-      .spyOn(documentListViewService, 'excluded', 'get')
-      .mockReturnValue(new Set([4]))
-    jest
-      .spyOn(documentListViewService, 'filterRules', 'get')
-      .mockReturnValue([{ rule_type: FILTER_TITLE, value: 'apple' }])
-    jest
-      .spyOn(documentListViewService, 'selectedCount', 'get')
-      .mockReturnValue(2)
-    const adjustedSelectionData: SelectionData = {
-      ...selectionData,
-      selected_tags: [{ id: 12, document_count: 2 }],
-    }
-    const getSelectionDataSpy = jest
-      .spyOn(documentService, 'getSelectionData')
-      .mockReturnValue(of(adjustedSelectionData))
-
-    component.openTagsDropdown()
-
-    expect(getSelectionDataSpy).toHaveBeenCalledWith({
-      all: true,
-      filters: { title_search: 'apple' },
-      excluded_documents: [4],
-    })
-    expect(component.tagDocumentCounts()).toEqual(
-      adjustedSelectionData.selected_tags
-    )
-    expect(component.tagSelectionModel.selectionSize()).toEqual(1)
-  })
-
   it('should apply list selection data to document types menu when all filtered documents are selected', () => {
     jest.spyOn(permissionsService, 'currentUserCan').mockReturnValue(true)
     fixture.detectChanges()
@@ -496,47 +455,6 @@ describe('BulkEditorComponent', () => {
     )
   })
 
-  it('should request selection data for the other metadata menus when documents are excluded', () => {
-    jest.spyOn(permissionsService, 'currentUserCan').mockReturnValue(true)
-    fixture.detectChanges()
-    jest
-      .spyOn(documentListViewService, 'allSelected', 'get')
-      .mockReturnValue(true)
-    jest
-      .spyOn(documentListViewService, 'excluded', 'get')
-      .mockReturnValue(new Set([4]))
-    jest
-      .spyOn(documentListViewService, 'filterRules', 'get')
-      .mockReturnValue([{ rule_type: FILTER_TITLE, value: 'apple' }])
-    const getSelectionDataSpy = jest
-      .spyOn(documentService, 'getSelectionData')
-      .mockReturnValue(of(selectionData))
-
-    component.openDocumentTypeDropdown()
-    component.openCorrespondentDropdown()
-    component.openStoragePathDropdown()
-    component.openCustomFieldsDropdown()
-
-    expect(getSelectionDataSpy).toHaveBeenCalledTimes(4)
-    expect(getSelectionDataSpy).toHaveBeenCalledWith({
-      all: true,
-      filters: { title_search: 'apple' },
-      excluded_documents: [4],
-    })
-    expect(component.documentTypeDocumentCounts()).toEqual(
-      selectionData.selected_document_types
-    )
-    expect(component.correspondentDocumentCounts()).toEqual(
-      selectionData.selected_correspondents
-    )
-    expect(component.storagePathDocumentCounts()).toEqual(
-      selectionData.selected_storage_paths
-    )
-    expect(component.customFieldDocumentCounts()).toEqual(
-      selectionData.selected_custom_fields
-    )
-  })
-
   it('should execute modify tags bulk operation', () => {
     jest.spyOn(permissionsService, 'currentUserCan').mockReturnValue(true)
     jest
@@ -578,19 +496,16 @@ describe('BulkEditorComponent', () => {
       .mockReturnValue([{ id: 3 }, { id: 4 }])
     jest
       .spyOn(documentListViewService, 'selected', 'get')
-      .mockReturnValue(new Set([3]))
+      .mockReturnValue(new Set([3, 4]))
     jest
       .spyOn(documentListViewService, 'allSelected', 'get')
       .mockReturnValue(true)
-    jest
-      .spyOn(documentListViewService, 'excluded', 'get')
-      .mockReturnValue(new Set([4]))
     jest
       .spyOn(documentListViewService, 'filterRules', 'get')
       .mockReturnValue([{ rule_type: FILTER_TITLE, value: 'apple' }])
     jest
       .spyOn(documentListViewService, 'selectedCount', 'get')
-      .mockReturnValue(24)
+      .mockReturnValue(25)
     jest
       .spyOn(permissionsService, 'currentUserHasObjectPermissions')
       .mockReturnValue(true)
@@ -609,7 +524,6 @@ describe('BulkEditorComponent', () => {
     expect(req.request.body).toEqual({
       all: true,
       filters: { title_search: 'apple' },
-      excluded_documents: [4],
       method: 'modify_tags',
       parameters: { add_tags: [101], remove_tags: [] },
     })
@@ -1910,9 +1824,9 @@ describe('BulkEditorComponent', () => {
       },
     }
 
-    const openSpy = jest
+    const openSpy = jest.spyOn(modalService, 'open')
-      .spyOn(modalService, 'open')
+    openSpy.mockReturnValueOnce(modalRef as NgbModalRef)
-      .mockReturnValueOnce(modalRef as NgbModalRef)
+    openSpy.mockReturnValueOnce({} as NgbModalRef)
     ;(shareLinkBundleService.createBundle as jest.Mock).mockReturnValueOnce(
       of({ id: 42 })
     )
@@ -1946,9 +1860,11 @@ describe('BulkEditorComponent', () => {
 
     dialogInstance.onOpenManage()
     expect(modalRef.close).toHaveBeenCalled()
-    expect(router.navigate).toHaveBeenCalledWith(['/share-links'], {
+    expect(openSpy).toHaveBeenNthCalledWith(
-      queryParams: { type: 'bundles' },
+      2,
-    })
+      ShareLinkBundleManageDialogComponent,
+      expect.objectContaining({ backdrop: 'static', size: 'lg' })
+    )
     openSpy.mockRestore()
   })
 
@@ -2001,10 +1917,13 @@ describe('BulkEditorComponent', () => {
     openSpy.mockRestore()
   })
 
-  it('should navigate to share link bundle management', () => {
+  it('should open share link bundle management dialog', () => {
+    const openSpy = jest.spyOn(modalService, 'open')
     component.manageShareLinkBundles()
-    expect(router.navigate).toHaveBeenCalledWith(['/share-links'], {
+    expect(openSpy).toHaveBeenCalledWith(
-      queryParams: { type: 'bundles' },
+      ShareLinkBundleManageDialogComponent,
-    })
+      expect.objectContaining({ backdrop: 'static', size: 'lg' })
+    )
+    openSpy.mockRestore()
   })
 })

src-ui/src/app/components/document-list/bulk-editor/bulk-editor.component.ts

@@ -12,7 +12,6 @@ import {
   FormsModule,
   ReactiveFormsModule,
 } from '@angular/forms'
-import { Router } from '@angular/router'
 import {
   NgbDropdownModule,
   NgbModal,
@@ -70,6 +69,7 @@ import {
 import { ToggleableItemState } from '../../common/filterable-dropdown/toggleable-dropdown-button/toggleable-dropdown-button.component'
 import { PermissionsDialogComponent } from '../../common/permissions-dialog/permissions-dialog.component'
 import { ShareLinkBundleDialogComponent } from '../../common/share-link-bundle-dialog/share-link-bundle-dialog.component'
+import { ShareLinkBundleManageDialogComponent } from '../../common/share-link-bundle-manage-dialog/share-link-bundle-manage-dialog.component'
 import { ComponentWithPermissions } from '../../with-permissions/with-permissions.component'
 import { CustomFieldsBulkEditDialogComponent } from './custom-fields-bulk-edit-dialog/custom-fields-bulk-edit-dialog.component'
 
@@ -104,7 +104,6 @@ export class BulkEditorComponent
   public readonly permissionService = inject(PermissionsService)
   private savedViewService = inject(SavedViewService)
   private readonly shareLinkBundleService = inject(ShareLinkBundleService)
-  private readonly router = inject(Router)
 
   tagSelectionModel = new FilterableDropdownSelectionModel(true)
   correspondentSelectionModel = new FilterableDropdownSelectionModel()
@@ -361,7 +360,6 @@ export class BulkEditorComponent
       return {
         all: true,
         filters: queryParamsFromFilterRules(this.list.filterRules),
-        excluded_documents: Array.from(this.list.excluded),
       }
     }
 
@@ -375,8 +373,7 @@ export class BulkEditorComponent
   }
 
   openTagsDropdown() {
-    // If none excluded, use the selection data already available in the list view, otherwise fetch
+    if (this.list.allSelected) {
-    if (this.list.allSelected && this.list.excluded.size === 0) {
       const selectionData = this.list.selectionData
       this.tagDocumentCounts.set(selectionData?.selected_tags ?? [])
       this.applySelectionData(this.tagDocumentCounts(), this.tagSelectionModel)
@@ -384,7 +381,7 @@ export class BulkEditorComponent
     }
 
     this.documentService
-      .getSelectionData(this.getSelectionQuery())
+      .getSelectionData(Array.from(this.list.selected))
       .pipe(first())
       .subscribe((s) => {
         this.tagDocumentCounts.set(s.selected_tags)
@@ -393,7 +390,7 @@ export class BulkEditorComponent
   }
 
   openDocumentTypeDropdown() {
-    if (this.list.allSelected && this.list.excluded.size === 0) {
+    if (this.list.allSelected) {
       const selectionData = this.list.selectionData
       this.documentTypeDocumentCounts.set(
         selectionData?.selected_document_types ?? []
@@ -406,7 +403,7 @@ export class BulkEditorComponent
     }
 
     this.documentService
-      .getSelectionData(this.getSelectionQuery())
+      .getSelectionData(Array.from(this.list.selected))
       .pipe(first())
       .subscribe((s) => {
         this.documentTypeDocumentCounts.set(s.selected_document_types)
@@ -418,7 +415,7 @@ export class BulkEditorComponent
   }
 
   openCorrespondentDropdown() {
-    if (this.list.allSelected && this.list.excluded.size === 0) {
+    if (this.list.allSelected) {
       const selectionData = this.list.selectionData
       this.correspondentDocumentCounts.set(
         selectionData?.selected_correspondents ?? []
@@ -431,7 +428,7 @@ export class BulkEditorComponent
     }
 
     this.documentService
-      .getSelectionData(this.getSelectionQuery())
+      .getSelectionData(Array.from(this.list.selected))
       .pipe(first())
       .subscribe((s) => {
         this.correspondentDocumentCounts.set(s.selected_correspondents)
@@ -443,7 +440,7 @@ export class BulkEditorComponent
   }
 
   openStoragePathDropdown() {
-    if (this.list.allSelected && this.list.excluded.size === 0) {
+    if (this.list.allSelected) {
       const selectionData = this.list.selectionData
       this.storagePathDocumentCounts.set(
         selectionData?.selected_storage_paths ?? []
@@ -456,7 +453,7 @@ export class BulkEditorComponent
     }
 
     this.documentService
-      .getSelectionData(this.getSelectionQuery())
+      .getSelectionData(Array.from(this.list.selected))
       .pipe(first())
       .subscribe((s) => {
         this.storagePathDocumentCounts.set(s.selected_storage_paths)
@@ -468,7 +465,7 @@ export class BulkEditorComponent
   }
 
   openCustomFieldsDropdown() {
-    if (this.list.allSelected && this.list.excluded.size === 0) {
+    if (this.list.allSelected) {
       const selectionData = this.list.selectionData
       this.customFieldDocumentCounts.set(
         selectionData?.selected_custom_fields ?? []
@@ -481,7 +478,7 @@ export class BulkEditorComponent
     }
 
     this.documentService
-      .getSelectionData(this.getSelectionQuery())
+      .getSelectionData(Array.from(this.list.selected))
       .pipe(first())
       .subscribe((s) => {
         this.customFieldDocumentCounts.set(s.selected_custom_fields)
@@ -1138,8 +1135,9 @@ export class BulkEditorComponent
   }
 
   manageShareLinkBundles() {
-    void this.router.navigate(['/share-links'], {
+    this.modalService.open(ShareLinkBundleManageDialogComponent, {
-      queryParams: { type: 'bundles' },
+      backdrop: 'static',
+      size: 'lg',
     })
   }
 

src-ui/src/app/components/document-list/filter-editor/filter-editor.component.html

@@ -17,12 +17,6 @@
             }
           </select>
         }
-        @if (advancedSearchEditorAvailable) {
-          <button class="btn btn-sm btn-outline-primary" type="button" (click)="openAdvancedSearchEditor()"
-            title="Edit query" i18n-title [disabled]="disabled">
-            <i-bs name="sliders"></i-bs>
-          </button>
-        }
         @if (_textFilter) {
           <button class="btn btn-link btn-sm px-2 position-absolute top-0 end-0 z-10" (click)="resetTextField()" aria-label="Clear search" i18n-aria-label>
             <i-bs width="1em" height="1em" name="x"></i-bs>

src-ui/src/app/components/document-list/filter-editor/filter-editor.component.spec.ts

@@ -12,8 +12,6 @@ import {
   NgbDatepickerModule,
   NgbDropdownItem,
   NgbDropdownModule,
-  NgbModal,
-  NgbModalRef,
   NgbTypeaheadModule,
 } from '@ng-bootstrap/ng-bootstrap'
 import { NgSelectComponent, NgSelectModule } from '@ng-select/ng-select'
@@ -1425,16 +1423,11 @@ describe('FilterEditorComponent', () => {
     ])
   })
 
-  const clickTextFilterTarget = (name: string) => {
-    const item = fixture.debugElement
-      .queryAll(By.directive(NgbDropdownItem))
-      .find((el) => el.nativeElement.textContent.trim() === name)
-    expect(item).not.toBeUndefined()
-    item.triggerEventHandler('click')
-  }
-
   it('should convert duplicate target input to the correct filter rule', () => {
-    clickTextFilterTarget('Duplicates')
+    const textFieldTargetDropdown = fixture.debugElement.queryAll(
+      By.directive(NgbDropdownItem)
+    )[5]
+    textFieldTargetDropdown.triggerEventHandler('click')
     fixture.detectChanges()
 
     expect(component.textFilterTarget).toEqual('duplicates')
@@ -1460,7 +1453,10 @@ describe('FilterEditorComponent', () => {
   it('should convert user input to correct filter rules on full text query', () => {
     component.textFilterInput.nativeElement.value = 'foo'
     component.textFilterInput.nativeElement.dispatchEvent(new Event('input'))
-    clickTextFilterTarget('Advanced search')
+    const textFieldTargetDropdown = fixture.debugElement.queryAll(
+      By.directive(NgbDropdownItem)
+    )[4]
+    textFieldTargetDropdown.triggerEventHandler('click') // TEXT_FILTER_TARGET_FULLTEXT_QUERY
     fixture.detectChanges()
     tick(400)
     expect(component.textFilterTarget).toEqual('fulltext-query')
@@ -1929,7 +1925,10 @@ describe('FilterEditorComponent', () => {
   it('should leave relative dates not in quick list intact', () => {
     component.textFilterInput.nativeElement.value = 'created:[-2 week to now]'
     component.textFilterInput.nativeElement.dispatchEvent(new Event('input'))
-    clickTextFilterTarget('Advanced search')
+    const textFieldTargetDropdown = fixture.debugElement.queryAll(
+      By.directive(NgbDropdownItem)
+    )[4]
+    textFieldTargetDropdown.triggerEventHandler('click')
     fixture.detectChanges()
     tick(400)
     expect(component.filterRules).toEqual([
@@ -2517,45 +2516,4 @@ describe('FilterEditorComponent', () => {
 
     expect(component.textFilter).toEqual('help ')
   })
-
-  it('should open the advanced search editor with the current query and apply the result', () => {
-    const modalService: NgbModal = TestBed.inject(NgbModal)
-    let modal: NgbModalRef
-    modalService.activeInstances.subscribe(
-      (instances) => (modal = instances[0])
-    )
-    component.textFilterTarget = 'fulltext-query'
-    component.updateTextFilter('title:invoice')
-    fixture.detectChanges()
-
-    const editorButton = fixture.debugElement.query(
-      By.css('button[title="Edit query"]')
-    )
-    expect(editorButton).not.toBeNull()
-    editorButton.triggerEventHandler('click')
-    fixture.detectChanges()
-
-    expect(modal.componentInstance.query).toEqual('title:invoice')
-
-    const rulesSpy = jest.spyOn(component.filterRulesChange, 'next')
-    modal.componentInstance.queryApplied.emit('title:invoice AND NOT tag:paid')
-    expect(component.textFilter).toEqual('title:invoice AND NOT tag:paid')
-    expect(documentService.searchQuery).toEqual(
-      'title:invoice AND NOT tag:paid'
-    )
-    expect(rulesSpy).toHaveBeenCalledWith([
-      {
-        rule_type: FILTER_FULLTEXT_QUERY,
-        value: 'title:invoice AND NOT tag:paid',
-      },
-    ])
-  })
-
-  it('should not offer the advanced search editor for other targets', () => {
-    component.textFilterTarget = 'title-content'
-    fixture.detectChanges()
-    expect(
-      fixture.debugElement.query(By.css('button[title="Edit query"]'))
-    ).toBeNull()
-  })
 })

src-ui/src/app/components/document-list/filter-editor/filter-editor.component.ts

@@ -15,13 +15,12 @@ import {
 import { FormsModule, ReactiveFormsModule } from '@angular/forms'
 import {
   NgbDropdownModule,
-  NgbModal,
   NgbTypeahead,
   NgbTypeaheadModule,
 } from '@ng-bootstrap/ng-bootstrap'
 import { NgxBootstrapIconsModule } from 'ngx-bootstrap-icons'
 import { TourNgBootstrap } from 'ngx-ui-tour-ng-bootstrap'
-import { first, Observable, Subject, from } from 'rxjs'
+import { Observable, Subject, from } from 'rxjs'
 import {
   catchError,
   debounceTime,
@@ -122,7 +121,6 @@ import {
   PermissionsFilterDropdownComponent,
   PermissionsSelectionModel,
 } from '../../common/permissions-filter-dropdown/permissions-filter-dropdown.component'
-import { AdvancedSearchDialogComponent } from '../../common/advanced-search-dialog/advanced-search-dialog.component'
 import { LoadingComponentWithPermissions } from '../../loading-component/loading.component'
 
 const TEXT_FILTER_TARGET_TITLE = 'title'
@@ -207,11 +205,11 @@ const DEFAULT_TEXT_FILTER_TARGET_OPTIONS = [
   },
   { id: TEXT_FILTER_TARGET_ASN, name: $localize`ASN` },
   { id: TEXT_FILTER_TARGET_MIME_TYPE, name: $localize`File type` },
-  { id: TEXT_FILTER_TARGET_DUPLICATES, name: $localize`Duplicates` },
   {
     id: TEXT_FILTER_TARGET_FULLTEXT_QUERY,
     name: $localize`Advanced search`,
   },
+  { id: TEXT_FILTER_TARGET_DUPLICATES, name: $localize`Duplicates` },
 ]
 
 const DEPRECATED_CUSTOM_FIELDS_TEXT_FILTER_TARGET_OPTION = {
@@ -288,7 +286,6 @@ export class FilterEditorComponent
   permissionsService = inject(PermissionsService)
   private customFieldService = inject(CustomFieldsService)
   private searchService = inject(SearchService)
-  private modalService = inject(NgbModal)
 
   generateFilterName() {
     if (this.filterRules.length == 1) {
@@ -1375,23 +1372,6 @@ export class FilterEditorComponent
     }
   }
 
-  get advancedSearchEditorAvailable(): boolean {
-    return this.textFilterTarget === TEXT_FILTER_TARGET_FULLTEXT_QUERY
-  }
-
-  openAdvancedSearchEditor() {
-    const modal = this.modalService.open(AdvancedSearchDialogComponent, {
-      backdrop: 'static',
-      size: 'lg',
-    })
-    modal.componentInstance.query = this._textFilter ?? ''
-    modal.componentInstance.queryApplied
-      .pipe(first())
-      .subscribe((query: string) => {
-        this.updateTextFilter(query)
-      })
-  }
-
   textFilterKeydown(event: KeyboardEvent) {
     if (event.key == 'Enter') {
       if (event.defaultPrevented) {

src-ui/src/app/components/manage/share-links/share-link-list/share-link-list.component.html

@@ -1,110 +0,0 @@
-<div class="border border-top-0 rounded-bottom p-3">
-  @if (!loading() && error()) {
-    <div class="alert alert-danger mb-0" role="alert">{{ error() }}</div>
-  }
-  @if (!loading() && !error() && links().length === 0) {
-    <p class="mb-0 text-muted fst-italic" i18n>
-      No document share links currently exist.
-    </p>
-  }
-  @if (!loading() && !error() && links().length > 0) {
-    <div class="table-responsive">
-      <table class="table table-sm align-middle mb-0 bg-body">
-        <thead>
-          <tr>
-            <th scope="col" class="fw-normal" pngxSortable="document__title" [currentSortField]="sortField()" [currentSortReverse]="sortReverse()" (sort)="onSort($event)" i18n>Document</th>
-            <th scope="col" class="fw-normal" pngxSortable="created" [currentSortField]="sortField()" [currentSortReverse]="sortReverse()" (sort)="onSort($event)" i18n>Created</th>
-            <th scope="col" class="fw-normal" pngxSortable="expiration" [currentSortField]="sortField()" [currentSortReverse]="sortReverse()" (sort)="onSort($event)" i18n>Expires</th>
-            <th scope="col" i18n>File version</th>
-            <th scope="col" class="text-end" i18n>Actions</th>
-          </tr>
-        </thead>
-        <tbody>
-          @for (link of links(); track link.id) {
-            <tr>
-              <td>
-                <a routerLink="/documents/{{ link.document }}">{{ link.document_title | documentTitle }}</a>
-                <span class="badge bg-primary text-primary-text-contrast ms-3 small fs-normal cursor-pointer" (click)="copyDocumentID(link.document)">
-                  @if (copiedDocumentID() === link.document) {
-                    <i-bs width="1em" height="1em" name="clipboard-check" class="me-1"></i-bs><ng-container i18n>Copied!</ng-container>
-                  } @else {
-                    ID: {{link.document}}
-                  }
-                </span>
-              </td>
-              <td>{{ link.created | date: 'short' }}</td>
-              <td>
-                @if (link.expiration) {
-                  {{ link.expiration | date: 'short' }}
-                  @if (isExpired(link.expiration)) {
-                    <span class="badge text-bg-danger ms-2" i18n>Expired</span>
-                  }
-                } @else {
-                  <span i18n>Never</span>
-                }
-              </td>
-              <td>{{ fileVersionLabel(link.file_version) }}</td>
-              <td class="text-end">
-                <div class="d-inline-block position-relative">
-                  <span
-                    class="badge bg-primary small fade position-absolute top-50 end-100 translate-middle-y me-2 pe-none z-3 text-nowrap"
-                    [class.show]="copiedID() === link.id"
-                    i18n
-                  >Copied!</span>
-                  <div class="btn-group btn-group-sm">
-                    <button
-                      type="button"
-                      class="btn btn-outline-primary"
-                      (click)="copy(link)"
-                      title="Copy share link"
-                      i18n-title
-                    >
-                      @if (copiedID() === link.id) {
-                        <i-bs name="clipboard-check"></i-bs>
-                      } @else {
-                        <i-bs name="clipboard"></i-bs>
-                      }
-                      <span class="visually-hidden" i18n>Copy share link</span>
-                    </button>
-                    <pngx-confirm-button
-                      *pngxIfPermissions="{ action: PermissionAction.Delete, type: PermissionType.ShareLink }"
-                      buttonClasses="btn btn-sm btn-outline-danger"
-                      (confirm)="delete(link)"
-                      iconName="trash"
-                    >
-                      <span class="visually-hidden" i18n>Delete share link</span>
-                    </pngx-confirm-button>
-                  </div>
-                </div>
-              </td>
-            </tr>
-          }
-        </tbody>
-      </table>
-    </div>
-    <div class="d-flex flex-wrap justify-content-end align-items-center gap-3 mt-3 ms-auto">
-      <div class="d-flex align-items-center">
-        <label class="small text-muted me-2" for="shareLinkPageSize" i18n>Show:</label>
-        <select id="shareLinkPageSize" class="form-select form-select-sm w-auto" [(ngModel)]="pageSize">
-          <option [ngValue]="25">25</option>
-          <option [ngValue]="50">50</option>
-          <option [ngValue]="100">100</option>
-        </select>
-        <span class="small text-muted ms-2 d-none d-md-inline" i18n>per page</span>
-      </div>
-      @if (total() > pageSize) {
-        <ngb-pagination
-          class="mb-0"
-          [pageSize]="pageSize"
-          [collectionSize]="total()"
-          [page]="page()"
-          [maxSize]="5"
-          (pageChange)="setPage($event)"
-          size="sm"
-          aria-label="Share links pagination"
-          i18n-aria-label
-        ></ngb-pagination>
-      }
-    </div>
-  }
-</div>

src-ui/src/app/components/manage/share-links/share-link-list/share-link-list.component.spec.ts

@@ -1,155 +0,0 @@
-import { Clipboard } from '@angular/cdk/clipboard'
-import { ComponentFixture, TestBed } from '@angular/core/testing'
-import { RouterTestingModule } from '@angular/router/testing'
-import { NgxBootstrapIconsModule, allIcons } from 'ngx-bootstrap-icons'
-import { of, throwError } from 'rxjs'
-import { FileVersion, ShareLink } from 'src/app/data/share-link'
-import { SETTINGS_KEYS } from 'src/app/data/ui-settings'
-import { ShareLinkService } from 'src/app/services/rest/share-link.service'
-import { SettingsService } from 'src/app/services/settings.service'
-import { ToastService } from 'src/app/services/toast.service'
-import { ShareLinkListComponent } from './share-link-list.component'
-
-describe('ShareLinkListComponent', () => {
-  let component: ShareLinkListComponent
-  let fixture: ComponentFixture<ShareLinkListComponent>
-  let service: jest.Mocked<Pick<ShareLinkService, 'list' | 'delete'>>
-  let clipboard: Clipboard
-  let toastService: jest.Mocked<Pick<ToastService, 'showInfo' | 'showError'>>
-
-  const link = {
-    id: 1,
-    document: 42,
-    document_title: 'Test document',
-    slug: 'share-slug',
-    created: new Date().toISOString(),
-    expiration: null,
-    file_version: FileVersion.Archive,
-  } as ShareLink
-
-  beforeEach(() => {
-    service = {
-      list: jest.fn().mockReturnValue(of({ count: 1, results: [link] })),
-      delete: jest.fn().mockReturnValue(of(true)),
-    }
-    toastService = {
-      showInfo: jest.fn(),
-      showError: jest.fn(),
-    }
-
-    TestBed.configureTestingModule({
-      imports: [
-        ShareLinkListComponent,
-        NgxBootstrapIconsModule.pick(allIcons),
-        RouterTestingModule,
-      ],
-      providers: [
-        { provide: ShareLinkService, useValue: service },
-        { provide: ToastService, useValue: toastService },
-      ],
-    })
-
-    fixture = TestBed.createComponent(ShareLinkListComponent)
-    component = fixture.componentInstance
-    clipboard = TestBed.inject(Clipboard)
-  })
-
-  afterEach(() => {
-    jest.clearAllTimers()
-    jest.useRealTimers()
-  })
-
-  it('loads and renders document share links', () => {
-    fixture.detectChanges()
-
-    expect(service.list).toHaveBeenCalledWith(1, 25, 'created', true)
-    expect(component.links()).toEqual([link])
-    expect(fixture.nativeElement.textContent).toContain('Test document')
-    expect(fixture.nativeElement.textContent).toContain('ID: 42')
-  })
-
-  it('loads another page', () => {
-    fixture.detectChanges()
-    component.setPage(2)
-
-    expect(service.list).toHaveBeenLastCalledWith(2, 25, 'created', true)
-  })
-
-  it('sorts links and returns to the first page', () => {
-    fixture.detectChanges()
-    component.page.set(2)
-
-    component.onSort({ column: 'expiration', reverse: false })
-
-    expect(component.page()).toBe(1)
-    expect(service.list).toHaveBeenLastCalledWith(1, 25, 'expiration', false)
-  })
-
-  it('marks expired share links', () => {
-    service.list.mockReturnValue(
-      of({
-        count: 1,
-        results: [
-          {
-            ...link,
-            expiration: '2000-01-01T00:00:00.000Z',
-          },
-        ],
-      })
-    )
-
-    fixture.detectChanges()
-
-    expect(fixture.nativeElement.textContent).toContain('Expired')
-  })
-
-  it('stores a changed page size and reloads from the first page', () => {
-    const settingsService = TestBed.inject(SettingsService)
-    jest.spyOn(settingsService, 'get').mockReturnValueOnce({ share_links: 25 })
-    const setSpy = jest.spyOn(settingsService, 'set')
-    jest.spyOn(settingsService, 'storeSettings').mockReturnValue(of({}))
-    const reloadSpy = jest.spyOn(component, 'reload')
-    component.page.set(2)
-
-    component.pageSize = 50
-
-    expect(setSpy).toHaveBeenCalledWith(SETTINGS_KEYS.OBJECT_LIST_SIZES, {
-      share_links: 50,
-    })
-    expect(component.page()).toBe(1)
-    expect(reloadSpy).toHaveBeenCalled()
-  })
-
-  it('shows local copy feedback without a toast', () => {
-    jest.useFakeTimers()
-    jest.spyOn(clipboard, 'copy').mockReturnValue(true)
-    fixture.detectChanges()
-
-    component.copy(link)
-    fixture.detectChanges()
-
-    expect(component.copiedID()).toBe(link.id)
-    expect(fixture.nativeElement.querySelector('.badge.show')).not.toBeNull()
-    expect(toastService.showInfo).not.toHaveBeenCalled()
-
-    jest.advanceTimersByTime(3000)
-    expect(component.copiedID()).toBeNull()
-  })
-
-  it('deletes a link and reloads the list', () => {
-    fixture.detectChanges()
-    component.delete(link)
-
-    expect(service.delete).toHaveBeenCalledWith(link)
-    expect(service.list).toHaveBeenCalledTimes(2)
-    expect(toastService.showInfo).toHaveBeenCalled()
-  })
-
-  it('shows an error when loading fails', () => {
-    service.list.mockReturnValue(throwError(() => new Error('load failed')))
-    fixture.detectChanges()
-
-    expect(component.error()).toContain('Failed to load share links.')
-    expect(toastService.showError).toHaveBeenCalled()
-  })
-})

src-ui/src/app/components/manage/share-links/share-link-list/share-link-list.component.ts

@@ -1,172 +0,0 @@
-import { Clipboard } from '@angular/cdk/clipboard'
-import { CommonModule } from '@angular/common'
-import { Component, OnInit, inject, signal } from '@angular/core'
-import { FormsModule } from '@angular/forms'
-import { RouterModule } from '@angular/router'
-import { NgbPaginationModule } from '@ng-bootstrap/ng-bootstrap'
-import { NgxBootstrapIconsModule } from 'ngx-bootstrap-icons'
-import { takeUntil } from 'rxjs'
-import { ConfirmButtonComponent } from 'src/app/components/common/confirm-button/confirm-button.component'
-import { LoadingComponentWithPermissions } from 'src/app/components/loading-component/loading.component'
-import { FileVersion, ShareLink } from 'src/app/data/share-link'
-import { SHARE_LINK_BUNDLE_FILE_VERSION_LABELS } from 'src/app/data/share-link-bundle'
-import { SETTINGS_KEYS } from 'src/app/data/ui-settings'
-import { IfPermissionsDirective } from 'src/app/directives/if-permissions.directive'
-import {
-  SortEvent,
-  SortableDirective,
-} from 'src/app/directives/sortable.directive'
-import { DocumentTitlePipe } from 'src/app/pipes/document-title.pipe'
-import {
-  PermissionAction,
-  PermissionType,
-} from 'src/app/services/permissions.service'
-import { ShareLinkService } from 'src/app/services/rest/share-link.service'
-import { SettingsService } from 'src/app/services/settings.service'
-import { ToastService } from 'src/app/services/toast.service'
-import { environment } from 'src/environments/environment'
-
-@Component({
-  selector: 'pngx-share-link-list',
-  templateUrl: './share-link-list.component.html',
-  imports: [
-    CommonModule,
-    ConfirmButtonComponent,
-    DocumentTitlePipe,
-    FormsModule,
-    IfPermissionsDirective,
-    NgbPaginationModule,
-    NgxBootstrapIconsModule,
-    RouterModule,
-    SortableDirective,
-  ],
-})
-export class ShareLinkListComponent
-  extends LoadingComponentWithPermissions
-  implements OnInit
-{
-  private readonly clipboard = inject(Clipboard)
-  private readonly shareLinkService = inject(ShareLinkService)
-  private readonly settingsService = inject(SettingsService)
-  private readonly toastService = inject(ToastService)
-
-  readonly links = signal<ShareLink[]>([])
-  readonly total = signal(0)
-  readonly page = signal(1)
-  readonly sortField = signal('created')
-  readonly sortReverse = signal(true)
-  readonly copiedID = signal<number | null>(null)
-  readonly copiedDocumentID = signal<number | null>(null)
-  readonly error = signal<string | null>(null)
-  readonly PermissionAction = PermissionAction
-  readonly PermissionType = PermissionType
-
-  get pageSize(): number {
-    return (
-      this.settingsService.get(SETTINGS_KEYS.OBJECT_LIST_SIZES)?.share_links ||
-      25
-    )
-  }
-
-  set pageSize(pageSize: number) {
-    this.settingsService.set(SETTINGS_KEYS.OBJECT_LIST_SIZES, {
-      ...this.settingsService.get(SETTINGS_KEYS.OBJECT_LIST_SIZES),
-      share_links: pageSize,
-    })
-    this.settingsService.storeSettings().subscribe({
-      next: () => {
-        this.page.set(1)
-        this.reload()
-      },
-      error: (error) => {
-        this.toastService.showError($localize`Error saving settings`, error)
-      },
-    })
-  }
-
-  ngOnInit(): void {
-    this.reload()
-  }
-
-  reload(): void {
-    this.loading.set(true)
-    this.error.set(null)
-    this.shareLinkService
-      .list(this.page(), this.pageSize, this.sortField(), this.sortReverse())
-      .pipe(takeUntil(this.unsubscribeNotifier))
-      .subscribe({
-        next: (results) => {
-          this.links.set(results.results)
-          this.total.set(results.count)
-          this.loading.set(false)
-        },
-        error: (error) => {
-          this.loading.set(false)
-          this.error.set($localize`Failed to load share links.`)
-          this.toastService.showError(
-            $localize`Error retrieving share links.`,
-            error
-          )
-        },
-      })
-  }
-
-  setPage(page: number): void {
-    this.page.set(page)
-    this.reload()
-  }
-
-  onSort(event: SortEvent): void {
-    this.sortField.set(event.column || 'created')
-    this.sortReverse.set(event.column ? event.reverse : true)
-    this.page.set(1)
-    this.reload()
-  }
-
-  getShareUrl(link: ShareLink): string {
-    const apiURL = new URL(environment.apiBaseUrl)
-    return `${apiURL.origin}${apiURL.pathname.replace(/\/api\/$/, '/share/')}${
-      link.slug
-    }`
-  }
-
-  fileVersionLabel(version: FileVersion): string {
-    return SHARE_LINK_BUNDLE_FILE_VERSION_LABELS[version] ?? version
-  }
-
-  isExpired(expiration?: string): boolean {
-    return !!expiration && Date.parse(expiration) <= Date.now()
-  }
-
-  copy(link: ShareLink): void {
-    if (this.clipboard.copy(this.getShareUrl(link))) {
-      this.copiedID.set(link.id)
-      setTimeout(() => this.copiedID.set(null), 3000)
-    }
-  }
-
-  delete(link: ShareLink): void {
-    this.shareLinkService.delete(link).subscribe({
-      next: () => {
-        if (this.links().length === 1 && this.page() > 1) {
-          this.page.update((page) => page - 1)
-        }
-        this.toastService.showInfo($localize`Share link deleted.`)
-        this.reload()
-      },
-      error: (error) => {
-        this.toastService.showError(
-          $localize`Error deleting share link.`,
-          error
-        )
-      },
-    })
-  }
-
-  copyDocumentID(documentID: number): void {
-    if (this.clipboard.copy(documentID.toString())) {
-      this.copiedDocumentID.set(documentID)
-      setTimeout(() => this.copiedDocumentID.set(null), 3000)
-    }
-  }
-}

src-ui/src/app/components/manage/share-links/share-links.component.html

@@ -1,34 +0,0 @@
-<pngx-page-header
-  title="Share links"
-  i18n-title
-  info="Manage public links to individual documents and document bundles."
-  i18n-info
-  [loading]="loading()"
-></pngx-page-header>
-
-<ul
-  ngbNav
-  #nav="ngbNav"
-  class="nav-tabs"
-  [activeId]="activeNavID()"
-  (activeIdChange)="selectTab($event)"
->
-  @if (canViewDocumentLinks) {
-    <li [ngbNavItem]="ShareLinksNavIDs.DocumentLinks">
-      <button ngbNavLink i18n>Document links</button>
-      <ng-template ngbNavContent>
-        <pngx-share-link-list></pngx-share-link-list>
-      </ng-template>
-    </li>
-  }
-  @if (canViewBundles) {
-    <li [ngbNavItem]="ShareLinksNavIDs.Bundles">
-      <button ngbNavLink i18n>Bundles</button>
-      <ng-template ngbNavContent>
-        <pngx-share-link-bundle-list></pngx-share-link-bundle-list>
-      </ng-template>
-    </li>
-  }
-</ul>
-
-<div class="bg-body" [ngbNavOutlet]="nav"></div>

src-ui/src/app/components/manage/share-links/share-links.component.spec.ts

@@ -1,102 +0,0 @@
-import { ComponentFixture, TestBed } from '@angular/core/testing'
-import { ActivatedRoute, convertToParamMap, Router } from '@angular/router'
-import { NgbNavModule } from '@ng-bootstrap/ng-bootstrap'
-import { NgxBootstrapIconsModule, allIcons } from 'ngx-bootstrap-icons'
-import { of } from 'rxjs'
-import {
-  PermissionAction,
-  PermissionsService,
-  PermissionType,
-} from 'src/app/services/permissions.service'
-import { ShareLinkBundleService } from 'src/app/services/rest/share-link-bundle.service'
-import { ShareLinkService } from 'src/app/services/rest/share-link.service'
-import { ToastService } from 'src/app/services/toast.service'
-import { PageHeaderComponent } from '../../common/page-header/page-header.component'
-import { ShareLinksComponent, ShareLinksNavIDs } from './share-links.component'
-
-describe('ShareLinksComponent', () => {
-  let fixture: ComponentFixture<ShareLinksComponent>
-  let permissionsService: PermissionsService
-  let router: Router
-
-  const configure = async (type: string = null) => {
-    await TestBed.configureTestingModule({
-      imports: [
-        ShareLinksComponent,
-        NgbNavModule,
-        NgxBootstrapIconsModule.pick(allIcons),
-        PageHeaderComponent,
-      ],
-      providers: [
-        PermissionsService,
-        {
-          provide: ActivatedRoute,
-          useValue: {
-            snapshot: { queryParamMap: convertToParamMap({ type }) },
-          },
-        },
-        {
-          provide: Router,
-          useValue: { navigate: jest.fn().mockResolvedValue(true) },
-        },
-        {
-          provide: ShareLinkBundleService,
-          useValue: {
-            list: jest.fn().mockReturnValue(of({ count: 0, results: [] })),
-            rebuildBundle: jest.fn(),
-            delete: jest.fn(),
-          },
-        },
-        {
-          provide: ShareLinkService,
-          useValue: {
-            list: jest.fn().mockReturnValue(of({ count: 0, results: [] })),
-            delete: jest.fn(),
-          },
-        },
-        {
-          provide: ToastService,
-          useValue: { showInfo: jest.fn(), showError: jest.fn() },
-        },
-      ],
-    }).compileComponents()
-
-    permissionsService = TestBed.inject(PermissionsService)
-    router = TestBed.inject(Router)
-  }
-
-  afterEach(() => TestBed.resetTestingModule())
-
-  it('uses the requested bundles tab when permitted', async () => {
-    await configure(ShareLinksNavIDs.Bundles)
-    jest
-      .spyOn(permissionsService, 'currentUserCan')
-      .mockImplementation(
-        (action, type) =>
-          action === PermissionAction.View &&
-          type === PermissionType.ShareLinkBundle
-      )
-
-    fixture = TestBed.createComponent(ShareLinksComponent)
-    fixture.detectChanges()
-
-    expect(fixture.componentInstance.activeNavID()).toBe(
-      ShareLinksNavIDs.Bundles
-    )
-    expect(fixture.nativeElement.textContent).not.toContain('Document links')
-  })
-
-  it('updates the URL when a tab is selected', async () => {
-    await configure()
-    jest.spyOn(permissionsService, 'currentUserCan').mockReturnValue(true)
-
-    fixture = TestBed.createComponent(ShareLinksComponent)
-    fixture.componentInstance.selectTab(ShareLinksNavIDs.Bundles)
-
-    expect(router.navigate).toHaveBeenCalledWith([], {
-      relativeTo: TestBed.inject(ActivatedRoute),
-      queryParams: { type: ShareLinksNavIDs.Bundles },
-      queryParamsHandling: 'merge',
-    })
-  })
-})

src-ui/src/app/components/manage/share-links/share-links.component.ts

@@ -1,78 +0,0 @@
-import { Component, computed, inject, signal, viewChild } from '@angular/core'
-import { ActivatedRoute, Router } from '@angular/router'
-import { NgbNavModule } from '@ng-bootstrap/ng-bootstrap'
-import {
-  PermissionAction,
-  PermissionsService,
-  PermissionType,
-} from 'src/app/services/permissions.service'
-import { PageHeaderComponent } from '../../common/page-header/page-header.component'
-import { ShareLinkBundleListComponent } from './share-link-bundle-list/share-link-bundle-list.component'
-import { ShareLinkListComponent } from './share-link-list/share-link-list.component'
-
-export enum ShareLinksNavIDs {
-  DocumentLinks = 'documents',
-  Bundles = 'bundles',
-}
-
-@Component({
-  selector: 'pngx-share-links',
-  templateUrl: './share-links.component.html',
-  imports: [
-    NgbNavModule,
-    PageHeaderComponent,
-    ShareLinkBundleListComponent,
-    ShareLinkListComponent,
-  ],
-})
-export class ShareLinksComponent {
-  private readonly route = inject(ActivatedRoute)
-  private readonly router = inject(Router)
-  private readonly permissionsService = inject(PermissionsService)
-
-  readonly ShareLinksNavIDs = ShareLinksNavIDs
-  readonly activeNavID = signal(this.getInitialNavID())
-  private readonly documentLinks = viewChild(ShareLinkListComponent)
-  private readonly bundles = viewChild(ShareLinkBundleListComponent)
-  readonly loading = computed(() => {
-    const activeList =
-      this.activeNavID() === ShareLinksNavIDs.DocumentLinks
-        ? this.documentLinks()
-        : this.bundles()
-    return activeList?.loading() ?? true
-  })
-
-  get canViewDocumentLinks(): boolean {
-    return this.permissionsService.currentUserCan(
-      PermissionAction.View,
-      PermissionType.ShareLink
-    )
-  }
-
-  get canViewBundles(): boolean {
-    return this.permissionsService.currentUserCan(
-      PermissionAction.View,
-      PermissionType.ShareLinkBundle
-    )
-  }
-
-  selectTab(tab: ShareLinksNavIDs): void {
-    this.activeNavID.set(tab)
-    void this.router.navigate([], {
-      relativeTo: this.route,
-      queryParams: { type: tab },
-      queryParamsHandling: 'merge',
-    })
-  }
-
-  private getInitialNavID(): ShareLinksNavIDs {
-    const requestedTab = this.route.snapshot.queryParamMap.get('type')
-    if (requestedTab === ShareLinksNavIDs.Bundles && this.canViewBundles) {
-      return ShareLinksNavIDs.Bundles
-    }
-    if (this.canViewDocumentLinks) {
-      return ShareLinksNavIDs.DocumentLinks
-    }
-    return ShareLinksNavIDs.Bundles
-  }
-}

src-ui/src/app/data/advanced-search-query.ts

@@ -1,262 +0,0 @@
-// Fields and forms documented in docs/usage.md > "Document searches"
-
-export enum AdvancedSearchField {
-  Any = '',
-  Title = 'title',
-  Content = 'content',
-  OriginalFilename = 'original_filename',
-  NoteText = 'notes.note',
-  NoteAuthor = 'notes.user',
-  CustomFieldName = 'custom_fields.name',
-  CustomFieldValue = 'custom_fields.value',
-  Correspondent = 'correspondent',
-  DocumentType = 'document_type',
-  StoragePath = 'storage_path',
-  Tag = 'tag',
-  ASN = 'asn',
-  PageCount = 'page_count',
-  NumNotes = 'num_notes',
-  Created = 'created',
-  Added = 'added',
-  Modified = 'modified',
-  Checksum = 'checksum',
-}
-
-export enum AdvancedSearchFieldKind {
-  Text = 'text',
-  Number = 'number',
-  Date = 'date',
-  Checksum = 'checksum',
-}
-
-export const ADVANCED_SEARCH_FIELD_KINDS: Record<
-  AdvancedSearchField,
-  AdvancedSearchFieldKind
-> = {
-  [AdvancedSearchField.Any]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.Title]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.Content]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.OriginalFilename]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.NoteText]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.NoteAuthor]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.CustomFieldName]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.CustomFieldValue]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.Correspondent]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.DocumentType]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.StoragePath]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.Tag]: AdvancedSearchFieldKind.Text,
-  [AdvancedSearchField.ASN]: AdvancedSearchFieldKind.Number,
-  [AdvancedSearchField.PageCount]: AdvancedSearchFieldKind.Number,
-  [AdvancedSearchField.NumNotes]: AdvancedSearchFieldKind.Number,
-  [AdvancedSearchField.Created]: AdvancedSearchFieldKind.Date,
-  [AdvancedSearchField.Added]: AdvancedSearchFieldKind.Date,
-  [AdvancedSearchField.Modified]: AdvancedSearchFieldKind.Date,
-  [AdvancedSearchField.Checksum]: AdvancedSearchFieldKind.Checksum,
-}
-
-export enum AdvancedSearchOperator {
-  AllWords = 'all',
-  AnyWord = 'any',
-  Phrase = 'phrase',
-  StartsWith = 'prefix',
-  Equals = 'eq',
-  AtLeast = 'gte',
-  AtMost = 'lte',
-  Between = 'between',
-  DateKeyword = 'keyword',
-  WithinLast = 'within',
-}
-
-export const ADVANCED_SEARCH_OPERATORS_BY_KIND: Record<
-  AdvancedSearchFieldKind,
-  AdvancedSearchOperator[]
-> = {
-  [AdvancedSearchFieldKind.Text]: [
-    AdvancedSearchOperator.AllWords,
-    AdvancedSearchOperator.AnyWord,
-    AdvancedSearchOperator.Phrase,
-    AdvancedSearchOperator.StartsWith,
-  ],
-  [AdvancedSearchFieldKind.Number]: [
-    AdvancedSearchOperator.Equals,
-    AdvancedSearchOperator.AtLeast,
-    AdvancedSearchOperator.AtMost,
-    AdvancedSearchOperator.Between,
-  ],
-  [AdvancedSearchFieldKind.Date]: [
-    AdvancedSearchOperator.DateKeyword,
-    AdvancedSearchOperator.WithinLast,
-    AdvancedSearchOperator.AtLeast,
-    AdvancedSearchOperator.AtMost,
-    AdvancedSearchOperator.Between,
-  ],
-  [AdvancedSearchFieldKind.Checksum]: [AdvancedSearchOperator.StartsWith],
-}
-
-export const ADVANCED_SEARCH_DATE_KEYWORDS = [
-  'today',
-  'yesterday',
-  'tomorrow',
-  'previous week',
-  'this month',
-  'previous month',
-  'previous quarter',
-  'this year',
-  'previous year',
-] as const
-
-export type AdvancedSearchDateKeyword =
-  (typeof ADVANCED_SEARCH_DATE_KEYWORDS)[number]
-
-export enum AdvancedSearchDateUnit {
-  Day = 'day',
-  Week = 'week',
-  Month = 'month',
-  Year = 'year',
-}
-
-export enum AdvancedSearchLogicalOperator {
-  And = 'AND',
-  Or = 'OR',
-  Not = 'NOT',
-}
-
-export enum AdvancedSearchQueryElementType {
-  Atom = 'atom',
-  Group = 'group',
-}
-
-export interface AdvancedSearchQueryAtom {
-  type: AdvancedSearchQueryElementType.Atom
-  field: AdvancedSearchField
-  operator: AdvancedSearchOperator
-  value?: string
-  valueTo?: string
-  unit?: AdvancedSearchDateUnit // for WithinLast
-}
-
-export interface AdvancedSearchQueryGroup {
-  type: AdvancedSearchQueryElementType.Group
-  operator: AdvancedSearchLogicalOperator
-  children: AdvancedSearchQueryElement[]
-}
-
-export type AdvancedSearchQueryElement =
-  AdvancedSearchQueryAtom | AdvancedSearchQueryGroup
-
-export const ADVANCED_SEARCH_MAX_DEPTH = 2
-export const ADVANCED_SEARCH_MAX_ATOMS = 10
-
-export const ADVANCED_SEARCH_FIELD_LABELS: Record<AdvancedSearchField, string> =
-  {
-    [AdvancedSearchField.Any]: $localize`Any field`,
-    [AdvancedSearchField.Title]: $localize`Title`,
-    [AdvancedSearchField.Content]: $localize`Content`,
-    [AdvancedSearchField.OriginalFilename]: $localize`File name`,
-    [AdvancedSearchField.NoteText]: $localize`Note text`,
-    [AdvancedSearchField.NoteAuthor]: $localize`Note author`,
-    [AdvancedSearchField.CustomFieldName]: $localize`Custom field name`,
-    [AdvancedSearchField.CustomFieldValue]: $localize`Custom field value`,
-    [AdvancedSearchField.Correspondent]: $localize`Correspondent name`,
-    [AdvancedSearchField.DocumentType]: $localize`Document type name`,
-    [AdvancedSearchField.StoragePath]: $localize`Storage path name`,
-    [AdvancedSearchField.Tag]: $localize`Tag name`,
-    [AdvancedSearchField.ASN]: $localize`ASN`,
-    [AdvancedSearchField.PageCount]: $localize`Pages`,
-    [AdvancedSearchField.NumNotes]: $localize`Number of notes`,
-    [AdvancedSearchField.Created]: $localize`Created`,
-    [AdvancedSearchField.Added]: $localize`Added`,
-    [AdvancedSearchField.Modified]: $localize`Modified`,
-    [AdvancedSearchField.Checksum]: $localize`Checksum`,
-  }
-
-export const ADVANCED_SEARCH_FIELD_GROUPS: {
-  label: string
-  fields: AdvancedSearchField[]
-}[] = [
-  {
-    label: $localize`Text`,
-    fields: [
-      AdvancedSearchField.Any,
-      AdvancedSearchField.Title,
-      AdvancedSearchField.Content,
-      AdvancedSearchField.OriginalFilename,
-      AdvancedSearchField.NoteText,
-      AdvancedSearchField.NoteAuthor,
-      AdvancedSearchField.CustomFieldName,
-      AdvancedSearchField.CustomFieldValue,
-    ],
-  },
-  {
-    label: $localize`Names`,
-    fields: [
-      AdvancedSearchField.Correspondent,
-      AdvancedSearchField.DocumentType,
-      AdvancedSearchField.StoragePath,
-      AdvancedSearchField.Tag,
-    ],
-  },
-  {
-    label: $localize`Numbers`,
-    fields: [
-      AdvancedSearchField.ASN,
-      AdvancedSearchField.PageCount,
-      AdvancedSearchField.NumNotes,
-    ],
-  },
-  {
-    label: $localize`Dates`,
-    fields: [
-      AdvancedSearchField.Created,
-      AdvancedSearchField.Added,
-      AdvancedSearchField.Modified,
-    ],
-  },
-  { label: $localize`Other`, fields: [AdvancedSearchField.Checksum] },
-]
-
-export const ADVANCED_SEARCH_OPERATOR_LABELS: Record<
-  AdvancedSearchOperator,
-  string
-> = {
-  [AdvancedSearchOperator.AllWords]: $localize`contains all words`,
-  [AdvancedSearchOperator.AnyWord]: $localize`contains any word`,
-  [AdvancedSearchOperator.Phrase]: $localize`contains the phrase`,
-  [AdvancedSearchOperator.StartsWith]: $localize`starts with`,
-  [AdvancedSearchOperator.Equals]: $localize`is`,
-  [AdvancedSearchOperator.AtLeast]: $localize`is at least`,
-  [AdvancedSearchOperator.AtMost]: $localize`is at most`,
-  [AdvancedSearchOperator.Between]: $localize`is between`,
-  [AdvancedSearchOperator.DateKeyword]: $localize`is`,
-  [AdvancedSearchOperator.WithinLast]: $localize`is within the last`,
-}
-
-// Comparing dates reads differently than comparing counts
-export const ADVANCED_SEARCH_DATE_OPERATOR_LABELS: Partial<
-  Record<AdvancedSearchOperator, string>
-> = {
-  [AdvancedSearchOperator.AtLeast]: $localize`is on or after`,
-  [AdvancedSearchOperator.AtMost]: $localize`is on or before`,
-}
-
-export const ADVANCED_SEARCH_DATE_KEYWORD_LABELS: Record<string, string> = {
-  today: $localize`today`,
-  yesterday: $localize`yesterday`,
-  tomorrow: $localize`tomorrow`,
-  'previous week': $localize`previous week`,
-  'this month': $localize`this month`,
-  'previous month': $localize`previous month`,
-  'previous quarter': $localize`previous quarter`,
-  'this year': $localize`this year`,
-  'previous year': $localize`previous year`,
-}
-
-export const ADVANCED_SEARCH_DATE_UNIT_LABELS: Record<
-  AdvancedSearchDateUnit,
-  string
-> = {
-  [AdvancedSearchDateUnit.Day]: $localize`days`,
-  [AdvancedSearchDateUnit.Week]: $localize`weeks`,
-  [AdvancedSearchDateUnit.Month]: $localize`months`,
-  [AdvancedSearchDateUnit.Year]: $localize`years`,
-}

src-ui/src/app/data/document.ts

@@ -167,7 +167,6 @@ export interface Document extends ObjectWithPermissions {
 
   // Frontend only
   __changedFields?: string[]
-  __selectedVersionId?: number
 }
 
 export interface DocumentVersionInfo {

src-ui/src/app/data/paperless-task.ts

@@ -12,7 +12,6 @@ export enum PaperlessTaskType {
   ReprocessDocument = 'reprocess_document',
   BuildShareLink = 'build_share_link',
   BulkDelete = 'bulk_delete',
-  ApplyAiSuggestions = 'apply_ai_suggestions',
 }
 
 export enum PaperlessTaskTriggerSource {

src-ui/src/app/data/share-link.ts

@@ -26,7 +26,5 @@ export interface ShareLink extends ObjectWithPermissions {
 
   document: number // Document
 
-  document_title?: string
-
   file_version: string
 }

src-ui/src/app/data/ui-settings.ts

@@ -24,17 +24,6 @@ export enum CollapsibleSection {
   ATTRIBUTES = 'attributes',
 }
 
-export enum HideableSidebarItemID {
-  Dashboard = 'dashboard',
-  SavedViews = 'saved_views',
-  ShareLinks = 'share_links',
-  Workflows = 'workflows',
-  Mail = 'mail',
-  Documentation = 'documentation',
-}
-
-export const HIDEABLE_SIDEBAR_ITEM_IDS = Object.values(HideableSidebarItemID)
-
 export const PAPERLESS_GREEN_HEX = '#17541f'
 
 export const SETTINGS_KEYS = {
@@ -67,7 +56,6 @@ export const SETTINGS_KEYS = {
   NOTES_ENABLED: 'general-settings:notes-enabled',
   AUDITLOG_ENABLED: 'general-settings:auditlog-enabled',
   SLIM_SIDEBAR: 'general-settings:slim-sidebar',
-  SIDEBAR_HIDDEN_ITEMS: 'general-settings:sidebar:hidden-items',
   ATTRIBUTES_SECTIONS_COLLAPSED:
     'general-settings:attributes-sections-collapsed',
   UPDATE_CHECKING_ENABLED: 'general-settings:update-checking:enabled',
@@ -139,11 +127,6 @@ export const SETTINGS: UiSetting[] = [
     type: 'boolean',
     default: false,
   },
-  {
-    key: SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS,
-    type: 'array',
-    default: [],
-  },
   {
     key: SETTINGS_KEYS.ATTRIBUTES_SECTIONS_COLLAPSED,
     type: 'array',
@@ -247,8 +230,6 @@ export const SETTINGS: UiSetting[] = [
       document_types: 25,
       tags: 25,
       storage_paths: 25,
-      share_links: 25,
-      share_link_bundles: 25,
     },
   },
   {

src-ui/src/app/services/document-list-view.service.spec.ts

@@ -580,7 +580,7 @@ describe('DocumentListViewService', () => {
     expect(documentListViewService.isSelected(documents[3])).toBeTruthy()
   })
 
-  it('should exclude a toggled document while keeping all-selected mode', () => {
+  it('should clear all-selected mode when toggling a single document', () => {
     documentListViewService.reload()
     const req = httpTestingController.expectOne(
       `${environment.apiBaseUrl}documents/?page=1&page_size=50&ordering=-created&truncate_content=true&include_selection_data=true`
@@ -592,73 +592,8 @@ describe('DocumentListViewService', () => {
 
     documentListViewService.toggleSelected(documents[0])
 
-    expect(documentListViewService.allSelected).toBeTruthy()
+    expect(documentListViewService.allSelected).toBeFalsy()
-    expect(documentListViewService.excluded).toEqual(new Set([documents[0].id]))
-    expect(documentListViewService.selectedCount).toEqual(documents.length - 1)
     expect(documentListViewService.isSelected(documents[0])).toBeFalsy()
-
-    documentListViewService.toggleSelected(documents[0])
-
-    expect(documentListViewService.excluded.size).toEqual(0)
-    expect(documentListViewService.selectedCount).toEqual(documents.length)
-    expect(documentListViewService.isSelected(documents[0])).toBeTruthy()
-  })
-
-  it('should preserve exclusions across pages', () => {
-    documentListViewService.pageSize = 3
-    let req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}documents/?page=1&page_size=3&ordering=-created&truncate_content=true&include_selection_data=true`
-    )
-    req.flush({ count: documents.length, results: documents.slice(0, 3) })
-
-    documentListViewService.selectAll()
-    documentListViewService.toggleSelected(documents[0])
-    documentListViewService.currentPage = 2
-    req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}documents/?page=2&page_size=3&ordering=-created&truncate_content=true&include_selection_data=true`
-    )
-    req.flush({ count: documents.length, results: documents.slice(3, 6) })
-
-    expect(documentListViewService.excluded).toEqual(new Set([documents[0].id]))
-    expect(documentListViewService.selectedCount).toEqual(documents.length - 1)
-    expect(documentListViewService.selected).toEqual(
-      new Set(documents.slice(3, 6).map((document) => document.id))
-    )
-
-    documentListViewService.currentPage = 1
-    req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}documents/?page=1&page_size=3&ordering=-created&truncate_content=true&include_selection_data=true`
-    )
-    req.flush({ count: documents.length, results: documents.slice(0, 3) })
-
-    expect(documentListViewService.isSelected(documents[0])).toBeFalsy()
-    expect(documentListViewService.isSelected(documents[1])).toBeTruthy()
-  })
-
-  it('should clear exclusions when filters change', () => {
-    documentListViewService.reload()
-    let req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}documents/?page=1&page_size=50&ordering=-created&truncate_content=true&include_selection_data=true`
-    )
-    req.flush(full_results)
-    documentListViewService.selectAll()
-    documentListViewService.toggleSelected(documents[0])
-
-    documentListViewService.setFilterRules(filterRules)
-    req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}documents/?page=1&page_size=50&ordering=-created&truncate_content=true&include_selection_data=true&tags__id__all=9`
-    )
-    req.flush({ count: 3, results: documents.slice(0, 3) })
-
-    expect(documentListViewService.allSelected).toBeTruthy()
-    expect(documentListViewService.excluded.size).toEqual(0)
-    expect(documentListViewService.selectedCount).toEqual(3)
-
-    documentListViewService.setFilterRules([])
-    req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}documents/?page=1&page_size=50&ordering=-created&truncate_content=true&include_selection_data=true`
-    )
-    req.flush(full_results)
   })
 
   it('should clear all-selected mode when selecting a range', () => {

src-ui/src/app/services/document-list-view.service.ts

@@ -85,11 +85,6 @@ export interface ListViewState {
    */
   allSelected?: boolean
 
-  /**
-   * Document IDs excluded from the full filtered result set.
-   */
-  excluded?: Set<number>
-
   /**
    * The page size of the list view.
    */
@@ -220,7 +215,6 @@ export class DocumentListViewService {
       filterRules: [],
       selected: new Set<number>(),
       allSelected: false,
-      excluded: new Set<number>(),
     }
   }
 
@@ -230,9 +224,7 @@ export class DocumentListViewService {
     }
 
     this.selected.clear()
-    this.documents
+    this.documents?.forEach((doc) => this.selected.add(doc.id))
-      ?.filter((doc) => !this.excluded.has(doc.id))
-      .forEach((doc) => this.selected.add(doc.id))
 
     if (!this.collectionSize) {
       this.selectNone()
@@ -499,23 +491,14 @@ export class DocumentListViewService {
     return this.activeListViewState.allSelected ?? false
   }
 
-  get excluded(): Set<number> {
-    this.trackState()
-    if (!this.activeListViewState.excluded) {
-      this.activeListViewState.excluded = new Set<number>()
-    }
-    return this.activeListViewState.excluded
-  }
-
   get selectedCount(): number {
-    if (!this.allSelected || this.collectionSize == null) {
+    return this.allSelected
-      return this.selected.size
+      ? (this.collectionSize ?? this.selected.size)
-    }
+      : this.selected.size
-    return Math.max(0, this.collectionSize - this.excluded.size)
   }
 
   get hasSelection(): boolean {
-    return this.selectedCount > 0
+    return this.allSelected || this.selected.size > 0
   }
 
   setSort(field: string, reverse: boolean) {
@@ -680,14 +663,12 @@ export class DocumentListViewService {
   selectNone() {
     this.activeListViewState.allSelected = false
     this.selected.clear()
-    this.excluded.clear()
     this.rangeSelectionAnchorIndex = this.lastRangeSelectionToIndex = null
     this.markChanged()
   }
 
   reduceSelectionToFilter() {
     if (this.allSelected) {
-      this.excluded.clear()
       return
     }
 
@@ -707,7 +688,6 @@ export class DocumentListViewService {
 
   selectAll() {
     this.activeListViewState.allSelected = true
-    this.excluded.clear()
     this.syncSelectedToCurrentPage()
     this.markChanged()
   }
@@ -715,7 +695,6 @@ export class DocumentListViewService {
   selectPage() {
     this.activeListViewState.allSelected = false
     this.selected.clear()
-    this.excluded.clear()
     this.documents.forEach((doc) => {
       this.selected.add(doc.id)
     })
@@ -723,23 +702,15 @@ export class DocumentListViewService {
   }
 
   isSelected(d: Document) {
-    return this.allSelected ? !this.excluded.has(d.id) : this.selected.has(d.id)
+    return this.allSelected || this.selected.has(d.id)
   }
 
   toggleSelected(d: Document): void {
     if (this.allSelected) {
-      if (this.excluded.has(d.id)) {
+      this.activeListViewState.allSelected = false
-        this.excluded.delete(d.id)
-        this.selected.add(d.id)
-      } else {
-        this.excluded.add(d.id)
-        this.selected.delete(d.id)
-      }
-    } else if (this.selected.has(d.id)) {
-      this.selected.delete(d.id)
-    } else {
-      this.selected.add(d.id)
     }
+    if (this.selected.has(d.id)) this.selected.delete(d.id)
+    else this.selected.add(d.id)
     this.rangeSelectionAnchorIndex = this.documentIndexInCurrentView(d.id)
     this.lastRangeSelectionToIndex = null
     this.markChanged()
@@ -748,7 +719,6 @@ export class DocumentListViewService {
   selectRangeTo(d: Document) {
     if (this.allSelected) {
       this.activeListViewState.allSelected = false
-      this.excluded.clear()
     }
 
     if (this.rangeSelectionAnchorIndex !== null) {

src-ui/src/app/services/open-documents.service.spec.ts

@@ -221,25 +221,6 @@ describe('OpenDocumentsService', () => {
     expect(openDocumentsService.getOpenDocuments()).toHaveLength(1)
   })
 
-  it('should refresh documents in place and keep unsaved edits', () => {
-    const openDoc = { ...documents[0] }
-    subscriptions.push(openDocumentsService.openDocument(openDoc).subscribe())
-    openDoc.title = 'Unsaved title'
-    openDocumentsService.setDirty(openDoc, true, { title: openDoc.title })
-
-    openDocumentsService.refreshDocument(openDoc.id)
-    httpTestingController
-      .expectOne(
-        `${environment.apiBaseUrl}documents/${openDoc.id}/?full_perms=true`
-      )
-      .flush({ ...documents[0], tags: [4] })
-
-    const refreshed = openDocumentsService.getOpenDocument(openDoc.id)
-    expect(refreshed).toBe(openDoc)
-    expect(refreshed.title).toEqual('Unsaved title')
-    expect(refreshed.tags).toEqual([4])
-  })
-
   it('should handle error on refresh documents', () => {
     subscriptions.push(
       openDocumentsService.openDocument(documents[1]).subscribe()

src-ui/src/app/services/open-documents.service.ts

@@ -50,15 +50,7 @@ export class OpenDocumentsService {
     if (index > -1) {
       this.documentService.get(id).subscribe({
         next: (doc) => {
-          const openDoc = this.openDocuments.find((d) => d.id == id)
+          this.openDocuments[index] = doc
-          if (!openDoc) return
-          const unsavedEdits = Object.fromEntries(
-            (openDoc.__changedFields ?? []).map((field) => [
-              field,
-              openDoc[field],
-            ])
-          )
-          Object.assign(openDoc, doc, unsavedEdits)
           this.save()
         },
         error: () => {

src-ui/src/app/services/rest/document.service.spec.ts

@@ -175,7 +175,7 @@ describe(`DocumentService`, () => {
 
   it('should call appropriate api endpoint for getting selection data', () => {
     const ids = [documents[0].id]
-    subscription = service.getSelectionData({ documents: ids }).subscribe()
+    subscription = service.getSelectionData(ids).subscribe()
     const req = httpTestingController.expectOne(
       `${environment.apiBaseUrl}${endpoint}/selection_data/`
     )
@@ -185,20 +185,6 @@ describe(`DocumentService`, () => {
     })
   })
 
-  it('should get selection data with all, filters, and exclusions', () => {
-    const selection = {
-      all: true,
-      filters: { title__icontains: 'apple' },
-      excluded_documents: [2, 3],
-    }
-    subscription = service.getSelectionData(selection).subscribe()
-    const req = httpTestingController.expectOne(
-      `${environment.apiBaseUrl}${endpoint}/selection_data/`
-    )
-    expect(req.request.method).toEqual('POST')
-    expect(req.request.body).toEqual(selection)
-  })
-
   it('should call appropriate api endpoint for getting suggestions', () => {
     subscription = service.getSuggestions(documents[0].id).subscribe()
     const req = httpTestingController.expectOne(
@@ -254,7 +240,7 @@ describe(`DocumentService`, () => {
     })
   })
 
-  it('should call appropriate api endpoint for bulk edit with all, filters, and exclusions', () => {
+  it('should call appropriate api endpoint for bulk edit with all and filters', () => {
     const method = 'modify_tags'
     const parameters = {
       add_tags: [15],
@@ -263,7 +249,6 @@ describe(`DocumentService`, () => {
     const selection = {
       all: true,
       filters: { title__icontains: 'apple' },
-      excluded_documents: [2, 3],
     }
     subscription = service.bulkEdit(selection, method, parameters).subscribe()
     const req = httpTestingController.expectOne(
@@ -273,7 +258,6 @@ describe(`DocumentService`, () => {
     expect(req.request.body).toEqual({
       all: true,
       filters: { title__icontains: 'apple' },
-      excluded_documents: [2, 3],
       method,
       parameters,
     })

src-ui/src/app/services/rest/document.service.ts

@@ -72,7 +72,6 @@ export interface DocumentSelectionQuery {
   documents?: number[]
   all?: boolean
   filters?: { [key: string]: any }
-  excluded_documents?: number[]
 }
 
 @Injectable({
@@ -408,12 +407,10 @@ export class DocumentService extends AbstractPaperlessService<Document> {
     })
   }
 
-  getSelectionData(
+  getSelectionData(ids: number[]): Observable<SelectionData> {
-    selection: DocumentSelectionQuery
-  ): Observable<SelectionData> {
     return this.http.post<SelectionData>(
       this.getResourceUrl(null, 'selection_data'),
-      selection
+      { documents: ids }
     )
   }
 

src-ui/src/app/services/rest/share-link-bundle.service.spec.ts

@@ -48,4 +48,13 @@ describe('ShareLinkBundleService', () => {
     expect(req.request.body).toEqual({})
     req.flush({})
   })
+
+  it('lists bundles with expected parameters', () => {
+    subscription = service.listAllBundles().subscribe()
+    const req = httpTestingController.expectOne(
+      `${environment.apiBaseUrl}${endpoint}/?page=1&page_size=1000&ordering=-created`
+    )
+    expect(req.request.method).toBe('GET')
+    req.flush({ results: [] })
+  })
 })

src-ui/src/app/services/rest/share-link-bundle.service.ts

@@ -1,5 +1,6 @@
 import { Injectable } from '@angular/core'
 import { Observable } from 'rxjs'
+import { map } from 'rxjs/operators'
 import {
   ShareLinkBundleCreatePayload,
   ShareLinkBundleSummary,
@@ -31,4 +32,10 @@ export class ShareLinkBundleService extends AbstractNameFilterService<ShareLinkB
       {}
     )
   }
+
+  listAllBundles(): Observable<ShareLinkBundleSummary[]> {
+    return this.list(1, 1000, 'created', true).pipe(
+      map((response) => response.results)
+    )
+  }
 }

src-ui/src/app/services/settings.service.spec.ts

@@ -14,11 +14,7 @@ import { CustomFieldDataType } from '../data/custom-field'
 import { DEFAULT_DISPLAY_FIELDS, DisplayField } from '../data/document'
 import { SavedView } from '../data/saved-view'
 import { RemoteOCRModeConfig } from '../data/paperless-config'
-import {
+import { SETTINGS_KEYS, UiSettings } from '../data/ui-settings'
-  HideableSidebarItemID,
-  SETTINGS_KEYS,
-  UiSettings,
-} from '../data/ui-settings'
 import { PermissionsService } from './permissions.service'
 import { CustomFieldsService } from './rest/custom-fields.service'
 import { SettingsService } from './settings.service'
@@ -234,35 +230,6 @@ describe('SettingsService', () => {
     expect(notesEnabled()).toBeFalsy()
   })
 
-  it('updates sidebar item visibility', () => {
-    httpTestingController
-      .expectOne(`${environment.apiBaseUrl}ui_settings/`)
-      .flush(ui_settings)
-
-    expect(
-      settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows)
-    ).toBe(false)
-
-    settingsService.updateSidebarItemVisibility(
-      HideableSidebarItemID.Workflows,
-      false
-    )
-
-    expect(
-      settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows)
-    ).toBe(true)
-    expect(settingsService.get(SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS)).toEqual([])
-
-    settingsService.updateSidebarItemVisibility(
-      HideableSidebarItemID.Workflows,
-      true
-    )
-
-    expect(
-      settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows)
-    ).toBe(false)
-  })
-
   it('updates setting signals when settings are reinitialized', () => {
     let req = httpTestingController.expectOne(
       `${environment.apiBaseUrl}ui_settings/`

src-ui/src/app/services/settings.service.ts

@@ -24,7 +24,6 @@ import { DEFAULT_DISPLAY_FIELDS, DisplayField } from '../data/document'
 import { RemoteOCRModeConfig } from '../data/paperless-config'
 import { SavedView } from '../data/saved-view'
 import {
-  HideableSidebarItemID,
   PAPERLESS_GREEN_HEX,
   SETTINGS,
   SETTINGS_KEYS,
@@ -314,18 +313,6 @@ export class SettingsService {
   readonly globalDropzoneEnabled = signal(true)
   readonly globalDropzoneActive = signal(false)
   readonly organizingSidebarSavedViews = signal(false)
-  readonly sidebarHiddenItemsEditing = signal<HideableSidebarItemID[] | null>(
-    null
-  )
-  readonly organizingSidebarItems = computed(
-    () => this.sidebarHiddenItemsEditing() !== null
-  )
-  readonly sidebarHiddenItemsEditingChanged = new EventEmitter<
-    HideableSidebarItemID[]
-  >()
-  readonly hiddenSidebarItems = this.getSignal<HideableSidebarItemID[]>(
-    SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS
-  )
 
   readonly allDisplayFields = signal<Array<{ id: DisplayField; name: string }>>(
     DEFAULT_DISPLAY_FIELDS
@@ -762,29 +749,6 @@ export class SettingsService {
     return this.storeSettings()
   }
 
-  sidebarItemIsHidden(item: HideableSidebarItemID): boolean {
-    return (
-      this.sidebarHiddenItemsEditing() ?? this.hiddenSidebarItems()
-    ).includes(item)
-  }
-
-  updateSidebarItemVisibility(
-    item: HideableSidebarItemID,
-    visible: boolean
-  ): void {
-    const hiddenItems = new Set(
-      this.sidebarHiddenItemsEditing() ?? this.hiddenSidebarItems()
-    )
-    if (visible) {
-      hiddenItems.delete(item)
-    } else {
-      hiddenItems.add(item)
-    }
-    const updatedHiddenItems = [...hiddenItems]
-    this.sidebarHiddenItemsEditing.set(updatedHiddenItems)
-    this.sidebarHiddenItemsEditingChanged.emit(updatedHiddenItems)
-  }
-
   updateSavedViewsVisibility(
     dashboardVisibleViewIds: number[],
     sidebarVisibleViewIds: number[]

src-ui/src/app/utils/advanced-search-query.spec.ts

@@ -1,515 +0,0 @@
-import {
-  AdvancedSearchDateUnit,
-  AdvancedSearchField,
-  AdvancedSearchLogicalOperator,
-  AdvancedSearchOperator,
-  AdvancedSearchQueryAtom,
-  AdvancedSearchQueryElement,
-  AdvancedSearchQueryElementType,
-  AdvancedSearchQueryGroup,
-} from '../data/advanced-search-query'
-import {
-  parseAdvancedSearchQuery,
-  serializeAdvancedSearchQuery,
-} from './advanced-search-query'
-
-const atom = (
-  field: AdvancedSearchField,
-  operator: AdvancedSearchOperator,
-  value?: string,
-  extra: Partial<AdvancedSearchQueryAtom> = {}
-): AdvancedSearchQueryAtom => ({
-  type: AdvancedSearchQueryElementType.Atom,
-  field,
-  operator,
-  value,
-  ...extra,
-})
-
-const group = (
-  operator: AdvancedSearchLogicalOperator,
-  ...children: AdvancedSearchQueryElement[]
-): AdvancedSearchQueryGroup => ({
-  type: AdvancedSearchQueryElementType.Group,
-  operator,
-  children,
-})
-
-const { And, Or, Not } = AdvancedSearchLogicalOperator
-
-describe('serializeAdvancedSearchQuery', () => {
-  describe('text fields', () => {
-    it.each([
-      [AdvancedSearchOperator.AllWords, 'invoice', 'title:invoice'],
-      [
-        AdvancedSearchOperator.AllWords,
-        '  invoice   unpaid ',
-        'title:invoice AND title:unpaid',
-      ],
-      [
-        AdvancedSearchOperator.AnyWord,
-        'invoice unpaid',
-        'title:invoice OR title:unpaid',
-      ],
-      [
-        AdvancedSearchOperator.Phrase,
-        'quick brown fox',
-        'title:"quick brown fox"',
-      ],
-      [AdvancedSearchOperator.Phrase, 'say "hi"', 'title:"say hi"'],
-      [AdvancedSearchOperator.StartsWith, 'invoi', 'title:invoi*'],
-      [AdvancedSearchOperator.StartsWith, 'in*v?oi', 'title:invoi*'],
-    ])('%s %j writes %s', (operator, value, expected) => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(AdvancedSearchField.Title, operator, value)
-        )
-      ).toBe(expected)
-    })
-
-    it('writes bare words for the Any field', () => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(AdvancedSearchField.Any, AdvancedSearchOperator.AllWords, 'a b')
-        )
-      ).toBe('a AND b')
-    })
-
-    it.each([
-      ['A-1312/99', 'custom_fields.value:A-1312/99'],
-      ["O'Brien", "custom_fields.value:O'Brien"],
-      ["'quoted'", `custom_fields.value:"'quoted'"`],
-      ['foo:bar', 'custom_fields.value:"foo:bar"'],
-      ['(x)', 'custom_fields.value:"(x)"'],
-      ['2024*', 'custom_fields.value:"2024*"'],
-      ['a,b', 'custom_fields.value:"a,b"'],
-      ['OR', 'custom_fields.value:"OR"'],
-      ['or', 'custom_fields.value:or'],
-    ])(
-      'quotes %j only when the grammar would read it as syntax',
-      (value, expected) => {
-        expect(
-          serializeAdvancedSearchQuery(
-            atom(
-              AdvancedSearchField.CustomFieldValue,
-              AdvancedSearchOperator.AllWords,
-              value
-            )
-          )
-        ).toBe(expected)
-      }
-    )
-
-    it('uses the dotted names for custom fields', () => {
-      expect(
-        serializeAdvancedSearchQuery(
-          group(
-            And,
-            atom(
-              AdvancedSearchField.CustomFieldName,
-              AdvancedSearchOperator.Phrase,
-              'status'
-            ),
-            atom(
-              AdvancedSearchField.CustomFieldValue,
-              AdvancedSearchOperator.AllWords,
-              'paid'
-            )
-          )
-        )
-      ).toBe('custom_fields.name:"status" AND custom_fields.value:paid')
-    })
-
-    it('uses the dotted names for notes', () => {
-      expect(
-        serializeAdvancedSearchQuery(
-          group(
-            And,
-            atom(
-              AdvancedSearchField.NoteText,
-              AdvancedSearchOperator.AllWords,
-              'call'
-            ),
-            atom(
-              AdvancedSearchField.NoteAuthor,
-              AdvancedSearchOperator.AllWords,
-              'alice'
-            )
-          )
-        )
-      ).toBe('notes.note:call AND notes.user:alice')
-    })
-
-    it.each([
-      [AdvancedSearchOperator.AllWords, ''],
-      [AdvancedSearchOperator.AllWords, '  '],
-      [AdvancedSearchOperator.AllWords, '!! --'],
-      [AdvancedSearchOperator.Phrase, '""'],
-      [AdvancedSearchOperator.StartsWith, 'two words'],
-      [AdvancedSearchOperator.StartsWith, '***'],
-      [AdvancedSearchOperator.StartsWith, undefined],
-    ])('leaves out %s %j', (operator, value) => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(AdvancedSearchField.Title, operator, value)
-        )
-      ).toBe('')
-    })
-  })
-
-  describe('checksum', () => {
-    it('lowercases the prefix', () => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(
-            AdvancedSearchField.Checksum,
-            AdvancedSearchOperator.StartsWith,
-            '9F86D081'
-          )
-        )
-      ).toBe('checksum:9f86d081*')
-    })
-  })
-
-  describe('number fields', () => {
-    it.each([
-      [AdvancedSearchOperator.Equals, '42', undefined, 'asn:42'],
-      [AdvancedSearchOperator.AtLeast, '50', undefined, 'asn:[50 to]'],
-      [AdvancedSearchOperator.AtMost, '50', undefined, 'asn:[to 50]'],
-      [AdvancedSearchOperator.Between, '50', '150', 'asn:[50 to 150]'],
-      [AdvancedSearchOperator.Equals, '4.2', undefined, ''],
-      [AdvancedSearchOperator.Equals, '-1', undefined, ''],
-      [AdvancedSearchOperator.Equals, '2024-01-01', undefined, ''],
-      [AdvancedSearchOperator.Between, '50', '', ''],
-    ])('%s %j %j writes %j', (operator, value, valueTo, expected) => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(AdvancedSearchField.ASN, operator, value, { valueTo })
-        )
-      ).toBe(expected)
-    })
-  })
-
-  describe('date fields', () => {
-    it.each([
-      ['today', 'added:today'],
-      ['previous month', 'added:"previous month"'],
-      ['last tuesday', ''],
-      ['', ''],
-    ])('keyword %j writes %j', (value, expected) => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(
-            AdvancedSearchField.Added,
-            AdvancedSearchOperator.DateKeyword,
-            value
-          )
-        )
-      ).toBe(expected)
-    })
-
-    it.each([
-      ['1', AdvancedSearchDateUnit.Day, 'added:[-1 day to now]'],
-      ['3', AdvancedSearchDateUnit.Month, 'added:[-3 months to now]'],
-      ['2', AdvancedSearchDateUnit.Week, 'added:[-2 weeks to now]'],
-      ['0', AdvancedSearchDateUnit.Year, ''],
-      ['1.5', AdvancedSearchDateUnit.Year, ''],
-      ['3', undefined, ''],
-      ['3', 'fortnight' as AdvancedSearchDateUnit, ''],
-    ])('within the last %j %j writes %j', (value, unit, expected) => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(
-            AdvancedSearchField.Added,
-            AdvancedSearchOperator.WithinLast,
-            value,
-            {
-              unit,
-            }
-          )
-        )
-      ).toBe(expected)
-    })
-
-    it.each([
-      [
-        AdvancedSearchOperator.AtLeast,
-        '2024-01-01',
-        undefined,
-        'created:[2024-01-01 to]',
-      ],
-      [
-        AdvancedSearchOperator.AtMost,
-        '2024-01-01',
-        undefined,
-        'created:[to 2024-01-01]',
-      ],
-      [
-        AdvancedSearchOperator.Between,
-        '2024-01-01',
-        '2024-03-31',
-        'created:[2024-01-01 to 2024-03-31]',
-      ],
-      [AdvancedSearchOperator.AtLeast, '2024', undefined, ''],
-      [AdvancedSearchOperator.Between, '2024-01-01', 'now', ''],
-    ])('%s %j %j writes %j', (operator, value, valueTo, expected) => {
-      expect(
-        serializeAdvancedSearchQuery(
-          atom(AdvancedSearchField.Created, operator, value, { valueTo })
-        )
-      ).toBe(expected)
-    })
-  })
-
-  describe('groups', () => {
-    const invoice = atom(
-      AdvancedSearchField.Content,
-      AdvancedSearchOperator.AllWords,
-      'invoice'
-    )
-    const letter = atom(
-      AdvancedSearchField.Title,
-      AdvancedSearchOperator.AllWords,
-      'letter'
-    )
-    const paid = atom(
-      AdvancedSearchField.Tag,
-      AdvancedSearchOperator.AllWords,
-      'paid'
-    )
-    const twoWords = atom(
-      AdvancedSearchField.Title,
-      AdvancedSearchOperator.AllWords,
-      'a b'
-    )
-    const anyWords = atom(
-      AdvancedSearchField.Title,
-      AdvancedSearchOperator.AnyWord,
-      'a b'
-    )
-    const empty = atom(
-      AdvancedSearchField.Title,
-      AdvancedSearchOperator.AllWords,
-      ''
-    )
-
-    it.each([
-      ['an empty group', group(And), ''],
-      ['a group of empty atoms', group(Or, empty, group(And, empty)), ''],
-      [
-        'a single child without parentheses',
-        group(Or, invoice),
-        'content:invoice',
-      ],
-      ['All', group(And, invoice, letter), 'content:invoice AND title:letter'],
-      ['Any', group(Or, invoice, letter), 'content:invoice OR title:letter'],
-      [
-        'skipped empty atoms',
-        group(And, empty, invoice, empty),
-        'content:invoice',
-      ],
-      ['Not with one child', group(Not, paid), 'NOT tag:paid'],
-      [
-        'Not as none of',
-        group(Not, paid, letter),
-        'NOT (tag:paid OR title:letter)',
-      ],
-      [
-        'Not with a compound child',
-        group(Not, twoWords),
-        'NOT (title:a AND title:b)',
-      ],
-      [
-        'Not inside All',
-        group(And, invoice, group(Not, paid)),
-        'content:invoice AND NOT tag:paid',
-      ],
-      [
-        'Not inside Any',
-        group(Or, invoice, group(Not, paid)),
-        'content:invoice OR NOT tag:paid',
-      ],
-      [
-        'Any inside All',
-        group(And, invoice, group(Or, letter, paid)),
-        'content:invoice AND (title:letter OR tag:paid)',
-      ],
-      [
-        'All inside Any',
-        group(Or, invoice, group(And, letter, paid)),
-        'content:invoice OR (title:letter AND tag:paid)',
-      ],
-      [
-        'All inside All flattened',
-        group(And, invoice, group(And, letter, paid)),
-        'content:invoice AND title:letter AND tag:paid',
-      ],
-      [
-        'an all-words atom inside Any',
-        group(Or, invoice, twoWords),
-        'content:invoice OR (title:a AND title:b)',
-      ],
-      [
-        'an any-word atom inside All',
-        group(And, invoice, anyWords),
-        'content:invoice AND (title:a OR title:b)',
-      ],
-      [
-        'an all-words atom inside Not with siblings',
-        group(Not, paid, twoWords),
-        'NOT (tag:paid OR (title:a AND title:b))',
-      ],
-    ])('writes %s', (_, tree, expected) => {
-      expect(serializeAdvancedSearchQuery(tree)).toBe(expected)
-    })
-
-    it('writes the mockup example', () => {
-      expect(
-        serializeAdvancedSearchQuery(
-          group(
-            And,
-            invoice,
-            group(
-              Or,
-              atom(
-                AdvancedSearchField.Correspondent,
-                AdvancedSearchOperator.Phrase,
-                'acme corp'
-              ),
-              atom(
-                AdvancedSearchField.Content,
-                AdvancedSearchOperator.Phrase,
-                'acme corporation'
-              )
-            ),
-            atom(
-              AdvancedSearchField.Added,
-              AdvancedSearchOperator.WithinLast,
-              '3',
-              {
-                unit: AdvancedSearchDateUnit.Month,
-              }
-            ),
-            group(Not, paid)
-          )
-        )
-      ).toBe(
-        'content:invoice AND (correspondent:"acme corp" OR content:"acme corporation") AND added:[-3 months to now] AND NOT tag:paid'
-      )
-    })
-  })
-})
-
-describe('parseAdvancedSearchQuery', () => {
-  const canonical = [
-    'title:invoice',
-    'title:invoice AND title:unpaid',
-    'content:invoice OR content:receipt',
-    'a AND b',
-    'title:"quick brown fox"',
-    'title:invoi*',
-    "custom_fields.value:O'Brien",
-    'custom_fields.value:"foo:bar"',
-    'custom_fields.name:"status" AND custom_fields.value:paid',
-    'notes.note:call AND notes.user:alice',
-    'checksum:9f86d081*',
-    'asn:42',
-    'asn:[50 to 150]',
-    'asn:[50 to]',
-    'asn:[to 50]',
-    'page_count:[10 to]',
-    'added:today',
-    'added:"previous month"',
-    'added:[-1 day to now]',
-    'added:[-3 months to now]',
-    'created:[2024-01-01 to 2024-03-31]',
-    'created:[2024-01-01 to]',
-    'created:[to 2024-01-01]',
-    'NOT tag:paid',
-    'NOT (tag:paid OR title:letter)',
-    'NOT (title:a AND title:b)',
-    'content:invoice OR NOT tag:paid',
-    'content:invoice AND (title:letter OR tag:paid)',
-    'content:invoice OR (title:letter AND tag:paid)',
-    'content:invoice AND (correspondent:"acme corp" OR content:"acme corporation") AND added:[-3 months to now] AND NOT tag:paid',
-  ]
-
-  it.each(canonical)('reads back %s unchanged', (query) => {
-    const tree = parseAdvancedSearchQuery(query)
-    expect(tree).not.toBeNull()
-    expect(serializeAdvancedSearchQuery(tree)).toBe(query)
-  })
-
-  it('reads surrounding whitespace', () => {
-    expect(
-      serializeAdvancedSearchQuery(
-        parseAdvancedSearchQuery('  title:invoice  ')
-      )
-    ).toBe('title:invoice')
-  })
-
-  it('puts the words of one condition back together', () => {
-    expect(parseAdvancedSearchQuery('title:invoice AND title:unpaid')).toEqual(
-      group(
-        And,
-        atom(
-          AdvancedSearchField.Title,
-          AdvancedSearchOperator.AllWords,
-          'invoice unpaid'
-        )
-      )
-    )
-  })
-
-  it('keeps words of different fields apart', () => {
-    expect(parseAdvancedSearchQuery('title:a OR content:b')).toEqual(
-      group(
-        Or,
-        atom(AdvancedSearchField.Title, AdvancedSearchOperator.AllWords, 'a'),
-        atom(AdvancedSearchField.Content, AdvancedSearchOperator.AllWords, 'b')
-      )
-    )
-  })
-
-  it('reads a range as its condition', () => {
-    expect(parseAdvancedSearchQuery('added:[-3 months to now]')).toEqual(
-      group(
-        And,
-        atom(
-          AdvancedSearchField.Added,
-          AdvancedSearchOperator.WithinLast,
-          '3',
-          {
-            unit: AdvancedSearchDateUnit.Month,
-          }
-        )
-      )
-    )
-  })
-
-  it.each([
-    ['', 'nothing'],
-    ['   ', 'whitespace'],
-    ['type:invoice', 'a field alias'],
-    ['notes:call', 'a bare notes prefix'],
-    ['unknown:x', 'an unknown field'],
-    ['title:a AND content:b OR title:c', 'AND and OR mixed at one level'],
-    ['title:a b', 'an implicit AND'],
-    ['title:"unterminated', 'an unterminated phrase'],
-    ['asn:[50 to', 'an unterminated range'],
-    ['title:a*b', 'a wildcard in the middle'],
-    ['title:invoice^2', 'a boost'],
-    ['asn:[50 to abc]', 'a non-numeric bound'],
-    ['asn:invoice', 'a word on a number field'],
-    ['created:[2024 to 2025]', 'a year-only date range'],
-    ['added:"last tuesday"', 'a date keyword paperless does not have'],
-    ['title:[a to b]', 'a range on a text field'],
-    ['checksum:9f86d081', 'a checksum without a wildcard'],
-    ['(title:invoice)', 'parentheses the editor would not write'],
-    ['title:invoice AND', 'a trailing operator'],
-    ['title:"a"b', 'a value running into the next'],
-    ['ADDED:today', 'an uppercase field name'],
-  ])('leaves %s alone, having %s', (query) => {
-    expect(parseAdvancedSearchQuery(query)).toBeNull()
-  })
-})

src-ui/src/app/utils/advanced-search-query.ts

@@ -1,500 +0,0 @@
-import {
-  ADVANCED_SEARCH_DATE_KEYWORDS,
-  ADVANCED_SEARCH_FIELD_KINDS,
-  AdvancedSearchDateUnit,
-  AdvancedSearchField,
-  AdvancedSearchFieldKind,
-  AdvancedSearchLogicalOperator,
-  AdvancedSearchOperator,
-  AdvancedSearchQueryAtom,
-  AdvancedSearchQueryElement,
-  AdvancedSearchQueryElementType,
-  AdvancedSearchQueryGroup,
-} from '../data/advanced-search-query'
-
-// Anything the query grammar would read as syntax rather than as a word
-const SYNTAX_CHARS = /[\s():"[\]*?,{}^~\\]/
-const SYNTAX_CHARS_GLOBAL = new RegExp(SYNTAX_CHARS, 'g')
-// A word wrapped in single quotes is also syntax, an apostrophe inside one is not
-const EDGE_SINGLE_QUOTE = /^'|'$/
-const RESERVED_WORDS = /^(AND|OR|NOT|TO)$/
-const HAS_WORD_CHAR = /[\p{L}\p{N}]/u
-const WHOLE_NUMBER = /^\d+$/
-const ISO_DATE = /^\d{4}-\d{2}-\d{2}$/
-
-interface Serialized {
-  text: string
-  // The operator joining the top level of `text`, null when it is self-delimiting
-  join:
-    AdvancedSearchLogicalOperator.And | AdvancedSearchLogicalOperator.Or | null
-}
-
-const prefix = (field: AdvancedSearchField) => (field ? `${field}:` : '')
-
-const quote = (text: string) => `"${text.replace(/"/g, '')}"`
-
-const words = (value: string) =>
-  value
-    .trim()
-    .split(/\s+/)
-    .filter((word) => HAS_WORD_CHAR.test(word))
-
-const word = (w: string) =>
-  SYNTAX_CHARS.test(w) || EDGE_SINGLE_QUOTE.test(w) || RESERVED_WORDS.test(w)
-    ? quote(w)
-    : w
-
-const atomic = (text: string): Serialized => ({ text, join: null })
-
-function serializeWords(
-  atom: AdvancedSearchQueryAtom,
-  join: AdvancedSearchLogicalOperator.And | AdvancedSearchLogicalOperator.Or
-): Serialized {
-  // A field applies only to the word right after it, so repeat it per word
-  const terms = words(atom.value ?? '').map(
-    (w) => `${prefix(atom.field)}${word(w)}`
-  )
-  if (terms.length === 0) return null
-  if (terms.length === 1) return atomic(terms[0])
-  return { text: terms.join(` ${join} `), join }
-}
-
-function serializeRange(
-  atom: AdvancedSearchQueryAtom,
-  isValid: (v: string) => boolean
-): Serialized {
-  const lo = atom.value?.trim() ?? ''
-  const hi = atom.valueTo?.trim() ?? ''
-  const field = prefix(atom.field)
-  switch (atom.operator) {
-    case AdvancedSearchOperator.Equals:
-      return isValid(lo) ? atomic(`${field}${lo}`) : null
-    case AdvancedSearchOperator.AtLeast:
-      return isValid(lo) ? atomic(`${field}[${lo} to]`) : null
-    case AdvancedSearchOperator.AtMost:
-      return isValid(lo) ? atomic(`${field}[to ${lo}]`) : null
-    case AdvancedSearchOperator.Between:
-      return isValid(lo) && isValid(hi)
-        ? atomic(`${field}[${lo} to ${hi}]`)
-        : null
-  }
-  return null
-}
-
-function serializeAtom(atom: AdvancedSearchQueryAtom): Serialized {
-  const field = prefix(atom.field)
-  const value = atom.value?.trim() ?? ''
-
-  switch (atom.operator) {
-    case AdvancedSearchOperator.AllWords:
-      return serializeWords(atom, AdvancedSearchLogicalOperator.And)
-    case AdvancedSearchOperator.AnyWord:
-      return serializeWords(atom, AdvancedSearchLogicalOperator.Or)
-    case AdvancedSearchOperator.Phrase:
-      return HAS_WORD_CHAR.test(value)
-        ? atomic(`${field}${quote(value)}`)
-        : null
-    case AdvancedSearchOperator.StartsWith: {
-      if (/\s/.test(value)) return null
-      let stem = value.replace(SYNTAX_CHARS_GLOBAL, '')
-      if (!HAS_WORD_CHAR.test(stem)) return null
-      // checksum is indexed as-is, in lowercase
-      if (atom.field === AdvancedSearchField.Checksum) {
-        stem = stem.toLowerCase()
-      }
-      return atomic(`${field}${stem}*`)
-    }
-    case AdvancedSearchOperator.DateKeyword:
-      return (ADVANCED_SEARCH_DATE_KEYWORDS as readonly string[]).includes(
-        value
-      )
-        ? atomic(`${field}${word(value)}`)
-        : null
-    case AdvancedSearchOperator.WithinLast: {
-      const amount = parseInt(value, 10)
-      const units = Object.values(AdvancedSearchDateUnit) as string[]
-      if (!WHOLE_NUMBER.test(value) || amount < 1) return null
-      if (!units.includes(atom.unit)) return null
-      const unit = amount === 1 ? atom.unit : `${atom.unit}s`
-      return atomic(`${field}[-${amount} ${unit} to now]`)
-    }
-    case AdvancedSearchOperator.Equals:
-    case AdvancedSearchOperator.AtLeast:
-    case AdvancedSearchOperator.AtMost:
-    case AdvancedSearchOperator.Between:
-      return serializeRange(
-        atom,
-        ADVANCED_SEARCH_FIELD_KINDS[atom.field] === AdvancedSearchFieldKind.Date
-          ? (v) => ISO_DATE.test(v)
-          : (v) => WHOLE_NUMBER.test(v)
-      )
-  }
-  return null
-}
-
-function wrap(
-  child: Serialized,
-  parentJoin: AdvancedSearchLogicalOperator
-): string {
-  return child.join === null || child.join === parentJoin
-    ? child.text
-    : `(${child.text})`
-}
-
-function serializeGroup(group: AdvancedSearchQueryGroup): Serialized {
-  const children = group.children.map(serializeElement).filter(Boolean)
-  if (children.length === 0) return null
-
-  if (group.operator === AdvancedSearchLogicalOperator.Not) {
-    // A Not group matches documents matching none of its children
-    if (children.length === 1 && children[0].join === null) {
-      return atomic(`NOT ${children[0].text}`)
-    }
-    const inner =
-      children.length === 1
-        ? children[0].text
-        : children
-            .map((c) => wrap(c, AdvancedSearchLogicalOperator.Or))
-            .join(' OR ')
-    return atomic(`NOT (${inner})`)
-  }
-
-  if (children.length === 1) return children[0]
-  return {
-    text: children
-      .map((c) => wrap(c, group.operator))
-      .join(` ${group.operator} `),
-    join: group.operator,
-  }
-}
-
-function serializeElement(element: AdvancedSearchQueryElement): Serialized {
-  return element.type === AdvancedSearchQueryElementType.Group
-    ? serializeGroup(element)
-    : serializeAtom(element)
-}
-
-/**
- * Writes an editor tree as a full-text query. Atoms that are not filled in
- * (or not valid) are left out, and empty groups with them.
- */
-export function serializeAdvancedSearchQuery(
-  element: AdvancedSearchQueryElement
-): string {
-  return serializeElement(element)?.text ?? ''
-}
-
-// --- Reading a query back into the editor --------------------------------
-//
-// Deliberately narrow: this reads the forms serializeAdvancedSearchQuery
-// writes, and nothing else. A query it cannot read is not a failure, it just
-// stays text, so there is never a lossy or surprising conversion. The final
-// round-trip check below is what holds that promise: a tree is only returned
-// when writing it out again reproduces the query exactly.
-
-const RELATIVE_BOUND = /^-(\d+) (day|week|month|year)s?$/
-const FIELD_PREFIX = /^([a-z_]+(?:\.[a-z_]+)?):/
-const KEYWORD_TOKEN = /^(AND|OR|NOT)(?=[\s(]|$)/
-// Either bound may be missing: [50 to 150], [50 to], [to 50]
-const RANGE_BOUNDS = /^(?:(.+?) )?to(?: (.+))?$/
-const TRAILING_WILDCARD = /^([^*?]+)\*$/
-
-class UnreadableQuery extends Error {}
-
-interface Token {
-  type: 'term' | 'AND' | 'OR' | 'NOT' | '(' | ')'
-  field?: string
-  value?: string
-  quoted?: boolean
-  range?: boolean
-}
-
-// A parsed element, plus what it takes to merge the per-word terms the
-// serializer writes for "contains all words" back into a single condition
-interface Parsed {
-  element: AdvancedSearchQueryElement
-  word?: { field: AdvancedSearchField; text: string }
-}
-
-function tokenize(query: string): Token[] {
-  const tokens: Token[] = []
-  let i = 0
-  while (i < query.length) {
-    const rest = query.slice(i)
-    if (/^\s/.test(rest)) {
-      i++
-      continue
-    }
-    if (rest[0] === '(' || rest[0] === ')') {
-      tokens.push({ type: rest[0] as '(' | ')' })
-      i++
-      continue
-    }
-    const keyword = KEYWORD_TOKEN.exec(rest)
-    if (keyword) {
-      tokens.push({ type: keyword[1] as 'AND' | 'OR' | 'NOT' })
-      i += keyword[1].length
-      continue
-    }
-    const fieldMatch = FIELD_PREFIX.exec(rest)
-    const field = fieldMatch ? fieldMatch[1] : ''
-    i += fieldMatch ? fieldMatch[0].length : 0
-
-    const value = query.slice(i)
-    if (value.startsWith('"')) {
-      const end = query.indexOf('"', i + 1)
-      if (end < 0) throw new UnreadableQuery()
-      tokens.push({
-        type: 'term',
-        field,
-        value: query.slice(i + 1, end),
-        quoted: true,
-      })
-      i = end + 1
-    } else if (value.startsWith('[')) {
-      const end = query.indexOf(']', i + 1)
-      if (end < 0) throw new UnreadableQuery()
-      tokens.push({
-        type: 'term',
-        field,
-        value: query.slice(i + 1, end),
-        range: true,
-      })
-      i = end + 1
-    } else {
-      const bare = /^[^\s()]+/.exec(value)
-      if (!bare) throw new UnreadableQuery()
-      tokens.push({ type: 'term', field, value: bare[0] })
-      i += bare[0].length
-    }
-    // Nothing may run on directly after a value, e.g. title:"a"b
-    if (i < query.length && !/[\s)]/.test(query[i])) throw new UnreadableQuery()
-  }
-  return tokens
-}
-
-function resolveField(name: string): AdvancedSearchField {
-  const fields = Object.values(AdvancedSearchField) as string[]
-  // Aliases (type:, path:, notes:) are left to the text box on purpose:
-  // reading one would mean rewriting the user's query as it was read
-  if (!fields.includes(name)) throw new UnreadableQuery()
-  return name as AdvancedSearchField
-}
-
-function atomFrom(
-  field: AdvancedSearchField,
-  operator: AdvancedSearchOperator,
-  value: string,
-  extra: Partial<AdvancedSearchQueryAtom> = {}
-): AdvancedSearchQueryAtom {
-  return {
-    type: AdvancedSearchQueryElementType.Atom,
-    field,
-    operator,
-    value,
-    ...extra,
-  }
-}
-
-function parseRange(
-  field: AdvancedSearchField,
-  kind: AdvancedSearchFieldKind,
-  body: string
-): AdvancedSearchQueryAtom {
-  const bounds = RANGE_BOUNDS.exec(body)
-  if (!bounds) throw new UnreadableQuery()
-  const lo = bounds[1] ?? ''
-  const hi = bounds[2] ?? ''
-
-  if (kind === AdvancedSearchFieldKind.Date) {
-    const relative = RELATIVE_BOUND.exec(lo)
-    if (relative && hi === 'now') {
-      return atomFrom(field, AdvancedSearchOperator.WithinLast, relative[1], {
-        unit: relative[2] as AdvancedSearchDateUnit,
-      })
-    }
-  } else if (kind !== AdvancedSearchFieldKind.Number) {
-    throw new UnreadableQuery()
-  }
-
-  const isValid =
-    kind === AdvancedSearchFieldKind.Date
-      ? (v: string) => ISO_DATE.test(v)
-      : (v: string) => WHOLE_NUMBER.test(v)
-
-  if (lo && hi) {
-    if (!isValid(lo) || !isValid(hi)) throw new UnreadableQuery()
-    return atomFrom(field, AdvancedSearchOperator.Between, lo, { valueTo: hi })
-  }
-  if (lo && isValid(lo))
-    return atomFrom(field, AdvancedSearchOperator.AtLeast, lo)
-  if (hi && isValid(hi))
-    return atomFrom(field, AdvancedSearchOperator.AtMost, hi)
-  throw new UnreadableQuery()
-}
-
-function parseTerm(token: Token): Parsed {
-  const field = resolveField(token.field)
-  const kind = ADVANCED_SEARCH_FIELD_KINDS[field]
-  const value = token.value
-  const isKeyword = (
-    ADVANCED_SEARCH_DATE_KEYWORDS as readonly string[]
-  ).includes(value)
-
-  if (token.range) {
-    return { element: parseRange(field, kind, value) }
-  }
-
-  if (kind === AdvancedSearchFieldKind.Date) {
-    if (!isKeyword) throw new UnreadableQuery()
-    return {
-      element: atomFrom(field, AdvancedSearchOperator.DateKeyword, value),
-    }
-  }
-
-  if (token.quoted) {
-    if (kind !== AdvancedSearchFieldKind.Text) throw new UnreadableQuery()
-    return { element: atomFrom(field, AdvancedSearchOperator.Phrase, value) }
-  }
-
-  const wildcard = TRAILING_WILDCARD.exec(value)
-  if (wildcard) {
-    if (kind === AdvancedSearchFieldKind.Number) throw new UnreadableQuery()
-    return {
-      element: atomFrom(field, AdvancedSearchOperator.StartsWith, wildcard[1]),
-    }
-  }
-
-  if (kind === AdvancedSearchFieldKind.Number) {
-    if (!WHOLE_NUMBER.test(value)) throw new UnreadableQuery()
-    return { element: atomFrom(field, AdvancedSearchOperator.Equals, value) }
-  }
-  // A checksum is only ever searched by its first characters
-  if (kind === AdvancedSearchFieldKind.Checksum) throw new UnreadableQuery()
-
-  return {
-    element: atomFrom(field, AdvancedSearchOperator.AllWords, value),
-    word: { field, text: value },
-  }
-}
-
-// The serializer repeats the field for every word, because a field applies
-// only to the word after it. Put those back together into one condition.
-function mergeWords(
-  parts: Parsed[],
-  operator: AdvancedSearchLogicalOperator.And | AdvancedSearchLogicalOperator.Or
-): AdvancedSearchQueryElement[] {
-  const merged: AdvancedSearchQueryElement[] = []
-  for (let i = 0; i < parts.length; i++) {
-    const run = [parts[i]]
-    while (
-      parts[i].word &&
-      parts[i + 1]?.word &&
-      parts[i + 1].word.field === parts[i].word.field
-    ) {
-      run.push(parts[++i])
-    }
-    if (run.length === 1) {
-      merged.push(run[0].element)
-      continue
-    }
-    merged.push(
-      atomFrom(
-        run[0].word.field,
-        operator === AdvancedSearchLogicalOperator.And
-          ? AdvancedSearchOperator.AllWords
-          : AdvancedSearchOperator.AnyWord,
-        run.map((part) => part.word.text).join(' ')
-      )
-    )
-  }
-  return merged
-}
-
-interface Cursor {
-  tokens: Token[]
-  at: number
-}
-
-function parseExpression(cursor: Cursor): Parsed {
-  const parts: Parsed[] = [parseOperand(cursor)]
-  let operator:
-    AdvancedSearchLogicalOperator.And | AdvancedSearchLogicalOperator.Or
-  while (
-    cursor.tokens[cursor.at]?.type === 'AND' ||
-    cursor.tokens[cursor.at]?.type === 'OR'
-  ) {
-    const next = cursor.tokens[cursor.at++].type as
-      AdvancedSearchLogicalOperator.And | AdvancedSearchLogicalOperator.Or
-    // One level mixing AND and OR is never something the editor wrote
-    if (operator && next !== operator) throw new UnreadableQuery()
-    operator = next
-    parts.push(parseOperand(cursor))
-  }
-  if (parts.length === 1) return parts[0]
-
-  const children = mergeWords(parts, operator)
-  if (children.length === 1) return { element: children[0] }
-  return {
-    element: {
-      type: AdvancedSearchQueryElementType.Group,
-      operator,
-      children,
-    },
-  }
-}
-
-function parseOperand(cursor: Cursor): Parsed {
-  const token = cursor.tokens[cursor.at++]
-  if (!token) throw new UnreadableQuery()
-
-  if (token.type === 'NOT') {
-    const child = parseOperand(cursor)
-    return {
-      element: {
-        type: AdvancedSearchQueryElementType.Group,
-        operator: AdvancedSearchLogicalOperator.Not,
-        children: [child.element],
-      },
-    }
-  }
-  if (token.type === '(') {
-    const inner = parseExpression(cursor)
-    if (cursor.tokens[cursor.at++]?.type !== ')') throw new UnreadableQuery()
-    return { element: inner.element }
-  }
-  if (token.type !== 'term') throw new UnreadableQuery()
-  return parseTerm(token)
-}
-
-/**
- * Reads a query the editor could have written back into an editor tree, or
- * returns null when the editor cannot show it, in which case the query stays
- * text. Never returns a tree that would be written back differently.
- */
-export function parseAdvancedSearchQuery(
-  query: string
-): AdvancedSearchQueryGroup | null {
-  const trimmed = query?.trim() ?? ''
-  if (!trimmed) return null
-
-  let parsed: Parsed
-  try {
-    const cursor: Cursor = { tokens: tokenize(trimmed), at: 0 }
-    parsed = parseExpression(cursor)
-    if (cursor.at !== cursor.tokens.length) throw new UnreadableQuery()
-  } catch {
-    return null
-  }
-
-  const root =
-    parsed.element.type === AdvancedSearchQueryElementType.Group
-      ? parsed.element
-      : {
-          type: AdvancedSearchQueryElementType.Group as const,
-          operator: AdvancedSearchLogicalOperator.And,
-          children: [parsed.element],
-        }
-
-  return serializeAdvancedSearchQuery(root) === trimmed ? root : null
-}

src-ui/src/environments/environment.prod.ts

@@ -8,7 +8,7 @@ export const environment = {
   apiVersion: '10', // match src/paperless/settings.py
   appTitle: DEFAULT_APP_TITLE,
   tag: 'prod',
-  version: '3.2.0',
+  version: '3.1.3',
   webSocketHost: window.location.host,
   webSocketProtocol: window.location.protocol == 'https:' ? 'wss:' : 'ws:',
   webSocketBaseUrl: base_url.pathname + 'ws/',