Bug #1946
closed
/usr/local/www/services_captiveportal.php
100%
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;
Updated by Cyrill B about 13 years ago
line 642
replace second with third
line 661
replace second with fourth
Updated by Anonymous about 13 years ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset d568e38d0275efebbe0c1969e26b31bdf125570e.
Updated by Anonymous about 13 years ago
Applied in changeset 164a152577ccce56f896ea5ef020fcfbf60fe896.
Updated by Cyrill B about 13 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']
Updated by Scott Ullrich about 13 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.
Updated by Chris Buechler about 13 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.
Updated by Chris Buechler almost 13 years ago
- Affected Version changed from 2.1-IPv6 to 2.1
Updated by Chris Buechler over 12 years ago
- Status changed from New to Closed
#2322 open to fix what this broke and clear up the reason for the confusion.