The box can be unchecked but the author information always shows

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

komal.savla’s picture

Hi,
In node.tpl.php file of Rubik theme replace the following code :

 <?php if (!empty($submitted)): ?>
    <div class='<?php print $hook ?>-submitted clearfix'><?php print $submitted ?></div>
  <?php endif; ?>

With this :

 <?php if (!empty($display_submitted)): ?> // Modified this line
    <div class='<?php print $hook ?>-submitted clearfix'><?php print $submitted ?></div>
  <?php endif; ?>

Thanks,
Komal

tommyent’s picture

Thanks. I just wasn't sure if it was overlooked.

CoffeyMachine’s picture

Status: Active » Fixed

I believe this is actually an issue with the base theme Tao. There is a fix for that in the Tao issue queue which has been committed to the dev branch. #932744: Hide title in node pages.
Try updating Tao and let me know if you still have the problem.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Johann Wagner’s picture

Version: 7.x-4.0-beta5 » 7.x-4.0-beta8

Hello,

Rubik used as administration theme actually messed up apache solr search by providing hidden data : author and publication date.

Here's the scoundrel in template.php :

/**
 * Preprocessor for theme('node').
 */
function rubik_preprocess_node(&$vars) {
  $vars['layout'] = TRUE;
  $vars['submitted'] = _rubik_submitted($vars['node']);
}

I suggest to check $vars['display_submitted'] before injecting the submitted information.

Have a good day.

hanoii’s picture

Version: 7.x-4.0-beta8 » 7.x-4.x-dev
Status: Closed (fixed) » Needs review
FileSize
795 bytes

I believe the theme should honor node-type settings, at least by default. I can understand a "by design" statement that an admin theme should always display post information, but if that's the case, let's document it somehow. Attached is the patch if someone is interested enough.

hanoii’s picture

Status: Needs review » Needs work
FileSize
417 bytes

Sorry, patch is not great, it should properly need to be the efollowing, but still needs to work to hide the column side altogether if nothing is there.

haydeniv’s picture

Issue summary: View changes
Status: Needs work » Fixed

I don't have a problem with the column still showing at this point. This is an admin theme and you really should not be using it to display node views. This fixes the problem with the search index so I think this is fixed.

Committed: 8cd1e43

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.