Very simple PHP code, checking if user is the author in page.tpl.php

By Manamarak on 30 Aug 2014 at 21:51 UTC

I need to write a very simple piece of PHP code.
All I need is in a page-content_type.tpl.php in Drupal 6 add a simple if command that only calls the tabs variable IF the user is the author of the node.

I think it should look something like this:

<?php if [THE VIEWER IS THE AUTHOR OF THE NODE] ?>
            <?php if ($tabs): ?>
              <div class="tabs"><?php print $tabs; ?></div>
            <?php endif; ?>
<?php endif; ?>

I'm hoping I can use $classes>node_by_viewer to achieve this.

Could anyone tell me if I'm going to bang my head against a wall all evening, or if this is actually possible, please?

Comments

The tabs are only shown if

nevets commented 30 August 2014 at 22:07

The tabs are only shown if the person viewing the content has access to one of the other tabs soI have to wonder what/who you are trying to block access to.

Module quirk

Manamarak commented 30 August 2014 at 23:18

Thank you for your comment.
In this particular case, it is a module that is only allowing me the choice of either
-Users can edit all (types of nodes created by module)
-Users can edit only own (types of node created by module)
-Users cannot edit.

But I have one content type created by this module which is shared and everyone must be able to edit it, and another which only the author must be able to edit.

I would like to - as a quick workaround until I can fix the problem properly - find a way of at least stopping users seeing that they have access to editing other people's private nodes.

_

WorldFallz commented 2 September 2014 at 14:08

Simply hiding the tabs via a template file is a bad way to approach this-- it won't actually prevent the path from being accessed so anyone familiar with the paths on your site will be able to easily circumvent it.

That said, the page.tpl.php file should have the $node variable which means you can check the author of a node using $node->uid. However, you should really do this properly either with a module like content_access or with hook_menu_alter.

Thank you, WorldFallz. I

Manamarak commented 3 September 2014 at 12:32

Thank you, WorldFallz. I realise it is quick and dirty. I think I'm leaning towards an even worse solution - hiding the links via CSS.

I know it is not perfect, but I'm not worried so much about malicious behaviour as I am about ineptitude and a site steering people in the wrong direction.

In the longer term, I'm starting looking at getting someone with actual knowledge of coding to maintain the site. Suggestions of someone on an hourly rate welcome.