26 Ответ от Zevs

  • Zevs
  • Мастер
  • Неактивен
  • Откуда: Киев
  • Регистрация: 30.05.2006
  • Сообщений: 412
  • Репутация : [2|0]

Re: [Релиз] Репутация

Поотключал все расширения - все равно
Переставлял  - тоже не помогло

Добавлено спустя 1 час 2 минуты 59 секунд:

На другом форуме при увеличении репы  - тоже такая же ссылка получается

А при удалении - вот такая ошибка:

The error occurred on line 181 in /extensions/reputation/model/reputation.php

Database reported: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'rep_minus= WHERE id = 530' at line 1 (Errno: 1064).

Добавлено спустя 1 час 6 минут 26 секунд:

на test.  при удалении репы - ошибка вот такая 

Failed query: UPDATE users SET rep_plus=,rep_minus= WHERE id = 805
http://www.logan.in.ua/

Сайт Zevs

Поделиться

27 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

ок, будем рзбираться

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

28 Ответ от kmbxxx

  • kmbxxx
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

2.60906 секундд    
SELECT u.email, u.title, u.url, u.location, u.signature, u.email_setting, u.num_posts, u.registered, u.admin_note, u.avatar, u.avatar_width, u.avatar_height, p.id, p.poster AS username, p.poster_id, p.poster_ip, p.poster_email, p.message, p.hide_smilies, p.posted, p.edited, p.edited_by, g.g_id, g.g_user_title, o.user_id AS is_online, u.rep_plus, u.rep_minus, u.rep_enable, u.rep_disable_adm, r.id as rep_id FROM posts AS p INNER JOIN users AS u ON u.id=p.poster_id INNER JOIN groups AS g ON g.g_id=u.group_id LEFT JOIN online AS o ON (o.user_id=u.id AND o.user_id!=1 AND o.idle=0) LEFT JOIN reputation AS r ON (r.post_id = p.id AND r.from_user_id = 2) OR (r.user_id = u.id AND r.from_user_id = 2 AND r.time > 1320599357) WHERE p.id IN (440716,440717,440718,440726,440731) ORDER BY p.id

Не хорошо :-/

Поделиться

29 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

Это конечно не самый легкий запрос, но выборка по прямым индексам - 

LEFT JOIN reputation AS r ON (r.post_id = p.id AND r.from_user_id = 2) OR (r.user_id = u.id AND r.from_user_id = 2 AND r.time > 1320599357) WHERE p.id IN (440716,440717,440718,440726,440731)

Проверь наличие индексов в репутации у полей: post_id, from_user_id, user_id, time
Возможно проблема в другом месте? Например mysql в данный момент обрабатывает 5000 запросов, озу нет, физически захлебывается.

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

30 Ответ от floop (06.11.2011 22:50 отредактировано floop)

  • floop
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

при просмотре репутации админом криво отображается таблица. проблему нашел в файле Oxygen.min.css
в пятой строке

table{width:100%;table-layout:fixed;empty-cells:show;border-spacing:0;line-height:1.3333em;border-collapse:collapse}

изменил на table-layout:inherit и жизнь наладилась

http://punbb.ru/extensions/hcs_image_uploader/uploads/30000/1500/31786/thumb/p16duc9ii81o6241f14gg10gqjof1.jpg

https://github.com/floop-as/punbb/commi … 8#comments


+ Вот тут нашел ошибку с ссылками на профиль плюсующего/минусующего
https://github.com/floop-as/punbb_exten … f6e13cab2c

Buy Me a Coffee
+ hcs

Поделиться

31 Ответ от dimkalinux

  • dimkalinux
  • dimkalinux
  • Разработчик
  • Неактивен
  • Откуда: Киев
  • Регистрация: 15.01.2006
  • Сообщений: 635

Re: [Релиз] Репутация

Зря вы стиль таблиц ядра меняете для одной таблицы из расширения. Лучше изменить в стилях самого расширения.

Смайл. Дата. Подпись.

Поделиться

32 Ответ от floop (07.11.2011 00:05 отредактировано floop)

  • floop
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

Расширение сложное для моего понимания smile не пойму как сделано.. брр)

мне казалось оно должно хуком в mics.php подключаться, но нет, не вижу таких хуков

Buy Me a Coffee

Поделиться

33 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

Да, расширение немного усложнилось и в тоже время упростилось. Все вызовы хуков вынесены в диспетчер как одноименные с хуками методы. Т.е. вместо хука в манифесте - метод в классе.
Вся логика реализована в контроллерах. Контроллеры наследуются от базового контроллера определенного в developer_helper'е. Инициализирует котнроллеры и вызывает их методы тоже developer_helper. За счет его использования реализован паттерн MVC. По моему всё стало проще smile
Переопределить стиль для таблицы можно в методе view контроллера reputation/contrroller/reputation.php добавив туда чтото типа:

App::$forum_loader->add_css('#brd-reputation table{table-layout:inherit;}', array('type' => 'inline'));

Или в файле reputation/view/view_admin.php для таблицы вписать стиль.

Добавлено спустя 5 минут 6 секунд:

ЗЫ спасибо за найденные ошибки

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

