 |
Сделал, можно проверять на тестовом сервере. - При создании вирта, нарушающего лимит, он отправляется на премодерацию. - При возвращении вирта из архива, но с нарушением лимита, он отправляется на премодерацию. Если надо выдавать сообщение об ошибке вместо отправки на премодерацию, пишите Также немного поменял функцию возвращения тайника из архива, возможно, на мой взгляд, логика измениться не должна была. Но если вдруг вы знаете какие-нибудь тайные знания про то, как используются соавторы при восстановлении тайника из архива, напишите, т.к. я эту логику снес для упрощения кода. |
 |
Замечания по итогам тестирования: 1. Выбрал вместо традиционного виртуальный, говорю сохранить, а он мне выбрасывает пустое окошко "расположение тайника:" (картинку тут не могу прицепить). Что это и что с ним делать не понятно. Может выскакивающее окно сделать с другим заголовком и поясняющим текстом? 2. Надо помимо отправки на модерацию сообщать игрокам что-то вроде "Вы превысили лимит на создание виртуальных тайников, тайник отправлен на модерацию. В случае одобрения модератором он может быть опубликован" или "Вы превысили лимит на создание виртуальных тайников. Прежде, чем создать виртуальный тайник вам необходимо создать традиционный. Ваш тайник отправлен на модерацию." А модераторам отправлять не "новый тайник на модерации", как для новичков, а "новый виртуальный тайник на модерации". 3. Как лимит считается? |
|
|
>1. Выбрал вместо традиционного виртуальный, говорю сохранить, а он мне выбрасывает пустое окошко "расположение тайника:" (картинку тут не могу прицепить). Что это и что с ним делать не понятно. Может выскакивающее окно сделать с другим заголовком и поясняющим текстом? Видел какое-то такое окошечко, но вроде оно никак не мешает создать тайник.. Не знаю, вероятно, стоит убрать, то не в рамках данного фикса. > 3. Как лимит считается? Если количество созданных традиционок <= количеству виртов => отправить на премодерацию. Собственно, как и просили. Насчет сообщения понял, надо подумать, что можно сделать. А насчет нового статуса - более глубокие изменения требуются, т.к. сейчас имеются только вот эти: 1 статус по умолчанию 2 удален владельцем 3 удален администрацией 4 удален автоматически 5 неопределенный 6 премодерация 7 премодерация (архив) |
|
|
Добавил сообщение про лимит. При восстановлении из архива: print_message("Тайник отправлен на премодерацию, т.к. вы превысили лимит на создание виртуальных тайников"); При создании нового: print_message("Тайник отправлен на премодерацию. Возможные причины: это ваш первый тайник или вы создаете виртуальный тайник, хотя у вас недостаточно традиционных"); |
|
|
Кстати, насчет "расположение тайника" - можно ее расхлопнуть и там появится гуглокарта, на которой можно тыкнуть. Странная фишка, ну да ладно - фишка :) |
|
|
Про "окошко" от Сапера: на скрине справа вверху кусочек закрывашной кнопки. Может оттого, что в браузере запретил использовать шрифты и стили сайтов — иногда всё конкретно корячится. А карта не появилась, хотя окно провисело долго. |
|
|
Больше замечаний нет. Ну кроме фичи с окошком у Сапера. У меня это окошко работает нормально. Можно заливать на основной сайт. |
|
|
Влито в мастер |
|
|
Не до конца учел 5-й пункт, дофиксил, залил изменения на тестовый сервер: В случае, когда: - Автор изменяет(!) тип с традиционки на вирт - Тайник в игре (не в архиве) - Тайник действующий (не сомнительный) - У автора недостаточно традиционных тайников в игре Когда ВСЕ четыре условия выполняются, то тайник после редактирования отправляется на премодерацию Залито на тестовый сервер, просьба проверить Собственно, исходил из тех соображений, чтобы можно было сделать тайник сомнительным без премодерации, чтобы можно было тайники в архиве редактировать как угодно. |
|
|
Доп. фикс влит. |
- Анонимный Пользователь
