[beta] Мини календарь событий

[Sheer]

А шаблон не трогали?

Все изменения можно отследить https://github.com/AlexSheer/phpbb3.1-Events_Calendar/commit/af9f5876291e4cdd05d2382f6a67e3b1e5990ea3

пост уже не редактируется из-за даты прошедшей

Ну вообще-то так и задумано.

[Alecto]

в админке события: Автор - Anonymous
Автор темы - Drotik, редактировал я, но никак не анонимус.

Отправлено спустя 7 минут 4 секунды:
все, понял.
проставляется не автор темы, а тот кто устанавливал событие?
событие было еще с версии 1.0.0, поэтому всплыл анонимус.

[Sheer]

В таблицу events добавлено новое поле author_id. Его значение по умолчанию 1. Поскольку у тебя поле добавлено к уже существующей таблице с данными, то это поле так и заполнено. Если отредактировать пост, ничего в неи не меняя, то значение поля изменится на id пользователя, который отредактировал пост. Ну или в базе руками сменить 1 на нужный id.

[Alecto]

это лишнее

Код: Выделить всё

style="background-color: #CADCEB; border-color: #CADCEB; margin-bottom: 6px; padding: 10px;" 

айди с учетом оформления стиля, не вошедшего в css #minical

Код: Выделить всё

#minical {
  background-color: #CADCEB;
  border: 1px solid transparent;
  border-radius: 8px;
  margin: 0 auto 6px;
  max-width: 1152px;
  min-width: 625px;
  padding: 10px;
}

себе сделал еще и отступ строки с датой (для однодневных событий значения не имеет, для длительных - лучше воспринимается) и полную адаптацию под стиль при мастабировании.

Код: Выделить всё

<style>
#minical {
  background-color: #CADCEB;
  border: 1px solid transparent;
  border-radius: 8px;
  margin: 0 auto 6px;
  max-width: 1152px;
  min-width: 625px;
  padding: 10px;
}
#minical > .inner > span {
  padding-left: 20px;
}
@media only screen and (max-width: 700px), only screen and (max-device-width: 700px) {
#minical {
  border-radius: 0;
  margin: 0 -5px 6px;
}
}
</style>

[Sheer]

Это шелуха, а есть проблема. Нужно как-то извернуться, чтобы сменить id форума в таблице EVENTS, при переносе темы в другой форум, исходя из того, что надлежащего события для этого нет. Не иначе как в перехватчике нужно усложнять скуль-запрос. Не хотелось бы, а придется.

[Alecto]

а если просто убить событие при переносе? пусть заново создается?

[bozzafresco]

bozzafresco писал(а):
Источник цитаты пост уже не редактируется из-за даты прошедшей
Ну вообще-то так и задумано.

Не очень удачно
К примеру если создавать конкурс, то нельзя будет редактировать стартпост, пока не закончится таймер - изменятся сроки
Нельзя ли как-то эту функцию сделать отключаемой и оставить слежение за временем на совести авторов и модераторов?

[Alecto]

Не очень удачно

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

Отправлено спустя 3 минуты 26 секунд:

bozzafresco писал(а):Источник цитаты пост уже не редактируется из-за даты прошедшей
Ну вообще-то так и задумано.

еще один момент.
почему дата прошедшая? в длительном событии есть смысл проверять не по дате начала, а по дате окончания. вот если последняя истекла - тогда и вопрос снимается.
может быть такой вариант исправит ситуацию?

[Sheer]

Нельзя ли как-то эту функцию сделать отключаемой и оставить слежение за временем на совести авторов и модераторов?

В listener.php удалить или закомментировать

Код: Выделить всё

            else if (($event_end && $event_end < time()) || $event_start < (time() - (time() - mktime(0, 0, 0))))
            {
                $event['error'] = array($this->user->lang['START_DATE_ERROR']);
            } 

а если просто убить событие при переносе? пусть заново создается?

Невозможно, я же говорю: события нет при переносе или разделении темы. Но кажется я выкрутился. Ща погоняю еще проверю.

[Alecto]

В listener.php удалить или закомментировать

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

может так будет правильней? и не придется мучать ничью совесть?
просто немножко подправить условие.

[Sheer]

Так оно и проверяется

Код: Выделить всё

            if ($event_end && $event_end < $event_start)
            {
                $event['error'] = array($this->user->lang['END_DATE_ERROR']);
            }

Если событие длительное и дата окончания меньше даты начала - ошибка.

[Alecto]

если оставить только дату окончания - тогда пройдет однодневное событие с прошедшей датой.
если оставить как сейчас - эта часть по ИЛИ $event_start < (time() - (time() - mktime(0, 0, 0) заблокирует редактирование.
как-то бы разнести условие:
если есть дата окончания - то проверяем только ее, если даты окончания нет - проверяем только дату начала.

Отправлено спустя 9 минут 6 секунд:

Код: Выделить всё

            if ($event_end && $event_end < $event_start)
            {
                $event['error'] = array($this->user->lang['END_DATE_ERROR']);
            }
            else if ($event_end && $event_end < time())
            {
                $event['error'] = array($this->user->lang['START_DATE_ERROR']);
            }
            else if (!$event_end && $event_start < (time() - (time() - mktime(0, 0, 0))))
            {
                $event['error'] = array($this->user->lang['START_DATE_ERROR']);
            }

вот такой вариант вроде корректно работает с однодневными и многдневными событиями
но вот, если из расширенного события сделать однодневное - почему-то не срабатывает условие и сохранение проходит.
при повторном редактировании уже все работает правильно.
возможно, что при отключении РАСШИРЕННЫЙ РЕЖИМ, все еще есть дата окончания, хотя мы ее уже не видим, и эта часть условия !$event_end дает сбой.

[bozzafresco]

Еще один вариант, который я вижу - просто выводить предупреждение, но при этом сохранять данные анонса
Правда наверное тогда придется даты обрабатывать "на лету" при помощи jquery

[Sheer]

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

Без твоих правок событие преобразуется в единичное?

[Alecto]

Без твоих правок событие преобразуется в единичное?

конечно.
если оно было многодневным и выключить расширенный режим - оно станет однодневным.
с моими правками - тоже.

Отправлено спустя 2 минуты 51 секунду:
только событие превращается в 1-дневное не в момент отключения расширенного режима при редактировании, а при сохранении, и скорее всего уже после обсуждаемой проверки.
в этом и собака порылась.