[BUG 4.7.8] Debug code left inside a try/catch when viewing a topic without the proper permissions

[teraßyte]

The code I'm talking about is located in /applications/forums/modules/front/forums/topic.php on line 1290:

		catch ( \Exception $e )
		{
			\IPS\Log::log( $e, 'topic_call' );
			\IPS\Output::i()->error( 'node_error', '2F173/I', 404, '' );
		}

The \IPS\Log::log() call is executed when a user without permission tries to view a topic.

 

I found it only because I noticed a topic_call category while viewing the System Logs in ACP.

I think it was left there by mistake when debugging something. It should be removed or changed to use debug() instead.

[Marc Stridgen]

Thank you for bringing this issue to our attention! I can confirm this should be further reviewed and I have logged an internal bug report for our development team to investigate and address as necessary, in a future maintenance release.

 

[Marc Stridgen]

We have taken a look at this, and this is actually intentional in this case

[teraßyte]

I...see? Guess we'll just keep deleting these useless (for us) system logs. 🤨