How to get forum_id from topic_id?

[pierredu]

In one of my extensions, permission is given to use some bbcode only if the user is moderator of the forum.
When creating a topic, there is a f parameter on the command line.
When creating an answer or editing a message, there is only a t parameter.

It is not difficult to get the forum_id from the topic_id with a SQL query, but I'm quite sure that there is already a function in the core phpBB code, but I couldn't fond it.

Thanks in advance.

[Brf]

Near the top of viewtopic, it is getting forum_id by reading the topics table. If there were a function dedicated to that it would probably use that instead:

Code: Select all

		$sql = 'SELECT forum_id
			FROM ' . TOPICS_TABLE . "
			WHERE topic_id = $topic_id";
		$result = $db->sql_query($sql);
		$forum_id = (int) $db->sql_fetchfield('forum_id');
		$db->sql_freeresult($result);

		if (!$forum_id)
		{
			trigger_error('NO_TOPIC');
		}

[pierredu]

Thanks for the pointer.

[3Di]

In one of my extensions, permission is given to use some bbcode only if the user is moderator of the forum.
When creating a topic, there is a f parameter on the command line.
When creating an answer or editing a message, there is only a t parameter.

I tend to avoid extra queries if possible, a link to that part of code?

[pierredu]

https://github.com/pierrdu/lmdi_hidebbc ... stener.php
lines 56 & ss.

[3Di]

https://github.com/pierrdu/lmdi_hidebbc ... stener.php
lines 56 & ss.

Exact link https://github.com/pierrdu/lmdi_hidebbc ... er.php#L56

Are you still using request_var ?

Anyway, I will take a look at this later on.

[3Di]

In one of my extensions, permission is given to use some bbcode only if the user is moderator of the forum.
When creating a topic, there is a f parameter on the command line.
When creating an answer or editing a message, there is only a t parameter.

The first thing that comes to mind when looking at the event you're using is that you could be using custom forum permissions instead of the query. The second is that there are more effective methods using the s9e utilities for this. The latter would require a substantial rewrite of the code.

[pierredu]

Thanks for your remarks.