Restrict uploads

Настройка Компонент темы
25

image
image1202×382 16.5 KB

Спасибо, всё работает

26

Теперь можно ли настроить этот компонент так, чтобы он ограничивал доступ только группами подписчиков :P?

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

2 лайка
27

Функция «Ограничить загрузку» теперь полностью блокирует возможность загрузки файлов в новых сообщениях или ответах.

1 лайк
28

Верно, извините.

Похоже, что this._super() на самом деле не возвращает информацию о том, разрешена ли пользователю загрузка изображений в ядре. Я переиспользовал это из старого компонента и уже вчера заметил, что это также вызывает проблемы с логикой плейсхолдера.

Сейчас у меня нет времени углубляться в это.

Одна вещь, которую я заметил при тестировании сегодня: использование "*" в api.addComposerUploadHandler(["*"]) ломает регулярное выражение, используемое для сопоставления обработчиков загрузки.

1 лайк
29

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

2 лайка
30

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

import discourseComputed from "discourse-common/utils/decorators";

api.modifyClass("service:composer", 
  (SuperClass) => class extends SuperClass {
    @discourseComputed
    allowUpload() {
      return super.allowUpload && canUpload;
    }
  }
);

РЕДАКТИРОВАНИЕ:

Созданный PR:

4 лайка
Returning bumping after editing last post
31

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

9 лайков
33

@tshenry, вас интересует изменение компонента, чтобы он работал на основе групп?

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

1 лайк
34

@Moin Я только что оставил комментарий к твоему PR относительно того, на что, как я полагаю, ты ссылался в изменениях настроек группы. Если ты всё ещё заинтересован в работе над этим, не стесняйся отправлять обновление. Я уже настроил ветку твоего PR на своём тестовом сайте, так что смогу помочь продвинуть это вперёд.

2 лайка
35

Отличное обновление этого компонента от @Moin было объединено :tada: Спасибо, Moin!

Я постараюсь в ближайшее время обновить первый пост этой темы, чтобы отразить все улучшения, внесенные @Arkshine и @Moin в этом году.

4 лайка