Сохранение черновика новой темы

[Rhidian (Rhidian Bramley)]

Приоритет: Низкий

Платформа: Discourse mobile iOS (не тестировалось на других платформах). Воспроизводится на Discourse meta и try.Discourse.org.

Описание: При сохранении черновика новой темы по запросу кнопка «Открыть черновик» не работает.

Шаги для воспроизведения:

1. Перейдите на главную страницу Discourse.
2. Нажмите «+ Новая тема».
3. Введите тему «Testing Drafts» и текст в редакторе «Testing». Оставьте без категории.
4. Нажмите «Отмена» (иконка корзины в мобильной версии Discourse). Вам будет предложено сохранить черновик на потом — выберите этот вариант.
5. Кнопка «+ Новая тема» изменится на «+ Открыть черновик».
6. Нажмите «+ Открыть черновик». Ничего не происходит.
7. Проверьте папку «Черновики» — черновик не сохранён.

Последствия: Это временно сделало невозможным создание новой темы.

Обходное решение: Многократное нажатие неактивной кнопки «+ Открыть черновик» в конечном итоге приводит к сбою сайта Discourse в мобильном приложении и возврату к выбору домена. При возвращении в Discourse кнопка «+ Открыть черновик» сбрасывается и снова отображается как «+ Новая тема».

[jean.perez (Jean)]

Заголовок слишком короткий, чтобы пройти эту проверку

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/models/composer.js#L1184

Но я задаюсь вопросом: почему код переходит в состояние разрешения, а не отклонения с последующей обработкой ошибок?

Думаю, ошибки можно перехватить здесь

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/controllers/composer.js#L1244

что-то вроде этого

this._saveDraftPromise = model.saveDraft().then(() => {
    this._lastDraftSaved = Date.now();
    ...
}, error => {
    // обработка ошибок здесь
}).finally(() => {
    this._saveDraftPromise = null;
});

@eviltrout, есть какие-то мысли?


[eviltrout (Robin Ward)]

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

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

[SonsOfAhriman (Ryan Huddleston)]

Удалось воспроизвести эту ошибку в MacOS Chrome на Discourse meta, следуя шагам Rhidian, указанным ниже:

@Rhidian

Шаги для воспроизведения:

1. Перейдите на главную страницу Discourse.
2. Нажмите «+ New Topic».
3. Введите заголовок темы «Testing Drafts». Введите в теле композера «Testing». Оставьте без категории.
4. Нажмите «Cancel» (иконка корзины в мобильной версии Discourse). Вам будет предложено сохранить черновик для последующего использования — выберите этот вариант.
5. Кнопка «+ New Topic» изменится на «+ Open Draft».
6. Нажмите «+ Open Draft». Ничего не происходит.
7. Проверьте папку «Drafts» — черновик не сохранен.

[nbianca (Bianca)]

Здравствуйте,

Спасибо за сообщение об ошибке. Она была исправлена в последней версии Discourse. Проблема заключалась в том, что система предлагала сохранять очень короткие черновики, даже когда этого не должно происходить.