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

PathSync: определяет не все проблемные директории #3

Open
GoogleCodeExporter opened this issue Dec 5, 2015 · 9 comments

Comments

@GoogleCodeExporter
Copy link

Очень хотелось бы научить плагин 
информировать о всех проблемных 
директориях.
Пока что он никак не сообщает о 
директориях, оканчивающиеся на пробел, или 
точку.

Original issue reported on code.google.com by anton.si...@gmail.com on 1 Oct 2009 at 4:00

@GoogleCodeExporter
Copy link
Author

Уточнение: речь о FAT32.

Original comment by anton.si...@gmail.com on 1 Oct 2009 at 6:00

@GoogleCodeExporter
Copy link
Author

Проверил обе ситуации - обе 
диагностируются. OC - Vista'а
Проблемным считается такой путь, который 
не удалось установить в качестве текущего,
т.е. SetCurrentDirectory вернула False. Не знаю, что еще 
тут можно придумать.

Original comment by Max.Ru...@gmail.com on 1 Oct 2009 at 7:05

@GoogleCodeExporter
Copy link
Author

Разобрался как повторить ошибку. (FAT32 
непричём)
Надо чтобы в одном месте было 2 директории, 
например:
"Dir" и "Dir "
"Dir" и "Dir."
В этом случае SetCurrentDirectory не возвращает 
ошибку, но устанавливает конечно же 
не ту директорию.

Придумать тут можно кое-что: нужна 
"эвристика", чтобы определять проблемные 
пути и 
без SetCurrentDirectory.

Если плагин научиться это делать, то можно 
будет опцию "Notify error" переделать в  
"Notify problem path" (суть в том чтобы включать Notify 
при отключённом Sync)

  Sync folder
√ Notify problem path

Original comment by anton.si...@gmail.com on 2 Oct 2009 at 8:30

@GoogleCodeExporter
Copy link
Author

Можно и без "эвристики". Наверняка ведь 
можно проверить правильно ли отработал 
SetCurrentDirectory проверив текущий путь.

Original comment by anton.si...@gmail.com on 2 Oct 2009 at 8:34

@GoogleCodeExporter
Copy link
Author

Добавил сравнение каталога после установки

Original comment by Max.Ru...@gmail.com on 2 Oct 2009 at 9:28

@GoogleCodeExporter
Copy link
Author

Спасибо, теперь работает правильно.

А как насчёт этого:

  Sync folder
√ Notify problem path

?

Original comment by anton.si...@gmail.com on 3 Oct 2009 at 8:38

@GoogleCodeExporter
Copy link
Author

Ошибки проверяются путем установки 
каталога, так что установка второй галки 
без
первой - не имеет смысла. Эвристики делать 
не буду.

Original comment by Max.Ru...@gmail.com on 3 Oct 2009 at 8:55

@GoogleCodeExporter
Copy link
Author

Если не держать плагин включённым 
постоянно, то проявляется маленькое 
неудобство: 
синхронизация начинается не в тот момент, 
когда мы включили опцию, а только после 
того как мы меняем директорию в панели.

Смысл в выключении плагина простой: есть 
надежда что в фаре поправят баги вызванные 
последними изменениями, и есть желание 
поучаствовать в выявлении этих багов.

В связи с этим возникает ещё одно 
предложение: добавить в меню плагинов 
команду 
"Синхронизировать текущую директорию" 
(однократное действие)

Original comment by anton.si...@gmail.com on 7 Oct 2009 at 9:26

@GoogleCodeExporter
Copy link
Author

Если вдруг у вас появится желание 
реализовать предложенный выше пункт 
"Синхронизировать текущую директорию", то 
хотелось бы видеть его не только в 
панелях, но и в редакторе (возможно и в 
других областях)

Original comment by anton.si...@gmail.com on 9 Oct 2009 at 6:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant