Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Goto Anything fails for long strings #11409

Open
laurent22 opened this issue Nov 18, 2024 · 0 comments
Open

Goto Anything fails for long strings #11409

laurent22 opened this issue Nov 18, 2024 · 0 comments
Labels
bug It's a bug desktop All desktop platforms high High priority issues

Comments

@laurent22
Copy link
Owner

Operating system

Linux

Joplin version

3.1.23

Desktop version info

Joplin 3.1.23 (prod, linux)

Client ID: 7bc2d95cc9fd43eb99fe1b31a80e59df
Sync Version: 3
Profile Version: 47
Keychain Supported: No

Revision: b3ea127

Admonition markdown extension: 1.1.0
Backup: 1.4.2
Freehand Drawing: 2.12.0
Markmap: 1.7.0
Note encryption: 1.1.1
Note Tabs: 1.4.0
Quick Links: 1.3.2
Resume Note: 0.1.1

What issue do you have?

"Goto Anything... Ctrl+P" doesn't work if the text to seach is longer than 31 characters.
Steps to reproduce:

  • add new note
  • enter the text "12345678901234567890123456789012"
  • open other note
  • press Ctrl-P and enter "12345678901234567890123456789012", returns nothing.
  • enter instead "1234567890123456789012345678901" and it finds the note.

Log shows this when trying to search with the long string:

2024-11-18 15:25:03: AsyncActionQueue: [warn] Unhandled error: TypeError: Cannot read properties of undefined (reading 'replace')
TypeError: Cannot read properties of undefined (reading 'replace')
at removeDiacritics (/tmp/.mount_JoplinD4mskR/resources/app.asar/node_modules/@joplin/lib/string-utils.js:97:19)
at /tmp/.mount_JoplinD4mskR/resources/app.asar/plugins/GotoAnything.js:276:154
at Array.map ()
at DialogComponent.updateList (/tmp/.mount_JoplinD4mskR/resources/app.asar/plugins/GotoAnything.js:276:83)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async AsyncActionQueue.processQueue (/tmp/.mount_JoplinD4mskR/resources/app.asar/node_modules/@joplin/lib/AsyncActionQueue.js:73:25)

Ref: https://discourse.joplinapp.org/t/goto-anything-fails-for-long-strings/42019

@laurent22 laurent22 added bug It's a bug desktop All desktop platforms high High priority issues labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug desktop All desktop platforms high High priority issues
Projects
None yet
Development

No branches or pull requests

1 participant