Various (minor) style issues

Freso - May 3, 2008 - 09:37
Project:Pathauto
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I tried to get a look at the module to implement #247758: Use Transliteration module for transliteration, but there were too many wee issues that made me itch. So I swept over the code and fixed what I could find. This includes

  • Excessive white-space (including (but possibly not limited to) trailing white-space, double new-lines, double spaces in strings (and comments))
  • Double quoted strings where they might as well have been single quoted
  • Arrays not conforming to the Drupal coding standards

Not that some of these were also done one way some places and another way other places (like using double spaces in strings after periods), so this patch should also provide some more consistency within the module's code.

AttachmentSize
pathauto_style.d6.patch32.29 KB

#1

greggles - May 4, 2008 - 22:03
Status:needs review» fixed

Applied to HEAD and DRUPAL-5--2. Thanks, Freso!

(I also removed the two files from the contrib/ directory since they are no longer necessary...).

#2

Anonymous (not verified) - May 18, 2008 - 22:12
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.