Module: User Thank/Flame

Quote
freereign
The phorum_db_search() function will not work for my needs. It searches for subject, message, author, or user_id. I need to search meta.

It would be better to store flame/thanks data in a table of it's own to be able to search it. Searching meta data is not going to be easy or fast.

---
-=[ Panu ]=-

User custom fields is already moved into its own table for 5.2. It will be done for messages in 5.3 as well. phorum_user_custom_fields is the table in 5.2. In 5.3 that will be renamed to phorum_custom_fields and will hold both user and message custom info.

Now, searching that is a whole other thing.

Brian - Personal Blog

Quote
Brian Moon
Now, searching that is a whole other thing.

Not really, there is
phorum_db_search_custom_profile_field
already :).
it just needs to be generalized too.

---

Thomas Seifert

Quote
justingaynor
Yes, the User/Thank Flame module has a folder of templates files it uses for displaying ratings on various pages, and it is calling for a thread.tpl files within it's own /mods/user_thank_flame/templates/emerald folder, but it does not exist. I did not mean to suggest that this file was left out of the phorum build, simply the latest build of this particular add-on.

same problem here with latest version of mod.
what did you do in order to solve this?

Garghhhh, Flaming works fine but when i try to thank...

Fatal error: Call to undefined function phorum_db_update_messages() in /home/thepitc/public_html/mods/user_thank_flame/functions.php on line 51

Help anyone... I havent even changed anything yet

Also i didnt get any minimilistic template files..

[www.ipodbb.net]



Edited 1 time(s). Last edit at 07/03/2008 04:01PM by Alex1912.

Quote
freereign
LANG support is complete for all pages except the settings.php page. I have been unable to get $PHORUM['DATA']['LANG']['mod_user_thank_flame']['DisplayMode'] to show up as anything (although printing it in read.php or profile.php works fine).

As I know, a general phorum style is to not translate admin messages, so there is no need to provide localization support in settings.php.

While looking through code, I find that you define short_time in module english lang file. That’s not very good, as it doesn’t reflect the global setting and could be wrong for other languages. I’m guessing, you should reuse the following code from event_logging module:
[code="php"]
if (!isset($PHORUM['short_time'])) {
$f = str_replace($PHORUM['short_date'],'',$PHORUM['short_date_time']);
$f = preg_replace('/^\s+|\s+$/', '', $f);
$PHORUM['short_time'] = $f;
}
[/code]

Thanks for the module! :)

It's been awhile since I looked at this. I will look into the problems you are having (as well as the short_time in the lang file. I don't remember that...)

Another thing: I have been testing in SVN, so I need to test in stable (5.2.8) as well

Also you need to change phorum_db_update_messages to phorum_db_update_message in functions.php, as there is no such function (at least in 5.2.8).

Another suggestion is to use hooks instead of includes. I. e. write in templates something like {HOOK "tpl_user_thank_flame_profile"} and add that hook to your module. That way, if module will be disabled in admin, it will be transparently disabled in all templates.
Also, you should use css_register hook to add your css.



Edited 1 time(s). Last edit at 08/07/2008 09:56AM by Alexey Torkhov.

Thanks for the tips. The hook-template idea is a good one that I will incorporate. I also didn't know about the css_register hook. That won't take long to implement (neither will).

The phorum_db_update_messages were accidentally left unchanged to phorum_db_update_message. I have fixed them (there were 2 instances of it).