[maven-release-plugin][skip ci] prepare release 25.1.0.10

[APPS-3130] reverting changes of MNT-24137 (#3076 )
* [APPS-3130] reverting changes of MNT-24137 * reverting * reverting the change of mnt-24137 * reverting the change of mnt-24137 * reverting the change of mnt-24137 * reverting the changes of mnt-24137 * reverting the changes of mnt-24137 * reverting the changes of mnt-24137 * [APPS-3130] reverting changes of MNT-24137 --------- Co-authored-by: Sathish Kumar <ST28@ford.com>
2025-10-08 14:51:49 +00:00 · 2024-12-06 11:52:47 +00:00 · 2024-12-06 16:40:38 +05:30 · 2024-12-05 15:46:52 +00:00 · 2024-12-05 15:46:49 +00:00 · 2024-12-05 09:33:04 +01:00
43 changed files with 3819 additions and 3736 deletions
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -44,14 +44,14 @@ jobs:
      - uses: actions/checkout@v4
        with:
          fetch-depth: 0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - id: changed-files
-        uses: Alfresco/alfresco-build-tools/.github/actions/github-list-changes@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/github-list-changes@v8.2.0
        with:
          write-list-to-env: true
-      - uses: Alfresco/alfresco-build-tools/.github/actions/pre-commit@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/pre-commit@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Prepare maven cache and check compilation"
@@ -69,12 +69,12 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
-      - uses: Alfresco/alfresco-build-tools/.github/actions/veracode@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/veracode@v8.2.0
        continue-on-error: true
        with:
          srcclr-api-token: ${{ secrets.SRCCLR_API_TOKEN }}
@@ -92,10 +92,10 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/github-download-file@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/github-download-file@v8.2.0
        with:
          token: ${{ secrets.BOT_GITHUB_TOKEN }}
          repository: "Alfresco/veracode-baseline-archive"
@@ -142,9 +142,9 @@ jobs:
      !contains(github.event.head_commit.message, '[skip tests]') &&
      !contains(github.event.head_commit.message, '[force]')
    steps:
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - uses: Alfresco/ya-pmd-scan@v4.1.0
        with:
          classpath-build-command: "mvn test-compile -ntp -Pags -pl \"-:alfresco-community-repo-docker\""
@@ -175,14 +175,14 @@ jobs:
            testAttributes: "-Dtest=AllMmtUnitTestSuite"
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} - ${{ matrix.testModule }}
@@ -213,7 +213,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -255,9 +255,9 @@ jobs:
      REQUIRES_INSTALLED_ARTIFACTS: true
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Build"
        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
        run: |
@@ -270,7 +270,7 @@ jobs:
        run: docker compose -f ./scripts/ci/docker-compose/docker-compose.yaml --profile ${{ matrix.compose-profile }} up -d
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} - ${{ matrix.testSuite }}
@@ -301,7 +301,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -334,9 +334,9 @@ jobs:
        version: ['10.2.18', '10.4', '10.5']
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: Run MariaDB ${{ matrix.version }} database
@@ -345,7 +345,7 @@ jobs:
          MARIADB_VERSION: ${{ matrix.version }}
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} - ${{ matrix.version }}
@@ -376,7 +376,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -405,9 +405,9 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run MariaDB 10.6 database"
@@ -416,7 +416,7 @@ jobs:
          MARIADB_VERSION: 10.6
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -447,7 +447,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -476,9 +476,9 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run MySQL 8 database"
@@ -487,7 +487,7 @@ jobs:
          MYSQL_VERSION: 8
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -518,7 +518,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -546,9 +546,9 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run PostgreSQL 13.12 database"
@@ -557,7 +557,7 @@ jobs:
          POSTGRES_VERSION: 13.12
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -588,7 +588,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -616,9 +616,9 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run PostgreSQL 14.9 database"
@@ -627,7 +627,7 @@ jobs:
          POSTGRES_VERSION: 14.9
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -658,7 +658,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -686,9 +686,9 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run PostgreSQL 15.4 database"
@@ -697,7 +697,7 @@ jobs:
          POSTGRES_VERSION: 15.4
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -728,7 +728,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -754,16 +754,16 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run ActiveMQ"
        run: docker compose -f ./scripts/ci/docker-compose/docker-compose.yaml --profile activemq up -d
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -794,7 +794,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -854,9 +854,9 @@ jobs:
            mvn-options: '-Dencryption.ssl.keystore.location=${CI_WORKSPACE}/keystores/alfresco/alfresco.keystore -Dencryption.ssl.truststore.location=${CI_WORKSPACE}/keystores/alfresco/alfresco.truststore'
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Set transformers tag"
@@ -879,7 +879,7 @@ jobs:
        run: docker compose -f ./scripts/ci/docker-compose/docker-compose.yaml --profile ${{ matrix.compose-profile }} up -d
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} - ${{ matrix.testSuite }} ${{ matrix.idp }}
@@ -910,7 +910,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -968,9 +968,9 @@ jobs:
      REQUIRES_LOCAL_IMAGES: true
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Build"
        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
        run: |
@@ -986,7 +986,7 @@ jobs:
        run: mvn install -pl :alfresco-community-repo-integration-test -am -DskipTests -Pall-tas-tests
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} - ${{ matrix.test-name }}
@@ -1024,7 +1024,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.tests.outcome }}
@@ -1050,16 +1050,16 @@ jobs:
      !contains(github.event.head_commit.message, '[force')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
      - name: "Run Postgres 15.4 database"
        run: docker compose -f ./scripts/ci/docker-compose/docker-compose.yaml --profile postgres up -d
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -1090,7 +1090,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -1124,9 +1124,9 @@ jobs:
      REQUIRES_INSTALLED_ARTIFACTS: true
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Build"
        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
        run: |
@@ -1134,7 +1134,7 @@ jobs:
          bash ./scripts/ci/build.sh
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} 0${{ matrix.part }} - (PostgreSQL) ${{ matrix.test-name }}
@@ -1170,9 +1170,9 @@ jobs:
      REQUIRES_INSTALLED_ARTIFACTS: true
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Build"
        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
        run: |
@@ -1180,7 +1180,7 @@ jobs:
          bash ./scripts/ci/build.sh
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }} 0${{ matrix.part }} - (MySQL) ${{ matrix.test-name }}
@@ -1212,9 +1212,9 @@ jobs:
      REQUIRES_LOCAL_IMAGES: true
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Build"
        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
        run: |
@@ -1228,7 +1228,7 @@ jobs:
          mvn -B install -pl :alfresco-governance-services-automation-community-rest-api -am -Pags -Pall-tas-tests -DskipTests
      - name: "Prepare Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-prepare@v8.2.0
        id: rp-prepare
        with:
          rp-launch-prefix: ${{ env.RP_LAUNCH_PREFIX }}
@@ -1260,7 +1260,7 @@ jobs:
        continue-on-error: true
      - name: "Summarize Report Portal"
        if: github.ref_name == 'master'
-        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v7.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/reportportal-summarize@v8.2.0
        id: rp-summarize
        with:
          tests-outcome: ${{ steps.run-tests.outcome }}
@@ -1302,9 +1302,9 @@ jobs:
      !contains(github.event.head_commit.message, '[force]')
    steps:
      - uses: actions/checkout@v4
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Build"
        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
        run: |
--- a/.github/workflows/master_release.yml
+++ b/.github/workflows/master_release.yml
@@ -34,12 +34,12 @@ jobs:
      - uses: actions/checkout@v4
        with:
          persist-credentials: false
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
-      - uses: Alfresco/alfresco-build-tools/.github/actions/configure-git-author@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/configure-git-author@v8.2.0
        with:
          username: ${{ env.GIT_USERNAME }}
          email: ${{ env.GIT_EMAIL }}
@@ -63,12 +63,12 @@ jobs:
      - uses: actions/checkout@v4
        with:
          persist-credentials: false
-      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.2.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.2.0
      - name: "Init"
        run: bash ./scripts/ci/init.sh
-      - uses: Alfresco/alfresco-build-tools/.github/actions/configure-git-author@v7.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/configure-git-author@v8.2.0
        with:
          username: ${{ env.GIT_USERNAME }}
          email: ${{ env.GIT_EMAIL }}
--- a/amps/ags/pom.xml
+++ b/amps/ags/pom.xml
@@ -7,7 +7,7 @@
   <parent>
      <groupId>org.alfresco</groupId>
      <artifactId>alfresco-community-repo-amps</artifactId>
-      <version>23.4.0.67</version>
+      <version>25.1.0.10</version>
   </parent>
   <modules>
--- a/amps/ags/rm-automation/pom.xml
+++ b/amps/ags/rm-automation/pom.xml
@@ -7,7 +7,7 @@
   <parent>
      <groupId>org.alfresco</groupId>
      <artifactId>alfresco-governance-services-community-parent</artifactId>
-      <version>23.4.0.67</version>
+      <version>25.1.0.10</version>
   </parent>
   <modules>
--- a/amps/ags/rm-automation/rm-automation-community-rest-api/pom.xml
+++ b/amps/ags/rm-automation/rm-automation-community-rest-api/pom.xml
@@ -7,7 +7,7 @@
   <parent>
      <groupId>org.alfresco</groupId>
      <artifactId>alfresco-governance-services-automation-community-repo</artifactId>
-      <version>23.4.0.67</version>
+      <version>25.1.0.10</version>
   </parent>
   <build>
--- a/amps/ags/rm-community/pom.xml
+++ b/amps/ags/rm-community/pom.xml
@@ -7,7 +7,7 @@
   <parent>
      <groupId>org.alfresco</groupId>
      <artifactId>alfresco-governance-services-community-parent</artifactId>
-      <version>23.4.0.67</version>
+      <version>25.1.0.10</version>
   </parent>
   <modules>
--- a/amps/ags/rm-community/rm-community-repo/pom.xml
+++ b/amps/ags/rm-community/rm-community-repo/pom.xml
@@ -8,7 +8,7 @@
   <parent>
      <groupId>org.alfresco</groupId>
      <artifactId>alfresco-governance-services-community-repo-parent</artifactId>
-      <version>23.4.0.67</version>
+      <version>25.1.0.10</version>
   </parent>
   <properties>
--- a/amps/ags/rm-community/rm-community-repo/test/resources/alfresco/version.properties
+++ b/amps/ags/rm-community/rm-community-repo/test/resources/alfresco/version.properties
@@ -3,8 +3,8 @@
 #
 # Version label
-version.major=23
+version.major=25
-version.minor=4
+version.minor=1
 version.revision=0
 version.label=
