Cleanup code formatting in simpletestauto module

allisterbeharry - December 21, 2007 - 15:09
Project:Simpletest automation
Version:HEAD
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:active
Description

Reviewing the simpletestauto module with the coder module reveals two minor formatting issues - spacing when doing string concatenation with the . operator, and a slightly more major issue: camel-case names. In the simpletestauto.module file the camel-case names only occur in XML-RPC handlers. The attached patch normalizes the white-spacing in simpletestauto.module, enable_module.inc, and install_drupal.inc and converts camel-case function names in simpletestauto.module - it doesn't change any names in the other two files as these are unit test class files, functionally separate from Drupal, and follow standard naming conventions including those from SimpleTest. This patch is for the version of simpletestauto in SVN on t.d.o. revision 19.

AttachmentSize
simpletestauto_formatting.patch17.12 KB

#1

allisterbeharry - December 23, 2007 - 20:34

This is a standalone patch for the simpletestauto_server.module file.

AttachmentSize
simpletestauto_server_formatting.patch 3.41 KB

#2

boombatower - October 31, 2008 - 07:39
Project:Test driven development infrastructure» Simpletest automation
Version:<none>» HEAD
Assigned to:allisterbeharry» Anonymous
Status:needs review» active
 
 

Drupal is a registered trademark of Dries Buytaert.