kapai69 / far-plugins

Automatically exported from code.google.com/p/far-plugins
0 stars 0 forks source link

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

Open GoogleCodeExporter opened 9 years ago

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

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

GoogleCodeExporter commented 9 years ago
Уточнение: речь о FAT32.

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

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

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

GoogleCodeExporter commented 9 years ago
Разобрался как повторить ошибку. (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 commented 9 years ago
Можно и без "эвристики". Наверняка ведь 
можно проверить правильно ли отработал 
SetCurrentDirectory проверив текущий путь.

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

GoogleCodeExporter commented 9 years ago
Добавил сравнение каталога после установки

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

GoogleCodeExporter commented 9 years ago
Спасибо, теперь работает правильно.

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

  Sync folder
√ Notify problem path

?

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

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

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

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

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

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

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

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

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