[drupal-docs] [bug] Ambiguity re: how to write header comment blocks
webchick
drupal-docs at drupal.org
Sun Sep 11 14:41:32 UTC 2005
Issue status update for
http://drupal.org/node/28251
Post a follow up:
http://drupal.org/project/comments/add/28251
Project: Documentation
Version: <none>
Component: Developer Guide
Category: bug reports
Priority: minor
Assigned to: Anonymous
Reported by: webchick
Updated by: webchick
-Status: active
+Status: fixed
Hey cool! Something I can actually do! :D Updated.
webchick
Previous comments:
------------------------------------------------------------------------
Fri, 05 Aug 2005 04:10:24 +0000 : webchick
The coding standards recommend two different methods of inserting header
comment blocks:
- http://drupal.org/node/318 (Coding standards page) recommends: /*
$Id$ */
- http://drupal.org/node/546 (Header comment blocks page) recommends:
// $Id$
I checked the files in core, and it seems that the // $Id$ notation is
more common, so is probably supposed to be the standard followed? If
that's the case, 318 should probably be updated to reflect this since
that appears on the surface at least to be an "all-in-one" document for
coding standards... I would guess that most users don't even hit the
second page. But in either case, it would be nice to know which method
is preferred.
------------------------------------------------------------------------
Fri, 12 Aug 2005 02:35:39 +0000 : webchick
I noticed today that all the files in core are now changed to the //
$Id$ style for consistency, so the fix for this issue would be:
http://drupal.org/node/318
Change:
<?php
/* $Id$ */
to:
<?php
// $Id$
and:
<?php
/* $Id: CODING_STANDARDS.html,v 1.4 2004/10/27 11:55:32 uwe Exp $ */
to:
<?php
// $Id: CODING_STANDARDS.html,v 1.4 2004/10/27 11:55:32 uwe Exp $
------------------------------------------------------------------------
Thu, 08 Sep 2005 14:56:08 +0000 : cel4145
It appears that the particular handbook page in question uses php to
pull the coding standards from a location in Drupal CVS:
http://cvs.drupal.org/viewcvs/*checkout*/drupal/contributions/CODING_STANDARDS.html
Could someone with CVS access implement this update to the coding
standards text?
More information about the drupal-docs
mailing list