[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