lreminder: use URI instead of File::Basename

[lugs.git] / lreminder / reminder.pl

diff --git a/lreminder/reminder.pl b/lreminder/reminder.pl
index 4ecc1552b0df69aeaba085b3a6e3e24e2d8f6954..81d898ef98f75362215bcc0e298381a3175324c1 100755 (executable)
--- a/lreminder/reminder.pl
+++ b/lreminder/reminder.pl
@@ -15,7 +15,7 @@
 # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 #
 # Author: Steven Schubiger <stsc@refcnt.org>
-# Last modified: Mon Jan 13 22:07:51 CET 2014
+# Last modified: Wed Jul 16 16:17:32 CEST 2014
 
 use strict;
 use warnings;
@@ -34,9 +34,10 @@ use Hook::Output::File ();
 use LUGS::Events::Parser ();
 use Mail::Sendmail qw(sendmail);
 use Text::Wrap::Smart::XS qw(fuzzy_wrap);
+use URI ();
 use WWW::Mechanize ();
 
-my $VERSION = '0.43';
+my $VERSION = '0.45';
 
 #-----------------------
 # Start of configuration
@@ -56,7 +57,7 @@ my $Config = {
 #---------------------
 
 my $dbh  = DBI->connect("dbi:mysql(RaiseError=>1):$Config->{dbase_name}", $Config->{dbase_user}, $Config->{dbase_pass});
-my $file = File::Basename::basename($Config->{events_url});
+my $file = (URI->new($Config->{events_url})->path_segments)[-1];
 
 my ($test, $run) = (false, false);
 
@@ -90,7 +91,7 @@ sub fetch_and_write_events
     my $mech = WWW::Mechanize->new;
     my $http = $mech->get($Config->{events_url});
 
-    open(my $fh, '>', $file) or die "Cannot open $file: $!\n";
+    open(my $fh, '>', $file) or die "Cannot open $file for writing: $!\n";
     print {$fh} $http->content;
     close($fh);
 }
@@ -207,7 +208,7 @@ MSG
             To      => $mail_subscriber,
             Subject => encode('MIME-Q', "LUGS Reminder - $title"),
             Message => $message,
-        ) or die $Mail::Sendmail::error;
+        ) or die "Cannot send mail: $Mail::Sendmail::error";
     }
     elsif ($test) {
         printf "[%s] <$mail_subscriber> ($color)\n", scalar localtime;