1 (01.05.2009 21:04 отредактировано flashcat)

Тема: [Запрос] "Thanks"

Расширение, благодарности за посты.
(Please log in or register to see this URL)

И сразу вопрос к тем кто поставил, у меня при нажатие на кнопку спасибо выдает

 "The combination of the user that you want to thank and post does not exist!" 

и у всех в подписи появилась просто не переводимая для меня фраза
"zit ik hier wel goed? "

Из постов автора я понял, что релиз не готов до конца но у них как то уже работает. (Я просто не силен в англ.)

Подскажите пожалуйста как это лечиться???

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

заменил в thanks.php

    if ($forum_db->num_rows($result) == '0')
на

    if ($forum_db->num_rows($result) == '1')

Начали добавляться цыферки , но я не думаю что это правильно.

Во общем я все таки жду тех кто более понимает и разберается в кодах)

Сайт flashcat

Поделиться

2

Re: [Запрос] "Thanks"

if ($forum_db->num_rows($result) == '1')

почему 1 в кавычках?
функция num_rows возвращает количество строк, т.е. цифру, т.е. в любом случае сравнивать нужно с цифрой без кавычек...

или я снова торможу жутко?

Поделиться

3

Re: [Запрос] "Thanks"

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

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

Поделиться

4

Re: [Запрос] "Thanks"

убедиться что количество полученных строк больше нуля

или равно нулю как в строке до редактирования, но опять же без кавычек.

Поделиться

5

Re: [Запрос] "Thanks"

не нашел тама ссылки для скачивания.
Так бы может поразбирался )

(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

Поделиться

6 (03.05.2009 19:35 отредактировано S1LVER)

Re: [Запрос] "Thanks"

KANekT пишет:

не нашел тама ссылки для скачивания.
Так бы может поразбирался )


(Please log in or register to see this URL)

или

(Please log in or register to see this URL)

Поделиться

7 (03.05.2009 23:34 отредактировано flashcat)

Re: [Запрос] "Thanks"

knyshow пишет:

или равно нулю как в строке до редактирования, но опять же без кавычек.

Побывал ноль без кавычек ошибка запроса

"The combination of the user that you want to thank and post does not exist!" 

у меня только с единицей работает иначе ошибка. (с кавычками или без не играет роли)
У меня просто плохо получается читать чужие коды.
Но то что я вычитал должно получаться так.
При нажатие спасибо
1. Добавляется +1 под аватар
2. В посту за который сказали спасибо под подписью добавляется Ник кто сказал спасибо.
3. При нажатие у себя или у кого либо на число спасибо выводиться thanks.php  где написано кто за какой пост сказал пользователю спасибо.
Может там что то еще есть, это то что увидел я.
Пока что у меня работает с единицей только первый пункт. Со всем остальным пытаюсь разобраться но есть подозрение что оно не дописано и надо либо дописать либо ждать еще месяц два релиза от автора. (ну может я преувеличил и скоро все будет)
Или оно всё работает только у меня проблемы, так же как с нулем и единицей.

Сайт flashcat

Поделиться

8 (04.05.2009 18:40 отредактировано KANekT)

Re: [Запрос] "Thanks"

Релиз расширения punBB Thanks k0.01 (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)

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

Сайт KANekT

Поделиться

9

Re: [Запрос] "Thanks"

KANekT пишет:

Релиз расширения punBB Thanks v 0.01k (Please log in or register to see this URL)

smile Отлично и очень оперативно, огромное тебе спасибо супер программист... )

Сайт flashcat

Поделиться

10 (04.05.2009 18:41 отредактировано KANekT)

Re: [Запрос] "Thanks"

Для тех кто установил стандарт версию от kierownik. Исправление бага.

flashcat пишет:

if ($forum_db->num_rows($result) == '0')
на
if ($forum_db->num_rows($result) == '1')
$result =     if ($forum_db->num_rows($result) == '1')

было

    $result = $forum_db->query_build($query) or error(__FILE__, __LINE__);
    $result = $forum_db->fetch_assoc($result);

    if ($forum_db->num_rows($result) == '0')

ошибка в переопределении переменной $result
исправление

    $res = $forum_db->query_build($query) or error(__FILE__, __LINE__);
    $result = $forum_db->fetch_assoc($res);

    if (!$forum_db->num_rows($result) > 0)
