- (API) Refactored all plugin classes to use the new getName and getId methods.

- (API) Added getName and getId methods to the PluginInterface class

Author: objectivehtml

CHANGELOG.md

@@ -13,6 +13,8 @@
 - (Feature) Added new Sent Messages dashboard widget to view sent messages statistics right from the Craft dashboard
 - (Feature) Added new notification for user inactivity like login reminders and reminders to change passwords
 - (Feature) Added Expired Entries notification type that will notify users after entries are expired.
+- (Feature) Added ability to create a front-end form that will trigger the "Front-end Email Form" parcel type to send emails dynamically from users
+- (API) Added new craft.postmaster.email template tag to return a new Postmaster_EmailModel
 - (API) Added new PostmasterHelper class to keep code dry that need to perform misc common functions to validate data
 - (API) Refactored BasePlugin class to make the settings property public.
 - (API) Refactored template fields into a new Twig Macro for convenience
@@ -22,6 +24,7 @@
 - (Bug Fix) Fixed a link on the notification index page that was going to the wrong url
 - (Bug Fix) Fixed an issue with templates not translating text strings properly
 - (Bug Fix) Fixed an issue with the Extra Conditionals not working in the default notification type
+- (Bug Fix) Fixed an issue with the templates path being wrong. Replace with the CRAFT_TEMPLATES_PATH constant.
 
 #### 0.4.2
 ##### 12/16/2014

PostmasterPlugin.php

@@ -10,7 +10,7 @@ public function getName()
 
     public function getVersion()
     {
-        return '0.4.2.1';
+        return '0.5.0';
     }
 
     public function getDeveloper()

core/components/BaseNotificationType.php

@@ -8,10 +8,6 @@
 
 abstract class BaseNotificationType extends BasePlugin implements NotificationTypeInterface {
 
-	public $name;
-
-	public $id;
-
 	protected $service;
 
 	protected $notification;

core/components/BaseParcelType.php

@@ -8,10 +8,6 @@
 
 abstract class BaseParcelType extends BasePlugin implements ParcelTypeInterface {
 
-	public $name;
-
-	public $id;
-
 	protected $service;
 
 	protected $parcel;

core/components/BasePlugin.php

@@ -41,7 +41,7 @@ public function registerCpRoutes()
 
 	public function is($id)
 	{
-		if($this->id == $id || $this->name == $id)
+		if($this->getId() == $id || $this->getName() == $id)
 		{
 			return true;
 		}

core/components/BaseSchedule.php

@@ -8,10 +8,6 @@
 
 abstract class BaseSchedule extends BasePlugin implements ScheduleInterface {
 
-	public $name;
-
-	public $id;
-
 	public function onBeforeSend(Postmaster_TransportModel $model)
 	{
 		return true;

core/components/BaseService.php

@@ -8,10 +8,6 @@
 
 abstract class BaseService extends BasePlugin implements ServiceInterface {
 
-	public $name;
-
-	public $id;
-
 	protected $requireModels = array();
 
 	public function getInputHtml(Array $data = array())

core/interfaces/PluginInterface.php

@@ -3,6 +3,10 @@
 
 interface PluginInterface {
 
+	public function getName();
+
+	public function getId();
+
 	public function init();
 
 	public function is($id);

core/notification_schedules/AnyTimeNotificationSchedule.php

@@ -1,23 +1,24 @@
 <?php
 namespace Craft\Plugins\Postmaster\NotificationSchedules;
 
-use Carbon\Carbon;
-use Craft\Postmaster_TransportModel;
+use Craft\Craft;
 use Craft\Plugins\Postmaster\Components\BaseNotificationSchedule;
 
 class AnyTimeNotificationSchedule extends BaseNotificationSchedule {
 
-	public $name = 'Any Time';
-
-	public $id = 'anytime';
-
-	protected $now;
-
 	public function __construct($attributes = null)
 	{
 		parent::__construct($attributes);
+	}
 
-		$this->now = Carbon::now(new \DateTimeZone(\Craft\craft()->getTimezone()));
+	public function getName()
+	{
+		return Craft::t('Any Time');
+	}
+
+	public function getId()
+	{
+		return 'anytime';
 	}
 
 	public function getSettingsModelClassName()

core/notification_schedules/DefaultNotificationSchedule.php

@@ -2,15 +2,12 @@
 namespace Craft\Plugins\Postmaster\NotificationSchedules;
 
 use Carbon\Carbon;
+use Craft\Craft;
 use Craft\Postmaster_TransportModel;
 use Craft\Plugins\Postmaster\Components\BaseNotificationSchedule;
 
 class DefaultNotificationSchedule extends BaseNotificationSchedule {
 
-	public $name = 'Basic Reoccuring';
-
-	public $id = 'default';
-
 	protected $now;
 
 	public function __construct($attributes = null)
@@ -20,6 +17,16 @@ public function __construct($attributes = null)
 		$this->now = Carbon::now(new \DateTimeZone(\Craft\craft()->getTimezone()));
 	}
 
+    public function getName()
+    {
+        return Craft::t('Basic Reoccurring');
+    }
+
+    public function getId()
+    {
+        return 'default';
+    }
+
 	public function shouldSend($lastSent = false)
 	{
 		if(!$this->hasElapsedTimePassed($lastSent))