Project

General

Profile

Actions

Bug #5694

closed

Dynamic gateways' gateway IP shouldn't be editable

Added by Chris Buechler almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Category:
Web Interface
Target version:
Start date:
12/24/2015
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.3
Affected Architecture:

Description

Dynamic gateways have "dynamic" filled in for the gateway IP upon edit, which is correct. But they allow changing that "dynamic" text to any IP. Setting a specific gateway IP on a dynamic gateway entry breaks all kinds of things. That field was grayed out previously for dynamic gateways so it couldn't be broken. Needs to come back.

Actions #1

Updated by Steve Beaver almost 6 years ago

Does this pertain to the system_gateways_edit.php page?

Actions #2

Updated by Steve Beaver almost 6 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Steve Beaver to Chris Buechler
Actions #3

Updated by Chris Buechler almost 6 years ago

  • Status changed from Feedback to Confirmed
  • Assignee changed from Chris Buechler to Steve Beaver

Correct, system_gateways_edit.php. When it fills in "dynamic" in the gateway field, that field should be grayed out so it can't be changed.

Actions #4

Updated by Steve Beaver almost 6 years ago

OK. I was unable to locate that functionality in 2.2.x, but it is easy enough to add now.

Actions #5

Updated by Steve Beaver almost 6 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Steve Beaver to Chris Buechler

Element now set to readonly if $pconfig['dynamic'] is true.

Actions #6

Updated by Steve Beaver almost 6 years ago

  • % Done changed from 0 to 100
Actions #7

Updated by Chris Buechler almost 6 years ago

  • Status changed from Feedback to Resolved

Indeed this wasn't prevented in 2.2.x, that js must have gotten messed up at some point, swear that used to behave the same as 2.3 does now. This is correct now, thanks!

Actions

Also available in: Atom PDF