Content Downloader Plugin: баги и предложения

[GoogleCodeExporter]

Дополнительная информация по проблеме:

Original issue reported on code.google.com by vladimir.shelukhin on 4 Sep 2010 at 9:07

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Можно ли указать локальный путь для 
сохранения указанных ресурсов?

Original comment by vladimir.shelukhin on 19 Sep 2010 at 10:49

[GoogleCodeExporter]

нет нельзя указать. а для чего? 

Original comment by drizt@land.ru on 20 Sep 2010 at 5:05

[GoogleCodeExporter]

- Добавить галку "Проверять обновления", 
дальше если файл ресурсов меняется, то 
уведомлять юзера.
- Если галка не установлена, то когда юзер 
открывает плагин выводить в статусном поле 
инфу, если файл обновлён относительно 
прошлого открытия.
- Показывать дерево ресурсов раскрытым 
(лучше мышкой проскроллить, чем каждый раз 
кликать раскрывая дерево), или по крайней 
мере открыть сделать раскрытой 1ю ветку.
- Не активировать кнопку дайнлоад если 
ничего не выбрано для скачивания
- Если ресурс скачали, делать галку 
"затемнённой", что бы нельзя было снять

Original comment by maksim.maj on 20 Sep 2010 at 5:09

[GoogleCodeExporter]

я уеду меня неделю не будет. потом что 
нибудь поправлю 

Original comment by drizt@land.ru on 20 Sep 2010 at 5:14

[GoogleCodeExporter]

Для того, чтобы выбирать куда складывать 
скачанный контент. Сейчас плагин тупо срёт 
в C:\Documents and Settings\%username%\PsiData, что не есть гуд.

Original comment by vladimir.shelukhin on 20 Sep 2010 at 5:38

[GoogleCodeExporter]

во первых не срет, а записывает, а во вторых 
это та папка в которой Psi ищет контент, 
разве нет? 

Original comment by drizt@land.ru on 20 Sep 2010 at 6:07

[GoogleCodeExporter]

есть же PSIDATADIR и она не всегда C:\Documents and 
Settings\%username%\PsiData

Original comment by givan101 on 20 Sep 2010 at 6:48

[GoogleCodeExporter]

ещё есть 0720-psi-win32-portable.diff и в нём глюк если 
пся лежит в папке с кириллическим 
названием так и непофикшен

Original comment by givan101 on 20 Sep 2010 at 6:52

[GoogleCodeExporter]

ИМХО, в обычной версии должно туда и 
складываться, т.к. там юзерские данные.
В портативной версии у меня складывается 
куда надо - в папку с Псей.
Если датадир указан другой, он может 
задаваться куда угодно, то туда и должен 
сохраняться контент.
Т.е. в любом случае плагин сохраняет туда, 
откуда берёт псина.
p.s. лично я все эти 3 режима не тестил, но 
вроде бы задумано как я описал, пусть автор 
скажет

Original comment by maksim.maj on 20 Sep 2010 at 7:12

[GoogleCodeExporter]

drizt@land.ru:
1. Именно что срёт, т.к. без всяческих 
запросов заливает контент хз куда (юзер 
этого не знает и у него никто не спрашивал 
разрешения). Любой даунлоадер имеет 
настройку пути сохранения того, что 
скачивается.
2. Psi юзает контент из папки с psi.exe. Именно 
там пользователь и ищет иконсеты. Именно 
туда ссылаются многочисленные хелпы и вики.

maksim.maj: По умолчанию в венде PSIDATADIR нигде не 
указана (ни в реестре, ни в переменных 
окружения). У меня, как и у подавляющего 
большинства пользователей Psi+ берёт 
иконсеты из папки с psi.exe (непортабельная 
версия), а плагин без спроса закачивает в 
%userprofile%\PsiData. В общем, бардак полный... Даже 
специально сделали воркэраунд в скрипте 
инсталлятора, чтобы он не трогал при 
деинсталляции юзерский контент в папке с 
psi.exe.

Original comment by vladimir.shelukhin on 20 Sep 2010 at 9:57

[GoogleCodeExporter]

"Psi+ берёт иконсеты из папки с psi.exe"
Это конечно здорово, но у пси нет 
возможности туда писать (права доступа как 
бы), поэтому она и должна сохранять в %userprofile%

Original comment by tishka17 on 21 Sep 2010 at 2:13

[GoogleCodeExporter]

tishka17: Речь не о том, куда у Psi есть доступ, а о 
том, что должен быть предложен выбор пути 
сохранения скачиваемого контента (как у 
любого нормального скачивальщика).

Original comment by vladimir.shelukhin on 21 Sep 2010 at 6:30

[GoogleCodeExporter]