(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

Поделиться

11

Re: [Запрос] "Thanks"

Вы можете стать Первым сказавшим спасибо за это сообщение.

Эта строчка напрягает smile

PunBB.Ru ☭

Сайт nobody

Поделиться

12

Re: [Запрос] "Thanks"

Согласен. Требуем внести соответсвующие корректировки с учетом мнения пользователей!

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

Поделиться

13 (05.05.2009 20:19 отредактировано KANekT)

Re: [Запрос] "Thanks"

nobody пишет:

Вы можете стать Первым сказавшим спасибо за это сообщение.

Эта строчка напрягает smile

Предложи на что заменить ) В общем-то она правилься в ланг файле smile
это так еще первая версия. Жду предложений\ошипков и т.д. и буду ваять 2 )

(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

Поделиться

14

Re: [Запрос] "Thanks"

Вы можете стать Первым сказавшим спасибо за это сообщение.

punbb.ru стал последним ресурсом  который пал под натиском кармической кармы. smile Плачу. :')

Сайт private_joker

Поделиться

15

Re: [Запрос] "Thanks"

KANekT пишет:

Предложи на что заменить )

Убрать нах )) Ещё заметил, что когда заходишь гостем внизу каждого сообщения вместо "Вы можете стать Первым сказавшим спасибо за это сообщение." написано "Поблагодарили:". И пустота. Тоже не красиво. Может тоже убрать нафиг. Но тогда не будет видно, кто спасибо сказал и сказали ли вообще, то есть, потеряется весь смысл.

Вообще, на официальном форуме я писал о несколько другой реализации. В идеале я хотел, чтобы в юзеринфо была ссылка "спасибо". Кликая на нее счетчик рядом со ссылкой считал бы количество "спасиб" за конкретное сообщение, а в профиле можно было бы посмотреть общее количество сказанных и полученных благодарностей. И чтоб счётчик без обновления страницы считал, на аяксе или яве. А кликая на количество "спасиб" за сообщение выскакивало бы всплывающее окошко, показывающее список благодарных за пост юзеров. Я (Please log in or register to see this URL) что-то похожее сформулировать на ломанном английском ))

Ну это я так... Размечтался smile

Добавлено спустя 4 минуты 28 секунд:

private_joker пишет:

punbb.ru стал последним ресурсом  который пал под натиском кармической кармы.

Не согласен. "Спасибы" нельзя назвать кармой, а т.к. ресурс технической направленности, типа "вопрос - ответ", то благодарности полезная фича.

Конечно, в идеале для punbb.ru хорошо бы, если бы при нажатии на кнопку "Поблагодарить" на кошелёк перечислялась n-ая сумма big_smile Но это ж опенсорс, поэтому просто "Спасибо, брат" smile

PunBB.Ru ☭

Сайт nobody

Поделиться

16 (04.05.2009 20:31 отредактировано KANekT)

Re: [Запрос] "Thanks"

ok.
Будем исправлять)

Я пытался что-то похожее сформулировать на ломанном английском ))

Ну надо было на крепком русском формулировать )

(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

Поделиться

17

Re: [Запрос] "Thanks"

nobody пишет:

Не согласен. "Спасибы" нельзя назвать кармой, а т.к. ресурс технической направленности, типа "вопрос - ответ", то благодарности полезная фича.
Конечно, в идеале для punbb.ru хорошо бы, если бы при нажатии на кнопку "Поблагодарить" на кошелёк перечислялась n-ая сумма big_smile Но это ж опенсорс, поэтому просто "Спасибо, брат" smile

...и по  достижении 1500 "пасиб" выдавался статус 13-го апостола и загорался золотой нимб в предаватврье. smile А что, будет куда стремиться, вопросов без ответов много ((Please log in or register to see this URL)), непаханно поле wink
играть в фантики так по-взрослому.

Сайт private_joker

Поделиться

18 (05.05.2009 20:19 отредактировано KANekT)

Re: [Запрос] "Thanks"

Обновление по заявкам пользователей.
Убрано "Вы можете стать Первым сказавшим спасибо за это сообщение"
Исправлено отображение Благодаривших для гостей.
(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)

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

Сайт KANekT

Поделиться

19

Re: [Запрос] "Thanks"

Кстати, расширение можно сделать кросс-SQL если в запросе $query_thanks использовать substring:

'SELECT'    => 't.id, t.post_id, t.thank_date, u.username, substring(p.message FROM 1 FOR 70) || \'...\' as subject', 

Насколько я знаю substring есть и в мускуле и в постгре и работает одинаково

Сайт Crazy_LionHeart

Поделиться

20

Re: [Запрос] "Thanks"

Просьба обсужать расширение в теме (Please log in or register to see this URL)
по поводу substring исправлю в след ревизии.

(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

Поделиться