Клуб Веселых Дайверов



Дамы и Господа!
Напоминаю, что имеются в наличии наклейки и футболки с нашей эмблемой.
Стоимость наклейки 17 шек, размер - 140 мм, cтоимость футболки 35 шек.
Для приобретения обращайтесь к Ike


  Страницы: (33) « Первая ... 23 24 [25] 26 27 ... Последняя » ( Перейти к первому непрочитанному сообщению ) ОтветитьНовая темаГолосование 

> Замечания по сайту
M3ndax
Сообщение от: 20/05/12, 21:22
Цитата


Активист
***

Профиль | Цитата

Группа: Пользователи
Сообщений: 437
Регистрация: 21/12/09
Откуда: Нешер

Рейтинг:
< -5 ( 0 ) +5 >


Цитата (Vlad @ 20/05/12, 20:59)
А твои посты превышают 10 минут, смотри сам - 17 минут между постами

Влад тогда смысл склейки??? если на протяжение 15 минут я могу отредактировать свой пост сам!!!

я что то не въеду в эту логику.

обычно логика склейки проста: если последний пост в теме твой, то новый пост просто приклеивается к твоему (последний пост) и все, без времени и другой мороки.
Личное сообщениеОтправить письмо
Top
Vlad
Сообщение от: 20/05/12, 21:47
Цитата


ГенСек
Group Icon

Профиль | Цитата

Группа: Администраторы
Сообщений: 4807
Регистрация: 30/05/06
Откуда: Ришон ле Цион, Израиль



Цитата (M3ndax @ 20/05/12, 22:22)
обычно логика склейки проста: если последний пост в теме твой, то новый пост просто приклеивается к твоему (последний пост) и все, без времени и другой мороки

Ну так в чем же дело? Переделай, отмени ограничение по времени и будет нам всем счастье.
Я это сделать уж точно не смогу. Вот тебе мод.

Присоединённый файл ( Кол-во скачиваний: 3 )
Присоединённый файл  postmerge1.2.txt
Личное сообщениеОтправить письмоВебсайт пользователя
Top
M3ndax
Сообщение от: 20/05/12, 21:58
Цитата


Активист
***

Профиль | Цитата

Группа: Пользователи
Сообщений: 437
Регистрация: 21/12/09
Откуда: Нешер

Рейтинг:
< -5 ( 0 ) +5 >


