For Marital Status Thread Display this right?
$thread['postusername'] .= $marital_status_threadids[$thread['threadid']];
For Marital Status Thread Query, this?
($hook = vBulletinHook::fetch_hook('forumdisplay_query')) ? eval($hook) : false;
// Query to check if/what threadowner's marital status is
$marital_status_threadids = array();
$marital_status_threads_query = sprintf("SELECT thread.threadid, userfield.field17 AS maritalstatus FROM thread LEFT JOIN userfield ON(thread.postuserid=userfield.userid) WHERE thread.threadid IN (0%s)", $ids);
$marital_status_threads_result = $vbulletin->db->query_read($marital_status_threads_query);
while ($marital_status_threads_row = $vbulletin->db->fetch_array($marital_status_threads_result))
{$marital_status_threadids[$marital_status_threads_row['threadid']] = $marital_status_threads_row['maritalstatus'];}
I'm still getting the big error.
kk just to let you know I change it with the new code you gave us and I still got the same error.
Hmmm... There is no doubt that it is my code that is causing your error. Line 866 in forumdisplay is where the forumdisplay_query hook is located, and that's where I told you to make that first plugin. I just can't seem to figure out WHY.
This field17 of yours, it's just a string, correct? Like a link to an image or something, not the actual image data, right? I'm not understanding why it's trying to allocate such a huge chunk of memory. I mean, if it was actual binary data (like if you stored the image itself on the database) I guess I could understand...
Could you do me a favour and select five or ten random users, get their userid, run this query (phpMyAdmin, the vBulletin run query bit, or even by the command line, it matters not where), and paste the output for me?
SELECT field17 AS maritalstatus FROM userfield WHERE userid IN (XXX1,XXX2,XXX3,...,XXXN)
Replace the XXX1,XXX2,XXX3,...,XXXN with the userids. An example could be "IN (34562,43529,11324,74923,2514)".
--------------- Added 1246376862 at 1246376862 ---------------
don't want to hijack this thread or anything, but just wanted to ask if i only want to look for the usergroup(s) the threadstarter belongs to, do I also need to work with all that code?
I want to show an image in the threadbit, next to the username of the threadstarter if the threadstarter belongs to usergroup 22 ... that usergroup is an "additional" usergroup.
This can actually be done with just a little modification to the $hook_query_fields and $hook_query_joins variables for the call to threads. :)]
You will be making a plugin on the hook forumdisplay_query. Title it however you want.
Now, what you want is to join the user table with the thread table, so that you can select values from the user table. The specific field you want is membergroupids. PLEASE NOTE, however, that if for some reason you change around the users' PRIMARY usergroup (something I never do), that this will be an entirely different story! I'm just assuming here that you use secondary usergroups, so let me know if that's not the case!
ALSO NOTE, this first code is assuming that you want to use more than the usergroup '22' at some point in the future. We can uncomplicate things (heh, kind of) if you KNOW that you will only ever want to use usergroup 22, and I'll explain that a bit later.
So, here's the code that needs to go in the plugin, assuming the above:$hook_query_fields .= ', user.membergroupids '; // This adds the membergroupids field to the data returned
$hook_query_joins .= ' LEFT JOIN user ON(thread.postuserid=user.userid) '; // This adds the user table to the query, and searches only the record that matches the postuserid
Now, this data will be returned in the $thread variable as $thread['membergroupids']. The data itself is a comma-separated list, like "2,5,34,22,78", so in out next plugin, we'll explode that into an array, which will be a bit easier for us to handle with PHP.
Our NEXT plugin goes on the hook, threadbit_display. Title it what you want. :3
The code we'll use will make that list into an array, and will then search the array for the value '22'.
$thread['membergroupids'] = explode(',', $thread['membergroupids']);
$isgroup22 = in_array(22, $thread['membergroupids']);
Now, $isgroup22 will be true if the usergroup 22 is included. You can use this variable in an <if condition="$isgroup22">Print this<else />Print That</if> statement in the template, OR (if you have a variable you want to append it to) you can modify it directly in the plugin.
Consider this alternative to the above plugin:
$thread['membergroupids'] = explode(',', $thread['membergroupids']);
if (in_array(22, $thread['membergroupids'])) {$thread['postusername'] .= '<img src="/my/supercool/usergroup/image.png" />';}
This will just append the image to the username for the thread starter without the need for touching templates. :3
I'll be back in a bit with explanation for if you ONLY ever wanted to use usergroup 22.
--------------- Added 1246382814 at 1246382814 ---------------
Y'know, I really hate this effing "Auto-merged Double-Post" crap. :( A lot of posts can be read better if their content was seperated better.
* DragonBlade shakes a fist in defiance of the Admins. And is then struck down with the Almighty Ban Hammer. https://vborg.vbsupport.ru/external/2009/06/1.jpg
ANYWAYS, back to what I was saying. I went over it, and really, the efficiency wouldn't be much better in searching only for usergroup22 in the query, so what I posted is probably best. HOWEVER, there might be a slight bug with the second plugin.
The line, $thread['membergroupids'] = explode(',', $thread['membergroupids']);Assumes that there will be a comma-seperated list, but this is not always the case, so a better way to write it would be:$thread['membergroupids'] = ($thread['membergroupids']) ? explode(',', $thread['membergroupids']) : array();This makes it so that if there is no value in $thread['membergroupids'], it will return an empty array. It would do the same the other way, but could also throw warnings that would ugly up your page. XP