You are viewing the PKP Support Forum | PKP Home Wiki

[PATCH] Function docs for constructor of MailArticleTemplate

OJS development discussion, enhancement requests, third-party patches and plug-ins.

Moderators: jmacgreg, btbell, michael, bdgregg, barbarah, 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.

[PATCH] Function docs for constructor of MailArticleTemplate

Postby 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!

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
Posts: 113
Joined: Fri Mar 30, 2007 3:32 pm

Re: [PATCH] Function docs for constructor of MailArticleTemplate

Postby 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!

Posts: 952
Joined: Mon May 05, 2008 10:29 am
Location: Vancouver, BC

Return to OJS Development

Who is online

Users browsing this forum: No registered users and 1 guest