Project

General

Profile

Bug #1106

Error in boot process

Added by Yehuda Katz over 8 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Captive Portal
Target version:
Start date:
12/15/2010
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.0
Affected Architecture:

Description

I am not sure how important this is, but I saw this error flash by during boot:

Enabling voucher support...
Warning: Invalid argument supplied for foreach() in /etc/inc.voucher.inc on line 298
done

2.0-BETA4 (i386)
built on Wed Dec 15 07:09:54 EST 2010

Associated revisions

Revision 451e4a05 (diff)
Added by Erik Fonnesbeck over 8 years ago

Fix condition that needed to be negated after a recent layout change here and prevent a PHP warning if there are no voucher rolls. Fixes #1106

History

#1 Updated by Erik Fonnesbeck over 8 years ago

Looking at the source code, it looks like that probably means you don't have any voucher rolls yet. However, this is still a bug that needs to be fixed.

#2 Updated by Chris Buechler over 8 years ago

  • Category set to Captive Portal
  • Target version set to 2.0

#3 Updated by Yehuda Katz over 8 years ago

Erik Fonnesbeck wrote:

Looking at the source code, it looks like that probably means you don't have any voucher rolls yet. However, this is still a bug that needs to be fixed.

This is on a brand new install, I don't have anything yet.

#4 Updated by Erik Fonnesbeck over 8 years ago

I found that there was a change yesterday that unintentionally made it so that code would run when vouchers are disabled (and won't run when they are enabled). The case I mentioned is still something that could happen if vouchers were enabled and no voucher rolls were created yet, though.

#5 Updated by Erik Fonnesbeck over 8 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#6 Updated by Jon Gerdes over 8 years ago

I have not seen this error on yesterday's BETA4.

I have rebooted twice just to be sure so I think it is safe to close this bug.

Cheers
Jon

#7 Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF