[drupal-devel] [bug] drupal_set_message docs should be updated
puregin
puregin at puregin.org
Fri May 13 07:01:40 UTC 2005
Hi, I am working on this at this very moment :)
I would like to submit my proposed guidelines to
the doc list for discussion prior to submitting as a patch,
so please go ahead and apply Stefan's patch
now if that makes sense.
Djun
On 12 May 2005, at 11:32 PM, stefan nagtegaal wrote:
> Issue status update for http://drupal.org/node/22652
>
> Project: Drupal
> Version: cvs
> Component: base system
> Category: bug reports
> Priority: normal
> Assigned to: Anonymous
> Reported by: stefan nagtegaal
> Updated by: stefan nagtegaal
> Status: patch
>
> I did not see a patch of Djun yet.. Maybe it's better to let him build
> hi patch on top of this one?
>
>
> Some documentation and guidelines are better than none at all...
>
>
>
>
> stefan nagtegaal
>
>
>
> Previous comments:
> -----------------------------------------------------------------------
> -
>
> May 12, 2005 - 10:32 : stefan nagtegaal
>
> I updated the Doxygen comments of drupal_set_message() so it reflects
> the latest changes and we can keep the messages consistent..
>
>
> As noted in the Doxygen comments, we *always* begin a message with a
> Capital letter, and end the message with a period ('.').
>
>
> I hope one of you guys can spellcheck the documentation for me,
> because... Well, you probably know English isn't my strongest point..
> ;-)
>
>
> Please review, update/comment and apply...
>
>
>
>
> -----------------------------------------------------------------------
> -
>
> May 12, 2005 - 10:32 : stefan nagtegaal
>
> Attachment: http://drupal.org/files/issues/drupal_set_message.patch
> (1.26 KB)
>
> And, unfortunatly, I forgot the patch... :-(
>
>
>
>
> -----------------------------------------------------------------------
> -
>
> May 12, 2005 - 16:42 : Dries
>
> I believe Djun had some additional guidelines.
>
More information about the drupal-devel
mailing list