First, I want to thank those who have updated the module to avoid issues with PHP 5.4 for example.

There are a few presentation errors that have been there for a while. I presume most event.module users correct them by hand, but here goes.

I include a snapshot which should help see the issue ...

In both Month and Week tables, the date is presented with the time just before the event name, but the date shown there is redundant: these tables already show the month name and the day on the tables. (see the image)

In the Week table, the code results in an odd (erroneous) presentation :
it attempts to show the node Summary and other links, but this is a small weekly summary. They have no place there.

The following files should be patched as follows :
event-node-month.tpl.php
event-node-week.tpl.php

- <?php print t('Start: ') . $start_date_formatted .' '. $start_time_formatted ?>
+ <?php print t('Start: ') . $start_time_formatted ?>

- <?php print t('End: ') . $end_date_formatted .' '. $end_time_formatted ?>
+ <?php print t('End: ') . $end_time_formatted ?>

event-node-week.tpl.php needs the following lines removed :

- <div class="content description"><?php print $teaser ?></div> 
- <div class="links"><?php print $links ?></div> 
- <div class="links"><?php print $calendarsignup ?></div>

One last oddity. The file event-node-month.tpl.php names the CCS class "title summary", but the class "summary" is irrelevant for this table and does not exist in event.css. It has no effect, but might confuse some webmasters.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Michael_Lessard_micles.biz’s picture

Issue summary: View changes
Michael_Lessard_micles.biz’s picture

Issue summary: View changes
Michael_Lessard_micles.biz’s picture

Issue summary: View changes
FileSize
38.71 KB
Michael_Lessard_micles.biz’s picture

Component: Miscellaneous » Code
japerry’s picture

Status: Active » Closed (outdated)

Event for Drupal 8 is unrelated to older versions. If an issue similar to this one exists, please open a new issue with the 8.x branch.