[drupal-devel] [bug] drupal_set_message docs should be updated
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 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... stefan nagtegaal
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 Attachment: http://drupal.org/files/issues/drupal_set_message.patch (1.26 KB) And, unfortunatly, I forgot the patch... :-( 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...
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: Dries Status: patch I believe Djun had some additional guidelines. Dries 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... :-(
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.
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.
participants (3)
-
Dries -
puregin -
stefan nagtegaal