Making Quick Reply available only to x Usergroups

[Yahhoon23]

Hi, as the title basically says it all.

I want to setup quick reply to be used for only certain usergroups, and others won't even see the box there so they won't have to worry about complaining that they can't use it.

I bet it'd be just a simple addition of some code, but after looking, there's alot of areas it could be in.

Also, maybe not sure if I'd have to make quick reply where you don't have to click one of the quick reply links or vise versa???

So any help would be appreciated. Thank you.

[deathemperor]

well, this is very easy via using template conditions:
in showthread template: search for:

</div>

<br />

$quickreply

<!-- currently active users -->
<if condition="$show['activeusers']">

around $quickreply, insert this:

<if condition="is_member_of($post, specific id here)">$quickreply</if>

insert the usergroups' id that you want to let them use the quick reply box. That should be all.
Thanks Xenon for guiding me this. It's really useful, isn't it ?

[Yahhoon23]

Thanks for the reply. I'm still a little novice with coding so I'm sure I needed something else than I added.

I just seperated usergroups with commas, but that didn't work. Still all users were able to use quick reply.

Here is the code I used:

<if condition="is_member_of($post, 5,6,7,9)">$quickreply</if>

[Boofo]

Unless you use the hack that merk made for mutiple ids in that code, it has to be 1 id per instamce. So you should have:

<if condition="is_member_of($post, 5) OR is_member_of($post, 6) OR is_member_of($post, 7) OR is_member_of($post, 9)">$quickreply</if>

[deathemperor]

I'm newbie too. Template conditions are really funny. I'm sure you can't miss them dude.
Thanks Boofo for that, I haven't tried with multi usergroup.

[Yahhoon23]

Alright, went with that code and no change either.

Sorry for being such a pain this early in the morning.

I have the code directly below $quickreply

It needs to be BEFORE quickreply. ;)

[Zachery]

Unless you use the hack that merk made for mutiple ids in that code, it has to be 1 id per instamce. So you should have:

<if condition="is_member_of($post, 5) OR is_member_of($post, 6) OR is_member_of($post, 7) OR is_member_of($post, 9)">$quickreply</if>
Last time i check you can use inarray :)

<if condition="in_array(is_member_of($bbuserinfo, array(5,6,7,9)))">$quickreply</if>

And note that post only works in the postbit, thus why we use bbuserinfo

[deathemperor]

what happened may I know ?

[Boofo]

No problem. Glad I could help. ;)

Check out merk's addition to this that makes it so you can use commas to separate the ids in 1 varaiable, like the above example:

<if condition="is_member_of($post, 5,6,7,9)">$quickreply</if>

This makes it alot easier to use. ;)

[Zachery]

No problem. Glad I could help. ;)

Check out merk's addition to this that makes it so you can use commas to separate the ids in 1 varaiable, like the above example:

<if condition="is_member_of($post, 5,6,7,9)">$quickreply</if>

This makes it alot easier to use. ;)
Boofo you still need to user $bbuserinfo ;)

[Boofo]

Last time i check you can use inarray :)

<if condition="in_array(is_member_of($bbuserinfo, array(5,6,7,9)))">$quickreply</if>

And note that post only works in the postbit, thus why we use bbuserinfo

Yes, you can do it this way. But I didn't think that someone new to this would understand all of that yet so I started with the basics. ;)

[deathemperor]

hey Zachery, how to check if someone's id is X (X is specifiec number)
and what is the result of you 3 ?

[Boofo]

Boofo you still need to user $bbuserinfo ;)

Not if the showthread is the only place they will use this. But, yes, $bbuserinfo is the better way to do it board-wide. ;)

Sorry about the edit, Zachery. I hit the wrong button. ;)

[Zachery]

Not if the showthread is the only place they will use this. But, yes, $bbuserinfo is the better way to do it board-wide. ;)

Sorry about the edit, Zackery. I hit the wrong button. ;)
post is done per postbit part, and post isnt a varible that should be active on the showthread page at the bottom.

[Boofo]

I was going by this from the second post:

Thanks Xenon for guiding me this.

But I think you are right. $bbuserinfo would be the safest way to do it all-around. I stand corrected! ;)

[Yahhoon23]

Wow, I just got completely lost.

So I use

<if condition="in_array(is_member_of($bbuserinfo, array(5,6,7,9)))">$quickreply</if>

in the same spot around $quickreply in the showthread template ?

[Boofo]

Yes. ;)

[Yahhoon23]

Alright, because when I tried that on the showthread page, and being logged in as one of those usergroups, I'd go to the thread and at the very top it displays...

Warning: Wrong parameter count for in_array() in /home/wireless/public_html/forum/showthread.php(1735) : eval()'d code on line 122

[Boofo]

Does it work for you alright now?

[Yahhoon23]

Nope. It gives me that error at the top, and quick reply is still able to be used by admins which should be, and regular registered users which shouldn't be.

[Boofo]

Then use the code I posted with bbuserinfo instead of post and see if that works.

[Yahhoon23]

Yeah I did.

Maybe I didn't explain myself too well after u responded saying yes.

I asked if "<if condition="in_array(is_member_of($bbuserinfo, array(5,6,7,9)))">$quickreply</if>

in the same spot around $quickreply in the showthread template ?" was what I was supposed to use.

You replied with Yes.

So then I said well it still didn't work and it was giving me that error message at the top when viewing thread.

