Skip to content
Browse files

Broadband - do not error if no MAC provided

  • Loading branch information...
1 parent b655027 commit c7a65b9c7b72b35a8cc279b3a1c691c19ff5219c @ukfsn ukfsn committed Aug 27, 2010
Showing with 3 additions and 4 deletions.
  1. +3 −3 Kirin/Plugin/Broadband.pm
  2. +0 −1 templates/plugins/broadband/get-clid
View
6 Kirin/Plugin/Broadband.pm
@@ -24,12 +24,12 @@ sub order {
my $mac = uc $mm->param("mac");
my $stage = $mm->param("stage");
goto "stage_$stage" if $stage;
-
+
stage_1:
if (!$clid) {
return $mm->respond("plugins/broadband/get-clid");
}
- if (defined $mac and $mac !~ MAC_RE) {
+ if ($mac and $mac !~ MAC_RE) {
$mm->message("That MAC was not well-formed; please check.");
return $mm->respond("plugins/broadband/get-clid");
}
@@ -99,7 +99,6 @@ sub request_mac {
broadband => $bb,
timestamp => Time::Piece->new(),
class => "mac",
- token => $out,
description => "Request for MAC"
});
$mm->respond("plugins/broadband/mac-requested",
@@ -248,6 +247,7 @@ sub provider_handle {
user => Kirin->args->{"${p}_username"},
pass => Kirin->args->{"${p}_password"},
clientid => Kirin->args->{"${p}_clientid"},
+ debug => 1, # XXX
});
}
View
1 templates/plugins/broadband/get-clid
@@ -1,7 +1,6 @@
<h2>Broadband</h2>
<p>
-You do not currently have a broadband connection with us.
In order to see which services we can supply to you, please enter your
telephone number here:
</p>

0 comments on commit c7a65b9

Please sign in to comment.
Something went wrong with that request. Please try again.