fix: encode hash character in app route URLs to prevent truncation (#3075)

* fix: encode hash character in file paths for app URLs

Files and folders containing hash characters (#) in their names were causing
issues when opened in apps (text-editor, preview) because the browser treats
# as a URL fragment identifier, truncating the URL.

Changes:
- Import and use encodePath in useFileActions to encode path segments
- Encode hash characters segment-wise in getEditorRouteOpts()
- Build URLs manually in useFileActionsOpenWithApp and SaveAsModal to avoid
  double encoding from router.resolve()
- Add unit tests to verify correct encoding behavior

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* format

* react to code review

* fix: opening document from inside collabora

* cleanup

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Jannik Stehle <j.stehle@opencloud.eu>

Author: 3 people

packages/web-pkg/src/composables/actions/files/useFileActions.ts

@@ -184,8 +184,8 @@ export const useFileActions = () => {
     if (!routeName || !router.hasRoute(routeName)) {
       return null
     }
-    const routeOpts = getEditorRouteOpts(routeName, space, resource, remoteItemId)
-    return router.resolve(routeOpts)
+
+    return getEditorRouteOpts(routeName, space, resource, remoteItemId)
   }
   const getEditorRouteOpts = (
     routeName: RouteRecordName,

packages/web-pkg/tests/unit/composables/actions/files/useFileActions.spec.ts

@@ -103,12 +103,51 @@ describe('fileActions', () => {
         }
       })
       mocks.$router.hasRoute.mockReturnValue(true)
+
+      const mockSpace = mock<SpaceResource>({
+        getDriveAliasAndItem: () => 'personal/admin/test.txt'
+      })
+
       const result = editorAction.route({
-        space: mock<SpaceResource>(),
+        space: mockSpace,
         resources: actionOptions.resources
       })
       expect(result).not.toBeNull()
     })
+
+    it('returns unresolved route options so the router encodes the path', () => {
+      let editorAction: FileAction
+      const { mocks } = getWrapper({
+        setup: ({ getAllOpenWithActions }) => {
+          const actions = getAllOpenWithActions(actionOptions)
+          editorAction = actions.find((a) => a.name === 'editor-text-editor')
+        }
+      })
+      mocks.$router.hasRoute.mockReturnValue(true)
+
+      const mockSpace = mock<SpaceResource>({
+        getDriveAliasAndItem: () => 'personal/admin/ticket#1234/logs.txt'
+      })
+      const mockResource = mock<Resource>({
+        path: '/ticket#1234/logs.txt',
+        fileId: 'test-file-id'
+      })
+
+      const route = editorAction.route({
+        space: mockSpace,
+        resources: [mockResource]
+      })
+
+      // Resolving is left to the router. Handing it an already resolved location
+      // or a pre-encoded path both break the encoding of characters like `#`.
+      expect(route).toEqual(
+        expect.objectContaining({
+          name: 'text-editor',
+          params: { driveAliasAndItem: 'personal/admin/ticket#1234/logs.txt' }
+        })
+      )
+      expect(route).not.toHaveProperty('href')
+    })
   })
 
   describe('secure view context', () => {