Patch spotlight: daylight saving time for Drupal

Last modified: October 13, 2008 - 15:06


Help us test patches for core! This page will feature important core patches that need to be reviewed and tested. You can help with this important task even if you don't know how to code.

NOTE: This page just contains information about the current highest-priority core effort. For many more, check out Patch Spotlight 2.0

Currently in the spotlight: introduce daylight saving time for Drupal

#11077: Introduce Daylight Saving Time for Drupal
This patch adds support for daylight saving time, meaning users will get correct time representation in Drupal throughout the year.

For a synopsis of changes see: http://drupal.org/node/11077#comment-1053509

How to test this patch

Test the my account page, autodetection etc.

What do I do if I find a bug while testing a patch?

Try to reproduce the bug on a 'clean' copy of Drupal HEAD. For patches without database changes, you can "reverse" apply a patch to remove it:

patch -p0 -R < patchfile.patch

For patches that make changes to the database, you'll need a clean checkout elsewhere to compare against.

Check to see if the bug manifests itself without the patch. And note: don't forget to re-apply afterwards:

patch -p0 < patchfile.patch

If not, it's usually safe to assume it's caused by the current patch. Mark the issue "code needs work" and describe in detail what you experienced and how the developer can re-produce the steps.

If so, then there is a deeper problem with core... see if someone has already reported a bug on it by searching the issue queue. If not, feel free to file a bug.

General patch reviewing tips

Here is a list of general resources for applying and testing patches.

 
 

Drupal is a registered trademark of Dries Buytaert.