Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Search in directory: search in file with Windows-1251 (cyrillic) encoding #13838

Closed
ghost opened this issue Feb 17, 2017 · 6 comments
Closed

Comments

@ghost
Copy link

ghost commented Feb 17, 2017

Description

When you try to search for a latin substring in the list of files containing latin and cyrillic characters and saved in windows-1251 encoding, the search engine does not return any result.

Steps to Reproduce

  1. Create file with containing text:
    some text with кирилическими символами
  2. Save file with windows-1251 encoding in some directory
  3. Try to search in this directory substring some text with no opening file on new page.

Expected behavior: returns a list of files containing the searched substring

Actual behavior: the search engine does not return any result

Reproduces how often: always

Versions

OS: Windows 10
Atom: 1.14.2

@rsese
Copy link
Contributor

rsese commented Feb 26, 2017

I think I was able to reproduce but to clarify:

  1. Save file with windows-1251 encoding in some directory

Can you confirm how you perform this step?

@ousennin
Copy link

Yes, the bug is present. It is caused by bug in node module 'scandal'. It simply opens files for search in wrong codepage or even worse - as a binary...

@ghost
Copy link
Author

ghost commented Mar 3, 2017

to @rsese

Can you confirm how you perform this step?

Sure, you can do this in Atom, just open new file, select encoding windows-1251 (on the bottom of Atom panel), insert into file content some text with кирилическими символами, save them in some folder. Now you can try Search in direcory phrase 'some text'.

@Ben3eeE
Copy link
Contributor

Ben3eeE commented Mar 3, 2017

@rsese I found three issues related to find-and-replace not working with various encodings. atom/find-and-replace#741 and atom/find-and-replace#751 that can be duplicates for this and atom/find-and-replace#819 that is possibly related.

@rsese
Copy link
Contributor

rsese commented Mar 3, 2017

Thanks for the pointers @Ben3eeE ⚡ Not sure why I didn't search the find-and-replace package....those do seem like basically the same issue.

We'll go ahead and close this as a duplicate @st1gus but you can subscribe to any of those issues above if you want to follow along.

@rsese rsese closed this as completed Mar 3, 2017
@rsese rsese added the duplicate label Mar 3, 2017
@lock
Copy link

lock bot commented Apr 2, 2018

This issue has been automatically locked since there has not been any recent activity after it was closed. If you can still reproduce this issue in Safe Mode then please open a new issue and fill out the entire issue template to ensure that we have enough information to address your issue. Thanks!

@lock lock bot locked and limited conversation to collaborators Apr 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants