Follow Coding Standards

quicksketch - May 17, 2008 - 17:11
Project:MultiBlock
Version:5.x-1.0
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

My company is working on a large project for which we'll likely use Multiblock as a back-end for some advanced block management. I was really pleased to see such a nice module already in place for Drupal 5. We're targeting for D6 now, and we might need to add a little bit of functionality to make this work perfectly. However, "multiblock" is exactly what we need (multiple instances of the same block).

I'd like to start rolling a lot of fixes for this module, but before doing so I'd like to do a simple update to bring the module up to use the Drupal coding standards. The module is already really well formatted except for comments. This brings the module up to date with the following 2 changes:

- Comments start with a capital letter and end with a period.
- Function documentation uses the PHPdoc comment format, with two stars opening the comment block (/**).

#1

quicksketch - May 17, 2008 - 17:12
AttachmentSize
multiblock-style.patch 10.28 KB

#2

andrewlevine - May 18, 2008 - 00:46
Status:needs review» fixed

Glad someone else is finding a use for this module. I really do appreciate the patch. I committed it as is besides one typo fix to the DRUPAL-5 branch.

I'm not as familiar with D6 as I'd like to be, but I'll do the best I can with testing if you want to port to it. Let me know if there's anything you need from me.

#3

Anonymous (not verified) - June 1, 2008 - 00:52
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.