Removing the report button (Page 1 of 2)

So your saying instead of doing the source edits, use that to prevent it from displaying. I would prefer to rip it out from the source so that the server isn`t doing the work and then the CSS is hiding it.

Yeah I`d prefer to remove the function, do my edits do that or did I miss something?

Thank you so much Strofanto. I think you mean lines 164 to 243

To sum up for anyone wanting to completly strip the report function to save the server crunching it when it`s not going to be displayed, here`s what you need to do:

1. Remove the function.

Open misc.php and delete or comment out lines 164 to 243.

2. Remove the actual link.

viewtopic.php

FIND:

$post_actions[] = '<li class="postreport"><a href="misc.php?report='.$cur_post['id'].'">'.$lang_topic['Report'].'</a>';

REPLACE WITH:

$post_actions[] = '';

FIND:

$post_actions[] = '<li class="postreport"><a href="misc.php?report='.$cur_post['id'].'">'.$lang_topic['Report'].'</a>'.$lang_topic['Link separator'].'</li><li class="postdelete"><a href="delete.php?id='.$cur_post['id'].'">'.$lang_topic['Delete'].'</a>'.$lang_topic['Link separator'].'</li><li class="postedit"><a href="edit.php?id='.$cur_post['id'].'">'.$lang_topic['Edit'].'</a>'.$lang_topic['Link separator'].'</li><li class="postquote"><a href="post.php?tid='.$id.'&qid='.$cur_post['id'].'">'.$lang_topic['Quote'].'</a>';

REPLACE WITH:

$post_actions[] = '<li class="postdelete"><a href="delete.php?id='.$cur_post['id'].'">'.$lang_topic['Delete'].'</a>'.$lang_topic['Link separator'].'</li><li class="postedit"><a href="edit.php?id='.$cur_post['id'].'">'.$lang_topic['Edit'].'</a>'.$lang_topic['Link separator'].'</li><li class="postquote"><a href="post.php?tid='.$id.'&qid='.$cur_post['id'].'">'.$lang_topic['Quote'].'</a>';

FIND:

<div class="postfootright"><?php echo (count($post_actions)) ? '<ul>'.implode($lang_topic['Link separator'].'</li>', $post_actions).'</li></ul></div>'."\n" : '<div> </div></div>'."\n" ?>

REPLACE WITH:

<div class="postfootright"><?php echo (count($post_actions)) ? '<ul>'.implode($post_actions).'</li></ul></div>'."\n" : '<div> </div></div>'."\n" ?>

That`s it Once again thanks to Strofanto for advise on removing the function.

I'd still go for the CSS solution but it's just me.

Actually, isn't there an admin option to disable reporting?

I believe there's only an option to decide how to report (internal, email or both).

The changes you have posted so far in this thread would have absolutely no effect on performance. Also, some of the changes doesn't even make sense. Why would you change:

$post_actions[] = '<li class="postreport"><a href="misc.php?report='.$cur_post['id'].'">'.$lang_topic['Report'].'</a>';

into:

$post_actions[] = '';

?

Comment it out, don't just set it to null.

That`s pointless. If I`m not using the button (because I`m not getting the reports) then there`s no need for the button to be there.

The button is removed so the server doesn`t have to display it, thus the server has less to do improving performance oh so very very slightly.

I`ve already said I`m no coder, that`s the only safe way I know to remove the button with out breaking any code. Which is why I post here asking "is there a cleaner way than this?"

I`m open to suggestions how to do this, that`s why I posted in the first place. So how would you remove the report button from the code?

Thanks.

More than "absolutely no effect" then Seriously though, can we get back ontopic here. Performance it errelevant.. I just want to remove the link, if you know a better way than what I have done then please post it.

Yes it will but you`ve still got the extra pole next to it and besides the aim is to stop pun proccessing the report function not just hide it

Once again I recommend to do it through CSS, it will be just one line in your base.css file and it will avoid source code alteration, which is handy if you have to update your forum.