34 Ответ от kmbxxx

  • kmbxxx
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

hcs индексы есть. Возможно, но факт в том, что как только отключаю мод репутации время генерации страниц падает с 5-8 секунд до 0.5 секунд.

Поделиться

35 Ответ от Visman

  • Visman
  • Visman
  • Главсуслпромсбыт
  • Неактивен
  • Откуда: Сибирь
  • Регистрация: 01.01.2008
  • Сообщений: 1,515
  • Репутация : [176|0]

Re: [Релиз] Репутация

kmbxxx, убери из запроса 

AND r.time > 1320599357

и посмотри на время генерации.

Моя сборка FluxBB 1.5 * Parserus - BBCode parser

Поделиться

36 Ответ от kmbxxx

  • kmbxxx
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

Не, не сильно изменилось.... Вот если сократить до одного  (440716,440717,440718,440726,440731) то заметно падает до 0,3 секунд, хотя тоже много...

Поделиться

37 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

Ладно сделаю вариант с дополнительным запросом, без джойна

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

38 Ответ от kmbxxx

  • kmbxxx
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

анализ запроса средствами phpmyadmin:

id     select_type     table     type     possible_keys     key     key_len     ref     rows     Extra
1     SIMPLE     p     range     PRIMARY,posts_multi_idx     PRIMARY     4     NULL     9     Using where; Using temporary; Using filesort
1     SIMPLE     u     eq_ref     PRIMARY     PRIMARY     4     new_forum2.p.poster_id     1      
1     SIMPLE     g     eq_ref     PRIMARY     PRIMARY     4     new_forum2.u.group_id     1      
1     SIMPLE     o     ALL     online_user_id_ident_idx     NULL     NULL     NULL     86      
1     SIMPLE     r     ALL     rep_post_id_idx,reputation_rep_time_idx     NULL     NULL     NULL     68397

Поделиться

39 Ответ от KANekT

  • KANekT
  • KANekT
  • Разработчик
  • Неактивен
  • Откуда: Брежнев
  • Регистрация: 11.03.2008
  • Сообщений: 2,110
  • Репутация : [218|0]

Re: [Релиз] Репутация

Аякс голосование jquery_ui почему не работает на sqlite3.

PunBB Extensions | Перед созданием темы => ФАК | Профессиональная разработка сложных сайтов

Расширения распространяются по Creative Commons Attribution-NonCommercial 4.0

Сайт KANekT

Поделиться

40 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

kmbxxx, все ясно спасибо

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

41 Ответ от PunBB Fun (03.12.2011 09:50 отредактировано PunBB Fun)

  • PunBB Fun
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

Вроде всё встало нормально, но почему то вместо всплывающих комментариев ссылка вида - http//mysite.ru/profile.php?id=525525

Естественно что при клике попадаем на несуществующую страницу. Что не так или где подправить?

Не забываем ставить плюсики дополнительно к «спасибам».

Поделиться

42 Ответ от PunBB Fun (06.12.2011 05:15 отредактировано PunBB Fun)

  • PunBB Fun
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

Нашел очередной баг - если из гостя пожмакать на "репу" (нижний бокс с кликабельными именами), после чего залогинится, выскочит ошибка примерно такого содержания:

{"message":"

\u0421\u043f\u0430\u0441\u0438\u0431\u043e \u0437\u0430 \u0441\u0438\u0433\u043d\u0430\u043b<\/p>"}
Не забываем ставить плюсики дополнительно к «спасибам».

Поделиться

43 Ответ от Deluxe

  • Deluxe
  • Новичок
  • Неактивен

Re: [Релиз] Репутация

Здравсвуйте, установил эту модификацию. При включении запрашивает "nya_jquery_ui". Как установить "nya_jquery_ui" ?

Поделиться

44 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

[Релиз] jQuery UI

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

45 Ответ от kmbxxx

  • kmbxxx
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

Ладно сделаю вариант с дополнительным запросом, без джойна

Его еще ждать? smile

Поделиться

46 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

Если кто-то предложит свой вариант, то я по-бырому его посмотрю. А так-то времени у меня увы пока нет.

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться

47 Ответ от kmbxxx

  • kmbxxx
  • Опытный
  • Неактивен

Re: [Релиз] Репутация

Подтверждаю [Релиз] Репутация , аналогичная ситуация. smile
hcs, будем ждать сэнсея %)

Поделиться

48 Ответ от SlayerNik

  • SlayerNik
  • Интересующийся
  • Неактивен

Re: [Релиз] Репутация

+1 к багу с несуществующей ссылкой,  обидный баг

Добавлено спустя 41 минуту 45 секунд:

а почему тогда на этом форуме работает без багов?

Поделиться

49 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

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

Захочешь — найдешь время, не захочешь — найдешь причину.
+ kmbxxx, SlayerNik

Поделиться

50 Ответ от hcs

  • hcs
  • hcs
  • Администратор
  • Неактивен
  • Откуда: СССР
  • Регистрация: 10.02.2006
  • Сообщений: 6,337
  • Репутация : [272|0]

Re: [Релиз] Репутация

Тормоза ушли?

Захочешь — найдешь время, не захочешь — найдешь причину.

Поделиться