Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Was at a loose end this evening, so I've gone through the entire codebase removing all trailing whitespace.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | whitespace_cleanup-1371122-3-D6.patch | 187.4 KB | sammyd56 |
| #1 | whitespace_cleanup-1371122-1.patch | 81.71 KB | sammyd56 |
Comments
Comment #1
sammyd56 CreditAttribution: sammyd56 commentedComment #2
dawehnerGreat! Thanks for your contributions!
I had some fears about whitespaces in regions which are changed by other patches, but as these whitespace fixes
are mostly in css files and html files, which aren't changed that often this shouldn't be a problem at all.
Committed to 7.x-3.x
I would be happy if you would like to backport these changes to 6.x-3.x as well. Do you use some kind of automatic tool for that?
Comment #3
sammyd56 CreditAttribution: sammyd56 commentedI have a gedit plugin that removes trailing whitespace when saving files.
It's really handy, but also a bit annoying, because when you make a patch to a file that already contained whitespace you end up including unrelated whitespace fixes and have to remove them from the patch.
So, I figured I could save people some time by opening all the files one by one and resaving them, thus stripping the whitespace.
Being a PHP novice I can't help with real coding at the moment, but at least I can help with something and show my appreciation to Drupal developers.
Here's a patch for 6.x-3.x.
Comment #4
dawehnerYour work is highly welcomed!
You know 2.5 years ago i started with patches like that and now i have the pleasure to help out with this module on daily base.
There will be no fixed for 6.x-2.x anymore, because it's a REAL stable version :)