So obviously it creates an error along with not working using that code provided "<if condition="in_array(is_member_of($bbuserinfo, array(5,6,7,9)))">$quickreply</if>"

[Wayne Luke]

post is done per postbit part, and post isnt a varible that should be active on the showthread page at the bottom.
Even if $post was available, it would contain the userid of the last person to post in the thread, not the person viewing the thread. ;)

[Yahhoon23]

Ok, I made one error. I had two instances of $quickreply. So I removed that, and now the quick reply box is gone for me, which isn't what I want, since I am the 7 usergroup.

The Warning: at the top still stays.

So basically it did the exact opposite of what I wanted it to do.

[Boofo]

You need to REPLACE $quickreply in the showthread template with that code.

[Yahhoon23]

Yeah, Re-read above post. I was posting it underneath $quickreply. So after removing that one, the Warning Parameter message stays when viewing thread, but the quick reply box is gone when it should still be there for my usergroup

[Boofo]

Even if $post was available, it would contain the userid of the last person to post in the thread, not the person viewing the thread. ;)

So, you're saying using $post in that code wouldn't work?

[Boofo]

Yeah, Re-read above post. I was posting it underneath $quickreply. So after removing that one, the Warning Parameter message stays when viewing thread, but the quick reply box is gone when it should still be there for my usergroup

Like I said, try using the code I posted that uses the separate instances with the OR and change $post to $bbuserinfo. ;)

[Wayne Luke]

<if condition="in_array(is_member_of($bbuserinfo, array(5,6,7,9)))">$quickreply</if>
This will not work because it is incorrect logic. the structure for in_array has two required variables:
bool in_array ( mixed needle, array haystack [, bool strict])

The code above is only submitting one variable parameter, the result of the is_member_of () call. Secondly, is_member_of() does not account for an array of ID's to be passed to it. But only takes a single userid.

So, you would need to either install the change by Merk which allows an array to be sent to is_member_of or use individual conditions as outlined above but matching against $bbuserinfo instead of $post.

[Wayne Luke]

So, you're saying using $post in that code wouldn't work? Not to provide a reliable answer based on the viewer's permissions.

$post is an array that contains specific information about a post which is the information from the post table joined with the user table information of the person who made the post.

[Boofo]

This will not work because it is incorrect logic. the structure for in_array has two required variables:
bool in_array ( mixed needle, array haystack [, bool strict])

The code above is only submitting one variable parameter, the result of the is_member_of () call. Secondly, is_member_of() does not account for an array of ID's to be passed to it. But only takes a single userid.

So, you would need to either install the change by Merk which allows an array to be sent to is_member_of or use individual conditions as outlined above but matching against $bbuserinfo instead of $post.

But you can use the in_array with calling forumids, right? Like this:

if (in_array($forumid, array(XX, XX, XX)))

[Wayne Luke]

But you can use the in_array with calling forumids, right? Like this:

if (in_array($forumid, array(XX, XX, XX)))
Yes, that is a different application though. If checking for specific forums that would work. If checking for primary usergroup only you could even use:

if (in_array($bbuserinfo['usergroupid'], array (4,5,6)))

And have it work... Just the is_member_of() function is limited to a single userid. I haven't looked at Merk's hack for this but I suspect he simply wrapped the checks of that function in a FOR...EACH loop that checks each value in an array. Wouldn't be a bad change for the core product in my opinion.

[Wayne Luke]

But we have gotten off track here... The solution to the original question is to open the SHOWTHREAD template and replace:

$quickreply

WITH

<if condition="is_member_of($bbuserinfo, 5) OR is_member_of($bbuserinfo, 6) OR is_member_of($bbuserinfo, 7) OR is_member_of($bbuserinfo, 9)">$quickreply</if>


I believe that will provide the solution you are looking for.

[Yahhoon23]

THANK YOU! :)

Working good now.

[Boofo]

Yes, that is a different application though. If checking for specific forums that would work. If checking for primary usergroup only you could even use:

if (in_array($bbuserinfo['usergroupid'], array (4,5,6)))

And have it work... Just the is_member_of() function is limited to a single userid. I haven't looked at Merk's hack for this but I suspect he simply wrapped the checks of that function in a FOR...EACH loop that checks each value in an array. Wouldn't be a bad change for the core product in my opinion.

merk's hack is great. I asked about that on vb.com, if I remember right. That would be a great addition to vbulletin, like you said. Mayve we'll see it in the next update? ;)

Ok, back to the code at hand. Basically, using the code I posted with the OR and changing the post to bbuserinfo is the same thing you posted, right?

* Boofo writes this down because he isn't right too often. ;)

[Wayne Luke]

Ok, back to the code at hand. Basically, using the code I posted with the OR and changing the post to bbuserinfo is the same thing you posted, right?
Correct.

[Boofo]

Correct.

Thank you, sir. ;)

* Boofo writes down that Wayne said he was correct which will probably be the last time that ever happens. ;)

[deathemperor]

so, can we use

if (in_array($bbuserinfo['userid'], array (5843,9948)))

to check if it's Wayne Luke or Boofo ?

[Wayne Luke]

Yes...

[deathemperor]

thank you Wayne Luke
template conditions....great ! ^ ^

[sketch42]

so whats the end result ???
https://vborg.vbsupport.ru/showpost.php?p=493024&postcount=33
well that worked but it says done but with errors on the page