Jump to content

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

Featured Replies

Posted
  • Community Expert

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.

Solved by Marc

Go to solution
  • Community Expert

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.

 

  • 2 months later...
  • Community Expert
  • Solution

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

  • Author
  • Community Expert

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

Guest
Reply to this topic...

Recently Browsing 0

  • No registered users viewing this page.