Skip to content

fix: apply files_fulltextsearch dotted-UID workaround on app startup - #87

Merged
vitormattos merged 1 commit into
mainfrom
fix/files-fulltextsearch-dotted-uid-hook
Oct 23, 2026
Merged

vitormattos merged 1 commit into
mainfrom
fix/files-fulltextsearch-dotted-uid-hook

Conversation

@vitormattos

Copy link
Copy Markdown
Member

Problem

The files_fulltextsearch 35.0.0 search-result postprocessor regex-escapes UIDs containing dots before resolving Nextcloud files. The PostgreSQL index can find the document, but the app drops the result. See upstream issue #389 and PR #390.

Change

  • Keep the upstream-compatible one-line fix as a versioned .patch file.
  • Apply it from the existing before-starting hook for affected 35.x installations. The hook skips missing apps or already-fixed code, is idempotent, checks patch applicability without fuzz, and runs PHP syntax validation.
  • Document that in-place App Store updates require restarting the application container to reapply the hook. Remove the workaround when the upstream 35.x release includes the fix.

post-installation would not work for an already-installed Nextcloud, and app source lives in a persistent volume, not the Docker image.

Verification

  • Synthetic pilot.dotted user in isolated Nextcloud 35.0.2 / files_fulltextsearch 35.0.0 laboratory: search returned 0 results before, 1 correct file after.
  • Re-running hook produces no changes (same SHA-256).
  • php -l, sh -n, and git diff --check passed.

No customer document content or credentials are included.

@vitormattos
vitormattos force-pushed the fix/files-fulltextsearch-dotted-uid-hook branch from 0dbd2c3 to 39411df Compare October 23, 2026 23:47
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos force-pushed the fix/files-fulltextsearch-dotted-uid-hook branch from 39411df to 2e0c4c4 Compare October 23, 2026 23:48
@vitormattos
vitormattos merged commit 060aff2 into main Oct 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant