Ticket #5822 (new Feature Requests)

Opened 1 year ago

Last modified 1 year ago

Add validation for pickupgroup field

Reported by: bcmike Assigned to: p_lindheimer
Priority: minor Milestone: 2.11
Component: Core - Users/Devices Version: 2.10-branch
Keywords: Cc: All
Confirmation: Unreviewed Distro:
Backend Engine: All Distro Ver:
Backend Ver: SVN Revision (if applicable):

Description

If you enter a value that is not 1-63 in the pickupgroup field of any SIP extension, and apply the changes, the SIP module will break along with various parts of the dial plan rendering the machine useless.

The value must then be manually removed from both the DB and conf files, as well asterisk then needs to be killed and restarted.

As far as I know this happens across all versions of both Asterisk and FreePBX

A validation for these field seems to be necessary.

Change History

06/15/12 14:59:22 changed by p_lindheimer

  • owner set to p_lindheimer.
  • priority changed from crash to minor.
  • type changed from Bugs to Feature Requests.
  • component changed from Web interface to Core - Users/Devices.
  • summary changed from No validation for pickupgroup field to Add validation for pickupgroup field.

well it's always valid to put in validation though the rules to validate this are trickier since the format is flexible. However, in testing this we could not repro the issue. It is also odd that such an issue would have any affect to the GUI. I could see Asterisk having a bug in not handling such a value, but to the GUI it's just a string in a field and there is no reason for it to be acting up.

I'll change this to a feature request to add validation though.