Project

General

Profile

Actions

Bug #1946

closed

/usr/local/www/services_captiveportal.php

Added by Cyrill B over 12 years ago. Updated almost 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Captive Portal
Target version:
Start date:
10/10/2011
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.1
Affected Architecture:
All

Description

/usr/local/www/services_captiveportal.php

lines 272 to 293
replace $config['captiveportal'] with $newcp

line 637
rename Primary to Tertiary

line 656
rename Secondary to Quaternary

line 369
Insert:

document.iform.radiusport3.disabled = radius_endis;
document.iform.radiusport4.disabled = radius_endis;

Actions #1

Updated by Cyrill B over 12 years ago

line 642
replace second with third

line 661
replace second with fourth

Actions #2

Updated by Anonymous over 12 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100
Actions #3

Updated by Anonymous over 12 years ago

Actions #4

Updated by Cyrill B over 12 years ago

Small error in commit (Revision 164a1525)

lines 272 to 293

replace $config['captiveportal'] with $newcp

e.g. $newcp['captiveportal']['radiusip'] should have been $newcp['radiusip']

Actions #5

Updated by Scott Ullrich over 12 years ago

Please open a new ticket for subsequent issues and please be sure the line #'s correspond with the prior changes.

Better yet - please fork our repo on github, fix the items and open a pull request.

Actions #6

Updated by Chris Buechler over 12 years ago

  • Status changed from Feedback to New
  • Target version set to 2.1

The description is wrong and the fix is wrong. Opening this because it appears committing a fix broke at least something. These aren't 4 RADIUS servers, the description probably needs clarified but it's NOT Tertiary and Quaternary. I couldn't revert the commits easily, someone will have to take a closer look.

Actions #7

Updated by Chris Buechler about 12 years ago

  • Affected Version changed from 2.1-IPv6 to 2.1
Actions #8

Updated by Chris Buechler almost 12 years ago

  • Status changed from New to Closed

#2322 open to fix what this broke and clear up the reason for the confusion.

Actions

Also available in: Atom PDF