26

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
(Please log in or register to see this URL)/

Сайт Zevs

Поделиться

27

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

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

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

Поделиться

28

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

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 (06.11.2011 22:50 отредактировано 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

(Please log in or register to see this URL)


+ Вот тут нашел ошибку с ссылками на профиль плюсующего/минусующего
(Please log in or register to see this URL)

(Please log in or register to see this URL)
+ hcs

Поделиться

31

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

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

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

Поделиться

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

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

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

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

(Please log in or register to see this URL)

Поделиться

33

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

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

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

Поделиться

35

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

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

AND r.time > 1320599357

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

Моя (Please log in or register to see this URL) FluxBB 1.5, (Please log in or register to see this URL), (Please log in or register to see this URL).

Поделиться

36

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

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

Поделиться

37

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

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

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

Поделиться

38

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

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

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

(Please log in or register to see this URL) | Перед созданием темы => (Please log in or register to see this URL) | (Please log in or register to see this URL)

Расширения распространяются по (Please log in or register to see this URL)

Сайт KANekT

Поделиться

40

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

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

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

Поделиться

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

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

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

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

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

Поделиться

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

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

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

{"message":"

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

Поделиться

43

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

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

Поделиться

44

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

(Please log in or register to see this URL)

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

Поделиться

45

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

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

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

Поделиться

46

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

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

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

Поделиться

47

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

Подтверждаю (Please log in or register to see this URL) , аналогичная ситуация. smile
hcs, будем ждать сэнсея %)

Поделиться

48

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

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

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

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

Поделиться

49

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

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

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

Поделиться

50

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

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

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

Поделиться