Changeset 5269 for branches/Release01x00/MailerContrib/lib/Foswiki/Contrib/MailerContrib/UpData.pm

Timestamp:

10/15/09 19:28:59 (3 years ago)

Author:

KennethLavrsen

Message:

Item2260: MailerContrib cannot find email templates based on skin or web
The checkin does the following

• It makes a tiny little fix in MailerContrib.pm where I pushTopicContext before loading mail template and pop it again afterwards
• Updates the the documentation to show by example how to tailor the mail message
• Merge over the code from trunk. This introduces a bug fix from Item1603 which for some reason was only applied to trunk
• And with this merge also merge over a lot of perl tinying.

It was too difficult to separate the two once I was this far. Look at next checkin in trunk to see only the actual code changes

File:

• branches/Release01x00/MailerContrib/lib/Foswiki/Contrib/MailerContrib/UpData.pm (modified) (2 diffs)

branches/Release01x00/MailerContrib/lib/Foswiki/Contrib/MailerContrib/UpData.pm

@@ -43 +43 @@
     my ( $class, $session, $web ) = @_;
     my $this = bless( {}, $class );
-    $this->{web} = $web;
+    $this->{web}     = $web;
     $this->{session} = $session;
     return $this;
@@ -58 +58 @@
     my ( $this, $topic ) = @_;
 
-    if ( ! defined( $this->{parent}{$topic} )) {
-        my( $meta, $text ) =
-          Foswiki::Func::readTopic( $this->{web}, $topic );
+    if ( !defined( $this->{parent}{$topic} ) ) {
+        my ( $meta, $text ) = Foswiki::Func::readTopic( $this->{web}, $topic );
         my $parent = $meta->get('TOPICPARENT');
         $this->{parent}{$topic} = $parent->{name} if $parent;