vladimir.shelukhin: да, PSIDATADIR нигде специально не 
указана, т.к. Psi, как и весь другой софт, 
юзерские настройки по умолчанию хранит в 
папке юзера, а не в папке куда установлена 
программа. Т.е. Psi сохраняет профиль юзера, 
историю переписки, аватары и прочее - у 
каждого юзера отдельно в его системном 
%userprofile%, что правильно с точки зрения 
безопасности и "многопользовательности". 
Просто большинство юзеров являясь 
админами прут иконки и другие личные файлы 
в програм файлз, а потом возмущаются что 
анинсталлер их удаляет :)) Вот и сделали 
костыль в инсталляторе.
В общем я считаю что сейчас всё сделано 
правильно. Другой вопрос что может быть 
необходим функционал выборочного 
сохранения в любое место - пусть решает 
автор.

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

Original comment by maksim.maj on 21 Sep 2010 at 9:19

[GoogleCodeExporter]

maksim.maj: принимать решение куда сохранять 
файлы должен не автор и не плагин, а 
пользователь (даже без админских прав).

Original comment by vladimir.shelukhin on 22 Sep 2010 at 5:40

[GoogleCodeExporter]

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

Если нужна просто качалка - надо разделить 
кнопку "Download and Install" на отдельную Download и 
отдельную Install, одна качает куда показывает 
пользователь, вторая ставит куда считает 
нужной. Или добавить Save as... :) Если это нужно 
я не против.

В общем мнения высказали.

Вообще плагин лучше переименовать в 
ResourceManager и пилить, чтобы мог добавлять 
ресурсы, удалять, включая плагины, звуки, 
скины и что там ещё есть, и чекать это всё 
дело на обновления сравнивая с локальными 
файлами.

Original comment by maksim.maj on 22 Sep 2010 at 10:10

[GoogleCodeExporter]

И что в итоге будет получаться? Инсталлер 
ставит контент в папку с psi.exe, плагин ставит 
контент в %userprofile%\PsiData. И как потом 
разбираться? Что и где лежит? И нет ли дупов 
и пр.???

Original comment by vladimir.shelukhin on 23 Sep 2010 at 7:24

[GoogleCodeExporter]

Инсталлер ставит стандартный пакет в 
програм файлз (если есть права админа), что 
предназначено для всех юзеров пк.
Дальше Psi каждому запустившуму юзеру 
создаёт свой личный профайл (%userprofile%\PsiData), в 
котором хранит личные настройки юзера 
(профиль, историю, автары..).
Дальше кажлый юзер может настроить пси по 
своему вкусу, добавляя в свою личную папку 
что хочет: языки, иконки, звуки.. То же самое 
сейчас делается через плагин.
Анинсталлер должен удалять то, что он 
ставил, т.е. стандартный пакет из папки 
програм файл, не трогая настройки юзеров 
(%userprofile%\PsiData), а так же изменения что мог 
сделать админ в папке с програм файлз (мог 
записать туда что то).
А за своей папкой юзер должен следить сам, 
оттуда в любом случае софт ничего не 
удаляет.
В случае портативной версии плагин 
сохраняет туда же куда и пси, в папку с пси, 
что фактически является папкой юзера.

Original comment by maksim.maj on 23 Sep 2010 at 11:17

[GoogleCodeExporter]

При использовании инсталлера можно 
указать путь куда ставить пакет, а в 
плагине нельзя. И если нет прав админа и 
инсталлер успешно запустился, то можно 
поставить пакет туда, куда есть права на 
запись/изменение/выполнение файлов.

Деинсталлятор удаляет только то что он сам 
ставил (это уже пофикшено давно и нет 
смысла это вновь и вновь проговаривать).

Все мануалы и факи ссылаются на то, что 
ресурсы хранятся в папке с psi.exe. Т.е. по 
умолчанию юзер вообще ничего не знает про 
PsiDataDir и думает, что вся работа 
осуществляется в рабочей папке с psi.exe. 
Сейчас плагин заливает по-тихому в PsiDataDir и 
не даёт никакой предупредительной и/или 
прочей ознакомительной информации о том, 
куда будет помещён скачанный контент. Надо 
тогда либо выводить инфу перед скачиванием 
(ставить юзера перед фактом), либо давать 
юзеру право выбора пути складирования 
контента. И если в плагине заявлено "Download and 
Install", то надо, как в любом уважающем себя 
инсталлере, делать проверку на наличие 
админских прав на установку софта/файлов.

И при таком "двойном" хранении ресурсов, как 
описывает maksim.maj, от появления дупов будут 
только проблемы. Поэтому надо делать 
проверку на наличие админских прав. Если 
имеем юзерские права, то складывать туда, 
куда инсталлер установил основной пакет 
Psi+, если есть админские права и/или 
портативная установка, то ставить туда же, 
где находится psi.exe.

Original comment by vladimir.shelukhin on 24 Sep 2010 at 10:59

