[OUT] Resource Downloader User Groups 1.0.0

[Deku]

Deku добавил(а) новый ресурс:

[MDV] Resource Downloader User Groups - Данный аддон добавляет на ваш форум систему ограничения скачивания ресурсов по группам пользователей

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

Функционал

• Выбор групп пользователей прямо при создании ресурса.
• Отображение предупреждающего блока о том, что пользователь не может скачать ресурс.
• Возможность выбора категорий XFRM в которых будет работать плагин.

Узнать больше об этом ресурсе...

 

[dimka1510]

Конфликтует с [OzzModz] XFRM: Require Connected Account to Download. При нажатии на "Добавить ресурс" или "Изменить ресурс" на пустой странице пишет ошибку если в настройках категории включить этот плагин, также ошибка присутствует в админке.

Fatal error: Access level to OzzModz\XFRMConnectedAccountRequirement\XFRM\Pub\Controller\Category::setupResourceCreate() must be public (as in class MDV\ResourceDownloaderUserGroups\XFRM\Pub\Controller\Category) in /var/www/username/data/www/test1.ru/src/addons/OzzModz/XFRMConnectedAccountRequirement/XFRM/Pub/Controller/Category.php on line 25

Трассировка стека​

#0 [internal function]: XF::handleFatalError()
#1 {main}

Содержимое запроса​

array(4) {
["url"] => string(42) "/resources/categories/banki-i-platezhi.81/"
["referrer"] => string(50) "https://test1.ru/resources/vtb-onlajn.6/edit"
["_GET"] => array(0) {
}
["_POST"] => array(0) {
}
}

 

[Deku]

Конфликтует с [OzzModz] XFRM: Require Connected Account to Download. При нажатии на "Добавить ресурс" на пустой странице пишет ошибку если в настройках категории включить этот плагин, также ошибка присутствует в админке.

Fatal error: Access level to OzzModz\XFRMConnectedAccountRequirement\XFRM\Pub\Controller\Category::setupResourceCreate() must be public (as in class MDV\ResourceDownloaderUserGroups\XFRM\Pub\Controller\Category) in /var/www/username/data/www/test1.ru/src/addons/OzzModz/XFRMConnectedAccountRequirement/XFRM/Pub/Controller/Category.php on line 25

Трассировка стека​

#0 [internal function]: XF::handleFatalError()
#1 {main}

Содержимое запроса​

array(4) {
["url"] => string(42) "/resources/categories/banki-i-platezhi.81/"
["referrer"] => string(50) "https://test1.ru/resources/vtb-onlajn.6/edit"
["_GET"] => array(0) {
}
["_POST"] => array(0) {
}
}

Гляну.

 

[inz]

Конфликтует с [OzzModz] XFRM: Require Connected Account to Download. При нажатии на "Добавить ресурс" на пустой странице пишет ошибку если в настройках категории включить этот плагин, также ошибка присутствует в админке.

Fatal error: Access level to OzzModz\XFRMConnectedAccountRequirement\XFRM\Pub\Controller\Category::setupResourceCreate() must be public (as in class MDV\ResourceDownloaderUserGroups\XFRM\Pub\Controller\Category) in /var/www/username/data/www/test1.ru/src/addons/OzzModz/XFRMConnectedAccountRequirement/XFRM/Pub/Controller/Category.php on line 25

Трассировка стека​

#0 [internal function]: XF::handleFatalError()
#1 {main}

Содержимое запроса​

array(4) {
["url"] => string(42) "/resources/categories/banki-i-platezhi.81/"
["referrer"] => string(50) "https://test1.ru/resources/vtb-onlajn.6/edit"
["_GET"] => array(0) {
}
["_POST"] => array(0) {
}
}

Всё же проблемка у этого плагина, у оззи правильный скоп функции стоит.

 

[Deku]

Deku обновил(а) ресурс [MDV] Resource Downloader User Groups новой записью:

1.0.1

• Исправлена ошибка совместимости с многими плагинами расширяющими setupResourceCreate()

Узнать больше об этом обновлении...

 

[dimka1510]

Deku, можешь сделать что-то типа скрытия ненужных групп из списка? Например чтобы "Забанен", "Ограничен" или какие-нибудь "Боты" не отображались.

 

[Deku]

Deku, можешь сделать что-то типа скрытия ненужных групп из списка? Например чтобы "Забанен", "Ограничен" или какие-нибудь "Боты" не отображались.

