[ABD] Group Template Variables

[martti]

Extension Name: Group Template Variables
Author: martti
Extension Description:
The extension provide a template variable for every group the current user is in. They are all put under the template variable marttiphpbb_grouptempvars

I.e:

Code: Select all

{%- if marttiphpbb_grouptempvars.4 -%}
    This content is only visible when the user is in group with id 4.
{%- endif -%}

To list all group ids a user is in:

Code: Select all

<div class="panel">
  <h3>
    all group ids:
  </h3>
  <p>
    {%- for group_id, value in marttiphpbb_grouptempvars -%}
      {{- group_id -}}
    	{%- if not loop.last -%},&nbsp;{%- endif -%}
    {%- endfor -%}
  </p>
</div>

This shows:



In the previous version (for phpBB 3.1) of this extension the variables were available in the form of:

Code: Select all

S_GROUP_ID

Where ID was to be replaced with the id number of the group.
This format is still available but depreciated.

This extension can be used together with my extension Custom Code for inserting custom template code in a non destructive way.

How to determine the group ids?

Extension Version: 1.1.0
Requirements:

• phpBB 3.3+
• PHP 7.1+

Extension Download: https://github.com/marttiphpbb/phpbb-ex ... master.zip
The files are to be put in ext/marttiphpbb/grouptempvars
Github repository: https://github.com/marttiphpbb/phpbb-ext-grouptempvars
Languages: n/a
Templates: n/a (all)

[Mauron]

Good!

[uruguayito]

Interesting extension . Thanks.

Please, tell me: where I put the sentence that you refer to ? (Also I have installed your other extension)
(I don't understand how to do)

[martti]

Interesting extension . Thanks.

Please, tell me: where I put the sentence that you refer to ? (Also I have installed your other extension)
(I don't understand how to do)

In ACP > Extensions > Custom Code > Edit : choose one of the template event locations.

[Kailey]

This only grabs the user's primary group - not all groups a user is a member of. It's best to use the group_memberships() function.

Nice idea though.

[Lumpy Burgertushie]

grabbing all the groups a member is a member of is not needed. this is only for showing or hiding something to one specific group and it works perfectly for that.

actually, it doesn't grab any group IDs , you have to supply the group ID yourself when using it and depending on how you word the switch variable it can apply to multiple groups at once.

robert

[RMcGirr83]

Not meaning to hijack this post but been done already not that anyone would know.

[Kailey]

grabbing all the groups a member is a member of is not needed. this is only for showing or hiding something to one specific group and it works perfectly for that.

actually, it doesn't grab any group IDs , you have to supply the group ID yourself when using it and depending on how you word the switch variable it can apply to multiple groups at once.

I'm well aware of how this extension works. I think you might have missed my point though. If you look at the code, they are grabbing the group_id from the users table, which is used to designate the primary group. An example: I am a member of group 1, 2, and 5, with my primary group being group 1. Using this extension, if I try to display something to members of group 2 only (using an <!-- IF S_GROUP_2 --> switch), I would expect it to show for me also, but it won't because it doesn't take into account the user_group table. This is why the group_memberships() function would be a better fit.

Not meaning to hijack this post but been done already not that anyone would know.

I certainly didn't know.

[martti]

updated: pending groups are left out correctly now.

[martti]

I'm well aware of how this extension works. I think you might have missed my point though. If you look at the code, they are grabbing the group_id from the users table, which is used to designate the primary group. An example: I am a member of group 1, 2, and 5, with my primary group being group 1. Using this extension, if I try to display something to members of group 2 only (using an <!-- IF S_GROUP_2 --> switch), I would expect it to show for me also, but it won't because it doesn't take into account the user_group table. This is why the group_memberships() function would be a better fit.

Have a closer look and see it's the USER_GROUP_TABLE that's being fetched from.

Code: Select all

		$sql = 'SELECT group_id 
			FROM ' . USER_GROUP_TABLE . '
			WHERE user_id = ' . $user_id . '
				AND user_pending = 0';
		$result = $this->db->sql_query($sql);
		while ($group_id = $this->db->sql_fetchfield('group_id'))
		{
			$template_vars['S_GROUP_' . $group_id] = true;
		}

https://github.com/marttiphpbb/phpbb-ex ... hp#L61-L71

We don't want to use the group_memberships() function because it JOINs the USER_TABLE and the USER_GROUP_TABLE and fetches all the colums of the USER_GROUP_TABLE and the username, username_clean and user_email columns of the USER_TABLE. But we are simply after all the group ids of the current user. It doesn't make sense to fetch all that data just to throw it away.
https://github.com/phpbb/phpbb/blob/dc8 ... .php#L3495

Not meaning to hijack this post but been done already not that anyone would know.

I certainly didn't know.

I didn't either. But I like my solution without the group_memberships() function more.

Edit: replaced the link to group_memberships() with a permalink.

[martti]

ALL groups have a template variable. The rumours are not true.

Code: Select all

<!-- check group variables -->
<p>
<!-- IF S_GROUP_5 --> 
    You're in the admin group.
<!-- ELSE --> 
    You're NOT in the admin group.
<!-- ENDIF -->
</p>
<p>
<!-- IF S_GROUP_4 --> 
    You're in the global moderators group.
<!-- ELSE --> 
    You're NOT in the global moderators group.
<!-- ENDIF -->
</p>

Logged in as the board owner:

Logged in as a normal user:

[Mess]

Very sexy.

[Kailey]

Have a closer look and see it's the USER_GROUP_TABLE that's being fetched from

I swear it didn't say that earlier! I apologize then.

[mrgoldy]

Hello,

I was wondering if I could use something similar in the Pages extension. I created a page and parsed the HTML with this extension enabled. I had part in the page: <!-- IF S_GROUP_1 -->This is only visible for guests<!-- ENDIF--> However, all registered members and administrators could see it. Is there any chance this does not work in the html parsing of the Pages extension?

[martti]

Hello,

I was wondering if I could use something similar in the Pages extension. I created a page and parsed the HTML with this extension enabled. I had part in the page: <!-- IF S_GROUP_1 -->This is only visible for guests<!-- ENDIF--> However, all registered members and administrators could see it. Is there any chance this does not work in the html parsing of the Pages extension?

Do you see the tag <!-- IF S_GROUP_1-->? If so, then it is not working. If not, maybe your registered members and administrators also belong to the guests group?