- Code: Select all
--- ojs-2.2.2_clean/classes/mail/ArticleMailTemplate.inc.php 2008-06-30 21:16:10.000000000 -0400
+++ ojs-2.2.2/classes/mail/ArticleMailTemplate.inc.php 2008-11-12 15:18:50.000000000 -0500
@@ -40,7 +40,7 @@
/**
* Constructor.
* @param $article object
- * @param $emailType int optional
+ * @param $emailKey string unique identifier for the template
* @param $locale string optional
* @param $enableAttachments boolean optional
* @param $journal object optional
[PATCH] Function docs for constructor of MailArticleTemplate
Moderators: jmacgreg, michael, jheckman, barbarah, btbell, bdgregg, asmecher
Forum rules
Developer Resources:
Documentation: The OJS Technical Reference and the OJS API Reference are both available from the OJS Documentation page.
Git: You can access our public Git Repository here. Comprehensive Git usage instructions are available on the wiki.
Bugzilla: You can access our Bugzilla report tracker here.
Search: You can use our Google Custom Search to search across our main website, the support forum, and Bugzilla.
Questions and discussion are welcome, but if you have a workflow or usability question you should probably post to the OJS Editorial Support and Discussion subforum; if you have a technical support question, try the OJS Technical Support subforum.
Developer Resources:
Documentation: The OJS Technical Reference and the OJS API Reference are both available from the OJS Documentation page.
Git: You can access our public Git Repository here. Comprehensive Git usage instructions are available on the wiki.
Bugzilla: You can access our Bugzilla report tracker here.
Search: You can use our Google Custom Search to search across our main website, the support forum, and Bugzilla.
Questions and discussion are welcome, but if you have a workflow or usability question you should probably post to the OJS Editorial Support and Discussion subforum; if you have a technical support question, try the OJS Technical Support subforum.
2 posts
• Page 1 of 1
[PATCH] Function docs for constructor of MailArticleTemplate
by rmichael » Wed Nov 12, 2008 1:35 pm
Here's a tiny one liner to fix the docs in the constructor of MailArticleTemplate. It should be the same as the parent, MailTemplate. It's of interest to me because otherwise, the doxygen generated docs are wrong and I use them quite a bit. Also, it's misleading because it's not an int, it's a string .. the template name!
Regards,
Richard
Richard
- rmichael
- Posts: 113
- Joined: Fri Mar 30, 2007 3:32 pm
Re: [PATCH] Function docs for constructor of MailArticleTemplate
by mcrider » Wed Nov 12, 2008 2:21 pm
Hi Richard,
Thanks, I'll make the change in CVS. Glad to hear people are using the Doxygen docs!
Cheers,
Matt
Thanks, I'll make the change in CVS. Glad to hear people are using the Doxygen docs!
Cheers,
Matt
Matthew Crider
Public Knowledge Project Team
Public Knowledge Project Team
- mcrider
- Posts: 952
- Joined: Mon May 05, 2008 10:29 am
- Location: Vancouver, BC
2 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 2 guests