Block or expire one voucher code?

namezero111111

Sorry to bump this, I have the same issue where I'd like to expire a bunch of vouchers.

I thought there was a way in the GUI, but I can't seen to find it!

l.capriotti

still trying to find my way, please provide a working example.

namezero111111

Me too. Maybe it will be implemented in a future version.
You can expire a whole list, but it doesn't seem there is a way to expire a single code.

eri--

You need a client application to send a login request to authenticate and after that send a POST with the variable i said before set and the voucher you want to expire.
Normally wget with cookies can do this.

namezero111111

Ahh I get it now. Like a special PHP page where you enter the code and then say "Expire" it sends a request with that var set.
Do you know it that's complicated to integrate into the GUI? If not, I'd like to do it.
Essentially, I'd just need another tab on the CP tab and then enter a list a vouchers (like on the voucher check page), that then goes and posts the above mentioned commands to the portal page, no?

eri--

Normally there is a tab for testing vouchers and you can do there this option.

namezero111111

Yes, that's a great idea.I'll try to see if I can add a checkbox on the test voucher page for expiration.

l.capriotti

many tks for the hints, I was able to code an expire voucher page (copying the test voucher page and calling voucher_expire instead):


if ($_POST) {
    if ($_POST['vouchers']) {
        $result = voucher_expire($_POST['vouchers']);
        echo "

\n";
        if ( $result) {
            echo "";
            echo "";
        } else {
            echo "";
            echo "";
        }
        echo "

| ![](\"/themes/{$g['theme']}/images/icons/icon_pass.gif\") | Success |
| ![](\"/themes/{$g['theme']}/images/icons/icon_block.gif\") | Error |

";
    }
}

However on the portal auth log I see the following entries:


php[59817]: /status_captiveportal_expire.php: Captive portal Array passed as unset index: Array ( )
logportalauth[59817]: 37xN4hp7umY (123/68) forced to expire

and I tracked it down to the call:


captiveportal_write_db($cpdb, false, $unsetindexes);

in /etc/inc/voucher.inc

but I'm not able to get rid of that php error.

EDIT: attaching the patch related to 2.0.1-RELEASE

voucherExpire.diff.txt

elratt0r

The php error came from an invalid initialized array.
See attached patch.


diff --git a/etc/inc/voucher.inc b/etc/inc/voucher.inc
index ea723ca..a0455d2 100644
--- a/etc/inc/voucher.inc
+++ b/etc/inc/voucher.inc
@@ -217,7 +217,7 @@ function voucher_expire($voucher_received) {
 	// split into an array. Useful for multiple vouchers given
 	$a_vouchers_received = preg_split("/[\t\n\r ]+/s", $voucher_received); 
 	$active_dirty = false;
-	$unsetindexes[] = array();
+	$unsetindexes = array();

 	// go through all received vouchers, check their valid and extract
 	// Roll# and Ticket# using the external readvoucher binary

Kind regards
Michael

voucher_array.txt

bardelot

@elratt0r:

The php error came from an invalid initialized array.
See attached patch.

Kind regards
Michael

Thanks, made a pull request on github.