Код
if ($this->post['author_id'] == $this->lastpost['author_id'] && $timedeff < 600 && !($this->post['attach_id']) && $this->post['author_id'])
       {


замени на
Код
if ($this->post['author_id'] == $this->lastpost['author_id'] && !($this->post['attach_id']) && $this->post['author_id'])
       {

и проверь :wink:

а это можно удалить

Код
$timedeff = time() - $this->lastpost['post_date'];
Личное сообщениеОтправить письмо
Top
Vlad
Сообщение от: 20/05/12, 22:49
Цитата


ГенСек
Group Icon

Профиль | Цитата

Группа: Администраторы
Сообщений: 4807
Регистрация: 30/05/06
Откуда: Ришон ле Цион, Израиль



Ок, изменил. Проверяйте, а я спать пошел :139_1:
Личное сообщениеОтправить письмоВебсайт пользователя
Top
M3ndax
Сообщение от: 20/05/12, 22:50
Цитата


Активист
***

Профиль | Цитата

Группа: Пользователи
Сообщений: 437
Регистрация: 21/12/09
Откуда: Нешер

Рейтинг:
< -5 ( 0 ) +5 >


споки будем тестить :99:
Личное сообщениеОтправить письмо
Top
Vlad
Сообщение от: 20/05/12, 22:51
Цитата


ГенСек
Group Icon

Профиль | Цитата

Группа: Администраторы
Сообщений: 4807
Регистрация: 30/05/06
Откуда: Ришон ле Цион, Израиль



Тимур, поставь в профиле птичку использование летнего времени.
У тебя неправильно показывает время постов.

Добавлено @ 22:52
А нет, это у меня глюки, сорри.
Личное сообщениеОтправить письмоВебсайт пользователя
Top
M3ndax
Сообщение от: 20/05/12, 22:53
Цитата


Активист
***

Профиль | Цитата

Группа: Пользователи
Сообщений: 437
Регистрация: 21/12/09
Откуда: Нешер

Рейтинг:
< -5 ( 0 ) +5 >


ну вот весь тест испортил :132:
так точно будет сделано :51:

Добавлено @ 23:09
такс тест спустя 16 минут

Добавлено @ 23:10
работает :148:
Личное сообщениеОтправить письмо
Top
Costa
Сообщение от: 21/05/12, 00:03
Цитата


Постоянный житель
Group Icon

Профиль | Цитата

Группа: Модераторы
Сообщений: 1192
Регистрация: 26/08/08
Откуда: Нетания, Израиль



Может все-таки оставить хоть какое-то ограничение по времени? Подклеивание поста к посту двух- или трехмесячной давности сильно искажает смысл.
Личное сообщениеОтправить письмо
Top
Roman
Сообщение от: 21/05/12, 00:20
Цитата


Постоянный житель
******

Профиль | Цитата

Группа: Пользователи
Сообщений: 2732
Регистрация: 09/10/10
Откуда: Dallas, Texas, USA

Рейтинг:
< -5 ( 1 ) +5 >


Цитата
Может все-таки оставить хоть какое-то ограничение по времени?

15 минут явно было мало. Совсем без ограничений может привести к другим казусам.
Может имеет смысл сделать что то на уровне 24 часов ?
Личное сообщениеОтправить письмо
Top
ika
Сообщение от: 21/05/12, 17:58
Цитата


Нач.Снаб.
Group Icon

Профиль | Цитата

Группа: Модераторы
Сообщений: 1676
Регистрация: 09/10/06
Откуда: Реховот, Израиль



Цитата (Costa @ 21/05/12, 00:03)
Может все-таки оставить хоть какое-то ограничение по времени?  Подклеивание поста к посту двух- или трехмесячной давности сильно искажает смысл.

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

Вот пример: Футболки: новый заказ (сообщение #53422)

Никто не узнает, что футболки готовы, поскольку тема не попадает в список обновленных тем!
Личное сообщениеОтправить письмоВебсайт пользователя
Top
Лев
Сообщение от: 21/05/12, 18:39
Цитата


Библиотекарь
******

Профиль | Цитата

Группа: Пользователи
Сообщений: 2715
Регистрация: 04/04/09
Откуда: Нес-Циона, Израиль

Рейтинг:
< -5 ( 1 ) +5 >


Влад как администратор может любую, даже самую старую тему обновить. Он это делает каждый раз когда не может что нибудь быстро продать.
Надо просто в личной связи попросить у него это сделать
Личное сообщение
Top
M3ndax
Сообщение от: 21/05/12, 18:41
Цитата


Активист
***

Профиль | Цитата

Группа: Пользователи
Сообщений: 437
Регистрация: 21/12/09
Откуда: Нешер

Рейтинг:
< -5 ( 0 ) +5 >


ika
тут есть два варианта.

1) поставить время на мерж но не 10 минут так как это смешно. скажем 24 часа.

2) с мержам обновлять дату поста.

оба варианта не проблема сделать.

Добавлено @ 18:48
первый вариант
вернем
Код
$timedeff = time() - $this->lastpost['post_date'];

добавим после
Код
$timeend = 86400; //тайм аут на склеивания в секундах (86400 сек = 24 часа)


и изменим
Код
if ($this->post['author_id'] == $this->lastpost['author_id'] && !($this->post['attach_id']) && $this->post['author_id'])


на

Код
if ($this->post['author_id'] == $this->lastpost['author_id'] && $timedeff < $timeend && !($this->post['attach_id']) && $this->post['author_id'])



Добавлено @ 18:50
Для второго варианта надо глянуть в post_reply_post.php.
Личное сообщениеОтправить письмо
Top
ika
Сообщение от: 21/05/12, 20:14
Цитата


Нач.Снаб.
Group Icon

Профиль | Цитата

Группа: Модераторы
Сообщений: 1676
Регистрация: 09/10/06
Откуда: Реховот, Израиль



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

Это сообщение отредактировал ika - 21/05/12, 20:35
Личное сообщениеОтправить письмоВебсайт пользователя
Top
Vlad
Сообщение от: 21/05/12, 21:08
Цитата


ГенСек
Group Icon

Профиль | Цитата

Группа: Администраторы
Сообщений: 4807
Регистрация: 30/05/06
Откуда: Ришон ле Цион, Израиль



Думается мне, что 24 часа для мержа слишком много, полагаю что 1 часа достаточно.
Для начала сделаю 1 час и будем посмотреть.
Личное сообщениеОтправить письмоВебсайт пользователя
Top
ika
Сообщение от: 21/05/12, 21:39
Цитата


Нач.Снаб.
Group Icon

Профиль | Цитата

Группа: Модераторы
Сообщений: 1676
Регистрация: 09/10/06
Откуда: Реховот, Израиль



Цитата (Vlad @ 21/05/12, 21:08)
Для начала сделаю 1 час и будем посмотреть.

:48:
Личное сообщениеОтправить письмоВебсайт пользователя
Top
1 посетитель читает эту тему, из них (1 гость)
0 пользователей:

  Страницы: (33) « Первая ... 23 24 [25] 26 27 ... Последняя » ОтветитьНовая темаГолосование