[GoogleCodeExporter]

>> - Не активировать кнопку дайнлоад если 
ничего не выбрано для скачивания
сделано
>> - Если ресурс скачали, делать галку 
"затемнённой", что бы нельзя было снять
вроде я изначально сделал чтоб калка 
затемнялась

Original comment by drizt@land.ru on 28 Sep 2010 at 7:25

[GoogleCodeExporter]

v0.1.5
  * кнопка Download and Install теперь неактивна, если ничего не выбрано.

в венде данная кнопка всё равно активна. 
даже при первоначальном открытии вкладки с 
тредом ресурсов.

  * поправил отображение описаний (раньше необходимо было выбирать мышкой)
работает как и раньше. пока не кликнешь на 
ресурсе, описалово не выводится.

описания выводятся только для moods и emoticons -- 
для элементов остальных групп ничего не 
выводится.

Original comment by vladimir.shelukhin on 29 Sep 2010 at 10:25

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Последняя версия уже очень даже ничего, мне 
нравится. Замечания такие:
1)Самая верхняя группа в списке iconsets - 
галочка напротив нее ведет себя 
неадекватно. У меня установлены все 
иконсеты, но она почему-то не затемнена. И я 
могу напротив нее поставить галочку, но при 
этом кнопка "Скачать и установить" активной 
не становится
2)Давай добавим возможность и удалять 
иконсеты.
3)В коде так и не вкурил, зачем используются 
psiOptions.

Original comment by wadea...@gmail.com on 12 Nov 2010 at 10:37

[GoogleCodeExporter]

Под виндой почему-то все иконсеты не 
отмечены. 

Original comment by wadea...@gmail.com on 14 Nov 2010 at 9:46

[GoogleCodeExporter]

а они у тебя в папке пользователя лежат? 

Original comment by drizt@land.ru on 14 Nov 2010 at 9:58

[GoogleCodeExporter]

C какой радости? Они лежат под виндой в 
папке, куда пси ставилась.

Original comment by wadea...@gmail.com on 5 Dec 2010 at 9:50

[GoogleCodeExporter]

вот поэтому и не отмечены, Content Downloader Plugin 
работает с папкой пользователя.

Original comment by drizt@land.ru on 5 Dec 2010 at 9:54

[GoogleCodeExporter]

drizt@land.ru: а можно сделать, чтобы плагин 
детектил и те иконсеты, которые в папке с 
psi.exe лежат??

Original comment by vladimir.shelukhin on 5 Dec 2010 at 11:27

[GoogleCodeExporter]

Конечно можно, и даже нужно. Зачем 
копировать по 2 раза иконсеты и туда и туда.

Original comment by wadea...@gmail.com on 5 Dec 2010 at 1:13

[GoogleCodeExporter]

Дело в том, что в windows vista и в windows 7 не так 
просто получить доступ на запись в каталог 
program files (куда по умолчанию ставит 
приложение инсталлятор).

Original comment by vladimir.shelukhin on 5 Dec 2010 at 1:53

[GoogleCodeExporter]

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

Original comment by wadea...@gmail.com on 5 Dec 2010 at 2:49

[GoogleCodeExporter]

звучит разумно, посмотрю как время будет

Original comment by drizt@land.ru on 5 Dec 2010 at 6:15

[GoogleCodeExporter]

по мотивам #1122374 из жуика - необходимо 
отказаться от мгновенной загрузке 
контента при открытии настроек плагина. 
Вместо этого добавить кнопку, и только 
после нажатия на нее начинать что-то 
грузить

Original comment by wadea...@gmail.com on 22 Dec 2010 at 3:51

[GoogleCodeExporter]

что за мотивы? че то я вообще не в теме, пруф 
дай.

Original comment by drizt@land.ru on 22 Dec 2010 at 4:01

[GoogleCodeExporter]

http://juick.com/1122374

Original comment by vladimir.shelukhin on 22 Dec 2010 at 4:11

[GoogleCodeExporter]

Original comment by rion4ik on 3 May 2011 at 4:24

• Added labels: Component-Plugins

[GoogleCodeExporter]

Перестал работать плагин. Приложил пример 
скачанного файла .jisp
И все такие скачиваются.

Original comment by nik.van...@gmail.com on 10 Mar 2012 at 12:59

Attachments:

• psiplus-activities-16.jisp

[GoogleCodeExporter]

Да на самом деле есть такая проблема, буду 
разбираться.

Original comment by drizt@land.ru on 14 Mar 2012 at 3:11

[GoogleCodeExporter]

Fixed. Проблема была в том что плагин не 
обрабатывает редирект. Так что я в файле 
списка сделал прямые ссылки, так в любом 
случае лучше. Зачем лишние обращения в сеть?

Спс за багрепорт.

Original comment by drizt@land.ru on 14 Mar 2012 at 3:29