--- a/amps/ags/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/patch/v33/RMv33HoldAuditEntryValuesPatchUnitTest.java
+++ b/amps/ags/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/patch/v33/RMv33HoldAuditEntryValuesPatchUnitTest.java
@@ -33,14 +33,15 @@ import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 import org.alfresco.module.org_alfresco_module_rm.query.RecordsManagementQueryDAO;
 import org.alfresco.repo.domain.propval.PropertyStringValueEntity;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
 import org.alfresco.module.org_alfresco_module_rm.query.RecordsManagementQueryDAO;
 import org.alfresco.repo.domain.propval.PropertyStringValueEntity;
 /**
 * RM V3.3 Hold audit entries values patch unit test
 *
@@ -55,7 +56,6 @@ public class RMv33HoldAuditEntryValuesPatchUnitTest
    @InjectMocks
    private RMv33HoldAuditEntryValuesPatch patch;
    @Before
    public void setUp()
    {
@@ -93,15 +93,15 @@ public class RMv33HoldAuditEntryValuesPatchUnitTest
        verify(mockedRecordsManagementQueryDAO, times(1)).updatePropertyStringValueEntity(deleteHoldPropertyStringValueEntity);
        assertEquals("Add To Hold", addToHoldPropertyStringValueEntity.getStringValue());
-        assertEquals("add to hold", addToHoldPropertyStringValueEntity.getStringLower());
+        assertEquals("add to hold", addToHoldPropertyStringValueEntity.getStringEndLower());
        assertEquals(Long.valueOf(770_786_109L), addToHoldPropertyStringValueEntity.getStringCrc());
        assertEquals("Remove From Hold", removeFromHoldPropertyStringValueEntity.getStringValue());
-        assertEquals("remove from hold", removeFromHoldPropertyStringValueEntity.getStringLower());
+        assertEquals("remove from hold", removeFromHoldPropertyStringValueEntity.getStringEndLower());
        assertEquals(Long.valueOf(2_967_613_012L), removeFromHoldPropertyStringValueEntity.getStringCrc());
        assertEquals("Delete Hold", deleteHoldPropertyStringValueEntity.getStringValue());
-        assertEquals("delete hold", deleteHoldPropertyStringValueEntity.getStringLower());
+        assertEquals("delete hold", deleteHoldPropertyStringValueEntity.getStringEndLower());
        assertEquals(Long.valueOf(132_640_810L), deleteHoldPropertyStringValueEntity.getStringCrc());
    }
@@ -123,5 +123,3 @@ public class RMv33HoldAuditEntryValuesPatchUnitTest
        verify(mockedRecordsManagementQueryDAO, times(0)).updatePropertyStringValueEntity(any());
    }
 }
--- a/amps/ags/rm-community/rm-community-rest-api-explorer/pom.xml
+++ b/amps/ags/rm-community/rm-community-rest-api-explorer/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-governance-services-community-repo-parent</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <build>
--- a/amps/pom.xml
+++ b/amps/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <modules>
--- a/amps/share-services/pom.xml
+++ b/amps/share-services/pom.xml
@@ -8,7 +8,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-amps</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <properties>
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -7,7 +7,7 @@
   <parent>
      <groupId>org.alfresco</groupId>
      <artifactId>alfresco-community-repo</artifactId>
-      <version>23.4.0.67</version>
+      <version>25.1.0.10</version>
   </parent>
   <dependencies>
--- a/data-model/pom.xml
+++ b/data-model/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <properties>
@@ -118,6 +118,18 @@
            <groupId>org.jibx</groupId>
            <artifactId>jibx-run</artifactId>
            <version>1.4.2</version>
            <exclusions>
                <!-- [ACS-5371] Excluded to avoid conflict in JDK9+ as it includes javax.xml-->
                <exclusion>
                    <groupId>xpp3</groupId>
                    <artifactId>xpp3</artifactId>
                </exclusion>
            </exclusions>
        </dependency>
        <dependency>
            <groupId>net.sf.kxml</groupId>
            <artifactId>kxml2</artifactId>
            <version>${dependency.kxml2.version}</version>
        </dependency>
        <dependency> 
            <groupId>com.fasterxml.jackson.core</groupId> 
--- a/mmt/pom.xml
+++ b/mmt/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <dependencies>
--- a/packaging/distribution/pom.xml
+++ b/packaging/distribution/pom.xml
@@ -9,6 +9,6 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-packaging</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
 </project>
--- a/packaging/distribution/src/main/resources/licenses/notice.txt
+++ b/packaging/distribution/src/main/resources/licenses/notice.txt
@@ -146,6 +146,7 @@ libgif  http://giflib.sourceforge.net/
 libfreetype http://www.freetype.org/
 PostgreSQL  http://www.postgresql.org/
 PostgreSQL JDBC Driver  http://www.postgresql.org/
 kXML 2 	http://kxml.sourceforge.net/
 === CDDL 1.0 ===
--- a/packaging/docker-alfresco/pom.xml
+++ b/packaging/docker-alfresco/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-packaging</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <properties>
--- a/packaging/pom.xml
+++ b/packaging/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <modules>
--- a/packaging/tests/pom.xml
+++ b/packaging/tests/pom.xml
@@ -6,7 +6,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-packaging</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <modules>
--- a/packaging/tests/tas-cmis/pom.xml
+++ b/packaging/tests/tas-cmis/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-tests</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <organization>
--- a/packaging/tests/tas-email/pom.xml
+++ b/packaging/tests/tas-email/pom.xml
@@ -9,7 +9,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-tests</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <developers>
--- a/packaging/tests/tas-integration/pom.xml
+++ b/packaging/tests/tas-integration/pom.xml
@@ -9,7 +9,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-tests</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <developers>
--- a/packaging/tests/tas-restapi/pom.xml
+++ b/packaging/tests/tas-restapi/pom.xml
@@ -8,7 +8,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-tests</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <properties>
@@ -171,7 +171,7 @@
        <dependency>
            <groupId>org.codehaus.groovy</groupId>
            <artifactId>groovy</artifactId>
-            <version>3.0.22</version>
+            <version>3.0.23</version>
        </dependency>
        <!-- https://mvnrepository.com/artifact/org.codehaus.groovy/groovy-json-->
--- a/packaging/tests/tas-webdav/pom.xml
+++ b/packaging/tests/tas-webdav/pom.xml
@@ -9,7 +9,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-tests</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <developers>
--- a/packaging/war/pom.xml
+++ b/packaging/war/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo-packaging</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <properties>
--- a/pom.xml
+++ b/pom.xml
@@ -2,7 +2,7 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
    <modelVersion>4.0.0</modelVersion>
    <artifactId>alfresco-community-repo</artifactId>
-    <version>23.4.0.67</version>
+    <version>25.1.0.10</version>
    <packaging>pom</packaging>
    <name>Alfresco Community Repo Parent</name>
@@ -23,8 +23,8 @@
    </modules>
    <properties>
-        <acs.version.major>23</acs.version.major>
+        <acs.version.major>25</acs.version.major>
-        <acs.version.minor>4</acs.version.minor>
+        <acs.version.minor>1</acs.version.minor>
        <acs.version.revision>0</acs.version.revision>
        <acs.version.label />
        <amp.min.version>${acs.version.major}.0.0</amp.min.version>
@@ -64,38 +64,39 @@
        <dependency.cxf.version>4.0.5</dependency.cxf.version>
        <dependency.opencmis.version>1.0.0-jakarta-1</dependency.opencmis.version>
        <dependency.webscripts.version>9.4</dependency.webscripts.version>
-        <dependency.bouncycastle.version>1.78.1</dependency.bouncycastle.version>
+        <dependency.bouncycastle.version>1.79</dependency.bouncycastle.version>
        <dependency.mockito-core.version>5.14.1</dependency.mockito-core.version>
        <dependency.assertj.version>3.26.3</dependency.assertj.version>
        <dependency.org-json.version>20240303</dependency.org-json.version>
        <dependency.commons-dbcp.version>2.12.0</dependency.commons-dbcp.version>
-        <dependency.commons-io.version>2.17.0</dependency.commons-io.version>
+        <dependency.commons-io.version>2.18.0</dependency.commons-io.version>
        <dependency.gson.version>2.11.0</dependency.gson.version>
        <dependency.guava.version>33.3.1-jre</dependency.guava.version>
        <dependency.httpclient.version>4.5.14</dependency.httpclient.version>
        <dependency.httpcore.version>4.4.16</dependency.httpcore.version>
-        <dependency.httpcomponents-httpclient5.version>5.2.1</dependency.httpcomponents-httpclient5.version>
+        <dependency.httpcomponents-httpclient5.version>5.4.1</dependency.httpcomponents-httpclient5.version>
        <dependency.httpcomponents-httpcore5.version>5.3</dependency.httpcomponents-httpcore5.version>
        <dependency.commons-httpclient.version>3.1-HTTPCLIENT-1265</dependency.commons-httpclient.version>
        <dependency.xercesImpl.version>2.12.2</dependency.xercesImpl.version>
        <dependency.slf4j.version>2.0.16</dependency.slf4j.version>
-        <dependency.log4j.version>2.23.1</dependency.log4j.version>
+        <dependency.log4j.version>2.24.2</dependency.log4j.version>
-        <dependency.groovy.version>3.0.22</dependency.groovy.version>
+        <dependency.groovy.version>3.0.23</dependency.groovy.version>
        <dependency.tika.version>2.9.2</dependency.tika.version>
        <dependency.truezip.version>7.7.10</dependency.truezip.version>
        <dependency.poi.version>5.3.0</dependency.poi.version>
        <dependency.jboss.logging.version>3.5.0.Final</dependency.jboss.logging.version>
        <dependency.camel.version>4.6.0</dependency.camel.version> <!-- when bumping this version, please keep track/sync with included netty.io dependencies -->
        <dependency.netty.version>4.1.113.Final</dependency.netty.version> <!-- must be in sync with camels transitive dependencies, e.g.: netty-common -->
-        <dependency.activemq.version>5.18.3</dependency.activemq.version>
+        <dependency.activemq.version>5.18.6</dependency.activemq.version>
        <dependency.apache-compress.version>1.27.1</dependency.apache-compress.version>
        <dependency.awaitility.version>4.2.2</dependency.awaitility.version>
        <dependency.swagger-ui.version>4.1.3</dependency.swagger-ui.version>
-        <dependency.swagger-parser.version>1.0.71</dependency.swagger-parser.version>
+        <dependency.swagger-parser.version>1.0.72</dependency.swagger-parser.version>
        <dependency.maven-filtering.version>3.1.1</dependency.maven-filtering.version>
        <dependency.maven-artifact.version>3.8.6</dependency.maven-artifact.version>
        <dependency.jdom2.version>2.0.6.1</dependency.jdom2.version>
        <dependency.pooled-jms.version>3.1.6</dependency.pooled-jms.version>
        <dependency.kxml2.version>2.3.0</dependency.kxml2.version>
        <dependency.jakarta-ee-jaxb-api.version>4.0.2</dependency.jakarta-ee-jaxb-api.version>
        <dependency.jakarta-ee-jaxb-impl.version>4.0.5</dependency.jakarta-ee-jaxb-impl.version>
@@ -154,7 +155,7 @@
        <connection>scm:git:https://github.com/Alfresco/alfresco-community-repo.git</connection>
        <developerConnection>scm:git:https://github.com/Alfresco/alfresco-community-repo.git</developerConnection>
        <url>https://github.com/Alfresco/alfresco-community-repo</url>
-        <tag>23.4.0.67</tag>
+        <tag>25.1.0.10</tag>
    </scm>
    <distributionManagement>
@@ -371,7 +372,7 @@
            <dependency>
                <groupId>org.quartz-scheduler</groupId>
                <artifactId>quartz</artifactId>
-                <version>2.3.2</version>
+                <version>2.5.0</version>
                <!-- exclude c3p0 -->
                <!-- see https://issues.alfresco.com/jira/browse/REPO-3447 -->
                <exclusions>
@@ -465,7 +466,7 @@
            <dependency>
                <groupId>org.apache.xmlbeans</groupId>
                <artifactId>xmlbeans</artifactId>
-                <version>5.2.1</version>
+                <version>5.2.2</version>
            </dependency>
            <dependency>
                <groupId>org.json</groupId>
@@ -703,7 +704,7 @@
            <dependency>
                <groupId>com.networknt</groupId>
                <artifactId>json-schema-validator</artifactId>
-                <version>1.5.1</version>
+                <version>1.5.4</version>
            </dependency>
            <!-- upgrade dependency from TIKA -->
            <dependency>
@@ -950,7 +951,7 @@
            <dependency>
                <groupId>org.projectlombok</groupId>
                <artifactId>lombok</artifactId>
-                <version>1.18.34</version>
+                <version>1.18.36</version>
                <scope>provided</scope>
            </dependency>
            <dependency>
@@ -1015,7 +1016,7 @@
                </plugin>
                <plugin>
                    <artifactId>maven-failsafe-plugin</artifactId>
-                    <version>3.5.1</version>
+                    <version>3.5.2</version>
                </plugin>
                <plugin>
                    <artifactId>maven-jar-plugin</artifactId>
@@ -1028,7 +1029,7 @@
                <plugin>
                    <groupId>org.apache.maven.plugins</groupId>
                    <artifactId>maven-javadoc-plugin</artifactId>
-                    <version>3.10.1</version>
+                    <version>3.11.1</version>
                </plugin>
                <plugin>
                    <groupId>org.apache.maven.plugins</groupId>
--- a/remote-api/pom.xml
+++ b/remote-api/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <dependencies>
@@ -45,7 +45,7 @@
        <dependency>
            <groupId>org.apache.santuario</groupId>
            <artifactId>xmlsec</artifactId>
-            <version>4.0.2</version>
+            <version>4.0.3</version>
        </dependency>
        <!-- newer version, see REPO-3133 -->
        <dependency>
@@ -119,7 +119,7 @@
        <dependency>
            <groupId>org.eclipse.jetty</groupId>
            <artifactId>jetty-server</artifactId>
-            <version>11.0.15</version>
+            <version>11.0.24</version>
            <scope>test</scope>
            <exclusions>
                <exclusion>
--- a/remote-api/src/main/java/org/alfresco/rest/api/impl/AuditImpl.java
+++ b/remote-api/src/main/java/org/alfresco/rest/api/impl/AuditImpl.java
@@ -85,7 +85,7 @@ public class AuditImpl implements Audit
    // list of equals filter's auditEntry (via where clause)
    private final static Set<String> LIST_AUDIT_ENTRY_EQUALS_QUERY_PROPERTIES = new HashSet<>(
-            Arrays.asList(new String[] { CREATED_BY_USER, VALUES_KEY, VALUES_VALUE }));
+            Arrays.asList(new String[]{CREATED_BY_USER, VALUES_KEY, VALUES_VALUE}));
    // map of sort parameters for the moment one createdAt
    private final static Map<String, String> SORT_PARAMS_TO_NAMES;
@@ -295,23 +295,29 @@ public class AuditImpl implements Audit
        }
        else
        {
-            if (hasMoreItems) {
+            if (hasMoreItems)
-                if (q != null) {
+            {
                if (q != null)
                {
                    // filtering via "where" clause
                    AuditEntryQueryWalker propertyWalker = new AuditEntryQueryWalker();
                    QueryHelper.walk(q, propertyWalker);
                    totalItems = getAuditEntriesCountByAppAndProperties(auditApplication, propertyWalker);
-                } else {
+                }
                else
                {
                    totalItems = getAuditEntriesCountByApp(auditApplication);
                }
-            } else {
+            }
            else
            {
                totalItems = totalRetrievedItems;
            }
        }
        entriesAudit = (skipCount >= totalRetrievedItems)
-                        ? Collections.emptyList()
+                ? Collections.emptyList()
-                        : entriesAudit.subList(skipCount, end);
+                : entriesAudit.subList(skipCount, end);
        return CollectionWithPagingInfo.asPaged(paging, entriesAudit, hasMoreItems, totalItems);
    }
@@ -475,8 +481,7 @@ public class AuditImpl implements Audit
        final Map<String, UserInfo> mapUserInfo = new HashMap<>(10);
        // create the callback for auditQuery method
-        final AuditQueryCallback callback = new AuditQueryCallback()
+        final AuditQueryCallback callback = new AuditQueryCallback() {
        {
            public boolean valuesRequired()
            {
                return ((includeParam != null) && (includeParam.contains(PARAM_INCLUDE_VALUES)));
@@ -532,7 +537,7 @@ public class AuditImpl implements Audit
        params.setApplicationName(auditApplication.getName());
        params.setFromId(auditEntryId);
        params.setToId(auditEntryId + 1);
-        
+
        List<String> includeParam = new ArrayList<>();
        if (parameters != null)
        {
@@ -545,8 +550,7 @@ public class AuditImpl implements Audit
        final List<AuditEntry> results = new ArrayList<>();
        // create the callback for auditQuery method
-        final AuditQueryCallback callback = new AuditQueryCallback()
+        final AuditQueryCallback callback = new AuditQueryCallback() {
        {
            public boolean valuesRequired()
            {
                return ((includeParam != null) && (includeParam.contains(PARAM_INCLUDE_VALUES)));
@@ -710,8 +714,7 @@ public class AuditImpl implements Audit
    @Override
    public CollectionWithPagingInfo<AuditEntry> listAuditEntriesByNodeId(String nodeId, Parameters parameters)
    {
-        AuthenticationUtil.runAs(new RunAsWork<Void>()
+        AuthenticationUtil.runAs(new RunAsWork<Void>() {
        {
            public Void doWork() throws Exception
            {
                checkEnabled();
@@ -772,8 +775,7 @@ public class AuditImpl implements Audit
        final List<AuditEntry> results = new ArrayList<>();
        String auditAppId = "alfresco-access";
-        String auditApplicationName = AuthenticationUtil.runAs(new RunAsWork<String>()
+        String auditApplicationName = AuthenticationUtil.runAs(new RunAsWork<String>() {
        {
            public String doWork() throws Exception
            {
                return findAuditAppByIdOr404(auditAppId).getName();
@@ -781,8 +783,7 @@ public class AuditImpl implements Audit
        }, AuthenticationUtil.getSystemUserName());
        // create the callback for auditQuery method
-        final AuditQueryCallback callback = new AuditQueryCallback()
+        final AuditQueryCallback callback = new AuditQueryCallback() {
        {
            public boolean valuesRequired()
            {
                return ((includeParam != null) && (includeParam.contains(PARAM_INCLUDE_VALUES)));
@@ -808,8 +809,7 @@ public class AuditImpl implements Audit
        Long toTime = propertyWalker.getToTime();
        validateWhereBetween(nodeRef.getId(), fromTime, toTime);
-        AuthenticationUtil.runAs(new RunAsWork<Object>()
+        AuthenticationUtil.runAs(new RunAsWork<Object>() {
        {
            public Object doWork() throws Exception
            {
                // QueryParameters
@@ -820,7 +820,7 @@ public class AuditImpl implements Audit
                pathParams.setFromTime(fromTime);
                pathParams.setToTime(toTime);
                pathParams.setApplicationName(auditApplicationName);
-                pathParams.addSearchKey("/"+auditAppId+"/transaction/path", nodePath);
+                pathParams.addSearchKey("/" + auditAppId + "/transaction/path", nodePath);
                auditService.auditQuery(callback, pathParams, limit);
                AuditQueryParameters copyFromPathParams = new AuditQueryParameters();
@@ -830,7 +830,7 @@ public class AuditImpl implements Audit
                copyFromPathParams.setFromTime(fromTime);
                copyFromPathParams.setToTime(toTime);
                copyFromPathParams.setApplicationName(auditApplicationName);
-                copyFromPathParams.addSearchKey("/"+auditAppId+"/transaction/copy/from/path", nodePath);
+                copyFromPathParams.addSearchKey("/" + auditAppId + "/transaction/copy/from/path", nodePath);
                auditService.auditQuery(callback, copyFromPathParams, limit);
                AuditQueryParameters moveFromPathParams = new AuditQueryParameters();
@@ -840,7 +840,7 @@ public class AuditImpl implements Audit
                moveFromPathParams.setFromTime(fromTime);
                moveFromPathParams.setToTime(toTime);
                moveFromPathParams.setApplicationName(auditApplicationName);
-                moveFromPathParams.addSearchKey("/"+auditAppId+"/transaction/move/from/path", nodePath);
+                moveFromPathParams.addSearchKey("/" + auditAppId + "/transaction/move/from/path", nodePath);
                auditService.auditQuery(callback, moveFromPathParams, limit);
                return null;
@@ -857,7 +857,7 @@ public class AuditImpl implements Audit
        public AuditEntriesByNodeIdQueryWalker()
        {
-            super(new HashSet<>(Arrays.asList(new String[] { CREATED_BY_USER })), null);
+            super(new HashSet<>(Arrays.asList(new String[]{CREATED_BY_USER})), null);
        }
        @Override
@@ -904,19 +904,22 @@ public class AuditImpl implements Audit
    public int getAuditEntriesCountByApp(AuditService.AuditApplication auditApplication)
    {
        final String applicationName = auditApplication.getKey().substring(1);
        return auditService.getAuditEntriesCountByApp(applicationName);
    }
    public int getAuditEntriesCountByAppAndProperties(AuditService.AuditApplication auditApplication, AuditEntryQueryWalker propertyWalker)
    {
        final String applicationName = auditApplication.getKey().substring(1);
        AuditQueryParameters parameters = new AuditQueryParameters();
-        parameters.setApplicationName(auditApplication.getName());
+        parameters.setApplicationName(applicationName);
        parameters.setFromTime(propertyWalker.getFromTime());
        parameters.setToTime(propertyWalker.getToTime());
        parameters.setFromId(propertyWalker.getFromId());
        parameters.setToId(propertyWalker.getToId());
        parameters.setUser(propertyWalker.getCreatedByUser());
-        return auditService.getAuditEntriesCountByAppAndProperties(parameters);
+        return auditService.getAuditEntriesCountByAppAndProperties(applicationName, parameters);
    }
 }
--- a/repository/pom.xml
+++ b/repository/pom.xml
@@ -7,7 +7,7 @@
    <parent>
        <groupId>org.alfresco</groupId>
        <artifactId>alfresco-community-repo</artifactId>
-        <version>23.4.0.67</version>
+        <version>25.1.0.10</version>
    </parent>
    <dependencies>
@@ -397,7 +397,7 @@
        <dependency>
            <groupId>org.mybatis</groupId>
            <artifactId>mybatis</artifactId>
-            <version>3.5.16</version>
+            <version>3.5.17</version>
        </dependency>
        <dependency>
            <groupId>org.mybatis</groupId>
@@ -850,12 +850,12 @@
                    </execution>
                </executions>
                <configuration>
-                    <complianceLevel>1.8</complianceLevel>
+                    <complianceLevel>17</complianceLevel>
                    <outxml>false</outxml>
                    <verbose>true</verbose>
                    <showWeaveInfo>true</showWeaveInfo>
-                    <source>1.8</source>
+                    <source>17</source>
-                    <target>1.8</target>
+                    <target>17</target>
                    <additionalCompilerArgs>
                        <arg>-parameters</arg>
                    </additionalCompilerArgs>
--- a/repository/src/main/java/org/alfresco/repo/audit/AuditComponent.java
+++ b/repository/src/main/java/org/alfresco/repo/audit/AuditComponent.java
@@ -1,286 +1,282 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software.
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ *
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ *
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ *
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.audit;
+package org.alfresco.repo.audit;
-
+
-import java.io.Serializable;
+import java.io.Serializable;
-import java.util.HashMap;
+import java.util.HashMap;
-import java.util.List;
+import java.util.List;
-import java.util.Map;
+import java.util.Map;
-
+
-import org.alfresco.repo.audit.model.AuditApplication;
+import org.alfresco.repo.audit.model.AuditApplication;
-import org.alfresco.repo.audit.model.AuditModelRegistry;
+import org.alfresco.repo.audit.model.AuditModelRegistry;
-import org.alfresco.repo.audit.model._3.AuditPath;
+import org.alfresco.repo.audit.model._3.AuditPath;
-import org.alfresco.service.cmr.audit.AuditQueryParameters;
+import org.alfresco.service.cmr.audit.AuditQueryParameters;
-import org.alfresco.service.cmr.audit.AuditService.AuditQueryCallback;
+import org.alfresco.service.cmr.audit.AuditService.AuditQueryCallback;
-
+
-/**
+/**
- * The audit component. Used by the AuditService and AuditMethodInterceptor to insert audit entries.
+ * The audit component. Used by the AuditService and AuditMethodInterceptor to insert audit entries.
- * 
+ *
- * @author Derek Hulley
+ * @author Derek Hulley
- */
+ */
-public interface AuditComponent
+public interface AuditComponent
-{
+{
-    /**
+    /**
-     * Determines whether audit is globally enabled or disabled.
+     * Determines whether audit is globally enabled or disabled.
-     * 
+     *
-     * @return                  Returns <code>true</code> if audit is enabled
+     * @return Returns <code>true</code> if audit is enabled
-     * 
+     *
-     * @since 3.3
+     * @since 3.3
-     */
+     */
-    public boolean isAuditEnabled();
+    public boolean isAuditEnabled();
-    
+
-    /**
+    /**
-     * Switch auditing on or off
+     * Switch auditing on or off
-     * 
+     *
-     * @param enable            <tt>true</tt> to enable auditing or <tt>false</tt> to disable
+     * @param enable
-     * 
+     *            <tt>true</tt> to enable auditing or <tt>false</tt> to disable
-     * @since 3.4
+     *
-     */
+     * @since 3.4
-    public void setAuditEnabled(boolean enable);
+     */
-
+    public void setAuditEnabled(boolean enable);
-    /**
+
-     * @param userAuditFilter UserAuditFilter
+    /**
-     * 
+     * @param userAuditFilter
-     * @since 4.2
+     *            UserAuditFilter
-     */
+     *
-    public void setUserAuditFilter(UserAuditFilter userAuditFilter);
+     * @since 4.2
-    
+     */
-    /**
+    public void setUserAuditFilter(UserAuditFilter userAuditFilter);
-     * Get all registered audit applications, whether active or not.
+
-     * 
+    /**
-     * @return                  Returns a map of registered audit applications keyed by name
+     * Get all registered audit applications, whether active or not.
-     * 
+     *
-     * @since 3.4
+     * @return Returns a map of registered audit applications keyed by name
-     */
+     *
-    public Map<String, AuditApplication> getAuditApplications();
+     * @since 3.4
-    
+     */
-    /**
+    public Map<String, AuditApplication> getAuditApplications();
-     * Determine whether the audit infrastructure expects audit values to be passed in.
+
-     * This is a helper method to allow optimizations in the client code.  Reasons why
+    /**
-     * this method might return <tt>false</tt> are: auditing is disabled; no audit applications
+     * Determine whether the audit infrastructure expects audit values to be passed in. This is a helper method to allow optimizations in the client code. Reasons why this method might return <tt>false</tt> are: auditing is disabled; no audit applications have been registered. Sometimes, depending on the log level, this method may always return <tt>true</tt>.
-     * have been registered.  Sometimes, depending on the log level, this method may always
+     *
-     * return <tt>true</tt>.
+     *
-     * 
+     * @return Returns <code>true</code> if the calling code (data producers) should go ahead and generate the data for {@link #recordAuditValues(String, Map) recording}.
-     * 
+     *
-     * @return                  Returns <code>true</code> if the calling code (data producers)
+     * @since 3.3
-     *                          should go ahead and generate the data for
+     */
-     *                          {@link #recordAuditValues(String, Map) recording}.
+    public boolean areAuditValuesRequired();
-     * 
+
-     * @since 3.3
+    /**
-     */
+     * Determines if audit values are required based on whether there are any audit applications registered to record data for the given path. This helper method gives data producers a shortcut in the event that nothing would be recorded in any event.
-    public boolean areAuditValuesRequired();
+     *
-    
+     * @param path
-    /**
+     *            the audit path
-     * Determines if audit values are required based on whether there are any audit applications
+     * @return Returns <tt>true</tt> if there is at least one audit application registered to handle the given path.
-     * registered to record data for the given path. This helper method gives data producers a 
+     *
-     * shortcut in the event that nothing would be recorded in any event.
+     * @since 3.4
-     * 
+     */
-     * @param path              the audit path
+    public boolean areAuditValuesRequired(String path);
-     * @return                  Returns <tt>true</tt> if there is at least one audit application
+
-     *                          registered to handle the given path.
+    /**
-     * 
+     * Delete audit entries for the given application and time range
-     * @since 3.4                         
+     *
-     */
+     * @param applicationName
-    public boolean areAuditValuesRequired(String path);
+     *            the name of the application
-    
+     * @param fromTime
-    /**
+     *            the start time of entries to remove (inclusive and optional)
-     * Delete audit entries for the given application and time range
+     * @param toTime
-     * 
+     *            the end time of entries to remove (exclusive and optional)
-     * @param applicationName   the name of the application
+     * @return Returns the number of entries deleted
-     * @param fromTime          the start time of entries to remove (inclusive and optional)
+     *
-     * @param toTime            the end time of entries to remove (exclusive and optional)
+     * @since 3.2
-     * @return                  Returns the number of entries deleted
+     */
-     * 
+    int deleteAuditEntries(String applicationName, Long fromTime, Long toTime);
-     * @since 3.2
+
-     */
+    /**
-    int deleteAuditEntries(String applicationName, Long fromTime, Long toTime);
+     * Delete audit entries for the given application and id range
-
+     *
-    /**
+     * @param applicationName
-     * Delete audit entries for the given application and id range
+     *            the name of the application
-     *
+     * @param fromId
-     * @param applicationName   the name of the application
+     *            the start time of entries to remove (inclusive and optional)
-     * @param fromId          the start time of entries to remove (inclusive and optional)
+     * @param toId
-     * @param toId            the end time of entries to remove (exclusive and optional)
+     *            the end time of entries to remove (exclusive and optional)
-     * @return                  Returns the number of entries deleted
+     * @return Returns the number of entries deleted
-     *
+     *
-     * @since 5.2.2
+     * @since 5.2.2
-     */
+     */
-    int deleteAuditEntriesByIdRange(String applicationName, Long fromId, Long toId);
+    int deleteAuditEntriesByIdRange(String applicationName, Long fromId, Long toId);
-
+
-    /**
+    /**
-     * Delete a discrete list of audit entries based on ID
+     * Delete a discrete list of audit entries based on ID
-     * 
+     *
-     * @param auditEntryIds     the audit entry IDs to delete
+     * @param auditEntryIds
-     * @return                  Returns the number of entries deleted
+     *            the audit entry IDs to delete
-     */
+     * @return Returns the number of entries deleted
-    int deleteAuditEntries(List<Long> auditEntryIds);
+     */
-    
+    int deleteAuditEntries(List<Long> auditEntryIds);
-    /**
+
-     * Check if an audit path is enabled.  The path will be disabled if it or any higher
+    /**
-     * path has been explicitly disabled.  Any disabled path will not be processed when
+     * Check if an audit path is enabled. The path will be disabled if it or any higher path has been explicitly disabled. Any disabled path will not be processed when data is audited.
-     * data is audited.
+     *
-     * 
+     * @param applicationName
-     * @param applicationName   the name of the application being logged to
+     *            the name of the application being logged to
-     * @param path              the audit path to check or <tt>null</tt> to assume the
+     * @param path
-     *                          application's root path
+     *            the audit path to check or <tt>null</tt> to assume the application's root path
-     * @return                  Returns <tt>true</tt> if the audit path has been disabled
+     * @return Returns <tt>true</tt> if the audit path has been disabled
-     * 
+     *
-     * @since 3.2
+     * @since 3.2
-     */
+     */
-    boolean isAuditPathEnabled(String applicationName, String path);
+    boolean isAuditPathEnabled(String applicationName, String path);
-    
+
-    /**
+    /**
-     * Enable auditing (if it is not already enabled) for all paths that contain the given path.
+     * Enable auditing (if it is not already enabled) for all paths that contain the given path. The path is the path as originally logged and not the path that the generated data may contain - although this would be similarly enabled.
-     * The path is the path as originally logged and
+     * <p>
-     * not the path that the generated data may contain - although this would be similarly
+     * If the enabled
-     * enabled.
+     *
-     * <p>
+     * @param applicationName
-     * If the enabled 
+     *            the name of the application being logged to
-     * 
+     * @param path
-     * @param applicationName   the name of the application being logged to
+     *            the audit path to check or <tt>null</tt> to assume the application's root path
-     * @param path              the audit path to check or <tt>null</tt> to assume the
+     *
-     *                          application's root path
+     * @since 3.2
-     * 
+     */
-     * @since 3.2
+    void enableAudit(String applicationName, String path);
-     */
+
-    void enableAudit(String applicationName, String path);
+    /**
-
+     * Disable auditing (if it is not already disabled) for all paths that contain the given path. The path is the path as originally logged and not the path that the generated data may contain - although this would be similarly disabled.
-    /**
+     * <p>
-     * Disable auditing (if it is not already disabled) for all paths that contain the given path.
+     * If the path is <b>/x/y</b> then any data paths that start with <b>/x/y</b> will be stripped out <u>before</u> data generators and data recorders are applied. If the path represents the root path of the application, then auditing for that application is effectively disabled.
-     * The path is the path as originally logged and
+     *
-     * not the path that the generated data may contain - although this would be similarly
+     * @param applicationName
-     * disabled.
+     *            the name of the application being logged to
-     * <p>
+     * @param path
-     * If the path is <b>/x/y</b> then any data paths that start with <b>/x/y</b> will be stripped
+     *            the audit path to check or <tt>null</tt> to assume the application's root path
-     * out <u>before</u> data generators and data recorders are applied.  If the path represents
+     *
-     * the root path of the application, then auditing for that application is effectively disabled. 
+     * @since 3.2
-     * 
+     */
-     * @param applicationName   the name of the application being logged to
+    void disableAudit(String applicationName, String path);
-     * @param path              the audit path to check or <tt>null</tt> to assume the
+
-     *                          application's root path
+    /**
-     * 
+     * Remove all disabled paths i.e. enable all per-path based auditing. Auditing may still be disabled globally. This is primarily for test purposes; applications should know which paths need {@link #enableAudit(String, String) enabling} or {@link #disableAudit(String, String) disabled}.
-     * @since 3.2
+     *
-     */
+     * @param applicationName
-    void disableAudit(String applicationName, String path);
+     *            the name of the application
-    
+     *
-    /**
+     * @since 3.2
-     * Remove all disabled paths i.e. enable all per-path based auditing.  Auditing may still be
+     */
-     * disabled globally.  This is primarily for test purposes; applications should know which
+    void resetDisabledPaths(String applicationName);
-     * paths need {@link #enableAudit(String, String) enabling} or
+
-     * {@link #disableAudit(String, String) disabled}.
+    /**
-     * 
+     * Create an audit entry for the given map of values. The map key is a path - starting with '/' ({@link AuditApplication#AUDIT_PATH_SEPARATOR}) - relative to the root path provided.
-     * @param applicationName   the name of the application
+     * <p/>
-     * 
+     * The root path and value keys are combined to produce a map of data keyed by full path. This fully-pathed map is then passed through the {@link AuditModelRegistry#getAuditPathMapper() audit path mapper}. The result may yield data destined for several different {@link AuditModelRegistry#getAuditApplicationByKey(String) audit applications}. depending on the data extraction and generation defined in the applications, values (or derived values) may be recorded against several audit entries (one per application represented).
-     * @since 3.2
+     * <p/>
-     */
+     * The return values reflect what was actually persisted and is controlled by the data extractors defined in the audit configuration.
-    void resetDisabledPaths(String applicationName);
+     * <p/>
-
+     * A new read-write transaction is started if there are values to write that there is not a viable transaction present.
-    /**
+     *
-     * Create an audit entry for the given map of values.  The map key is a path - starting with '/'
+     * @param rootPath
-     * ({@link AuditApplication#AUDIT_PATH_SEPARATOR}) - relative to the root path provided.
+     *            a base path of {@link AuditPath} key entries concatenated with the path separator '/' ({@link AuditApplication#AUDIT_PATH_SEPARATOR})
-     * <p/>
+     * @param values
-     * The root path and value keys are combined to produce a map of data keyed by full path.  This
+     *            the values to audit mapped by {@link AuditPath} key relative to root path (may be <tt>null</tt>)
-     * fully-pathed map is then passed through the
+     * @return Returns the values that were actually persisted, keyed by their full path.
-     * {@link AuditModelRegistry#getAuditPathMapper() audit path mapper}.  The result may yield data
+     * @throws IllegalStateException
-     * destined for several different
+     *             if the transaction state could not be determined
-     * {@link AuditModelRegistry#getAuditApplicationByKey(String) audit applications}.  depending on
+     *
-     * the data extraction and generation defined in the applications, values (or derived values) may
+     * @since 3.2
-     * be recorded against several audit entries (one per application represented).
+     */
-     * <p/>
+    Map<String, Serializable> recordAuditValues(String rootPath, Map<String, Serializable> values);
-     * The return values reflect what was actually persisted and is controlled by the data extractors
+
-     * defined in the audit configuration.
+    /**
-     * <p/>
+     * The same as {@link AuditComponent#recordAuditValues(String, Map)}, but with controlled usage of userFilter
-     * A new read-write transaction is started if there are values to write that there is not a viable
+     *
-     * transaction present.
+     * @param rootPath
-     * 
+     *            a base path of {@link AuditPath} key entries concatenated with the path separator '/' ({@link AuditApplication#AUDIT_PATH_SEPARATOR})
-     * @param rootPath          a base path of {@link AuditPath} key entries concatenated with the path separator
+     * @param values
-     *                          '/' ({@link AuditApplication#AUDIT_PATH_SEPARATOR})
+     *            the values to audit mapped by {@link AuditPath} key relative to root path (may be <tt>null</tt>)
-     * @param values            the values to audit mapped by {@link AuditPath} key relative to root path
+     * @param useUserFilter
-     *                          (may be <tt>null</tt>)
+     *            if <tt>false</tt> the user filter is disabled.
-     * @return                  Returns the values that were actually persisted, keyed by their full path.
+     * @return Returns the values that were actually persisted, keyed by their full path.
-     * @throws IllegalStateException if the transaction state could not be determined
+     * @throws IllegalStateException
-     * 
+     *             if the transaction state could not be determined
-     * @since 3.2
+     */
-     */
+    Map<String, Serializable> recordAuditValuesWithUserFilter(String rootPath, Map<String, Serializable> values, boolean useUserFilter);
-    Map<String, Serializable> recordAuditValues(String rootPath, Map<String, Serializable> values);
+
-    
+    /**
-    /**
+     * Find audit entries using the given parameters
-     * The same as {@link AuditComponent#recordAuditValues(String, Map)}, but with controlled usage of userFilter
+     *
-     * 
+     * @param callback
-     * @param rootPath          a base path of {@link AuditPath} key entries concatenated with the path separator
+     *            the data callback per entry
-     *                          '/' ({@link AuditApplication#AUDIT_PATH_SEPARATOR})
+     * @param parameters
-     * @param values            the values to audit mapped by {@link AuditPath} key relative to root path
+     *            the parameters for the query (may not be <tt>null</tt>)
-     *                          (may be <tt>null</tt>)
+     * @param maxResults
-     * @param useUserFilter     if <tt>false</tt> the user filter is disabled.
+     *            the maximum number of results to retrieve (must be greater than 0)
-     * @return                  Returns the values that were actually persisted, keyed by their full path.
+     *
-     * @throws IllegalStateException if the transaction state could not be determined
+     * @throws IllegalArgumentException
-     */
+     *             if maxResults less or equal to zero
-    Map<String, Serializable> recordAuditValuesWithUserFilter(String rootPath, Map<String, Serializable> values, boolean useUserFilter);
+     *
-    
+     * @since 3.2
-    /**
+     */
-     * Find audit entries using the given parameters
+    void auditQuery(AuditQueryCallback callback, AuditQueryParameters parameters, int maxResults);
-     * 
+
-     * @param callback          the data callback per entry
+    /**
-     * @param parameters        the parameters for the query (may not be <tt>null</tt>)
+     * Issue an audit query to retrieve min / max audit record id for a given application.
-     * @param maxResults        the maximum number of results to retrieve (must be greater than 0)
+     *
-     * 
+     * @param applicationName
-     * @throws IllegalArgumentException if maxResults less or equal to zero
+     *            the name of the application
-     * 
+     * @param extremes
-     * @since 3.2
+     *            a list containing min/max or both
-     */
+     * @return a map containing min/max and the associated value
-    void auditQuery(AuditQueryCallback callback, AuditQueryParameters parameters, int maxResults);
+     */
-
+    HashMap<String, Long> getAuditMinMaxByApp(String applicationName, List<String> extremes);
-    /**
+
-     * Issue an audit query to retrieve min / max audit record id for a given application.
+    /**
-     *
+     * Issue an audit query to retrieve count of records for a given application.
-     * @param applicationName               the name of the application
+     *
-     * @param extremes                      a list containing min/max or both
+     * @param applicationName
-     * @return                              a map containing min/max and the associated value
+     *            the name of the application
-     */
+     * @return a map containing min/max and the associated value
-    HashMap<String, Long> getAuditMinMaxByApp(String applicationName, List<String> extremes);
+     */
-
+    default int getAuditEntriesCountByApp(String applicationName)
-    /**
+    {
-     * Issue an audit query to retrieve count of records for a given application.
+        return -1;
-     *
+    }
-     * @param applicationName             the name of the application
+
-     * @return                  a map containing min/max and the associated value
+    /**
-     */
+     * Issue an audit query to retrieve count of records for a given application and properties
-    default int getAuditEntriesCountByApp(String applicationName)
+     *
-    {
+     * @param applicationName
-        return -1;
+     *            the name of the application
-    }
+     * @param parameters
-
+     *            audit parameters provided by the <code>where</code> clause on the ReST API
-    /**
+     * @return a map containing min/max and the associated value
-     * Issue an audit query to retrieve count of records for a given application and properties
+     */
-     *
+    default int getAuditEntriesCountByAppAndProperties(String applicationName, AuditQueryParameters parameters)
-     * @param parameters                  audit parameters provided by the <code>where</code> clause on the ReST API
+    {
-     * @return                            a map containing min/max and the associated value
+        return -1;
-     */
+    }
-    default int getAuditEntriesCountByAppAndProperties(AuditQueryParameters parameters)
+}
    {
        return -1;
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/audit/AuditComponentImpl.java
+++ b/repository/src/main/java/org/alfresco/repo/audit/AuditComponentImpl.java
--- a/repository/src/main/java/org/alfresco/repo/audit/AuditServiceImpl.java
+++ b/repository/src/main/java/org/alfresco/repo/audit/AuditServiceImpl.java
@@ -1,197 +1,210 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.audit;
+package org.alfresco.repo.audit;
-
+
-import java.util.HashMap;
+import java.util.HashMap;
-import java.util.List;
+import java.util.List;
-import java.util.Map;
+import java.util.Map;
-import java.util.TreeMap;
+import java.util.TreeMap;
-
+
-import org.alfresco.service.cmr.audit.AuditQueryParameters;
+import org.alfresco.service.cmr.audit.AuditQueryParameters;
-import org.alfresco.service.cmr.audit.AuditService;
+import org.alfresco.service.cmr.audit.AuditService;
-
+
-/**
+/**
- * The implementation of the AuditService for application auditing.
+ * The implementation of the AuditService for application auditing.
- * 
+ * 
- * @author Derek Hulley
+ * @author Derek Hulley
- * @since 3.2
+ * @since 3.2
- */
+ */
-public class AuditServiceImpl implements AuditService
+public class AuditServiceImpl implements AuditService
-{
+{
-    private AuditComponent auditComponent;
+    private AuditComponent auditComponent;
-
+
-    public AuditServiceImpl()
+    public AuditServiceImpl()
-    {
+    {
-        super();
+        super();
-    }
+    }
-
+
-    public void setAuditComponent(AuditComponent auditComponent)
+    public void setAuditComponent(AuditComponent auditComponent)
-    {
+    {
-        this.auditComponent = auditComponent;
+        this.auditComponent = auditComponent;
-    }
+    }
-
+
-    /**
+    /**
-     * {@inheritDoc}
+     * {@inheritDoc}
-     * @since 3.4
+     * 
-     */
+     * @since 3.4
-    public boolean isAuditEnabled()
+     */
-    {
+    public boolean isAuditEnabled()
-        return auditComponent.isAuditEnabled();
+    {
-    }
+        return auditComponent.isAuditEnabled();
-
+    }
-    /**
+
-     * {@inheritDoc}
+    /**
-     * @since 3.4
+     * {@inheritDoc}
-     */
+     * 
-    @Override
+     * @since 3.4
-    public void setAuditEnabled(boolean enable)
+     */
-    {
+    @Override
-        auditComponent.setAuditEnabled(enable);
+    public void setAuditEnabled(boolean enable)
-    }
+    {
-
+        auditComponent.setAuditEnabled(enable);
-    /**
+    }
-     * {@inheritDoc}
+
-     * @since 3.4
+    /**
-     */
+     * {@inheritDoc}
-    @Override
+     * 
-    public Map<String, AuditApplication> getAuditApplications()
+     * @since 3.4
-    {
+     */
-        Map<String, org.alfresco.repo.audit.model.AuditApplication> apps = auditComponent.getAuditApplications();
+    @Override
-
+    public Map<String, AuditApplication> getAuditApplications()
-        Map<String, AuditApplication> ret = new TreeMap<String, AuditApplication>();
+    {
-        for (String app : apps.keySet())
+        Map<String, org.alfresco.repo.audit.model.AuditApplication> apps = auditComponent.getAuditApplications();
-        {
+
-            String name = app;
+        Map<String, AuditApplication> ret = new TreeMap<String, AuditApplication>();
-            String key = org.alfresco.repo.audit.model.AuditApplication.AUDIT_PATH_SEPARATOR + apps.get(app).getApplicationKey();
+        for (String app : apps.keySet())
-            boolean enabled = auditComponent.isAuditPathEnabled(app, key);
+        {
-            AuditApplication auditApplication = new AuditApplication(name, key, enabled);
+            String name = app;
-            ret.put(name, auditApplication);
+            String key = org.alfresco.repo.audit.model.AuditApplication.AUDIT_PATH_SEPARATOR + apps.get(app).getApplicationKey();
-        }
+            boolean enabled = auditComponent.isAuditPathEnabled(app, key);
-        return ret;
+            AuditApplication auditApplication = new AuditApplication(name, key, enabled);
-    }
+            ret.put(name, auditApplication);
-
+        }
-    /**
+        return ret;
-     * {@inheritDoc}
+    }
-     * @since 3.2
+
-     */
+    /**
-    public boolean isAuditEnabled(String applicationName, String path)
+     * {@inheritDoc}
-    {
+     * 
-        return auditComponent.isAuditPathEnabled(applicationName, path);
+     * @since 3.2
-    }
+     */
-
+    public boolean isAuditEnabled(String applicationName, String path)
-    /**
+    {
-     * {@inheritDoc}
+        return auditComponent.isAuditPathEnabled(applicationName, path);
-     * @since 3.2
+    }
-     */
+
-    public void enableAudit(String applicationName, String path)
+    /**
-    {
+     * {@inheritDoc}
-        auditComponent.enableAudit(applicationName, path);
+     * 
-    }
+     * @since 3.2
-
+     */
-    /**
+    public void enableAudit(String applicationName, String path)
-     * {@inheritDoc}
+    {
-     * @since 3.2
+        auditComponent.enableAudit(applicationName, path);
-     */
+    }
-    public void disableAudit(String applicationName, String path)
+
-    {
+    /**
-        auditComponent.disableAudit(applicationName, path);
+     * {@inheritDoc}
-    }
+     * 
-
+     * @since 3.2
-    /**
+     */
-     * {@inheritDoc}
+    public void disableAudit(String applicationName, String path)
-     * @since 3.2
+    {
-     */
+        auditComponent.disableAudit(applicationName, path);
-    public int clearAudit(String applicationName)
+    }
-    {
+
-        Long now = Long.valueOf(System.currentTimeMillis());
+    /**
-        return auditComponent.deleteAuditEntries(applicationName, null, now);
+     * {@inheritDoc}
-    }
+     * 
-
+     * @since 3.2
-    /**
+     */
-     * {@inheritDoc}
+    public int clearAudit(String applicationName)
-     * @since 3.4
+    {
-     */
+        Long now = Long.valueOf(System.currentTimeMillis());
-    @Override
+        return auditComponent.deleteAuditEntries(applicationName, null, now);
-    public int clearAudit(String applicationName, Long fromTime, Long toTime)
+    }
-    {
+
-        toTime = (toTime == null) ? Long.valueOf(System.currentTimeMillis()) : toTime;
+    /**
-        return auditComponent.deleteAuditEntries(applicationName, fromTime, toTime);
+     * {@inheritDoc}
-    }
+     * 
-
+     * @since 3.4
-    /**
+     */
-     * {@inheritDoc}
+    @Override
-     * @since 5.2.2
+    public int clearAudit(String applicationName, Long fromTime, Long toTime)
-     */
+    {
-    @Override
+        toTime = (toTime == null) ? Long.valueOf(System.currentTimeMillis()) : toTime;
-    public int clearAuditByIdRange(String applicationName, Long fromId, Long toId)
+        return auditComponent.deleteAuditEntries(applicationName, fromTime, toTime);
-    {
+    }
-        return auditComponent.deleteAuditEntriesByIdRange(applicationName, fromId, toId);
+
-    }
+    /**
-
+     * {@inheritDoc}
-    /**
+     * 
-     * {@inheritDoc}
+     * @since 5.2.2
-     * @since 3.4
+     */
-     */
+    @Override
-    @Override
+    public int clearAuditByIdRange(String applicationName, Long fromId, Long toId)
-    public int clearAudit(List<Long> auditEntryIds)
+    {
-    {
+        return auditComponent.deleteAuditEntriesByIdRange(applicationName, fromId, toId);
-        return auditComponent.deleteAuditEntries(auditEntryIds);
+    }
-    }
+
-
+    /**
-    /**
+     * {@inheritDoc}
-     * {@inheritDoc}
+     * 
-     * @since 3.3
+     * @since 3.4
-     */
+     */
-    public void auditQuery(AuditQueryCallback callback, AuditQueryParameters parameters, int maxResults)
+    @Override
-    {
+    public int clearAudit(List<Long> auditEntryIds)
-        auditComponent.auditQuery(callback, parameters, maxResults);
+    {
-    }
+        return auditComponent.deleteAuditEntries(auditEntryIds);
-
+    }
-    /**
+
-     * {@inheritDoc}
+    /**
-     */
+     * {@inheritDoc}
-    public HashMap<String, Long> getAuditMinMaxByApp(String applicationName, List<String> extremes)
+     * 
-    {
+     * @since 3.3
-        return auditComponent.getAuditMinMaxByApp(applicationName, extremes);
+     */
-    }
+    public void auditQuery(AuditQueryCallback callback, AuditQueryParameters parameters, int maxResults)
-
+    {
-    /**
+        auditComponent.auditQuery(callback, parameters, maxResults);
-     * {@inheritDoc}
+    }
-     */
+
-    @Override
+    /**
-    public int getAuditEntriesCountByApp(String applicationName)
+     * {@inheritDoc}
-    {
+     */
-        return auditComponent.getAuditEntriesCountByApp(applicationName);
+    public HashMap<String, Long> getAuditMinMaxByApp(String applicationName, List<String> extremes)
-    }
+    {
-
+        return auditComponent.getAuditMinMaxByApp(applicationName, extremes);
-    /**
+    }
-     * {@inheritDoc}
+
-     */
+    /**
-    @Override public int getAuditEntriesCountByAppAndProperties(AuditQueryParameters parameters)
+     * {@inheritDoc}
-    {
+     */
-        return auditComponent.getAuditEntriesCountByAppAndProperties(parameters);
+    @Override
-    }
+    public int getAuditEntriesCountByApp(String applicationName)
-}
+    {
        return auditComponent.getAuditEntriesCountByApp(applicationName);
    }
    /**
     * {@inheritDoc}
     */
    @Override
    public int getAuditEntriesCountByAppAndProperties(String applicationName, AuditQueryParameters parameters)
    {
        return auditComponent.getAuditEntriesCountByAppAndProperties(applicationName, parameters);
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/content/AbstractRoutingContentStore.java
+++ b/repository/src/main/java/org/alfresco/repo/content/AbstractRoutingContentStore.java
@@ -1,493 +1,484 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2021 Alfresco Software Limited
+ * Copyright (C) 2005 - 2021 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L% 
- */
+ */
-package org.alfresco.repo.content;
+package org.alfresco.repo.content;
-
+
-import java.io.Serializable;
+import java.io.Serializable;
-import java.util.Collections;
+import java.util.Collections;
-import java.util.List;
+import java.util.List;
-import java.util.Map;
+import java.util.Map;
-import java.util.concurrent.locks.ReentrantReadWriteLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock;
-import java.util.concurrent.locks.ReentrantReadWriteLock.ReadLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock.ReadLock;
-import java.util.concurrent.locks.ReentrantReadWriteLock.WriteLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock.WriteLock;
-
+
-import org.alfresco.error.AlfrescoRuntimeException;
+import org.apache.commons.logging.Log;
-import org.alfresco.repo.cache.SimpleCache;
+import org.apache.commons.logging.LogFactory;
-import org.alfresco.service.cmr.repository.ContentIOException;
+
-import org.alfresco.service.cmr.repository.ContentReader;
+import org.alfresco.error.AlfrescoRuntimeException;
-import org.alfresco.service.cmr.repository.ContentWriter;
+import org.alfresco.repo.cache.SimpleCache;
-import org.alfresco.util.GUID;
+import org.alfresco.service.cmr.repository.ContentIOException;
-import org.alfresco.util.Pair;
+import org.alfresco.service.cmr.repository.ContentReader;
-import org.apache.commons.logging.Log;
+import org.alfresco.service.cmr.repository.ContentWriter;
-import org.apache.commons.logging.LogFactory;
+import org.alfresco.util.GUID;
-
+import org.alfresco.util.Pair;
-/**
+
- * A store providing support for content store implementations that provide
+/**
- * routing of content read and write requests based on context.
+ * A store providing support for content store implementations that provide routing of content read and write requests based on context.
- * 
+ * 
- * @see ContentContext
+ * @see ContentContext
- * 
+ * 
- * @since 2.1
+ * @since 2.1
- * @author Derek Hulley
+ * @author Derek Hulley
- */
+ */
-public abstract class AbstractRoutingContentStore implements ContentStore
+public abstract class AbstractRoutingContentStore implements ContentStore
-{
+{
-    private static Log logger = LogFactory.getLog(AbstractRoutingContentStore.class);
+    private static Log logger = LogFactory.getLog(AbstractRoutingContentStore.class);
-    
+
-    private String instanceKey = GUID.generate();
+    private String instanceKey = GUID.generate();
-    private SimpleCache<Pair<String, String>, ContentStore> storesByContentUrl;
+    private SimpleCache<Pair<String, String>, ContentStore> storesByContentUrl;
-    private ReadLock storesCacheReadLock;
+    private ReadLock storesCacheReadLock;
-    private WriteLock storesCacheWriteLock;
+    private WriteLock storesCacheWriteLock;
-    
+
-    protected AbstractRoutingContentStore()
+    protected AbstractRoutingContentStore()
-    {
+    {
-        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
+        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
-        storesCacheReadLock = lock.readLock();
+        storesCacheReadLock = lock.readLock();
-        storesCacheWriteLock = lock.writeLock();
+        storesCacheWriteLock = lock.writeLock();
-    }
+    }
-
+
-    /**
+    /**
-     * @param storesCache       cache of stores used to access URLs 
+     * @param storesCache
-     */
+     *            cache of stores used to access URLs
-    public void setStoresCache(SimpleCache<Pair<String, String>, ContentStore> storesCache)
+     */
-    {
+    public void setStoresCache(SimpleCache<Pair<String, String>, ContentStore> storesCache)
-        this.storesByContentUrl = storesCache;
+    {
-    }
+        this.storesByContentUrl = storesCache;
-
+    }
-    /**
+
-     * @return          Returns a list of all possible stores available for reading or writing
+    /**
-     */
+     * @return Returns a list of all possible stores available for reading or writing
-    protected abstract List<ContentStore> getAllStores();
+     */
-    
+    protected abstract List<ContentStore> getAllStores();
-    /**
+
-     * Get a content store based on the context provided.  The applicability of the
+    /**
-     * context and even the types of context allowed are up to the implementation, but
+     * Get a content store based on the context provided. The applicability of the context and even the types of context allowed are up to the implementation, but normally there should be a fallback case for when the parameters are not adequate to make a decision.
-     * normally there should be a fallback case for when the parameters are not adequate
+     * 
-     * to make a decision.
+     * @param ctx
-     * 
+     *            the context to use to make the choice
-     * @param ctx       the context to use to make the choice
+     * @return Returns the store most appropriate for the given context and <b>never <tt>null</tt></b>
-     * @return          Returns the store most appropriate for the given context and
+     */
-     *                  <b>never <tt>null</tt></b>
+    protected abstract ContentStore selectWriteStore(ContentContext ctx);
-     */
+
-    protected abstract ContentStore selectWriteStore(ContentContext ctx);
+    /**
-    
+     * Checks the cache for the store and ensures that the URL is in the store.
-    /**
+     * 
-     * Checks the cache for the store and ensures that the URL is in the store.
+     * @param contentUrl
-     * 
+     *            the content URL to search for
-     * @param contentUrl    the content URL to search for
+     * @return Returns the store matching the content URL
-     * @return              Returns the store matching the content URL
+     */
-     */
+    private ContentStore selectReadStore(String contentUrl)
-    private ContentStore selectReadStore(String contentUrl)
+    {
-    {
+        Pair<String, String> cacheKey = new Pair<String, String>(instanceKey, contentUrl);
-        Pair<String, String> cacheKey = new Pair<String, String>(instanceKey, contentUrl);
+        storesCacheReadLock.lock();
-        storesCacheReadLock.lock();
+        try
-        try
+        {
-        {
+            // Check if the store is in the cache
-            // Check if the store is in the cache
+            ContentStore store = storesByContentUrl.get(cacheKey);
-            ContentStore store = storesByContentUrl.get(cacheKey);
+            if (store != null)
-            if (store != null)
+            {
-            {
+                // We found a store that was previously used
-                // We found a store that was previously used
+                try
-                try
+                {
-                {
+                    // It is possible for content to be removed from a store and
-                    // It is possible for content to be removed from a store and
+                    // it might have moved into another store.
-                    // it might have moved into another store.
+                    if (store.exists(contentUrl))
-                    if (store.exists(contentUrl))
+                    {
-                    {
+                        // We found a store and can use it
-                        // We found a store and can use it
+                        return store;
-                        return store;
+                    }
-                    }
+                }
-                }
+                catch (UnsupportedContentUrlException e)
-                catch (UnsupportedContentUrlException e)
+                {
-                {
+                    // This is odd. The store that previously supported the content URL
-                    // This is odd.  The store that previously supported the content URL
+                    // no longer does so. I can't think of a reason why that would be.
-                    // no longer does so.  I can't think of a reason why that would be.
+                    throw new AlfrescoRuntimeException(
-                    throw new AlfrescoRuntimeException(
+                            "Found a content store that previously supported a URL, but no longer does: \n" +
-                            "Found a content store that previously supported a URL, but no longer does: \n" +
+                                    "   Store:       " + store + "\n" +
-                            "   Store:       " + store + "\n" +
+                                    "   Content URL: " + contentUrl);
-                            "   Content URL: " + contentUrl);
+                }
-                }
+            }
-            }
+        }
-        }
+        finally
-        finally
+        {
-        {
+            storesCacheReadLock.unlock();
-            storesCacheReadLock.unlock();
+        }
-        }
+        // Get the write lock and double check
-        // Get the write lock and double check
+        storesCacheWriteLock.lock();
-        storesCacheWriteLock.lock();
+        try
-        try
+        {
-        {
+            // Double check
-            // Double check
+            ContentStore store = storesByContentUrl.get(cacheKey);
-            ContentStore store = storesByContentUrl.get(cacheKey);
+            if (store != null && store.exists(contentUrl))
-            if (store != null && store.exists(contentUrl))
+            {
-            {
+                // We found a store and can use it
-                // We found a store and can use it
+                if (logger.isDebugEnabled())
-                if (logger.isDebugEnabled())
+                {
-                {
+                    logger.debug(
-                    logger.debug(
+                            "Found mapped store for content URL: \n" +
-                            "Found mapped store for content URL: \n" +
+                                    "   Content URL: " + contentUrl + "\n" +
-                            "   Content URL: " + contentUrl + "\n" +
+                                    "   Store:       " + store);
-                            "   Store:       " + store);
+                }
-                }
+                return store;
-                return store;
+            }
-            }
+            else
-            else
+            {
-            {
+                store = null;
-                store = null;
+            }
-            }
+            // It isn't, so search all the stores
-            // It isn't, so search all the stores
+            List<ContentStore> stores = getAllStores();
-            List<ContentStore> stores = getAllStores();
+            // Keep track of the unsupported state of the content URL - it might be a rubbish URL
-            // Keep track of the unsupported state of the content URL - it might be a rubbish URL
+            boolean contentUrlSupported = false;
-            boolean contentUrlSupported = false;
+            for (ContentStore storeInList : stores)
-            for (ContentStore storeInList : stores)
+            {
-            {
+                boolean exists = false;
-                boolean exists = false;
+                try
-                try
+                {
-                {
+                    exists = storeInList.exists(contentUrl);
-                    exists = storeInList.exists(contentUrl);
+                    // At least the content URL was supported
-                    // At least the content URL was supported
+                    contentUrlSupported = true;
-                    contentUrlSupported = true;
+                }
-                }
+                catch (UnsupportedContentUrlException e)
-                catch (UnsupportedContentUrlException e)
+                {
-                {
+                    // The store can't handle the content URL
-                    // The store can't handle the content URL
+                }
-                }
+                if (!exists)
-                if (!exists)
+                {
-                {
+                    // It is not in the store
-                    // It is not in the store
+                    continue;
-                    continue;
+                }
-                }
+                // We found one
-                // We found one
+                store = storeInList;
-                store = storeInList;
+                // Put the value in the cache
-                // Put the value in the cache
+                storesByContentUrl.put(cacheKey, store);
-                storesByContentUrl.put(cacheKey, store);
+                break;
-                break;
+            }
-            }
+            // Check if the content URL was supported
-            // Check if the content URL was supported
+            if (!contentUrlSupported)
-            if (!contentUrlSupported)
+            {
-            {
+                throw new UnsupportedContentUrlException(this, contentUrl);
-                throw new UnsupportedContentUrlException(this, contentUrl);
+            }
-            }
+            // Done
-            // Done
+            if (logger.isDebugEnabled())
-            if (logger.isDebugEnabled())
+            {
-            {
+                logger.debug(
-                logger.debug(
+                        "Mapped content URL to store for reading: \n" +
-                        "Mapped content URL to store for reading: \n" +
+                                "   Content URL: " + contentUrl + "\n" +
-                        "   Content URL: " + contentUrl + "\n" +
+                                "   Store:       " + store);
-                        "   Store:       " + store);
+            }
-            }
+            return store;
-            return store;
+        }
-        }
+        finally
-        finally
+        {
-        {
+            storesCacheWriteLock.unlock();
-            storesCacheWriteLock.unlock();
+        }
-        }
+    }
-    }
+
-
+    /**
-    /**
+     * @return Returns <tt>true</tt> if the URL is supported by any of the stores.
-     * @return      Returns <tt>true</tt> if the URL is supported by any of the stores.
+     */
-     */
+    public boolean isContentUrlSupported(String contentUrl)
-    public boolean isContentUrlSupported(String contentUrl)
+    {
-    {
+        List<ContentStore> stores = getAllStores();
-        List<ContentStore> stores = getAllStores();
+        boolean supported = false;
-        boolean supported = false;
+        for (ContentStore store : stores)
-        for (ContentStore store : stores)
+        {
-        {
+            if (store.isContentUrlSupported(contentUrl))
-            if (store.isContentUrlSupported(contentUrl))
+            {
-            {
+                supported = true;
-                supported = true;
+                break;
-                break;
+            }
-            }
+        }
-        }
+        // Done
-        // Done
+        if (logger.isDebugEnabled())
-        if (logger.isDebugEnabled())
+        {
-        {
+            logger.debug("The url " + (supported ? "is" : "is not") + " supported by at least one store.");
-            logger.debug("The url " + (supported ? "is" : "is not") + " supported by at least one store.");
+        }
-        }
+        return supported;
-        return supported;
+    }
-    }
+
-
+    /**
-    /**
+     * @return Returns <tt>true</tt> if write is supported by any of the stores.
-     * @return      Returns <tt>true</tt> if write is supported by any of the stores.
+     */
-     */
+    public boolean isWriteSupported()
-    public boolean isWriteSupported()
+    {
-    {
+        List<ContentStore> stores = getAllStores();
-        List<ContentStore> stores = getAllStores();
+        boolean supported = false;
-        boolean supported = false;
+        for (ContentStore store : stores)
-        for (ContentStore store : stores)
+        {
-        {
+            if (store.isWriteSupported())
-            if (store.isWriteSupported())
+            {
-            {
+                supported = true;
-                supported = true;
+                break;
-                break;
+            }
-            }
+        }
-        }
+        // Done
-        // Done
+        if (logger.isDebugEnabled())
-        if (logger.isDebugEnabled())
+        {
-        {
+            logger.debug("Writing " + (supported ? "is" : "is not") + " supported by at least one store.");
-            logger.debug("Writing " + (supported ? "is" : "is not") + " supported by at least one store.");
+        }
-        }
+        return supported;
-        return supported;
+    }
-    }
+
-
+    /**
-    /**
+     * @return Returns <b>.</b> always
-     * @return      Returns <b>.</b> always
+     */
-     */
+    public String getRootLocation()
-    public String getRootLocation()
+    {
-    {
+        return ".";
-        return ".";
+    }
-    }
+
-
+    /**
-    /**
+     * @return Returns <tt>-1</tt> always
-     * @return      Returns <tt>-1</tt> always
+     */
-     */
+    @Override
-    @Override
+    public long getSpaceFree()
-    public long getSpaceFree()
+    {
-    {
+        return -1L;
-        return -1L;
+    }
-    }
+
-
+    /**
-    /**
+     * @return Returns <tt>-1</tt> always
-     * @return      Returns <tt>-1</tt> always
+     */
-     */
+    @Override
-    @Override
+    public long getSpaceTotal()
-    public long getSpaceTotal()
+    {
-    {
+        return -1L;
-        return -1L;
+    }
-    }
+
-
+    /**
-    /**
+     * @see #selectReadStore(String)
-     * @see #selectReadStore(String)
+     */
-     */
+    public boolean exists(String contentUrl) throws ContentIOException
-    public boolean exists(String contentUrl) throws ContentIOException
+    {
-    {
+        ContentStore store = selectReadStore(contentUrl);
-        ContentStore store = selectReadStore(contentUrl);
+        return (store != null);
-        return (store != null);
+    }
-    }
+
-
+    /**
-    /**
+     * @return Returns a valid reader from one of the stores otherwise a {@link EmptyContentReader} is returned.
-     * @return  Returns a valid reader from one of the stores otherwise
+     */
-     *          a {@link EmptyContentReader} is returned.
+    public ContentReader getReader(String contentUrl) throws ContentIOException
-     */
+    {
-    public ContentReader getReader(String contentUrl) throws ContentIOException
+        ContentStore store = selectReadStore(contentUrl);
-    {
+        if (store != null)
-        ContentStore store = selectReadStore(contentUrl);
+        {
-        if (store != null)
+            if (logger.isDebugEnabled())
-        {
+            {
-            if (logger.isDebugEnabled())
+                logger.debug("Getting reader from store: \n" +
-            {
+                        "   Content URL: " + contentUrl + "\n" +
-                logger.debug("Getting reader from store: \n" +
+                        "   Store:       " + store);
-                        "   Content URL: " + contentUrl + "\n" +
+            }
-                        "   Store:       " + store);
+            return store.getReader(contentUrl);
-            }
+        }
-            return store.getReader(contentUrl);
+        else
-        }
+        {
-        else
+            if (logger.isDebugEnabled())
-        {
+            {
-            if (logger.isDebugEnabled())
+                logger.debug("Getting empty reader for content URL: " + contentUrl);
-            {
+            }
-                logger.debug("Getting empty reader for content URL: " + contentUrl);
+            return new EmptyContentReader(contentUrl);
-            }
+        }
-            return new EmptyContentReader(contentUrl);
+    }
-        }
+
-    }
+    /**
-
+     * Selects a store for the given context and caches store that was used.
-    /**
+     * 
-     * Selects a store for the given context and caches store that was used.
+     * @see #selectWriteStore(ContentContext)
-     * 
+     */
-     * @see #selectWriteStore(ContentContext)
+    public ContentWriter getWriter(ContentContext context) throws ContentIOException
-     */
+    {
-    public ContentWriter getWriter(ContentContext context) throws ContentIOException
+        String contentUrl = context.getContentUrl();
-    {
+        Pair<String, String> cacheKey = new Pair<String, String>(instanceKey, contentUrl);
-        String contentUrl = context.getContentUrl();
+        if (contentUrl != null)
-        Pair<String, String> cacheKey = new Pair<String, String>(instanceKey, contentUrl);
+        {
-        if (contentUrl != null)
+            // Check to see if it is in the cache
-        {
+            storesCacheReadLock.lock();
-            // Check to see if it is in the cache
+            try
-            storesCacheReadLock.lock();
+            {
-            try
+                // Check if the store is in the cache
-            {
+                ContentStore store = storesByContentUrl.get(cacheKey);
-                // Check if the store is in the cache
+                if (store != null)
-                ContentStore store = storesByContentUrl.get(cacheKey);
+                {
-                if (store != null)
+                    throw new ContentExistsException(this, contentUrl);
-                {
+                }
-                    throw new ContentExistsException(this, contentUrl);
+                /* We could go further and check each store for the existence of the URL, but that would be overkill. The main problem we need to prevent is the simultaneous access of the same store. The router represents a single store and therefore if the URL is present in any of the stores, it is effectively present in all of them. */
-                }
+            }
-                /*
+            finally
-                 * We could go further and check each store for the existence of the URL,
+            {
-                 * but that would be overkill.  The main problem we need to prevent is
+                storesCacheReadLock.unlock();
-                 * the simultaneous access of the same store.  The router represents
+            }
-                 * a single store and therefore if the URL is present in any of the stores,
+        }
-                 * it is effectively present in all of them.
+        // Select the store for writing
-                 */
+        ContentStore store = selectWriteStore(context);
-            }
+        // Check that we were given a valid store
-            finally
+        if (store == null)
-            {
+        {
-                storesCacheReadLock.unlock();
+            throw new NullPointerException(
-            }
+                    "Unable to find a writer.  'selectWriteStore' may not return null: \n" +
-        }
+                            "   Router: " + this + "\n" +
-        // Select the store for writing
+                            "   Chose:  " + store);
-        ContentStore store = selectWriteStore(context);
+        }
-        // Check that we were given a valid store
+        else if (!store.isWriteSupported())
-        if (store == null)
+        {
-        {
+            throw new AlfrescoRuntimeException(
-            throw new NullPointerException(
+                    "A write store was chosen that doesn't support writes: \n" +
-                    "Unable to find a writer.  'selectWriteStore' may not return null: \n" +
+                            "   Router: " + this + "\n" +
-                    "   Router: " + this + "\n" +
+                            "   Chose:  " + store);
-                    "   Chose:  " + store);
+        }
-        }
+        ContentWriter writer = store.getWriter(context);
-        else if (!store.isWriteSupported())
+        String newContentUrl = writer.getContentUrl();
-        {
+        Pair<String, String> newCacheKey = new Pair<String, String>(instanceKey, newContentUrl);
-            throw new AlfrescoRuntimeException(
+        // Cache the store against the URL
-                    "A write store was chosen that doesn't support writes: \n" +
+        storesCacheWriteLock.lock();
-                    "   Router: " + this + "\n" +
+        try
-                    "   Chose:  " + store);
+        {
-        }
+            storesByContentUrl.put(newCacheKey, store);
-        ContentWriter writer = store.getWriter(context);
+        }
-        String newContentUrl = writer.getContentUrl();
+        finally
-        Pair<String, String> newCacheKey = new Pair<String, String>(instanceKey, newContentUrl);
+        {
-        // Cache the store against the URL
+            storesCacheWriteLock.unlock();
-        storesCacheWriteLock.lock();
+        }
-        try
+        // Done
-        {
+        if (logger.isDebugEnabled())
-            storesByContentUrl.put(newCacheKey, store);
+        {
-        }
+            logger.debug(
-        finally
+                    "Got writer and cache URL from store: \n" +
-        {
+                            "   Context: " + context + "\n" +
-            storesCacheWriteLock.unlock();
+                            "   Writer:  " + writer + "\n" +
-        }
+                            "   Store:   " + store);
-        // Done
+        }
-        if (logger.isDebugEnabled())
+        return writer;
-        {
+    }
-            logger.debug(
+
-                    "Got writer and cache URL from store: \n" +
+    public ContentWriter getWriter(ContentReader existingContentReader, String newContentUrl) throws ContentIOException
-                    "   Context: " + context + "\n" +
+    {
-                    "   Writer:  " + writer + "\n" +
+        return getWriter(new ContentContext(existingContentReader, newContentUrl));
-                    "   Store:   " + store);
+    }
-        }
+
-        return writer;
+    /**
-    }
+     * This operation has to be performed on all the stores in order to maintain the {@link ContentStore#exists(String)} contract.
-
+     */
-    public ContentWriter getWriter(ContentReader existingContentReader, String newContentUrl) throws ContentIOException
+    public boolean delete(String contentUrl) throws ContentIOException
-    {
+    {
-        return getWriter(new ContentContext(existingContentReader, newContentUrl));
+        boolean deleted = true;
-    }
+        List<ContentStore> stores = getAllStores();
-
+        for (ContentStore store : stores)
-    /**
+        {
-     * This operation has to be performed on all the stores in order to maintain the
+            if (store.isWriteSupported())
-     * {@link ContentStore#exists(String)} contract.
+            {
-     */
+                deleted &= store.delete(contentUrl);
-    public boolean delete(String contentUrl) throws ContentIOException
+            }
-    {
+        }
-        boolean deleted = true;
+        // Done
-        List<ContentStore> stores = getAllStores();
+        if (logger.isDebugEnabled())
-        for (ContentStore store : stores)
+        {
-        {
+            logger.debug("Deleted content URL from stores: \n" +
-            if (store.isWriteSupported())
+                    "   Stores:  " + stores.size() + "\n" +
-            {
+                    "   Deleted: " + deleted);
-                deleted &= store.delete(contentUrl);
+        }
-            }
+        return deleted;
-        }
+    }
-        // Done
+
-        if (logger.isDebugEnabled())
+    /**
-        {
+     * {@inheritDoc}
-            logger.debug("Deleted content URL from stores: \n" +
+     */
-                    "   Stores:  " + stores.size() + "\n" +
+    @Override
-                    "   Deleted: " + deleted);
+    public Map<String, String> getStorageProperties(String contentUrl)
-        }
+    {
-        return deleted;
+        ContentStore contentStore = selectReadStore(contentUrl);
-    }
+
-
+        if (contentStore == null)
-    /**
+        {
-     * {@inheritDoc}
+            logNoContentStore(contentUrl);
-     */
+            return Collections.emptyMap();
-    @Override
+        }
-    public Map<String, String> getStorageProperties(String contentUrl)
+        final String message = "Getting storage properties from store: ";
-    {
+        logExecution(contentUrl, contentStore, message);
-        ContentStore contentStore = selectReadStore(contentUrl);
+
-
+        return contentStore.getStorageProperties(contentUrl);
-        if (contentStore == null)
+    }
-        {
+
-            logNoContentStore(contentUrl);
+    /**
-            return Collections.emptyMap();
+     * {@inheritDoc}
-        }
+     */
-        final String message = "Getting storage properties from store: ";
+    @Override
-        logExecution(contentUrl, contentStore, message);
+    public boolean requestSendContentToArchive(String contentUrl, Map<String, Serializable> archiveParams)
-
+    {
-        return contentStore.getStorageProperties(contentUrl);
+        final ContentStore contentStore = selectReadStore(contentUrl);
-    }
+        if (contentStore == null)
-
+        {
-    /**
+            logNoContentStore(contentUrl);
-     * {@inheritDoc}
+            return ContentStore.super.requestSendContentToArchive(contentUrl, archiveParams);
-     */
+        }
-    @Override
+        final String message = "Sending content to archive: ";
-    public boolean requestSendContentToArchive(String contentUrl, Map<String, Serializable> archiveParams)
+        logExecution(contentUrl, contentStore, message);
-    {
+        return contentStore.requestSendContentToArchive(contentUrl, archiveParams);
-        final ContentStore contentStore = selectReadStore(contentUrl);
+    }
-        if (contentStore == null)
+
-        {
+    /**
-            logNoContentStore(contentUrl);
+     * {@inheritDoc}
-            return ContentStore.super.requestSendContentToArchive(contentUrl, archiveParams);
+     */
-        }
+    @Override
-        final String message = "Sending content to archive: ";
+    public boolean requestRestoreContentFromArchive(String contentUrl, Map<String, Serializable> restoreParams)
-        logExecution(contentUrl, contentStore, message);
+    {
-        return contentStore.requestSendContentToArchive(contentUrl, archiveParams);
+        final ContentStore contentStore = selectReadStore(contentUrl);
-    }
+        if (contentStore == null)
-
+        {
-    /**
+            logNoContentStore(contentUrl);
-     * {@inheritDoc}
+            return ContentStore.super.requestRestoreContentFromArchive(contentUrl, restoreParams);
-     */
+        }
-    @Override
+        final String message = "Restoring content from archive: ";
-    public boolean requestRestoreContentFromArchive(String contentUrl, Map<String, Serializable> restoreParams)
+        logExecution(contentUrl, contentStore, message);
-    {
+        return contentStore.requestRestoreContentFromArchive(contentUrl, restoreParams);
-        final ContentStore contentStore = selectReadStore(contentUrl);
+    }
-        if (contentStore == null)
+
-        {
+    private void logExecution(final String contentUrl, final ContentStore contentStore, final String message)
-            logNoContentStore(contentUrl);
+    {
-            return ContentStore.super.requestRestoreContentFromArchive(contentUrl, restoreParams);
+        if (logger.isTraceEnabled())
-        }
+        {
-        final String message = "Restoring content from archive: ";
+            logger.trace(message + "\n" +
-        logExecution(contentUrl, contentStore, message);
+                    "   Content URL: " + contentUrl + "\n" +
-        return ContentStore.super.requestRestoreContentFromArchive(contentUrl, restoreParams);
+                    "   Store:       " + contentStore);
-    }
+        }
-
+    }
-    private void logExecution(final String contentUrl, final ContentStore contentStore, final String message)
+
-    {
+    private void logNoContentStore(String contentUrl)
-        if (logger.isTraceEnabled())
+    {
-        {
+        if (logger.isTraceEnabled())
-            logger.trace(message + "\n" +
+        {
-                    "   Content URL: " + contentUrl + "\n" +
+            logger.trace("Content Store not found for content URL: " + contentUrl);
-                    "   Store:       " + contentStore);
+        }
-        }
+    }
-    }
+}
    private void logNoContentStore(String contentUrl)
    {
        if (logger.isTraceEnabled())
        {
            logger.trace("Content Store not found for content URL: " + contentUrl);
        }
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/domain/CrcHelper.java
+++ b/repository/src/main/java/org/alfresco/repo/domain/CrcHelper.java
@@ -1,123 +1,117 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-
+
-package org.alfresco.repo.domain;
+package org.alfresco.repo.domain;
-
+
-import java.io.UnsupportedEncodingException;
+import java.io.UnsupportedEncodingException;
-import java.util.zip.CRC32;
+import java.util.zip.CRC32;
-
+
-import org.alfresco.util.Pair;
+import org.alfresco.util.Pair;
-
+
-/**
+/**
- * Helper class to calculate CRC values for string persistence.
+ * Helper class to calculate CRC values for string persistence.
- * 
+ * 
- * @author Derek Hulley
+ * @author Derek Hulley
- * @since 3.2
+ * @since 3.2
- */
+ */
-public class CrcHelper
+public class CrcHelper
-{
+{
-    public static final String EMPTY_STRING = ".empty";
+    public static final String EMPTY_STRING = ".empty";
-    
+
-    /**
+    /**
-     * Calculate a persistable, unique pair of values that can be persisted in a database unique
+     * Calculate a persistable, unique pair of values that can be persisted in a database unique key and guarantee correct case-sensitivity.
-     * key and guarantee correct case-sensitivity.
+     * <p>
-     * <p>
+     * While the short-string version of the value is always lowercase, the CRC is calculated from the virgin string if case-sensitivity is enforced; in the case-insensitive case, the CRC is calculated from a lowercase version of the string.
-     * While the short-string version of the value is always lowercase, the CRC is
+     * <p>
-     * calculated from the virgin string if case-sensitivity is enforced; in the case-insensitive
+     * If the value is an empty string, then {@link #EMPTY_STRING} is used instead. This ensures that persisted values don't fall foul of the Oracle empty string comparison "behaviour" i.e you should never persist an empty string in Oracle as it equates to a SQL <b>NULL</b>.
-     * case, the CRC is calculated from a lowercase version of the string.
+     * 
-     * <p>
+     * @param value
-     * If the value is an empty string, then {@link #EMPTY_STRING} is used instead.  This ensures
+     *            the raw value that will be persisted
-     * that persisted values don't fall foul of the Oracle empty string comparison "behaviour" i.e
+     * @param dataLength
-     * you should never persist an empty string in Oracle as it equates to a SQL <b>NULL</b>.
+     *            the maximum number of characters that can be persisted
-     * 
+     * @param useCharsFromStart
-     * @param value             the raw value that will be persisted
+     *            <tt>true</tt> if the shortened string value must be made from the first characters of the string or <tt>false</tt> to use characters from the end of the string.
-     * @param dataLength        the maximum number of characters that can be persisted
+     * @param caseSensitive
-     * @param useCharsFromStart <tt>true</tt> if the shortened string value must be made from
+     *            <tt>true</tt> if the resulting pair must be case-sensitive or <tt>false</tt> if the pair must be case-insensitive.
-     *                          the first characters of the string or <tt>false</tt> to use
+     * @return Return the persistable pair. The result will never be <tt>null</tt>, but the individual pair values will be <tt>null</tt> if the value given is <tt>null</tt>
-     *                          characters from the end of the string.
+     */
-     * @param caseSensitive     <tt>true</tt> if the resulting pair must be case-sensitive or
+    public static Pair<String, Long> getStringCrcPair(
-     *                          <tt>false</tt> if the pair must be case-insensitive.
+            String value,
-     * @return                  Return the persistable pair.  The result will never be <tt>null</tt>,
+            int dataLength,
-     *                          but the individual pair values will be <tt>null</tt> if the
+            boolean useCharsFromStart,
-     *                          value given is <tt>null</tt>
+            boolean caseSensitive)
-     */
+    {
-    public static Pair<String, Long> getStringCrcPair(
+        String valueLowerCase;
-            String value,
+        if (value == null)
-            int dataLength,
+        {
-            boolean useCharsFromStart,
+            return new Pair<String, Long>(null, null);
-            boolean caseSensitive)
+        }
-    {
+        else if (value.length() == 0)
-        String valueLowerCase;
+        {
-        if (value == null)
+            value = CrcHelper.EMPTY_STRING;
-        {
+            valueLowerCase = value;
-            return new Pair<String, Long>(null, null);
+        }
-        }
+        else
-        else if (value.length() == 0)
+        {
-        {
+            valueLowerCase = value.toLowerCase();
-            value = CrcHelper.EMPTY_STRING;
+        }
-            valueLowerCase = value;
+        Long valueCrc;
-        }
+        try
-        else
+        {
-        {
+            CRC32 crc = new CRC32();
-            valueLowerCase = value.toLowerCase();
+            if (caseSensitive)
-        }
+            {
-        Long valueCrc;
+                crc.update(value.getBytes("UTF-8"));
-        try
+            }
-        {
+            else
-            CRC32 crc = new CRC32();
+            {
-            if (caseSensitive)
+                crc.update(valueLowerCase.getBytes("UTF-8"));
-            {
+            }
-                crc.update(value.getBytes("UTF-8"));
+            valueCrc = crc.getValue();
-            }
+        }
-            else
+        catch (UnsupportedEncodingException e)
-            {
+        {
-                crc.update(valueLowerCase.getBytes("UTF-8"));
+            throw new RuntimeException("UTF-8 encoding is not supported");
-            }
+
-            valueCrc = crc.getValue();
+        }
-        }
+        // Get the short value (case-sensitive or not)
-        catch (UnsupportedEncodingException e)
+        String valueShort = null;
-        {
+        int valueLen = valueLowerCase.length();
-            throw new RuntimeException("UTF-8 encoding is not supported");
+        if (valueLen < dataLength)
-        }
+        {
-        // Crc Value will change based on the case-sensitive, So we need to get the short value based on case-sensitive
+            valueShort = valueLowerCase;
-        String valueShort = null;
+        }
-        String currentValue = caseSensitive ? value : valueLowerCase;
+        else if (useCharsFromStart)
-        int valueLen = currentValue.length();
+        {
-        if (valueLen < dataLength)
+            valueShort = valueLowerCase.substring(0, dataLength - 1);
-        {
+        }
-            valueShort = currentValue;
+        else
-        }
+        {
-        else if (useCharsFromStart)
+            valueShort = valueLowerCase.substring(valueLen - dataLength);
-        {
+        }
-            valueShort = currentValue.substring(0, dataLength - 1);
+        return new Pair<String, Long>(valueShort, valueCrc);
-        }
+    }
-        else
+}
        {
            valueShort = currentValue.substring(valueLen - dataLength);
        }
        return new Pair<String, Long>(valueShort, valueCrc);
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/domain/audit/AuditDAO.java
+++ b/repository/src/main/java/org/alfresco/repo/domain/audit/AuditDAO.java
@@ -1,258 +1,285 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.domain.audit;
+package org.alfresco.repo.domain.audit;
-
+
-import java.io.Serializable;
+import java.io.Serializable;
-import java.net.URL;
+import java.net.URL;
-import java.util.HashMap;
+import java.util.HashMap;
-import java.util.List;
+import java.util.List;
-import java.util.Map;
+import java.util.Map;
-import java.util.Set;
+import java.util.Set;
-
+
-import org.alfresco.service.cmr.audit.AuditService.AuditQueryCallback;
+import org.alfresco.service.cmr.audit.AuditService.AuditQueryCallback;
-import org.alfresco.service.cmr.repository.ContentData;
+import org.alfresco.service.cmr.repository.ContentData;
-import org.alfresco.util.Pair;
+import org.alfresco.util.Pair;
-
+
-/**
+/**
- * DAO services for <b>alf_audit_XXX</b> tables.
+ * DAO services for <b>alf_audit_XXX</b> tables.
- * <p>
+ * <p>
- * The older methods are supported by a different implementation and will eventually
+ * The older methods are supported by a different implementation and will eventually be deprecated and phased out.
- * be deprecated and phased out.
+ * 
- * 
+ * @author Derek Hulley
- * @author Derek Hulley
+ * @since 3.2
- * @since 3.2
+ */
- */
+public interface AuditDAO
-public interface AuditDAO
+{
-{
+    /* V3.2 methods after here only, please */
-    /*
+
-     * V3.2 methods after here only, please
+    /**
-     */
+     * Information about the audit application to be passed in an out of the interface.
-
+     * 
-    /**
+     * @author Derek Hulley
-     * Information about the audit application to be passed in an out of the interface.
+     * @since 3.2
-     * 
+     */
-     * @author Derek Hulley
+    public static class AuditApplicationInfo
-     * @since 3.2
+    {
-     */
+        private Long id;
-    public static class AuditApplicationInfo
+        private String name;
-    {
+        private Long modelId;
-        private Long id;
+        private Long disabledPathsId;
-        private String name;
+
-        private Long modelId;
+        @Override
-        private Long disabledPathsId;
+        public String toString()
-        
+        {
-        @Override
+            StringBuilder sb = new StringBuilder();
-        public String toString()
+            sb.append("AuditApplicationInfo ")
-        {
+                    .append("[ id=").append(id)
-            StringBuilder sb = new StringBuilder();
+                    .append(", name=").append(name)
-            sb.append("AuditApplicationInfo ")
+                    .append(", modelId=").append(modelId)
-              .append("[ id=").append(id)
+                    .append(", disabledPathsId=").append(disabledPathsId)
-              .append(", name=").append(name)
+                    .append("]");
-              .append(", modelId=").append(modelId)
+            return sb.toString();
-              .append(", disabledPathsId=").append(disabledPathsId)
+        }
-              .append("]");
+
-            return sb.toString();
+        public Long getId()
-        }
+        {
-        
+            return id;
-        public Long getId()
+        }
-        {
+
-            return id;
+        public void setId(Long id)
-        }
+        {
-        public void setId(Long id)
+            this.id = id;
-        {
+        }
-            this.id = id;
+
-        }
+        public String getName()
-        public String getName()
+        {
-        {
+            return name;
-            return name;
+        }
-        }
+
-        public void setname(String name)
+        public void setname(String name)
-        {
+        {
-            this.name = name;
+            this.name = name;
-        }
+        }
-        public Long getModelId()
+
-        {
+        public Long getModelId()
-            return modelId;
+        {
-        }
+            return modelId;
-        public void setModelId(Long modelId)
+        }
-        {
+
-            this.modelId = modelId;
+        public void setModelId(Long modelId)
-        }
+        {
-        public Long getDisabledPathsId()
+            this.modelId = modelId;
-        {
+        }
-            return disabledPathsId;
+
-        }
+        public Long getDisabledPathsId()
-        public void setDisabledPathsId(Long disabledPathsId)
+        {
-        {
+            return disabledPathsId;
-            this.disabledPathsId = disabledPathsId;
+        }
-        }
+
-    }
+        public void setDisabledPathsId(Long disabledPathsId)
-    
+        {
-    /**
+            this.disabledPathsId = disabledPathsId;
-     * Creates a new audit model entry or finds an existing one
+        }
-     * 
+    }
-     * @param  url          the URL of the configuration
+
-     * @return              Returns the ID of the config matching the input stream and the
+    /**
-     *                      content storage details
+     * Creates a new audit model entry or finds an existing one
-     * @since 3.2
+     * 
-     */
+     * @param url
-    Pair<Long, ContentData> getOrCreateAuditModel(URL url);
+     *            the URL of the configuration
-    
+     * @return Returns the ID of the config matching the input stream and the content storage details
-    /**
+     * @since 3.2
-     * Get the audit application details.
+     */
-     * 
+    Pair<Long, ContentData> getOrCreateAuditModel(URL url);
-     * @param applicationName   the name of the application
+
-     * @return                  Returns details of an existing application or <tt>null</tt> if it doesn't exist
+    /**
-     * 
+     * Get the audit application details.
-     * @since 3.2
+     * 
-     */
+     * @param applicationName
-    AuditApplicationInfo getAuditApplication(String applicationName);
+     *            the name of the application
-
+     * @return Returns details of an existing application or <tt>null</tt> if it doesn't exist
-    /**
+     * 
-     * Creates a new audit application.  The application name must be unique.
+     * @since 3.2
-     * 
+     */
-     * @param application       the name of the application
+    AuditApplicationInfo getAuditApplication(String applicationName);
-     * @param modelId           the ID of the model configuration
+
-     * 
+    /**
-     * @since 3.2
+     * Creates a new audit application. The application name must be unique.
-     */
+     * 
-    AuditApplicationInfo createAuditApplication(String application, Long modelId);
+     * @param application
-    
+     *            the name of the application
-    /**
+     * @param modelId
-     * Update the audit application to refer to a new model.
+     *            the ID of the model configuration
-     * If the model did not change, then nothing will be done.
+     * 
-     * 
+     * @since 3.2
-     * @param id                the ID of the audit application
+     */
-     * @param modelId           the ID of the new model
+    AuditApplicationInfo createAuditApplication(String application, Long modelId);
-     * 
+
-     * @since 3.2
+    /**
-     */
+     * Update the audit application to refer to a new model. If the model did not change, then nothing will be done.
-    void updateAuditApplicationModel(Long id, Long modelId);
+     * 
-    
+     * @param id
-    /**
+     *            the ID of the audit application
-     * Update the audit application to hold a new set of disabled paths.
+     * @param modelId
-     * If the value did not change, then nothing will be done.
+     *            the ID of the new model
-     * 
+     * 
-     * @param id                the ID of the audit application
+     * @since 3.2
-     * @param disabledPaths     the new disabled paths
+     */
-     * 
+    void updateAuditApplicationModel(Long id, Long modelId);
-     * @since 3.2
+
-     */
+    /**
-    void updateAuditApplicationDisabledPaths(Long id, Set<String> disabledPaths);
+     * Update the audit application to hold a new set of disabled paths. If the value did not change, then nothing will be done.
-    
+     * 
-    /**
+     * @param id
-     * Delete audit entries for the application, possibly limiting the time range.
+     *            the ID of the audit application
-     * 
+     * @param disabledPaths
-     * @param applicationId     an existing audit application ID
+     *            the new disabled paths
-     * @param fromTime          the minimum entry time (inclusive, optional)
+     * 
-     * @param toTime            the maximum entry time (exclusive, optional)
+     * @since 3.2
-     * @return                  Returns the number of entries deleted
+     */
-     * 
+    void updateAuditApplicationDisabledPaths(Long id, Set<String> disabledPaths);
-     * @since 3.2
+
-     */
+    /**
-    int deleteAuditEntries(Long applicationId, Long fromTime, Long toTime);
+     * Delete audit entries for the application, possibly limiting the time range.
-
+     * 
-    /**
+     * @param applicationId
-     * Delete audit entries for the application for given id range.
+     *            an existing audit application ID
-     *
+     * @param fromTime
-     * @param applicationId     an existing audit application ID
+     *            the minimum entry time (inclusive, optional)
-     * @param fromId            the minimum fromId (inclusive, optional)
+     * @param toTime
-     * @param toId              the maximum toId (exclusive, optional)
+     *            the maximum entry time (exclusive, optional)
-     * @return                  Returns the number of entries deleted
+     * @return Returns the number of entries deleted
-     *
+     * 
-     * @since 5.2.2
+     * @since 3.2
-     */
+     */
-    int deleteAuditEntriesByIdRange(Long applicationId, Long fromId, Long toId);
+    int deleteAuditEntries(Long applicationId, Long fromTime, Long toTime);
-
+
-    /**
+    /**
-     * Delete a discrete list of audit entries.  Duplicate entries are collapsed
+     * Delete audit entries for the application for given id range.
-     * and the number of entries deleted will match the count of unique IDs in
+     *
-     * the list; otherwise a concurrency condition has occured and an exception
+     * @param applicationId
-     * will be generated.
+     *            an existing audit application ID
-     * 
+     * @param fromId
-     * @param auditEntryIds     the IDs of all audit entries to delete
+     *            the minimum fromId (inclusive, optional)
-     * @return                  Returns the number of entries deleted
+     * @param toId
-     */
+     *            the maximum toId (exclusive, optional)
-    int deleteAuditEntries(List<Long> auditEntryIds);
+     * @return Returns the number of entries deleted
-    
+     *
-    /**
+     * @since 5.2.2
-     * Create a new audit entry with the given map of values.
+     */
-     * 
+    int deleteAuditEntriesByIdRange(Long applicationId, Long fromId, Long toId);
-     * @param applicationId     an existing audit application ID
+
-     * @param time              the time (ms since epoch) to log the entry against
+    /**
-     * @param username          the authenticated user (<tt>null</tt> if not present)
+     * Delete a discrete list of audit entries. Duplicate entries are collapsed and the number of entries deleted will match the count of unique IDs in the list; otherwise a concurrency condition has occured and an exception will be generated.
-     * @param values            the values to record
+     * 
-     * @return                  Returns the unique entry ID
+     * @param auditEntryIds
-     * 
+     *            the IDs of all audit entries to delete
-     * @since 3.2
+     * @return Returns the number of entries deleted
-     */
+     */
-    Long createAuditEntry(Long applicationId, long time, String username, Map<String, Serializable> values);
+    int deleteAuditEntries(List<Long> auditEntryIds);
-    
+
-    /**
+    /**
-     * Find audit entries using the given parameters, any of which may be null
+     * Create a new audit entry with the given map of values.
-     * 
+     * 
-     * @param callback          the data callback per entry
+     * @param applicationId
-     * @param parameters        the parameters for the query (may not be <tt>null</tt>)
+     *            an existing audit application ID
-     * @param maxResults        the maximum number of results to retrieve (must be greater than 0)
+     * @param time
-     * 
+     *            the time (ms since epoch) to log the entry against
-     * @throws IllegalArgumentException if maxResults less or equal to zero
+     * @param username
-     */
+     *            the authenticated user (<tt>null</tt> if not present)
-    void findAuditEntries(
+     * @param values
-            AuditQueryCallback callback,
+     *            the values to record
-            org.alfresco.service.cmr.audit.AuditQueryParameters parameters,
+     * @return Returns the unique entry ID
-            int maxResults);
+     * 
-
+     * @since 3.2
-    /**
+     */
-     * Issue an audit query to retrieve min / max audit record id for a given application.
+    Long createAuditEntry(Long applicationId, long time, String username, Map<String, Serializable> values);
-     *
+
-     * @param appId             the database id of the application
+    /**
-     * @param extremes          a list containing min/max or both
+     * Find audit entries using the given parameters, any of which may be null
-     * @return                  a map containing min/max and the associated value
+     * 
-     */
+     * @param callback
-    HashMap<String, Long> getAuditMinMaxByApp(long appId, List<String> extremes);
+     *            the data callback per entry
-
+     * @param parameters
-    /**
+     *            the parameters for the query (may not be <tt>null</tt>)
-     * Issue an audit query to retrieve count of records for a given application.
+     * @param maxResults
-     *
+     *            the maximum number of results to retrieve (must be greater than 0)
-     * @param applicationId             the database id of the application
+     * 
-     * @return                  a map containing min/max and the associated value
+     * @throws IllegalArgumentException
-     */
+     *             if maxResults less or equal to zero
-    default int getAuditEntriesCountByApp(long applicationId)
+     */
-    {
+    void findAuditEntries(
-        return -1;
+            AuditQueryCallback callback,
-    }
+            org.alfresco.service.cmr.audit.AuditQueryParameters parameters,
-
+            int maxResults);
-    /**
+
-     * Issue an audit query to retrieve count of records for a given application and properties
+    /**
-     *
+     * Issue an audit query to retrieve min / max audit record id for a given application.
-     * @param parameters        audit parameters provided by the <code>where</code> clause on the ReST API
+     *
-     * @return                  a map containing min/max and the associated value
+     * @param appId
-     */
+     *            the database id of the application
-    default int getAuditEntriesCountByAppAndProperties(org.alfresco.service.cmr.audit.AuditQueryParameters parameters)
+     * @param extremes
-    {
+     *            a list containing min/max or both
-        return -1;
+     * @return a map containing min/max and the associated value
-    }
+     */
-}
+    HashMap<String, Long> getAuditMinMaxByApp(long appId, List<String> extremes);
    /**
     * Issue an audit query to retrieve count of records for a given application.
     *
     * @param applicationId
     *            the database id of the application
     * @return a map containing min/max and the associated value
     */
    default int getAuditEntriesCountByApp(long applicationId)
    {
        return -1;
    }
    /**
     * Issue an audit query to retrieve count of records for a given application and properties
     *
     * @param applicationName
     *            name of the application to be queried
     * @param parameters
     *            audit parameters provided by the <code>where</code> clause on the ReST API
     * @return a map containing min/max and the associated value
     */
    default int getAuditEntriesCountByAppAndProperties(String applicationName, org.alfresco.service.cmr.audit.AuditQueryParameters parameters)
    {
        return -1;
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/domain/audit/ibatis/AuditDAOImpl.java
+++ b/repository/src/main/java/org/alfresco/repo/domain/audit/ibatis/AuditDAOImpl.java
@@ -1,316 +1,315 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software.
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ *
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ *
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ *
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.domain.audit.ibatis;
+package org.alfresco.repo.domain.audit.ibatis;
-
+
-import java.io.Serializable;
+import java.io.Serializable;
-import java.util.ArrayList;
+import java.util.ArrayList;
-import java.util.HashMap;
+import java.util.HashMap;
-import java.util.List;
+import java.util.List;
-import java.util.Map;
+import java.util.Map;
-import java.util.stream.Collectors;
+import java.util.stream.Collectors;
-
+
-import org.alfresco.repo.domain.audit.AbstractAuditDAOImpl;
+import org.apache.ibatis.session.RowBounds;
-import org.alfresco.repo.domain.audit.AuditApplicationEntity;
+import org.mybatis.spring.SqlSessionTemplate;
-import org.alfresco.repo.domain.audit.AuditDeleteParameters;
+import org.springframework.dao.ConcurrencyFailureException;
-import org.alfresco.repo.domain.audit.AuditEntryEntity;
+
-import org.alfresco.repo.domain.audit.AuditModelEntity;
+import org.alfresco.repo.domain.audit.AbstractAuditDAOImpl;
-import org.alfresco.repo.domain.audit.AuditQueryParameters;
+import org.alfresco.repo.domain.audit.AuditApplicationEntity;
-import org.alfresco.repo.domain.audit.AuditQueryResult;
+import org.alfresco.repo.domain.audit.AuditDeleteParameters;
-import org.alfresco.repo.domain.propval.PropertyValueDAO.PropertyFinderCallback;
+import org.alfresco.repo.domain.audit.AuditEntryEntity;
-import org.alfresco.util.Pair;
+import org.alfresco.repo.domain.audit.AuditModelEntity;
-import org.apache.ibatis.session.RowBounds;
+import org.alfresco.repo.domain.audit.AuditQueryParameters;
-import org.mybatis.spring.SqlSessionTemplate;
+import org.alfresco.repo.domain.audit.AuditQueryResult;
-import org.springframework.dao.ConcurrencyFailureException;
+import org.alfresco.repo.domain.propval.PropertyValueDAO.PropertyFinderCallback;
-
+import org.alfresco.util.Pair;
-/**
+
- * iBatis-specific implementation of the DAO for <b>alf_audit_XXX</b> tables.
+/**
- * 
+ * iBatis-specific implementation of the DAO for <b>alf_audit_XXX</b> tables.
- * @author Derek Hulley
+ *
- * @since 3.2
+ * @author Derek Hulley
- */
+ * @since 3.2
-public class AuditDAOImpl extends AbstractAuditDAOImpl
+ */
-{
+public class AuditDAOImpl extends AbstractAuditDAOImpl
-    private static final String SELECT_MODEL_BY_CRC = "alfresco.audit.select_AuditModelByCrc";
+{
-    private static final String INSERT_MODEL = "alfresco.audit.insert.insert_AuditModel";
+    private static final String SELECT_MODEL_BY_CRC = "alfresco.audit.select_AuditModelByCrc";
-    
+    private static final String INSERT_MODEL = "alfresco.audit.insert.insert_AuditModel";
-    private static final String SELECT_APPLICATION_BY_ID = "alfresco.audit.select_AuditApplicationById";
+
-    private static final String SELECT_APPLICATION_BY_NAME_ID = "alfresco.audit.select_AuditApplicationByNameId";
+    private static final String SELECT_APPLICATION_BY_ID = "alfresco.audit.select_AuditApplicationById";
-    private static final String INSERT_APPLICATION = "alfresco.audit.insert.insert_AuditApplication";
+    private static final String SELECT_APPLICATION_BY_NAME_ID = "alfresco.audit.select_AuditApplicationByNameId";
-    private static final String UPDATE_APPLICATION = "alfresco.audit.update_AuditApplication";
+    private static final String INSERT_APPLICATION = "alfresco.audit.insert.insert_AuditApplication";
-    
+    private static final String UPDATE_APPLICATION = "alfresco.audit.update_AuditApplication";
-    private static final String DELETE_ENTRIES = "alfresco.audit.delete_AuditEntries";
+
-    private static final String DELETE_ENTRIES_BY_ID = "alfresco.audit.delete_AuditEntriesById";
+    private static final String DELETE_ENTRIES = "alfresco.audit.delete_AuditEntries";
-    private static final String INSERT_ENTRY = "alfresco.audit.insert.insert_AuditEntry";
+    private static final String DELETE_ENTRIES_BY_ID = "alfresco.audit.delete_AuditEntriesById";
-    private static final String SELECT_MINMAX_ENTRY_FOR_APP = "alfresco.audit.select_MinMaxAuditEntryId";
+    private static final String INSERT_ENTRY = "alfresco.audit.insert.insert_AuditEntry";
-    private static final String SELECT_COUNT_ENTRIES_FOR_APP = "alfresco.audit.select_CountAuditEntryId";
+    private static final String SELECT_MINMAX_ENTRY_FOR_APP = "alfresco.audit.select_MinMaxAuditEntryId";
-    private static final String SELECT_COUNT_ENTRIES_FOR_APP_WITH_PROPERTIES = "select_CountAuditEntryIdWithWhereClause";
+    private static final String SELECT_COUNT_ENTRIES_FOR_APP = "alfresco.audit.select_CountAuditEntryId";
-    
+    private static final String SELECT_COUNT_ENTRIES_FOR_APP_WITH_PROPERTIES = "select_CountAuditEntryIdWithWhereClause";
-    @SuppressWarnings("unused")
+
-    private static final String SELECT_ENTRIES_SIMPLE = "alfresco.audit.select_AuditEntriesSimple";
+    @SuppressWarnings("unused")
-    private static final String SELECT_ENTRIES_WITH_VALUES = "alfresco.audit.select_AuditEntriesWithValues";
+    private static final String SELECT_ENTRIES_SIMPLE = "alfresco.audit.select_AuditEntriesSimple";
-    private static final String SELECT_ENTRIES_WITHOUT_VALUES = "alfresco.audit.select_AuditEntriesWithoutValues";
+    private static final String SELECT_ENTRIES_WITH_VALUES = "alfresco.audit.select_AuditEntriesWithValues";
-    
+    private static final String SELECT_ENTRIES_WITHOUT_VALUES = "alfresco.audit.select_AuditEntriesWithoutValues";
-    private SqlSessionTemplate template;
+
-    
+    private SqlSessionTemplate template;
-    public final void setSqlSessionTemplate(SqlSessionTemplate sqlSessionTemplate) 
+
-    {
+    public final void setSqlSessionTemplate(SqlSessionTemplate sqlSessionTemplate)
-        this.template = sqlSessionTemplate;
+    {
-    }
+        this.template = sqlSessionTemplate;
-
+    }
-    @Override
+
-    protected AuditModelEntity getAuditModelByCrc(long crc)
+    @Override
-    {
+    protected AuditModelEntity getAuditModelByCrc(long crc)
-        AuditModelEntity entity = new AuditModelEntity();
+    {
-        entity.setContentCrc(crc);
+        AuditModelEntity entity = new AuditModelEntity();
-        entity = template.selectOne(
+        entity.setContentCrc(crc);
-                SELECT_MODEL_BY_CRC,
+        entity = template.selectOne(
-                entity);
+                SELECT_MODEL_BY_CRC,
-        // Done
+                entity);
-        return entity;
+        // Done
-    }
+        return entity;
-
+    }
-    @Override
+
-    protected AuditModelEntity createAuditModel(Long contentDataId, long crc)
+    @Override
-    {
+    protected AuditModelEntity createAuditModel(Long contentDataId, long crc)
-        AuditModelEntity entity = new AuditModelEntity();
+    {
-        entity.setContentDataId(contentDataId);
+        AuditModelEntity entity = new AuditModelEntity();
-        entity.setContentCrc(crc);
+        entity.setContentDataId(contentDataId);
-        template.insert(INSERT_MODEL, entity);
+        entity.setContentCrc(crc);
-        return entity;
+        template.insert(INSERT_MODEL, entity);
-    }
+        return entity;
-
+    }
-    @Override
+
-    protected AuditApplicationEntity getAuditApplicationById(Long id)
+    @Override
-    {
+    protected AuditApplicationEntity getAuditApplicationById(Long id)
-        Map<String, Object> params = new HashMap<String, Object>(11);
+    {
-        params.put("id", id);
+        Map<String, Object> params = new HashMap<String, Object>(11);
-        AuditApplicationEntity entity = template.selectOne(
+        params.put("id", id);
-                SELECT_APPLICATION_BY_ID,
+        AuditApplicationEntity entity = template.selectOne(
-                params);
+                SELECT_APPLICATION_BY_ID,
-        // Done
+                params);
-        if (logger.isDebugEnabled())
+        // Done
-        {
+        if (logger.isDebugEnabled())
-            logger.debug("Searched for audit application ID " + id + " and found: " + entity);
+        {
-        }
+            logger.debug("Searched for audit application ID " + id + " and found: " + entity);
-        return entity;
+        }
-    }
+        return entity;
-
+    }
-    @Override
+
-    protected AuditApplicationEntity getAuditApplicationByName(String appName)
+    @Override
-    {
+    protected AuditApplicationEntity getAuditApplicationByName(String appName)
-        // Resolve the name as a property ID
+    {
-        Pair<Long, Serializable> appNamePair = propertyValueDAO.getPropertyValue(appName);
+        // Resolve the name as a property ID
-        if (appNamePair == null)
+        Pair<Long, Serializable> appNamePair = propertyValueDAO.getPropertyValue(appName);
-        {
+        if (appNamePair == null)
-            // There will be no results
+        {
-            return null;
+            // There will be no results
-        }
+            return null;
-        
+        }
-        Map<String, Object> params = new HashMap<String, Object>(11);
+
-        params.put("id", appNamePair.getFirst());
+        Map<String, Object> params = new HashMap<String, Object>(11);
-        AuditApplicationEntity entity = template.selectOne(
+        params.put("id", appNamePair.getFirst());
-                SELECT_APPLICATION_BY_NAME_ID,
+        AuditApplicationEntity entity = template.selectOne(
-                params);
+                SELECT_APPLICATION_BY_NAME_ID,
-        // Done
+                params);
-        if (logger.isDebugEnabled())
+        // Done
-        {
+        if (logger.isDebugEnabled())
-            logger.debug("Searched for audit application '" + appName + "' and found: " + entity);
+        {
-        }
+            logger.debug("Searched for audit application '" + appName + "' and found: " + entity);
-        return entity;
+        }
-    }
+        return entity;
-
+    }
-    @Override
+
-    protected AuditApplicationEntity createAuditApplication(Long appNameId, Long modelId, Long disabledPathsId)
+    @Override
-    {
+    protected AuditApplicationEntity createAuditApplication(Long appNameId, Long modelId, Long disabledPathsId)
-        AuditApplicationEntity entity = new AuditApplicationEntity();
+    {
-        entity.setVersion((short)0);
+        AuditApplicationEntity entity = new AuditApplicationEntity();
-        entity.setApplicationNameId(appNameId);
+        entity.setVersion((short) 0);
-        entity.setAuditModelId(modelId);
+        entity.setApplicationNameId(appNameId);
-        entity.setDisabledPathsId(disabledPathsId);
+        entity.setAuditModelId(modelId);
-        template.insert(INSERT_APPLICATION, entity);
+        entity.setDisabledPathsId(disabledPathsId);
-        return entity;
+        template.insert(INSERT_APPLICATION, entity);
-    }
+        return entity;
-
+    }
-    @Override
+
-    protected AuditApplicationEntity updateAuditApplication(AuditApplicationEntity entity)
+    @Override
-    {
+    protected AuditApplicationEntity updateAuditApplication(AuditApplicationEntity entity)
-        AuditApplicationEntity updateEntity = new AuditApplicationEntity();
+    {
-        updateEntity.setId(entity.getId());
+        AuditApplicationEntity updateEntity = new AuditApplicationEntity();
-        updateEntity.setVersion(entity.getVersion());
+        updateEntity.setId(entity.getId());
-        updateEntity.incrementVersion();
+        updateEntity.setVersion(entity.getVersion());
-        updateEntity.setApplicationNameId(entity.getApplicationNameId());
+        updateEntity.incrementVersion();
-        updateEntity.setAuditModelId(entity.getAuditModelId());
+        updateEntity.setApplicationNameId(entity.getApplicationNameId());
-        updateEntity.setDisabledPathsId(entity.getDisabledPathsId());
+        updateEntity.setAuditModelId(entity.getAuditModelId());
-        
+        updateEntity.setDisabledPathsId(entity.getDisabledPathsId());
-        int updated = template.update(UPDATE_APPLICATION, updateEntity);
+
-        if (updated != 1)
+        int updated = template.update(UPDATE_APPLICATION, updateEntity);
-        {
+        if (updated != 1)
-            // unexpected number of rows affected
+        {
-            throw new ConcurrencyFailureException("Incorrect number of rows affected for updateAuditApplication: " + updateEntity + ": expected 1, actual " + updated);
+            // unexpected number of rows affected
-        }
+            throw new ConcurrencyFailureException("Incorrect number of rows affected for updateAuditApplication: " + updateEntity + ": expected 1, actual " + updated);
-        
+        }
-        // Done
+
-        return updateEntity;
+        // Done
-    }
+        return updateEntity;
-
+    }
-    public int deleteAuditEntries(Long applicationId, Long from, Long to)
+
-    {
+    public int deleteAuditEntries(Long applicationId, Long from, Long to)
-        AuditDeleteParameters params = new AuditDeleteParameters();
+    {
-        params.setAuditApplicationId(applicationId);
+        AuditDeleteParameters params = new AuditDeleteParameters();
-        params.setAuditFromTime(from);
+        params.setAuditApplicationId(applicationId);
-        params.setAuditToTime(to);
+        params.setAuditFromTime(from);
-        return template.delete(DELETE_ENTRIES, params);
+        params.setAuditToTime(to);
-    }
+        return template.delete(DELETE_ENTRIES, params);
-
+    }
-    public int deleteAuditEntriesByIdRange(Long applicationId, Long fromId, Long toId)
+
-    {
+    public int deleteAuditEntriesByIdRange(Long applicationId, Long fromId, Long toId)
-        AuditDeleteParameters params = new AuditDeleteParameters();
+    {
-        params.setAuditApplicationId(applicationId);
+        AuditDeleteParameters params = new AuditDeleteParameters();
-        params.setAuditFromId(fromId);
+        params.setAuditApplicationId(applicationId);
-        params.setAuditToId(toId);
+        params.setAuditFromId(fromId);
-        return template.delete(DELETE_ENTRIES, params);
+        params.setAuditToId(toId);
-    }
+        return template.delete(DELETE_ENTRIES, params);
-
+    }
-    @Override
+
-    protected int deleteAuditEntriesImpl(List<Long> auditEntryIds)
+    @Override
-    {
+    protected int deleteAuditEntriesImpl(List<Long> auditEntryIds)
-        AuditDeleteParameters params = new AuditDeleteParameters();
+    {
-        params.setAuditEntryIds(auditEntryIds);
+        AuditDeleteParameters params = new AuditDeleteParameters();
-        return template.delete(DELETE_ENTRIES_BY_ID, params);
+        params.setAuditEntryIds(auditEntryIds);
-    }
+        return template.delete(DELETE_ENTRIES_BY_ID, params);
-
+    }
-    @Override
+
-    protected AuditEntryEntity createAuditEntry(Long applicationId, long time, Long usernameId, Long valuesId)
+    @Override
-    {
+    protected AuditEntryEntity createAuditEntry(Long applicationId, long time, Long usernameId, Long valuesId)
-        AuditEntryEntity entity = new AuditEntryEntity();
+    {
-        entity.setAuditApplicationId(applicationId);
+        AuditEntryEntity entity = new AuditEntryEntity();
-        entity.setAuditTime(time);
+        entity.setAuditApplicationId(applicationId);
-        entity.setAuditUserId(usernameId);
+        entity.setAuditTime(time);
-        entity.setAuditValuesId(valuesId);
+        entity.setAuditUserId(usernameId);
-        template.insert(INSERT_ENTRY, entity);
+        entity.setAuditValuesId(valuesId);
-        return entity;
+        template.insert(INSERT_ENTRY, entity);
-    }
+        return entity;
-
+    }
-    public HashMap<String, Long> getAuditMinMaxByApp(long appId, List<String> extremes)
+
-    {
+    public HashMap<String, Long> getAuditMinMaxByApp(long appId, List<String> extremes)
-        // Build parameters to be used in the query. Filter the duplicates when inserting into map
+    {
-        Map<String, Object> params = extremes.stream().collect(Collectors.toMap(s -> s, s -> Boolean.TRUE, (s1, s2) -> s1));
+        // Build parameters to be used in the query. Filter the duplicates when inserting into map
-        params.put("auditAppId", appId);
+        Map<String, Object> params = extremes.stream().collect(Collectors.toMap(s -> s, s -> Boolean.TRUE, (s1, s2) -> s1));
-
+        params.put("auditAppId", appId);
-        HashMap<String, Long> result = template.selectOne(SELECT_MINMAX_ENTRY_FOR_APP, params);
+
-
+        HashMap<String, Long> result = template.selectOne(SELECT_MINMAX_ENTRY_FOR_APP, params);
-        return result;
+
-    }
+        return result;
-
+    }
-    @Override
+
-    public int getAuditEntriesCountByApp(long applicationId)
+    @Override
-    {
+    public int getAuditEntriesCountByApp(long applicationId)
-        Map<String, Object> params = new HashMap<>();
+    {
-        params.put("auditAppId", applicationId);
+        Map<String, Object> params = new HashMap<>();
-
+        params.put("auditAppId", applicationId);
-        int result = template.selectOne(SELECT_COUNT_ENTRIES_FOR_APP, params);
+
-
+        int result = template.selectOne(SELECT_COUNT_ENTRIES_FOR_APP, params);
-        return result;
+
-    }
+        return result;
-
+    }
-    @Override
+
-    public int getAuditEntriesCountByAppAndProperties(org.alfresco.service.cmr.audit.AuditQueryParameters parameters)
+    @Override
-    {
+    public int getAuditEntriesCountByAppAndProperties(String applicationName, org.alfresco.service.cmr.audit.AuditQueryParameters parameters)
-        AuditQueryParameters dbParameters = convertFromRestAuditQueryParameters(parameters);
+    {
-
+        AuditQueryParameters dbParameters = convertFromRestAuditQueryParameters(parameters);
-        int result = template.selectOne(SELECT_COUNT_ENTRIES_FOR_APP_WITH_PROPERTIES, dbParameters);
+
-
+        int result = template.selectOne(SELECT_COUNT_ENTRIES_FOR_APP_WITH_PROPERTIES, dbParameters);
-        return result;
+
-    }
+        return result;
-
+    }
-    @SuppressWarnings("unchecked")
+
-    @Override
+    @SuppressWarnings("unchecked")
-    protected void findAuditEntries(
+    @Override
-            final AuditQueryRowHandler rowHandler,
+    protected void findAuditEntries(
-            int maxResults,
+            final AuditQueryRowHandler rowHandler,
-            org.alfresco.service.cmr.audit.AuditQueryParameters restParameters)
+            int maxResults,
-    {
+            org.alfresco.service.cmr.audit.AuditQueryParameters restParameters)
-        AuditQueryParameters params = convertFromRestAuditQueryParameters(restParameters);
+    {
-        if (params==null)
+        AuditQueryParameters params = convertFromRestAuditQueryParameters(restParameters);
-        {
+        if (params == null)
-            return;
+        {
-        }
+            return;
-
+        }
-        if (maxResults > 0)
+
-        {
+        if (maxResults > 0)
-            // Query without getting the values.  We gather all the results and batch-fetch the audited
+        {
-            // values afterwards.
+            // Query without getting the values. We gather all the results and batch-fetch the audited
-            final Map<Long, AuditQueryResult> resultsByValueId = new HashMap<Long, AuditQueryResult>(173);
+            // values afterwards.
-            PropertyFinderCallback propertyFinderCallback = new PropertyFinderCallback()
+            final Map<Long, AuditQueryResult> resultsByValueId = new HashMap<Long, AuditQueryResult>(173);
-            {
+            PropertyFinderCallback propertyFinderCallback = new PropertyFinderCallback() {
-                public void handleProperty(Long id, Serializable value)
+                public void handleProperty(Long id, Serializable value)
-                {
+                {
-                    // get the row
+                    // get the row
-                    AuditQueryResult row = resultsByValueId.get(id);
+                    AuditQueryResult row = resultsByValueId.get(id);
-                    try
+                    try
-                    {
+                    {
-                        row.setAuditValue((Map<String, Serializable>) value);
+                        row.setAuditValue((Map<String, Serializable>) value);
-                    }
+                    }
-                    catch (ClassCastException e)
+                    catch (ClassCastException e)
-                    {
+                    {
-                        // The handler will deal with the entry
+                        // The handler will deal with the entry
-                    }
+                    }
-                }
+                }
-            };
+            };
-            
+
-            List<AuditQueryResult> rows = template.selectList(SELECT_ENTRIES_WITHOUT_VALUES, params, new RowBounds(0, maxResults));
+            List<AuditQueryResult> rows = template.selectList(SELECT_ENTRIES_WITHOUT_VALUES, params, new RowBounds(0, maxResults));
-            for (AuditQueryResult row : rows)
+            for (AuditQueryResult row : rows)
-            {
+            {
-                resultsByValueId.put(row.getAuditValuesId(), row);
+                resultsByValueId.put(row.getAuditValuesId(), row);
-                if (resultsByValueId.size() >= 100)
+                if (resultsByValueId.size() >= 100)
-                {
+                {
-                    // Fetch values for the results.  The treemap is ordered.
+                    // Fetch values for the results. The treemap is ordered.
-                    List<Long> valueIds = new ArrayList<Long>(resultsByValueId.keySet());
+                    List<Long> valueIds = new ArrayList<Long>(resultsByValueId.keySet());
-                    propertyValueDAO.getPropertiesByIds(valueIds, propertyFinderCallback);
+                    propertyValueDAO.getPropertiesByIds(valueIds, propertyFinderCallback);
-                    // Clear and continue
+                    // Clear and continue
-                    resultsByValueId.clear();
+                    resultsByValueId.clear();
-                }
+                }
-            }
+            }
-            // Process any remaining results
+            // Process any remaining results
-            if (resultsByValueId.size() > 0)
+            if (resultsByValueId.size() > 0)
-            {
+            {
-                // Fetch values for the results.  The treemap is ordered.
+                // Fetch values for the results. The treemap is ordered.
-                List<Long> valueIds = new ArrayList<Long>(resultsByValueId.keySet());
+                List<Long> valueIds = new ArrayList<Long>(resultsByValueId.keySet());
-                propertyValueDAO.getPropertiesByIds(valueIds, propertyFinderCallback);
+                propertyValueDAO.getPropertiesByIds(valueIds, propertyFinderCallback);
-            }
+            }
-            // Now pass the filled-out results to the row handler (order-preserved)
+            // Now pass the filled-out results to the row handler (order-preserved)
-            for (AuditQueryResult row : rows)
+            for (AuditQueryResult row : rows)
-            {
+            {
-                rowHandler.processResult(row);
+                rowHandler.processResult(row);
-            }
+            }
-        }
+        }
-        else
+        else
-        {
+        {
-            throw new IllegalArgumentException("maxResults must be greater than 0");
+            throw new IllegalArgumentException("maxResults must be greater than 0");
-        }
+        }
-    }
+    }
-}
+}
--- a/repository/src/main/java/org/alfresco/repo/domain/propval/PropertyStringValueEntity.java
+++ b/repository/src/main/java/org/alfresco/repo/domain/propval/PropertyStringValueEntity.java
@@ -1,173 +1,159 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.domain.propval;
+package org.alfresco.repo.domain.propval;
-
+
-import org.alfresco.repo.domain.CrcHelper;
+import org.alfresco.repo.domain.CrcHelper;
-import org.alfresco.util.EqualsHelper;
+import org.alfresco.util.EqualsHelper;
-import org.alfresco.util.Pair;
+import org.alfresco.util.Pair;
-
+
-/**
+/**
- * Entity bean for <b>alf_prop_string_value</b> table.
+ * Entity bean for <b>alf_prop_string_value</b> table.
- * 
+ * 
- * @author Derek Hulley
+ * @author Derek Hulley
- * @since 3.2
+ * @since 3.2
- */
+ */
-public class PropertyStringValueEntity
+public class PropertyStringValueEntity
-{
+{
-    public static final String EMPTY_STRING = "";
+    public static final String EMPTY_STRING = "";
-    public static final String EMPTY_STRING_REPLACEMENT = ".empty";
+    public static final String EMPTY_STRING_REPLACEMENT = ".empty";
-    
+
-    private Long id;
+    private Long id;
-    private String stringValue;
+    private String stringValue;
-    private String stringEndLower;
+    private String stringEndLower;
-    private Long stringCrc;
+    private Long stringCrc;
-    private String stringLower;
+
-    
+    public PropertyStringValueEntity()
-    public PropertyStringValueEntity()
+    {}
-    {
+
-    }
+    @Override
-    
+    public int hashCode()
-    @Override
+    {
-    public int hashCode()
+        return (stringValue == null ? 0 : stringValue.hashCode());
-    {
+    }
-        return (stringValue == null ? 0 : stringValue.hashCode());
+
-    }
+    @Override
-    
+    public boolean equals(Object obj)
-    @Override
+    {
-    public boolean equals(Object obj)
+        if (this == obj)
-    {
+        {
-        if (this == obj)
+            return true;
-        {
+        }
-            return true;
+        else if (obj != null && obj instanceof PropertyStringValueEntity)
-        }
+        {
-        else if (obj != null && obj instanceof PropertyStringValueEntity)
+            PropertyStringValueEntity that = (PropertyStringValueEntity) obj;
-        {
+            return EqualsHelper.nullSafeEquals(this.stringValue, that.stringValue);
-            PropertyStringValueEntity that = (PropertyStringValueEntity) obj;
+        }
-            return EqualsHelper.nullSafeEquals(this.stringValue, that.stringValue);
+        else
-        }
+        {
-        else
+            return false;
-        {
+        }
-            return false;
+    }
-        }
+
-    }
+    @Override
-    
+    public String toString()
-    @Override
+    {
-    public String toString()
+        StringBuilder sb = new StringBuilder(512);
-    {
+        sb.append("PropertyStringValueEntity")
-        StringBuilder sb = new StringBuilder(512);
+                .append("[ ID=").append(id)
-        sb.append("PropertyStringValueEntity")
+                .append(", stringValue=").append(stringValue)
-          .append("[ ID=").append(id)
+                .append("]");
-          .append(", stringValue=").append(stringValue)
+        return sb.toString();
-          .append("]");
+    }
-        return sb.toString();
+
-    }
+    public Pair<Long, String> getEntityPair()
-    
+    {
-    public Pair<Long, String> getEntityPair()
+        if (stringValue != null && stringValue.equals(PropertyStringValueEntity.EMPTY_STRING_REPLACEMENT))
-    {
+        {
-        if (stringValue != null && stringValue.equals(PropertyStringValueEntity.EMPTY_STRING_REPLACEMENT))
+            return new Pair<Long, String>(id, PropertyStringValueEntity.EMPTY_STRING);
-        {
+        }
-            return new Pair<Long, String>(id, PropertyStringValueEntity.EMPTY_STRING);
+        else
-        }
+        {
-        else
+            return new Pair<Long, String>(id, stringValue);
-        {
+        }
-            return new Pair<Long, String>(id, stringValue);
+    }
-        }
+
-    }
+    /**
-    
+     * Set the string and string-end values
-    /**
+     */
-     * Set the string and string-end values
+    public void setValue(String value)
-     */
+    {
-    public void setValue(String value)
+        if (value == null)
-    {
+        {
-        if (value == null)
+            throw new IllegalArgumentException("Null strings cannot be persisted");
-        {
+        }
-            throw new IllegalArgumentException("Null strings cannot be persisted");
+        if (value != null && value.equals(PropertyStringValueEntity.EMPTY_STRING))
-        }
+        {
-        if (value != null && value.equals(PropertyStringValueEntity.EMPTY_STRING))
+            // Oracle: We can't insert empty strings into the column.
-        {
+            value = PropertyStringValueEntity.EMPTY_STRING_REPLACEMENT;
-            // Oracle: We can't insert empty strings into the column.
+        }
-            value = PropertyStringValueEntity.EMPTY_STRING_REPLACEMENT;
+        stringValue = value;
-        }
+        // Calculate the crc value from the original value
-        stringValue = value;
+        Pair<String, Long> crcPair = CrcHelper.getStringCrcPair(value, 16, false, true);
-        // Calculate the crc value from the original value
+        stringEndLower = crcPair.getFirst();
-        Pair<String, Long> crcPair = CrcHelper.getStringCrcPair(value, 16, false, true);
+        stringCrc = crcPair.getSecond();
-        stringEndLower = crcPair.getFirst();
+
-        stringCrc = crcPair.getSecond();
+    }
-        // Calculate the crc value with case-insensitive
+
-        Pair<String, Long> crcPairWithCaseInSensitive = CrcHelper.getStringCrcPair(value, 16, false, false);
+    public Long getId()
-        stringLower = crcPairWithCaseInSensitive.getFirst();
+    {
-    }
+        return id;
-
+    }
-    public Long getId()
+
-    {
+    public void setId(Long id)
-        return id;
+    {
-    }
+        this.id = id;
-
+    }
-    public void setId(Long id)
+
-    {
+    public String getStringValue()
-        this.id = id;
+    {
-    }
+        return stringValue;
-
+    }
-    public String getStringValue()
+
-    {
+    public void setStringValue(String stringValue)
-        return stringValue;
+    {
-    }
+        this.stringValue = stringValue;
-
+    }
-    public void setStringValue(String stringValue)
+
-    {
+    public String getStringEndLower()
-        this.stringValue = stringValue;
+    {
-    }
+        return stringEndLower;
-    
+    }
-    public String getStringEndLower()
+
-    {
+    public void setStringEndLower(String stringEndLower)
-        return stringEndLower;
+    {
-    }
+        this.stringEndLower = stringEndLower;
-
+    }
-    public void setStringEndLower(String stringEndLower)
+
-    {
+    public Long getStringCrc()
-        this.stringEndLower = stringEndLower;
+    {
-    }
+        return stringCrc;
-
+    }
-    public Long getStringCrc()
+
-    {
+    public void setStringCrc(Long stringCrc)
-        return stringCrc;
+    {
-    }
+        this.stringCrc = stringCrc;
-
+    }
-    public void setStringCrc(Long stringCrc)
+}
    {
        this.stringCrc = stringCrc;
    }
    public String getStringLower()
    {
        return stringLower;
    }
    public void setStringLower(String stringLower)
    {
        this.stringLower = stringLower;
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/download/ZipDownloadExporter.java
+++ b/repository/src/main/java/org/alfresco/repo/download/ZipDownloadExporter.java
@@ -1,329 +1,344 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2016 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software.
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ *
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ *
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ *
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.download;
+package org.alfresco.repo.download;
-
+
-import java.io.File;
+import java.io.File;
-import java.io.FileNotFoundException;
+import java.io.FileNotFoundException;
-import java.io.FileOutputStream;
+import java.io.FileOutputStream;
-import java.io.IOException;
+import java.io.IOException;
-import java.io.InputStream;
+import java.io.InputStream;
-import java.io.OutputStream;
+import java.io.OutputStream;
-import java.nio.file.attribute.FileTime;
+import java.nio.file.attribute.FileTime;
-import java.util.Date;
+import java.util.Date;
-import java.util.Deque;
+import java.util.Deque;
-import java.util.Iterator;
+import java.util.Iterator;
-import java.util.LinkedList;
+import java.util.LinkedList;
-
+
-import org.alfresco.model.ContentModel;
+import org.apache.commons.compress.archivers.zip.ZipArchiveEntry;
-import org.alfresco.repo.transaction.RetryingTransactionHelper;
+import org.apache.commons.compress.archivers.zip.ZipArchiveOutputStream;
-import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
+import org.apache.commons.compress.archivers.zip.ZipArchiveOutputStream.UnicodeExtraFieldPolicy;
-import org.alfresco.service.cmr.coci.CheckOutCheckInService;
+import org.slf4j.Logger;
-import org.alfresco.service.cmr.dictionary.DictionaryService;
+import org.slf4j.LoggerFactory;
-import org.alfresco.service.cmr.download.DownloadStatus;
+
-import org.alfresco.service.cmr.download.DownloadStatus.Status;
+import org.alfresco.model.ContentModel;
-import org.alfresco.service.cmr.repository.ContentData;
+import org.alfresco.repo.transaction.RetryingTransactionHelper;
-import org.alfresco.service.cmr.repository.NodeRef;
+import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
-import org.alfresco.service.cmr.repository.NodeService;
+import org.alfresco.service.cmr.coci.CheckOutCheckInService;
-import org.alfresco.service.cmr.view.ExporterContext;
+import org.alfresco.service.cmr.dictionary.DictionaryService;
-import org.alfresco.service.cmr.view.ExporterException;
+import org.alfresco.service.cmr.download.DownloadStatus;
-import org.alfresco.service.namespace.QName;
+import org.alfresco.service.cmr.download.DownloadStatus.Status;
-import org.alfresco.util.Pair;
+import org.alfresco.service.cmr.repository.ContentData;
-import org.apache.commons.compress.archivers.zip.ZipArchiveEntry;
+import org.alfresco.service.cmr.repository.NodeRef;
-import org.apache.commons.compress.archivers.zip.ZipArchiveOutputStream;
+import org.alfresco.service.cmr.repository.NodeService;
-import org.apache.commons.compress.archivers.zip.ZipArchiveOutputStream.UnicodeExtraFieldPolicy;
+import org.alfresco.service.cmr.view.ExporterContext;
-import org.slf4j.Logger;
+import org.alfresco.service.cmr.view.ExporterException;
-import org.slf4j.LoggerFactory;
+import org.alfresco.service.namespace.QName;
-
+import org.alfresco.util.Pair;
-
+
-/**
+/**
- * Handler for exporting node content to a ZIP file
+ * Handler for exporting node content to a ZIP file
- * 
+ *
- * @author Alex Miller
+ * @author Alex Miller
- */
+ */
-public class ZipDownloadExporter extends BaseExporter
+public class ZipDownloadExporter extends BaseExporter
-{
+{
-    private static Logger log = LoggerFactory.getLogger(ZipDownloadExporter.class);
+    private static Logger log = LoggerFactory.getLogger(ZipDownloadExporter.class);
-    
+
-    private static final String PATH_SEPARATOR = "/";
+    private static final String PATH_SEPARATOR = "/";
-
+
-    protected ZipArchiveOutputStream zipStream;
+    protected ZipArchiveOutputStream zipStream;
-
+
-    private NodeRef downloadNodeRef;
+    private NodeRef downloadNodeRef;
-    private int sequenceNumber = 1;
+    private int sequenceNumber = 1;
-    private long total;
+    private long total;
-    private long done;
+    private long done;
-    private long totalFileCount;
+    private long totalFileCount;
-    private long filesAddedCount;
+    private long filesAddedCount;
-    
+
-    private RetryingTransactionHelper transactionHelper;
+    private RetryingTransactionHelper transactionHelper;
-    private DownloadStorage downloadStorage;
+    private DownloadStorage downloadStorage;
-    private DictionaryService dictionaryService;
+    private DictionaryService dictionaryService;
-    private DownloadStatusUpdateService updateService;
+    private DownloadStatusUpdateService updateService;
-
+
-    private Deque<Pair<String, NodeRef>> path = new LinkedList<Pair<String, NodeRef>>();
+    private Deque<Pair<String, NodeRef>> path = new LinkedList<Pair<String, NodeRef>>();
-    private String currentName;
+    private String currentName;
-
+
-    private OutputStream outputStream;
+    private OutputStream outputStream;
-    private Date zipTimestampCreated;
+    private Date zipTimestampCreated;
-    private Date zipTimestampModified;
+    private Date zipTimestampModified;
-
+
-    /**
+    /**
-     * Construct
+     * Construct
-     *
+     *
-     * @param zipFile File
+     * @param zipFile
-     * @param checkOutCheckInService CheckOutCheckInService
+     *            File
-     * @param nodeService NodeService
+     * @param checkOutCheckInService
-     * @param transactionHelper RetryingTransactionHelper
+     *            CheckOutCheckInService
-     * @param updateService DownloadStatusUpdateService
+     * @param nodeService
-     * @param downloadStorage DownloadStorage
+     *            NodeService
-     * @param dictionaryService DictionaryService
+     * @param transactionHelper
-     * @param downloadNodeRef NodeRef
+     *            RetryingTransactionHelper
-     * @param total long
+     * @param updateService
-     * @param totalFileCount long
+     *            DownloadStatusUpdateService
-     */
+     * @param downloadStorage
-    public ZipDownloadExporter(File zipFile, CheckOutCheckInService checkOutCheckInService, NodeService nodeService, RetryingTransactionHelper transactionHelper, DownloadStatusUpdateService updateService, DownloadStorage downloadStorage, DictionaryService dictionaryService, NodeRef downloadNodeRef, long total, long totalFileCount)
+     *            DownloadStorage
-    {
+     * @param dictionaryService
-        super(checkOutCheckInService, nodeService);
+     *            DictionaryService
-        try
+     * @param downloadNodeRef
-        {
+     *            NodeRef
-            this.outputStream = new FileOutputStream(zipFile);
+     * @param total
-            this.updateService = updateService;
+     *            long
-            this.transactionHelper = transactionHelper;
+     * @param totalFileCount
-            this.downloadStorage = downloadStorage;
+     *            long
-            this.dictionaryService = dictionaryService;
+     */
-            
+    public ZipDownloadExporter(File zipFile, CheckOutCheckInService checkOutCheckInService, NodeService nodeService,
-            this.downloadNodeRef = downloadNodeRef;
+            RetryingTransactionHelper transactionHelper, DownloadStatusUpdateService updateService,
-            this.total = total;
+            DownloadStorage downloadStorage, DictionaryService dictionaryService,
-            this.totalFileCount = totalFileCount;
+            NodeRef downloadNodeRef, long total, long totalFileCount)
-        }
+    {
-        catch (FileNotFoundException e)
+        super(checkOutCheckInService, nodeService);
-        {
+        try
-            throw new ExporterException("Failed to create zip file", e);
+        {
-        }
+            this.outputStream = new FileOutputStream(zipFile);
-    }
+            this.updateService = updateService;
-
+            this.transactionHelper = transactionHelper;
-    @Override
+            this.downloadStorage = downloadStorage;
-    public void start(final ExporterContext context)
+            this.dictionaryService = dictionaryService;
-    {
+
-        zipStream = new ZipArchiveOutputStream(outputStream);
+            this.downloadNodeRef = downloadNodeRef;
-        // NOTE: This encoding allows us to workaround bug...
+            this.total = total;
-        //       http://bugs.sun.com/bugdatabase/view_bug.do;:WuuT?bug_id=4820807
+            this.totalFileCount = totalFileCount;
-        zipStream.setEncoding("UTF-8");
+        }
-        zipStream.setCreateUnicodeExtraFields(UnicodeExtraFieldPolicy.ALWAYS);
+        catch (FileNotFoundException e)
-        zipStream.setUseLanguageEncodingFlag(true);
+        {
-        zipStream.setFallbackToUTF8(true);
+            throw new ExporterException("Failed to create zip file", e);
-    }
+        }
-
+    }
-    @Override
+
-    public void startNode(NodeRef nodeRef)
+    @Override
-    {
+    public void start(final ExporterContext context)
-        this.currentName = (String)nodeService.getProperty(nodeRef, ContentModel.PROP_NAME);
+    {
-        this.zipTimestampCreated = (Date)nodeService.getProperty(nodeRef, ContentModel.PROP_CREATED);
+        zipStream = new ZipArchiveOutputStream(outputStream);
-        this.zipTimestampModified = (Date)nodeService.getProperty(nodeRef, ContentModel.PROP_MODIFIED);
+        // NOTE: This encoding allows us to workaround bug...
-        path.push(new Pair<String, NodeRef>(currentName, nodeRef));
+        // http://bugs.sun.com/bugdatabase/view_bug.do;:WuuT?bug_id=4820807
-        if (dictionaryService.isSubClass(nodeService.getType(nodeRef), ContentModel.TYPE_FOLDER))
+        zipStream.setEncoding("UTF-8");
-        {
+        zipStream.setCreateUnicodeExtraFields(UnicodeExtraFieldPolicy.ALWAYS);
-            String path = getPath() + PATH_SEPARATOR;
+        zipStream.setUseLanguageEncodingFlag(true);
-            ZipArchiveEntry archiveEntry = new ZipArchiveEntry(path);
+        zipStream.setFallbackToUTF8(true);
-            try
+    }
-            {
+
-                archiveEntry.setTime(zipTimestampCreated.getTime());
+    @Override
-                archiveEntry.setCreationTime(FileTime.fromMillis(zipTimestampCreated.getTime()));
+    public void startNode(NodeRef nodeRef)
-                archiveEntry.setLastModifiedTime(FileTime.fromMillis(zipTimestampModified.getTime()));
+    {
-                zipStream.putArchiveEntry(archiveEntry);
+        this.currentName = (String) nodeService.getProperty(nodeRef, ContentModel.PROP_NAME);
-                zipStream.closeArchiveEntry();
+        this.zipTimestampCreated = (Date) nodeService.getProperty(nodeRef, ContentModel.PROP_CREATED);
-            }
+        this.zipTimestampModified = (Date) nodeService.getProperty(nodeRef, ContentModel.PROP_MODIFIED);
-            catch (IOException e)
+        path.push(new Pair<String, NodeRef>(currentName, nodeRef));
-            {
+        if (dictionaryService.isSubClass(nodeService.getType(nodeRef), ContentModel.TYPE_FOLDER))
-                throw new ExporterException("Unexpected IOException adding folder entry", e);
+        {
-            }
+            String path = getPath() + PATH_SEPARATOR;
-        }
+            ZipArchiveEntry archiveEntry = new ZipArchiveEntry(path);
-    }
+            try
-    
+            {
-    @Override
+                archiveEntry.setTime(zipTimestampCreated.getTime());
-    public void contentImpl(NodeRef nodeRef, QName property, InputStream content, ContentData contentData, int index)
+                archiveEntry.setCreationTime(FileTime.fromMillis(zipTimestampCreated.getTime()));
-    {
+                archiveEntry.setLastModifiedTime(FileTime.fromMillis(zipTimestampModified.getTime()));
-        // if the content stream to output is empty, then just return content descriptor as is
+                zipStream.putArchiveEntry(archiveEntry);
-        if (content == null)
+                zipStream.closeArchiveEntry();
-        {
+            }
-            return;
+            catch (IOException e)
-        }
+            {
-        
+                throw new ExporterException("Unexpected IOException adding folder entry", e);
-        try
+            }
-        {
+        }
-            if (log.isDebugEnabled())
+    }
-            {
+
-                log.debug("Archiving content for nodeRef: "+nodeRef+" with contentURL: "+contentData.getContentUrl());
+    @Override
-            }
+    public void contentImpl(NodeRef nodeRef, QName property, InputStream content, ContentData contentData, int index)
-            // ALF-2016
+    {
-            ZipArchiveEntry zipEntry=new ZipArchiveEntry(getPath());
+        // if the content stream to output is empty, then just return content descriptor as is
-            zipEntry.setTime(zipTimestampCreated.getTime());
+        if (content == null)
-            zipEntry.setCreationTime(FileTime.fromMillis(zipTimestampCreated.getTime()));
+        {
-            zipEntry.setLastModifiedTime(FileTime.fromMillis(zipTimestampModified.getTime()));
+            log.info("Archiving content has been removed or modified for the specified NodeReference: " + nodeRef
-            zipStream.putArchiveEntry(zipEntry);
+                    + ", and the size of the content is " + contentData.getSize());
-            
+            return;
-            // copy export stream to zip
+        }
-            copyStream(zipStream, content);
+
-            
+        try
-            zipStream.closeArchiveEntry();
+        {
-            filesAddedCount = filesAddedCount + 1;
+            if (log.isDebugEnabled())
-        }
+            {
-        catch (IOException e)
+                log.debug("Archiving content for nodeRef: " + nodeRef + " with contentURL: "
-        {
+                        + contentData.getContentUrl());
-            throw new ExporterException("Failed to zip export stream", e);
+            }
-        }
+            // ALF-2016
-    }
+            ZipArchiveEntry zipEntry = new ZipArchiveEntry(getPath());
-    
+            zipEntry.setTime(zipTimestampCreated.getTime());
-    @Override
+            zipEntry.setCreationTime(FileTime.fromMillis(zipTimestampCreated.getTime()));
-    public void endNode(NodeRef nodeRef)
+            zipEntry.setLastModifiedTime(FileTime.fromMillis(zipTimestampModified.getTime()));
-    {
+            zipStream.putArchiveEntry(zipEntry);
-        path.pop();
+
-    }
+            // copy export stream to zip
-
+            copyStream(zipStream, content);
-    @Override
+
-    public void end()
+            zipStream.closeArchiveEntry();
-    {
+            filesAddedCount = filesAddedCount + 1;
-        try
+        }
-        {
+        catch (IOException e)
-            zipStream.close();
+        {
-        }
+            throw new ExporterException("Failed to zip export stream", e);
-        catch (IOException error)
+        }
-        {
+    }
-            throw new ExporterException("Unexpected error closing zip stream!", error);
+
-        }
+    @Override
-    }
+    public void endNode(NodeRef nodeRef)
-
+    {
-    private String getPath()
+        path.pop();
-    {
+    }
-        if (path.size() < 1) 
+
-        {
+    @Override
-            throw new IllegalStateException("No elements in path!");    
+    public void end()
-        }
+    {
-        
+        try
-        Iterator<Pair<String, NodeRef>> iter = path.descendingIterator();
+        {
-        StringBuilder pathBuilder = new StringBuilder();
+            zipStream.close();
-        
+        }
-        while (iter.hasNext())
+        catch (IOException error)
-        {
+        {
-            Pair<String, NodeRef> element = iter.next();
+            throw new ExporterException("Unexpected error closing zip stream!", error);
-            
+        }
-            pathBuilder.append(element.getFirst());
+    }
-            if (iter.hasNext())
+
-            {
+    private String getPath()
-                pathBuilder.append(PATH_SEPARATOR);
+    {
-            }
+        if (path.size() < 1)
-        }
+        {
-        
+            throw new IllegalStateException("No elements in path!");
-        return pathBuilder.toString();
+        }
-    }
+
-
+        Iterator<Pair<String, NodeRef>> iter = path.descendingIterator();
-    /**
+        StringBuilder pathBuilder = new StringBuilder();
-     * Copy input stream to output stream
+
-     * 
+        while (iter.hasNext())
-     * @param output  output stream
+        {
-     * @param in  input stream
+            Pair<String, NodeRef> element = iter.next();
-     * @throws IOException
+
-     */
+            pathBuilder.append(element.getFirst());
-    private void copyStream(OutputStream output, InputStream in)
+            if (iter.hasNext())
-        throws IOException
+            {
-    {
+                pathBuilder.append(PATH_SEPARATOR);
-        byte[] buffer = new byte[2048 * 10];
+            }
-        int read = in.read(buffer, 0, 2048 *10);
+        }
-        int i = 0;
+
-        while (read != -1)
+        return pathBuilder.toString();
-        {
+    }
-            output.write(buffer, 0, read);
+
-            done = done + read;
+    /**
-            
+     * Copy input stream to output stream
-            // ALF-16289 - only update the status every 10MB
+     *
-            if (i++%500 == 0)
+     * @param output
-            {
+     *            output stream
-                updateStatus();
+     * @param in
-                checkCancelled();
+     *            input stream
-            }
+     * @throws IOException
-            
+     */
-            read = in.read(buffer, 0, 2048 *10);
+    private void copyStream(OutputStream output, InputStream in) throws IOException
-        }
+    {
-    }
+        byte[] buffer = new byte[2048 * 10];
-    
+        int read = in.read(buffer, 0, 2048 * 10);
-    private void checkCancelled()
+        int i = 0;
-    {
+        while (read != -1)
-        boolean downloadCancelled = transactionHelper.doInTransaction(new RetryingTransactionCallback<Boolean>()
+        {
-        {
+            output.write(buffer, 0, read);
-            @Override
+            done = done + read;
-            public Boolean execute() throws Throwable
+
-            {
+            // ALF-16289 - only update the status every 10MB
-                return downloadStorage.isCancelled(downloadNodeRef);                
+            if (i++ % 500 == 0)
-            }
+            {
-        }, true, true);
+                updateStatus();
-
+                checkCancelled();
-        if ( downloadCancelled == true)
+            }
-        {
+
-            log.debug("Download cancelled");
+            read = in.read(buffer, 0, 2048 * 10);
-            throw new DownloadCancelledException();
+        }
-        }
+    }
-    }
+
-
+    private void checkCancelled()
-    private void updateStatus()
+    {
-    {
+        boolean downloadCancelled = transactionHelper.doInTransaction(new RetryingTransactionCallback<Boolean>() {
-        transactionHelper.doInTransaction(new RetryingTransactionCallback<Object>()
+            @Override
-        {
+            public Boolean execute() throws Throwable
-            @Override
+            {
-            public Object execute() throws Throwable
+                return downloadStorage.isCancelled(downloadNodeRef);
-            {
+            }
-                DownloadStatus status = new DownloadStatus(Status.IN_PROGRESS, done, total, filesAddedCount, totalFileCount);
+        }, true, true);
-                
+
-                updateService.update(downloadNodeRef, status, getNextSequenceNumber());
+        if (downloadCancelled == true)
-                return null;
+        {
-            }
+            log.debug("Download cancelled");
-        }, false, true);
+            throw new DownloadCancelledException();
-    }
+        }
-
+    }
-    public int getNextSequenceNumber()
+
-    {
+    private void updateStatus()
-        return sequenceNumber++;
+    {
-    }
+        transactionHelper.doInTransaction(new RetryingTransactionCallback<Object>() {
-
+            @Override
-    public long getDone()
+            public Object execute() throws Throwable
-    {
+            {
-        return done;
+                DownloadStatus status = new DownloadStatus(Status.IN_PROGRESS, done, total, filesAddedCount, totalFileCount);
-    }
+
-
+                updateService.update(downloadNodeRef, status, getNextSequenceNumber());
-    public long getTotal()
+                return null;
-    {
+            }
-        return total;
+        }, false, true);
-    }
+    }
-
+
-    public long getFilesAdded()
+    public int getNextSequenceNumber()
-    {
+    {
-        return filesAddedCount;
+        return sequenceNumber++;
-    }
+    }
-
+
-    public long getTotalFiles()
+    public long getDone()
-    {
+    {
-        return totalFileCount;
+        return done;
-    }
+    }
-}
+
    public long getTotal()
    {
        return total;
    }
    public long getFilesAdded()
    {
        return filesAddedCount;
    }
    public long getTotalFiles()
    {
        return totalFileCount;
    }
 }
--- a/repository/src/main/java/org/alfresco/repo/site/SiteMembership.java
+++ b/repository/src/main/java/org/alfresco/repo/site/SiteMembership.java
@@ -2,7 +2,7 @@
 * #%L
 * Alfresco Repository
 * %%
- * Copyright (C) 2005 - 2016 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
 * %%
 * This file is part of the Alfresco software. 
 * If the software was purchased under a paid Alfresco license, the terms of 
@@ -25,14 +25,15 @@
 */
 package org.alfresco.repo.site;
 import java.util.Comparator;
 import java.util.List;
 import java.util.Optional;
 import org.alfresco.api.AlfrescoPublicApi;
 import org.alfresco.query.CannedQuerySortDetails;
 import org.alfresco.service.cmr.site.SiteInfo;
 import org.alfresco.util.Pair;
 import java.util.Comparator;
 import java.util.List;
 /**
 * Conveys information for a member of a site.
 *
@@ -52,7 +53,7 @@ public class SiteMembership extends AbstractSiteMembership
    public SiteMembership(SiteInfo siteInfo, String id, String firstName, String lastName,
            String role)
    {
-        super(siteInfo,id, role);
+        super(siteInfo, id, role);
        if (firstName == null)
        {
            throw new java.lang.IllegalArgumentException(
@@ -76,13 +77,8 @@ public class SiteMembership extends AbstractSiteMembership
            throw new java.lang.IllegalArgumentException(
                    "FirstName required building site membership of " + siteInfo.getShortName());
        }
        if (lastName == null)
        {
            throw new java.lang.IllegalArgumentException(
                    "LastName required building site membership of " + siteInfo.getShortName());
        }
        this.firstName = firstName;
-        this.lastName = lastName;
+        this.lastName = Optional.ofNullable(lastName).orElse("");
        this.isMemberOfGroup = isMemberOfGroup;
    }
@@ -164,7 +160,6 @@ public class SiteMembership extends AbstractSiteMembership
                ", isMemberOfGroup = " + isMemberOfGroup + "]";
    }
    static int compareTo(List<Pair<? extends Object, CannedQuerySortDetails.SortOrder>> sortPairs, SiteMembership o1, SiteMembership o2)
    {
        String personId1 = o1.getPersonId();
--- a/repository/src/main/java/org/alfresco/service/cmr/audit/AuditService.java
+++ b/repository/src/main/java/org/alfresco/service/cmr/audit/AuditService.java
@@ -1,266 +1,299 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2024 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.service.cmr.audit;
+package org.alfresco.service.cmr.audit;
-
+
-import java.io.Serializable;
+import java.io.Serializable;
-import java.util.HashMap;
+import java.util.HashMap;
-import java.util.List;
+import java.util.List;
-import java.util.Map;
+import java.util.Map;
-
+
-/**
+/**
- * The public API by which applications can query the audit logs and enable or disable auditing.
+ * The public API by which applications can query the audit logs and enable or disable auditing.
- * 
+ * 
- * @author Derek Hulley
+ * @author Derek Hulley
- */
+ */
-public interface AuditService
+public interface AuditService
-{
+{
-    /**
+    /**
-     * @return                  Returns <tt>true</tt> if auditing is globally enabled
+     * @return Returns <tt>true</tt> if auditing is globally enabled
-     * 
+     * 
-     * @since 3.4
+     * @since 3.4
-     */
+     */
-    boolean isAuditEnabled();
+    boolean isAuditEnabled();
-    
+
-    /**
+    /**
-     * Enable or disable the global auditing state
+     * Enable or disable the global auditing state
-     * 
+     * 
-     * @param enable            <tt>true</tt> to enable auditing globally or <tt>false</tt> to disable
+     * @param enable
-     * 
+     *            <tt>true</tt> to enable auditing globally or <tt>false</tt> to disable
-     * @since 3.4
+     * 
-     */
+     * @since 3.4
-    void setAuditEnabled(boolean enable);
+     */
-
+    void setAuditEnabled(boolean enable);
-    /**
+
-     * Helper bean to carry information about an audit application.
+    /**
-     * 
+     * Helper bean to carry information about an audit application.
-     * @author Derek Hulley
+     * 
-     * @since 3.4
+     * @author Derek Hulley
-     */
+     * @since 3.4
-    public static class AuditApplication
+     */
-    {
+    public static class AuditApplication
-        private final String name;
+    {
-        private final String key;
+        private final String name;
-        private final boolean enabled;
+        private final String key;
-        /**
+        private final boolean enabled;
-         * Constructor for final variables
+
-         */
+        /**
-        public AuditApplication(String name, String key, boolean enabled)
+         * Constructor for final variables
-        {
+         */
-            this.name = name;
+        public AuditApplication(String name, String key, boolean enabled)
-            this.key = key;
+        {
-            this.enabled = enabled;
+            this.name = name;
-        }
+            this.key = key;
-        public String getName()
+            this.enabled = enabled;
-        {
+        }
-            return name;
+
-        }
+        public String getName()
-        public String getKey()
+        {
-        {
+            return name;
-            return key;
+        }
-        }
+
-        public boolean isEnabled()
+        public String getKey()
-        {
+        {
-            return enabled;
+            return key;
-        }
+        }
-    }
+
-    
+        public boolean isEnabled()
-    /**
+        {
-     * Get all registered audit applications
+            return enabled;
-     * 
+        }
-     * @return                  Returns a map of audit applications keyed by their name
+    }
-     * 
+
-     * @since 3.4
+    /**
-     */
+     * Get all registered audit applications
-    Map<String, AuditApplication> getAuditApplications();
+     * 
-    
+     * @return Returns a map of audit applications keyed by their name
-    /**
+     * 
-     * @param applicationName   the name of the application to check 
+     * @since 3.4
-     * @param path              the path to check
+     */
-     * @return                  Returns <tt>true</tt> if auditing is enabled for the given path
+    Map<String, AuditApplication> getAuditApplications();
-     * 
+
-     * @since 3.2
+    /**
-     */
+     * @param applicationName
-    boolean isAuditEnabled(String applicationName, String path);
+     *            the name of the application to check
-    
+     * @param path
-    /**
+     *            the path to check
-     * Enable auditing for an application path
+     * @return Returns <tt>true</tt> if auditing is enabled for the given path
-     * 
+     * 
-     * @param applicationName   the name of the application to check 
+     * @since 3.2
-     * @param path              the path to enable
+     */
-     * 
+    boolean isAuditEnabled(String applicationName, String path);
-     * @since 3.2
+
-     */
+    /**
-    void enableAudit(String applicationName, String path);
+     * Enable auditing for an application path
-    
+     * 
-    /**
+     * @param applicationName
-     * Disable auditing for an application path
+     *            the name of the application to check
-     * 
+     * @param path
-     * @param applicationName   the name of the application to check 
+     *            the path to enable
-     * @param path              the path to disable
+     * 
-     * 
+     * @since 3.2
-     * @since 3.2
+     */
-     */
+    void enableAudit(String applicationName, String path);
-    void disableAudit(String applicationName, String path);
+
-    
+    /**
-    /**
+     * Disable auditing for an application path
-     * Remove all audit entries for the given application
+     * 
-     * 
+     * @param applicationName
-     * @param applicationName   the name of the application for which to remove entries
+     *            the name of the application to check
-     * @return                  Returns the number of audit entries deleted
+     * @param path
-     * 
+     *            the path to disable
-     * @since 3.2
+     * 
-     * 
+     * @since 3.2
-     * @deprecated          Use {@link #clearAudit(String, Long, Long)}
+     */
-     */
+    void disableAudit(String applicationName, String path);
-    int clearAudit(String applicationName);
+
-    
+    /**
-    /**
+     * Remove all audit entries for the given application
-     * Remove audit entries for the given application between the time ranges.  If no start
+     * 
-     * time is given then entries are deleted as far back as they exist.  If no end time is
+     * @param applicationName
-     * given then entries are deleted up until the current time.
+     *            the name of the application for which to remove entries
-     * 
+     * @return Returns the number of audit entries deleted
-     * @param applicationName   the name of the application for which to remove entries
+     * 
-     * @param fromTime          the start time of entries to remove (inclusive and optional)
+     * @since 3.2
-     * @param toTime            the end time of entries to remove (exclusive and optional)
+     * 
-     * @return                  Returns the number of audit entries deleted
+     * @deprecated Use {@link #clearAudit(String, Long, Long)}
-     * 
+     */
-     * @since 3.4
+    int clearAudit(String applicationName);
-     */
+
-    int clearAudit(String applicationName, Long fromTime, Long toTime);
+    /**
-
+     * Remove audit entries for the given application between the time ranges. If no start time is given then entries are deleted as far back as they exist. If no end time is given then entries are deleted up until the current time.
-    /**
+     * 
-     * Remove audit entries for the given application between the audit entry ids.
+     * @param applicationName
-     *
+     *            the name of the application for which to remove entries
-     * @param applicationName   the name of the application for which to remove entries
+     * @param fromTime
-     * @param fromId            the start time of entries to remove (inclusive and optional)
+     *            the start time of entries to remove (inclusive and optional)
-     * @param toId              the end time of entries to remove (exclusive and optional)
+     * @param toTime
-     * @return                  Returns the number of audit entries deleted
+     *            the end time of entries to remove (exclusive and optional)
-     *
+     * @return Returns the number of audit entries deleted
-     * @since 5.2.2
+     * 
-     */
+     * @since 3.4
-    int clearAuditByIdRange(String applicationName, Long fromId, Long toId);
+     */
-
+    int clearAudit(String applicationName, Long fromTime, Long toTime);
-    /**
+
-     * Delete a discrete list of audit entries.
+    /**
-     * <p/>
+     * Remove audit entries for the given application between the audit entry ids.
-     * This method should not be called <i>while</i> processing
+     *
-     * {@link #auditQuery(AuditQueryCallback, AuditQueryParameters, int) query results}.
+     * @param applicationName
-     * 
+     *            the name of the application for which to remove entries
-     * @param auditEntryIds     the IDs of all audit entries to delete
+     * @param fromId
-     * @return                  Returns the number of audit entries deleted
+     *            the start time of entries to remove (inclusive and optional)
-     * 
+     * @param toId
-     * @since 3.4
+     *            the end time of entries to remove (exclusive and optional)
-     */
+     * @return Returns the number of audit entries deleted
-    int clearAudit(List<Long> auditEntryIds);
+     *
-    
+     * @since 5.2.2
-    /**
+     */
-     * The interface that will be used to give query results to the calling code.
+    int clearAuditByIdRange(String applicationName, Long fromId, Long toId);
-     * 
+
-     * @since 3.2
+    /**
-     */
+     * Delete a discrete list of audit entries.
-    public static interface AuditQueryCallback
+     * <p/>
-    {
+     * This method should not be called <i>while</i> processing {@link #auditQuery(AuditQueryCallback, AuditQueryParameters, int) query results}.
-        /**
+     * 
-         * Determines whether this callback requires the values argument to be populated when {@link #handleAuditEntry}
+     * @param auditEntryIds
-         * is called.
+     *            the IDs of all audit entries to delete
-         * 
+     * @return Returns the number of audit entries deleted
-         * @return <code>true</code> if this callback requires the values argument to be populated
+     * 
-         */
+     * @since 3.4
-        boolean valuesRequired();
+     */
-        
+    int clearAudit(List<Long> auditEntryIds);
-        /**
+
-         * Handle a row of audit entry data.
+    /**
-         * 
+     * The interface that will be used to give query results to the calling code.
-         * @param entryId                   the unique audit entry ID
+     * 
-         * @param applicationName           the name of the application
+     * @since 3.2
-         * @param user                      the user that logged the entry
+     */
-         * @param time                      the time of the entry
+    public static interface AuditQueryCallback
-         * @param values                    the values map as created
+    {
-         * @return                          Return <tt>true</tt> to continue processing rows or <tt>false</tt> to stop
+        /**
-         */
+         * Determines whether this callback requires the values argument to be populated when {@link #handleAuditEntry} is called.
-        boolean handleAuditEntry(
+         * 
-                Long entryId,
+         * @return <code>true</code> if this callback requires the values argument to be populated
-                String applicationName,
+         */
-                String user,
+        boolean valuesRequired();
-                long time,
+
-                Map<String, Serializable> values);
+        /**
-        
+         * Handle a row of audit entry data.
-        /**
+         * 
-         * Handle audit entry failures
+         * @param entryId
-         * 
+         *            the unique audit entry ID
-         * @param entryId                   the entry ID
+         * @param applicationName
-         * @param errorMsg                  the error message
+         *            the name of the application
-         * @param error                     the exception causing the error (may be <tt>null</tt>)
+         * @param user
-         * @return                          Return <tt>true</tt> to continue processing rows or <tt>false</tt> to stop
+         *            the user that logged the entry
-         */
+         * @param time
-        boolean handleAuditEntryError(Long entryId, String errorMsg, Throwable error);
+         *            the time of the entry
-    }
+         * @param values
-    
+         *            the values map as created
-    /**
+         * @return Return <tt>true</tt> to continue processing rows or <tt>false</tt> to stop
-     * Issue an audit query using the given parameters and consuming results in the callback.
+         */
-     * Results are returned in entry order, corresponding to time order.
+        boolean handleAuditEntry(
-     * 
+                Long entryId,
-     * @param callback          the callback that will handle results
+                String applicationName,
-     * @param parameters        the parameters for the query (may not be <tt>null</tt>)
+                String user,
-     * @param maxResults        the maximum number of results to retrieve (must be greater than 0)
+                long time,
-     * 
+                Map<String, Serializable> values);
-     * @throws IllegalArgumentException if maxResults less or equal to zero
+
-     * 
+        /**
-     * @since 3.3
+         * Handle audit entry failures
-     */
+         * 
-    void auditQuery(AuditQueryCallback callback, AuditQueryParameters parameters, int maxResults);
+         * @param entryId
-
+         *            the entry ID
-    /**
+         * @param errorMsg
-     * Issue an audit query to retrieve min / max audit record id for a given application.
+         *            the error message
-     *
+         * @param error
-     * @param applicationName               the name of the application
+         *            the exception causing the error (may be <tt>null</tt>)
-     * @param extremes                      a list containing min/max or both
+         * @return Return <tt>true</tt> to continue processing rows or <tt>false</tt> to stop
-     * @return                              a map containing min/max and the associated value
+         */
-     */
+        boolean handleAuditEntryError(Long entryId, String errorMsg, Throwable error);
-    HashMap<String, Long> getAuditMinMaxByApp(String applicationName, List<String> extremes);
+    }
-
+
-    /**
+    /**
-     * Issue an audit query to retrieve min / max audit record id for a given application.
+     * Issue an audit query using the given parameters and consuming results in the callback. Results are returned in entry order, corresponding to time order.
-     *
+     * 
-     * @param applicationName               the name of the application
+     * @param callback
-     * @return                              a map containing min/max and the associated value
+     *            the callback that will handle results
-     */
+     * @param parameters
-    default int getAuditEntriesCountByApp(String applicationName)
+     *            the parameters for the query (may not be <tt>null</tt>)
-    {
+     * @param maxResults
-        return -1;
+     *            the maximum number of results to retrieve (must be greater than 0)
-    }
+     * 
-
+     * @throws IllegalArgumentException
-    /**
+     *             if maxResults less or equal to zero
-     * Issue an audit query to retrieve min / max audit record id for a given application and properties
+     * 
-     *
+     * @since 3.3
-     * @param parameters                    audit parameters provided by the <code>where</code> clause on the ReST API
+     */
-     * @return                              a map containing min/max and the associated value
+    void auditQuery(AuditQueryCallback callback, AuditQueryParameters parameters, int maxResults);
-     */
+
-    default int getAuditEntriesCountByAppAndProperties(AuditQueryParameters parameters)
+    /**
-    {
+     * Issue an audit query to retrieve min / max audit record id for a given application.
-        return -1;
+     *
-    }
+     * @param applicationName
-}
+     *            the name of the application
     * @param extremes
     *            a list containing min/max or both
     * @return a map containing min/max and the associated value
     */
    HashMap<String, Long> getAuditMinMaxByApp(String applicationName, List<String> extremes);
    /**
     * Issue an audit query to retrieve min / max audit record id for a given application.
     *
     * @param applicationName
     *            the name of the application
     * @return a map containing min/max and the associated value
     */
    default int getAuditEntriesCountByApp(String applicationName)
    {
        return -1;
    }
    /**
     * Issue an audit query to retrieve min / max audit record id for a given application and properties
     *
     * @param applicationName
     *            the name of the application
     * @param parameters
     *            audit parameters provided by the <code>where</code> clause on the ReST API
     * @return a map containing min/max and the associated value
     */
    default int getAuditEntriesCountByAppAndProperties(String applicationName, AuditQueryParameters parameters)
    {
        return -1;
    }
 }
--- a/repository/src/main/resources/alfresco/repository.properties
+++ b/repository/src/main/resources/alfresco/repository.properties
@@ -3,7 +3,7 @@
 repository.name=Main Repository
 # Schema number
-version.schema=19300
+version.schema=20000
 # Directory configuration
--- a/repository/src/test/java/org/alfresco/repo/site/SiteMembershipTest.java
+++ b/repository/src/test/java/org/alfresco/repo/site/SiteMembershipTest.java
@@ -1,136 +1,137 @@
-/*
+/*
- * #%L
+ * #%L
- * Alfresco Repository
+ * Alfresco Repository
- * %%
+ * %%
- * Copyright (C) 2005 - 2016 Alfresco Software Limited
+ * Copyright (C) 2005 - 2024 Alfresco Software Limited
- * %%
+ * %%
- * This file is part of the Alfresco software. 
+ * This file is part of the Alfresco software. 
- * If the software was purchased under a paid Alfresco license, the terms of 
+ * If the software was purchased under a paid Alfresco license, the terms of 
- * the paid license agreement will prevail.  Otherwise, the software is 
+ * the paid license agreement will prevail.  Otherwise, the software is 
- * provided under the following open source license terms:
+ * provided under the following open source license terms:
- * 
+ * 
- * Alfresco is free software: you can redistribute it and/or modify
+ * Alfresco is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
+ * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * (at your option) any later version.
- * 
+ * 
- * Alfresco is distributed in the hope that it will be useful,
+ * Alfresco is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
+ * GNU Lesser General Public License for more details.
- * 
+ * 
- * You should have received a copy of the GNU Lesser General Public License
+ * You should have received a copy of the GNU Lesser General Public License
- * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
- * #L%
+ * #L%
- */
+ */
-package org.alfresco.repo.site;
+package org.alfresco.repo.site;
-
+
-import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertEquals;
-
+
-import java.util.UUID;
+import java.util.UUID;
-
+
-import org.alfresco.service.cmr.site.SiteInfo;
+import org.junit.Before;
-import org.alfresco.service.cmr.site.SiteVisibility;
+import org.junit.Test;
-import org.junit.Before;
+
-import org.junit.Test;
+import org.alfresco.service.cmr.site.SiteInfo;
-
+import org.alfresco.service.cmr.site.SiteVisibility;
-/**
+
- * Test Membership constructor logs. Based on REPO-2520
+/**
- * 
+ * Test Membership constructor logs. Based on REPO-2520
- * @author Alexandru Epure
+ * 
- */
+ * @author Alexandru Epure
-public class SiteMembershipTest
+ */
-{
+public class SiteMembershipTest
-    SiteInfo siteInfo;
+{
-    SiteMembership siteMember;
+    SiteInfo siteInfo;
-    String personId = UUID.randomUUID().toString();
+    SiteMembership siteMember;
-    String firstName = UUID.randomUUID().toString();
+    String personId = UUID.randomUUID().toString();
-    String lastName = UUID.randomUUID().toString();
+    String firstName = UUID.randomUUID().toString();
-    String role = "Consumer";
+    String lastName = UUID.randomUUID().toString();
-
+    String role = "Consumer";
-    String idErrorMessage = "Id required building site membership";
+
-    String firstNameErrorMessage = "FirstName required building site membership of ";
+    String idErrorMessage = "Id required building site membership";
-    String lastNameErrorMessage = "LastName required building site membership of ";
+    String firstNameErrorMessage = "FirstName required building site membership of ";
-    String roleErrorMessage = "Role required building site membership";
+    String lastNameErrorMessage = "LastName required building site membership of ";
-
+    String roleErrorMessage = "Role required building site membership";
-    @Before
+
-    public void createSite()
+    @Before
-    {
+    public void createSite()
-        String sitePreset = "testSiteMembershipPreset";
+    {
-        String shortName = "testSiteMembershipShortName";
+        var sitePreset = "testSiteMembershipPreset";
-        String title = "testSiteMembershipTile";
+        var shortName = "testSiteMembershipShortName";
-        String description = "testSiteMembershipDescription";
+        var title = "testSiteMembershipTile";
-        siteInfo = new SiteInfoImpl(sitePreset, shortName, title, description,
+        var description = "testSiteMembershipDescription";
-                SiteVisibility.PUBLIC, null);
+        siteInfo = new SiteInfoImpl(sitePreset, shortName, title, description,
-    }
+                SiteVisibility.PUBLIC, null);
-
+    }
-    @Test
+
-    public void testNullpersonInfo() throws Exception
+    @Test
-    {
+    public void testNullpersonInfo() throws Exception
-        try
+    {
-        {
+        try
-            siteMember = new SiteMembership(siteInfo, null, firstName, lastName, role);
+        {
-        }
+            siteMember = new SiteMembership(siteInfo, null, firstName, lastName, role);
-        catch (IllegalArgumentException e)
+        }
-        {
+        catch (IllegalArgumentException e)
-            assertEquals(idErrorMessage, e.getMessage());
+        {
-        }
+            assertEquals(idErrorMessage, e.getMessage());
-
+        }
-        try
+
-        {
+        try
-            siteMember = new SiteMembership(siteInfo, null, role);
+        {
-        }
+            siteMember = new SiteMembership(siteInfo, null, role);
-        catch (IllegalArgumentException e)
+        }
-        {
+        catch (IllegalArgumentException e)
-            assertEquals(idErrorMessage, e.getMessage());
+        {
-        }
+            assertEquals(idErrorMessage, e.getMessage());
-    }
+        }
-
+    }
-    @Test
+
-    public void testNullRole() throws Exception
+    @Test
-    {
+    public void testNullRole() throws Exception
-        try
+    {
-        {
+        try
-            siteMember = new SiteMembership(siteInfo, personId, firstName, lastName, null);
+        {
-        }
+            siteMember = new SiteMembership(siteInfo, personId, firstName, lastName, null);
-        catch (IllegalArgumentException e)
+        }
-        {
+        catch (IllegalArgumentException e)
-            assertEquals(roleErrorMessage, e.getMessage());
+        {
-        }
+            assertEquals(roleErrorMessage, e.getMessage());
-
+        }
-        try
+
-        {
+        try
-            siteMember = new SiteMembership(siteInfo, personId, null);
+        {
-        }
+            siteMember = new SiteMembership(siteInfo, personId, null);
-        catch (IllegalArgumentException e)
+        }
-        {
+        catch (IllegalArgumentException e)
-            assertEquals(roleErrorMessage, e.getMessage());
+        {
-        }
+            assertEquals(roleErrorMessage, e.getMessage());
-    }
+        }
-
+    }
-    @Test
+
-    public void testNullFirstName() throws Exception
+    @Test
-    {
+    public void testNullFirstName() throws Exception
-        try
+    {
-        {
+        try
-            siteMember = new SiteMembership(siteInfo, personId, null, lastName, role);
+        {
-        }
+            siteMember = new SiteMembership(siteInfo, personId, null, lastName, role);
-        catch (IllegalArgumentException e)
+        }
-        {
+        catch (IllegalArgumentException e)
-            assertEquals(firstNameErrorMessage + siteInfo.getShortName(), e.getMessage());
+        {
-        }
+            assertEquals(firstNameErrorMessage + siteInfo.getShortName(), e.getMessage());
-    }
+        }
-
+    }
-    @Test
+
-    public void testNullLastName() throws Exception
+    @Test
-    {
+    public void testNullLastName() throws Exception
-        try
+    {
-        {
+        try
-            siteMember = new SiteMembership(siteInfo, personId, firstName, null, role);
+        {
-        }
+            siteMember = new SiteMembership(siteInfo, personId, firstName, null, role);
-        catch (IllegalArgumentException e)
+        }
-        {
+        catch (IllegalArgumentException e)
-            assertEquals(lastNameErrorMessage + siteInfo.getShortName(), e.getMessage());
+        {
-        }
+            assertEquals(lastNameErrorMessage + siteInfo.getShortName(), e.getMessage());
-    }
+        }
-}
+    }
 }
Author	SHA1	Message	Date
alfresco-build	69bb9a308a	[maven-release-plugin][skip ci] prepare release 25.1.0.10	2024-12-06 11:52:47 +00:00
SathishK-T	853d1bdc1b	[APPS-3130] reverting changes of MNT-24137 (#3076 ) * [APPS-3130] reverting changes of MNT-24137 * reverting * reverting the change of mnt-24137 * reverting the change of mnt-24137 * reverting the change of mnt-24137 * reverting the changes of mnt-24137 * reverting the changes of mnt-24137 * reverting the changes of mnt-24137 * [APPS-3130] reverting changes of MNT-24137 --------- Co-authored-by: Sathish Kumar <ST28@ford.com>	2024-12-06 16:40:38 +05:30
alfresco-build	f7d1bbcec3	[maven-release-plugin][skip ci] prepare for next development iteration	2024-12-05 15:46:52 +00:00
alfresco-build	731d580f7e	[maven-release-plugin][skip ci] prepare release 25.1.0.9	2024-12-05 15:46:49 +00:00
dependabot[bot]	ad4126ffea	Bump org.mybatis:mybatis from 3.5.16 to 3.5.17 (#3062 ) Bumps [org.mybatis:mybatis](https://github.com/mybatis/mybatis-3) from 3.5.16 to 3.5.17. - [Release notes](https://github.com/mybatis/mybatis-3/releases) - [Commits](https://github.com/mybatis/mybatis-3/compare/mybatis-3.5.16...mybatis-3.5.17) --- updated-dependencies: - dependency-name: org.mybatis:mybatis dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2024-12-05 09:33:04 +01:00
dependabot[bot]	933aced67c	Bump dependency.log4j.version from 2.23.1 to 2.24.2 (#3064 ) Bumps `dependency.log4j.version` from 2.23.1 to 2.24.2. Updates `org.apache.logging.log4j:log4j-slf4j2-impl` from 2.23.1 to 2.24.2 Updates `org.apache.logging.log4j:log4j-api` from 2.23.1 to 2.24.2 Updates `org.apache.logging.log4j:log4j-core` from 2.23.1 to 2.24.2 --- updated-dependencies: - dependency-name: org.apache.logging.log4j:log4j-slf4j2-impl dependency-type: direct:development update-type: version-update:semver-minor - dependency-name: org.apache.logging.log4j:log4j-api dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.apache.logging.log4j:log4j-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2024-12-05 09:31:49 +01:00
mohit-singh4	de0ba15a91	[Fix/MNT-24209] logging for download restapi (#3065 ) * [fix/MNT-24209] Adding loggers in ACS download restApi * [fix/MNT-24209] Adding loggers in ACS download restApi * [fix/MNT-24209] Adding loggers in ACS download restApi * Update ZipDownloadExporter.java * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI * Revert "Update ZipDownloadExporter.java" This reverts commit `2d97b34959`. * resolved Pre commit error * resolved Pre commit error * Bump io.swagger:swagger-parser from 1.0.71 to 1.0.72 (#3044) Bumps [io.swagger:swagger-parser](https://github.com/swagger-api/swagger-parser) from 1.0.71 to 1.0.72. - [Release notes](https://github.com/swagger-api/swagger-parser/releases) - [Commits](https://github.com/swagger-api/swagger-parser/compare/v1.0.71...v1.0.72) --- updated-dependencies: - dependency-name: io.swagger:swagger-parser dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [maven-release-plugin][skip ci] prepare release 25.1.0.4 * [maven-release-plugin][skip ci] prepare for next development iteration * Update the SiteMembership Class * Update the SiteMembership Class * Update the SiteMembership Class * Revert "Update the SiteMembership Class" This reverts commit `15045f9612`. * Update the SiteMembership Class * Update the SiteMembership Class * Update the license i SiteMembership * Revert "Update the license i SiteMembership" This reverts commit `ee8f3bec18`. * Update the license SiteMembership * Fix Precommit * Fix Precommit * Fix Precommit * Update the SiteMembership * [maven-release-plugin][skip ci] prepare release 25.1.0.5 * [maven-release-plugin][skip ci] prepare for next development iteration * Revert "Bump dependency.log4j.version from 2.23.1 to 2.24.2 (#3051)" This reverts commit `748be1f4a0`. * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI * resolved Pre commit error * resolved Pre commit error * reformatting ZipDownloadExporter file * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI * [fix/MNT-24209-logging-for-download-restapi] Adding loggers in downloads RestAPI --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: mohit-singh4 <mohit.singh@contractors.hyland.com> Co-authored-by: MohinishSah <88024811+MohinishSah@users.noreply.github.com> Co-authored-by: Mohinish Sah <Mohinish.Sah@contractors.onbase.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: alfresco-build <8039454+alfresco-build@users.noreply.github.com> Co-authored-by: varapathijanakiram <172787912+varapathijanakiram@users.noreply.github.com> Co-authored-by: vjanakiram <varapathi.janakiram@hyland.com> Co-authored-by: rrajoria <88024787+rrajoria@users.noreply.github.com> Co-authored-by: Cezary Witkowski <cezary.witkowski@hyland.com>	2024-12-04 14:24:43 +05:30
alfresco-build	00c36251d4	[maven-release-plugin][skip ci] prepare for next development iteration	2024-12-02 15:05:20 +00:00
alfresco-build	ea9ce0c0c2	[maven-release-plugin][skip ci] prepare release 25.1.0.8	2024-12-02 15:05:18 +00:00
Eva Vasques	37231e50a2	MNT-24753 Restore from Archive when we have multiple content stores (#3066 ) * Pre-commit changes * Method requestRestoreContentFromArchive needs to use injected ContentStore	2024-12-02 14:22:51 +00:00
alfresco-build	26d8c7dcbe	[maven-release-plugin][skip ci] prepare for next development iteration	2024-12-01 00:09:27 +00:00
alfresco-build	6ea97033d3	[maven-release-plugin][skip ci] prepare release 25.1.0.7	2024-12-01 00:09:23 +00:00
Alfresco CI User	cd0b2ffd24	[force] Force release for 2024-12-01.	2024-12-01 00:05:05 +00:00
alfresco-build	a14cbacb68	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-29 07:34:35 +00:00
alfresco-build	fc0b9863d4	[maven-release-plugin][skip ci] prepare release 25.1.0.6	2024-11-29 07:34:33 +00:00
Cezary Witkowski	29dc42d445	Merge pull request #3063 from Alfresco/revert-3051-dependabot/maven/dependency.log4j.version-2.24.2 Revert "Bump dependency.log4j.version from 2.23.1 to 2.24.2"	2024-11-28 15:26:55 +01:00
Cezary Witkowski	39da968d76	Revert "Bump dependency.log4j.version from 2.23.1 to 2.24.2 (#3051 )" This reverts commit `748be1f4a0`.	2024-11-28 14:36:47 +01:00
alfresco-build	6a4fecd46e	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-28 11:36:08 +00:00
alfresco-build	e083d4fcc2	[maven-release-plugin][skip ci] prepare release 25.1.0.5	2024-11-28 11:36:06 +00:00
varapathijanakiram	53e8a06a6e	Merge pull request #3058 from Alfresco/fix/MNT-24571-Site-Membership-API Fix/MNT-24571-Site Membership API	2024-11-28 16:23:20 +05:30
alfresco-build	c80e1fa94b	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-28 09:56:23 +00:00
alfresco-build	675c6f7aa1	[maven-release-plugin][skip ci] prepare release 25.1.0.4	2024-11-28 09:56:21 +00:00
dependabot[bot]	44d67847da	Bump io.swagger:swagger-parser from 1.0.71 to 1.0.72 (#3044 ) Bumps [io.swagger:swagger-parser](https://github.com/swagger-api/swagger-parser) from 1.0.71 to 1.0.72. - [Release notes](https://github.com/swagger-api/swagger-parser/releases) - [Commits](https://github.com/swagger-api/swagger-parser/compare/v1.0.71...v1.0.72) --- updated-dependencies: - dependency-name: io.swagger:swagger-parser dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2024-11-27 18:51:51 +01:00
vjanakiram	d405043c5d	Update the SiteMembership	2024-11-27 12:38:50 +05:30
rrajoria	da50f6271c	Fix Precommit	2024-11-27 11:20:04 +05:30
rrajoria	a5e713c176	Fix Precommit	2024-11-27 11:11:41 +05:30
rrajoria	2dcd2788b2	Fix Precommit	2024-11-27 11:02:51 +05:30
vjanakiram	6b738342c1	Update the license SiteMembership	2024-11-27 10:59:13 +05:30
vjanakiram	ba5f5418ab	Revert "Update the license i SiteMembership" This reverts commit `ee8f3bec18`.	2024-11-27 10:58:40 +05:30
vjanakiram	ee8f3bec18	Update the license i SiteMembership	2024-11-27 10:47:56 +05:30
vjanakiram	39395af528	Update the SiteMembership Class	2024-11-26 17:49:12 +05:30
vjanakiram	e6f9d88779	Update the SiteMembership Class	2024-11-26 17:27:33 +05:30
vjanakiram	70f70c6ed3	Revert "Update the SiteMembership Class" This reverts commit `15045f9612`.	2024-11-26 17:26:44 +05:30
vjanakiram	15045f9612	Update the SiteMembership Class	2024-11-26 17:25:23 +05:30
dependabot[bot]	c50c7c6a7e	Bump org.apache.activemq:activemq-client-jakarta from 5.18.3 to 5.18.6 (#2962 )	2024-11-26 11:19:23 +00:00
dependabot[bot]	60199141f5	Bump org.apache.httpcomponents.client5:httpclient5 from 5.2.1 to 5.4.1 (#3015 )	2024-11-26 11:19:14 +00:00
vjanakiram	b5e023950d	Update the SiteMembership Class	2024-11-26 16:48:11 +05:30
varapathijanakiram	d4591acf2c	Update the SiteMembership Class	2024-11-26 16:31:47 +05:30
dependabot[bot]	f34f2849dd	Bump org.eclipse.jetty:jetty-server in /remote-api (#2995 )	2024-11-26 09:37:26 +00:00
dependabot[bot]	52f2be49f3	Bump com.networknt:json-schema-validator from 1.5.3 to 1.5.4 (#3054 )	2024-11-26 09:15:25 +00:00
dependabot[bot]	748be1f4a0	Bump dependency.log4j.version from 2.23.1 to 2.24.2 (#3051 )	2024-11-26 09:14:44 +00:00
alfresco-build	52f1af2da6	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-25 10:35:23 +00:00
alfresco-build	626a97ff45	[maven-release-plugin][skip ci] prepare release 25.1.0.3	2024-11-25 10:35:21 +00:00
Damian Ujma	4a91132226	ACS-5371 Exclude xpp3 dependency to avoid conflict in JDK9+ (#3049 ) * ACS-5371 Exclude xpp3 dependency * ACS-5371 Set compiler compliance to 17 * ACS-5371 Update xmlpull version * ACS-5371 Update xmlpull version * ACS-5371 Use kxml2 * ACS-5371 Refactor	2024-11-25 09:49:47 +01:00
Piotr Żurek	b6fb8725ce	Revert "Bump org.springframework.security:spring-security-bom (#3048 )" This reverts commit `aacef0c949`.	2024-11-25 09:44:21 +01:00
alfresco-build	5fc0d9edde	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-24 00:10:05 +00:00
alfresco-build	44d394b1a0	[maven-release-plugin][skip ci] prepare release 25.1.0.2	2024-11-24 00:10:03 +00:00
Alfresco CI User	c1d62cca4d	[force] Force release for 2024-11-24.	2024-11-24 00:05:36 +00:00
Piotr Żurek	24ae4ce57c	Revert "Bump dependency.spring.version from 6.1.14 to 6.2.0 (#3038 )" This reverts commit `b8f2ec252d`.	2024-11-22 15:07:03 +01:00
dependabot[bot]	b8f2ec252d	Bump dependency.spring.version from 6.1.14 to 6.2.0 (#3038 ) Bumps `dependency.spring.version` from 6.1.14 to 6.2.0. Updates `org.springframework:spring-aop` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-beans` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-context` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-context-support` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-core` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-expression` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-jdbc` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-jms` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-test` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-orm` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-tx` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-web` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) Updates `org.springframework:spring-webmvc` from 6.1.14 to 6.2.0 - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](https://github.com/spring-projects/spring-framework/compare/v6.1.14...v6.2.0) --- updated-dependencies: - dependency-name: org.springframework:spring-aop dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-beans dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-context dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-context-support dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-core dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-expression dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-jdbc dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-jms dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-test dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-orm dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-tx dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-web dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.springframework:spring-webmvc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2024-11-22 14:22:19 +01:00
dependabot[bot]	ae721c86f6	Bump commons-io:commons-io from 2.17.0 to 2.18.0 (#3046 ) Bumps commons-io:commons-io from 2.17.0 to 2.18.0. --- updated-dependencies: - dependency-name: commons-io:commons-io dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2024-11-22 14:19:59 +01:00
dependabot[bot]	aacef0c949	Bump org.springframework.security:spring-security-bom (#3048 ) Bumps [org.springframework.security:spring-security-bom](https://github.com/spring-projects/spring-security) from 6.3.4 to 6.4.1. - [Release notes](https://github.com/spring-projects/spring-security/releases) - [Changelog](https://github.com/spring-projects/spring-security/blob/main/RELEASE.adoc) - [Commits](https://github.com/spring-projects/spring-security/compare/6.3.4...6.4.1) --- updated-dependencies: - dependency-name: org.springframework.security:spring-security-bom dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2024-11-22 14:19:15 +01:00
dependabot[bot]	e4edceda75	Bump org.apache.maven.plugins:maven-failsafe-plugin from 3.5.1 to 3.5.2 (#3019 )	2024-11-22 12:41:46 +00:00
dependabot[bot]	09ec6688c6	Bump org.quartz-scheduler:quartz from 2.3.2 to 2.5.0 (#3035 )	2024-11-22 12:41:36 +00:00
dependabot[bot]	14025f00de	Bump org.projectlombok:lombok from 1.18.34 to 1.18.36 (#3042 )	2024-11-22 12:41:16 +00:00
dependabot[bot]	22c5a5e134	Bump dependency.bouncycastle.version from 1.78.1 to 1.79 (#3016 )	2024-11-22 12:39:42 +00:00
dependabot[bot]	154cad1ff8	Bump org.apache.maven.plugins:maven-javadoc-plugin from 3.10.1 to 3.11.1 (#3021 )	2024-11-22 12:39:40 +00:00
dependabot[bot]	7749b89e2b	Bump org.apache.santuario:xmlsec from 4.0.2 to 4.0.3 (#3023 )	2024-11-22 12:39:35 +00:00
dependabot[bot]	bbb45e667b	Bump org.codehaus.groovy:groovy from 3.0.22 to 3.0.23 (#3030 )	2024-11-22 12:39:04 +00:00
dependabot[bot]	a5f01e0d94	Bump Alfresco/alfresco-build-tools from 7.1.0 to 8.2.0 (#3018 )	2024-11-22 12:39:02 +00:00
dependabot[bot]	0c6d455ec4	Bump com.networknt:json-schema-validator from 1.5.1 to 1.5.3 (#3017 )	2024-11-22 12:38:58 +00:00
dependabot[bot]	5e86f5163e	Bump org.apache.xmlbeans:xmlbeans from 5.2.1 to 5.2.2 (#3024 )	2024-11-22 12:38:23 +00:00
alfresco-build	4981be1564	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-20 15:49:36 +00:00
alfresco-build	9a30fbb75a	[maven-release-plugin][skip ci] prepare release 25.1.0.1	2024-11-20 15:49:34 +00:00
Piotr Żurek	9c86fdb8b5	Trigger CI	2024-11-20 15:42:59 +01:00
Kacper Magdziarz	c52f74832d	Updating master branch to 25.1.0 after 23.4.0 ACS release [skip ci]	2024-11-20 13:36:19 +01:00
alfresco-build	70dd7642f4	[maven-release-plugin][skip ci] prepare for next development iteration	2024-11-19 09:48:20 +00:00