Хорошо. При возможности реализую в ближайших обновах

 

[West14]

1. зачем? а ещё я бы обернул в $form->setup()

2. лучше унести в листенер entity_structure, как с практической точки зрения, так и с точки зрения пункта 20 стандартов

3. с расширением ResourceItem то же самое, что и в п.2
4. зачем?

5. почему вот этой проверки нет в контроллере?

6. два вопроса:
зачем для таких изменений в структуре два отдельных шага? так какие всё-таки кавычки? во втором шагу удаления очевидно копипаста, вопросов задавать не буду, исправляйте

7. зачем проверка empty?

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

 

[Deku]

зачем проверка empty?

По идее, тут проверяется то, указаны ли разрешенные группы в ресурсе, оттуда возвращается массив, если он пустой то соответственно ничего не указано. Или я не прав?

 

[inz]

По идее, тут проверяется то, указаны ли разрешенные группы в ресурсе, оттуда возвращается массив, если он пустой то соответственно ничего не указано. Или я не прав?

isMemberOf проверит уже у себя

 

[Deku]

isMemberOf проверит уже у себя

Понял. Спасибо.

 

[West14]

isMemberOf проверит уже у себя

Не, таки не проверит, isMemberOf проверяет аргумент на ложь, а пустой массив в неё кастуется. Следовательно, если у ресурса не заданы разрешенные группы, то не пустит вообще никого, что не является ожидаемым поведением. Так что проверка имеет место быть, был не прав.

 

[Deku]

Deku обновил(а) ресурс [MDV] Resource Downloader User Groups новой записью:

1.0.2

Журнал изменений:

• В настройки плагина добавлена возможность выбирать какие группы могут быть добавлены в список возможных для скачивания.
• Исправлен недочет с шагами установки.
• При создании ресурса добавлена проверка на то, включена ли возможность указания разрешенных групп.
• Исправлена ошибка с отображением кнопки скачивания. Ранее, если пользователь не выбрал ни одной группы из списка, то кнопка не отображалась.

Узнать больше об этом обновлении...

 

[dimka1510]

Deku, либо исключение ненужных групп работает наоборот, либо вместо exclude (исключить) нужно было в фразе написать include (включить). Отображаются только исключённые (на которых стоят галочки).

 

[West14]

зачем это вообще? ну вы хоть попробуйте движок посмотреть
where вполне себе принимает массив значений для поиска и строит соответствующее выражение
для отрицания это работает тоже


ну и на пачку других замечаний решили не реагировать, ладно

 

[Deku]

Deku, либо исключение ненужных групп работает наоборот, либо вместо exclude (исключить) нужно было в фразе написать include (включить). Отображаются только исключённые (на которых стоят галочки).

Перепутал фразу немного, оно включает группы а не исключает.

 

[Sadorimatsu]

Конфликтует с плагином. При установке этого при рабочем мульти префиксом, выводит тупо белую страницу, неважно какую. Забавный конфликт))

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

 

[Atikin]

Sadorimatsu, в журнале ошибок есть что-нибудь?

 

[Sadorimatsu]

Sadorimatsu, в журнале ошибок есть что-нибудь?

Спойлер

Журнал ошибки сервера

ErrorException: Fatal Error: Declaration of SV\MultiPrefix\XFRM\Entity\Category::getStructure(XF\Mvc\Entity\Structure $structure) must be compatible with MDV\ResourceDownloaderUserGroups\XFRM\Entity\Category::getStructure(XF\Mvc\Entity\Structure $structure): XF\Mvc\Entity\Structure src/addons/SV/MultiPrefix/XFRM/Entity/Category.php:108

Трассировка стека

#0 [internal function]: XF::handleFatalError()
#1 {main}

Содержимое запроса

array(4) {
  ["url"] => string(6) "/resources/"
  ["referrer"] => string(27) "https://site.ru/forums/"
  ["_GET"] => array(1) {
    ["/resources/"] => string(0) ""
  }
  ["_POST"] => array(0) {
  }
}

 

[dimka1510]

Deku, будет фикс этой проблемы? Данный плагин конфликтует со многими плагинами для менеджера ресурсов. И всегда одна и та же ошибка, меняется только конфликтующий плагин. Если не будет факса - хотя бы нужно в ресурсе указать что плагин конфликтует с другими плагинами.