Bug #8803
closed
PHP errors thrown in traffic shaper wizard multi
100%
Description
When attempting to save changes in order to proceed to the next step in the wizard following prioritizing VoIP, the following error(s) are thrown if any options in the step are enabled.
PHP ERROR: Type: 1, File: /usr/local/www/wizard.php(180) : eval()'d code, Line: 1, Message: Uncaught Error: Cannot create references to/from string offsets in /usr/local/www/wizard.php(180) : eval()'d code:1
Stack trace:
#0 /usr/local/www/wizard.php(180): eval()
#1 /usr/local/www/wizard.php(108): update_config_field('ezshaper->step5...', 'on', '', '', 'checkbox')
#2 {main}
thrown @ 2018-08-18 08:37:40
Updated by Matthew Deets over 6 years ago
Tested both the multi network and dedicated traffic shaper wizards and it appears that this issue is still occurring as of snapshot: 2.4.4.a.20180818.2240.
Updated by Jim Pingle over 6 years ago
- Project changed from pfSense Packages to pfSense
- Category set to Traffic Shaper (ALTQ)
- Target version set to 2.4.4
- Affected Architecture All added
- Affected Architecture deleted (
)
Updated by Anonymous over 6 years ago
- Status changed from 13 to Feedback
I am unable to reproduce this.
New VM running "out of the box" pfSense 2.4.4 snapshot from today
Number of wan conenctions: 1 => Next
Step 1 of 8 => accept defaults => Next
Step 2 of 8 Enable "Prioitize VOIP, set provider = generic, Upload and download rates set to 12 Mbps => Next
No errors observed.
Updated by Matthew Deets over 6 years ago
Steve, Were you able to get through the entire shaper wizard with no errors when selecting to prioritize other protocols after the voip step? On my system, running the latest snapshot with an SG-2440 , I get this error after the prioritze voip step - wizard allows me to prioritize voip traffic with no issues; if I select the checkbox to enable any higher or lower prioritized protocols in any of the later steps such as p2p or other protocols, it throws the PHP errors that I mentioned. I am able to get through the shaper wizard if I leave all of the steps after prioritize voip unchecked.
Updated by Matthew Deets over 6 years ago
Just to be clear, the error(s) that I encountered which I thought perhaps could be a bug, occurred if any checkbox is enabled in any step after the prioritize VoIP step. If I leave I’ll check boxes and checked, the wizard completes successfully with no errors
Updated by Anonymous over 6 years ago
- Status changed from Feedback to 13
Thanks for the extra detail. I was able to reproduce this after your clarification.
Updated by Matthew Deets over 6 years ago
Sorry, forgot to mention the step… If I check enable for step 3 The “penalty box” and press “next“, that’s where the error first occurs.
Updated by Anonymous over 6 years ago
The offending command is: $thisvar = &$config['ezshaper']['step4']['enable'];
Updated by Anonymous over 6 years ago
- Status changed from 13 to Feedback
- % Done changed from 0 to 100
Applied in changeset a34bab9f4b93091e147bcae18b79e6349c60496c.
Updated by Anonymous over 6 years ago
- % Done changed from 100 to 50
Please confirm that this change eliminates the PHP error messages. The Wizard should now test to see if a required $config element is present, and if not, create it..
Updated by Matthew Deets over 6 years ago
Verified with the latest snapshot that all steps are able to be completed in the traffic shaper wizard with no PHP errors. however, the traffic shaper wizard does not seem to be functioning - qothershigh, qotherslow, gaming, p2p, etc queues and floating rules were not created after the finish button was selected.
Updated by Anonymous over 6 years ago
I have retested and don't see any problems. Since I pushed the update in two commits an hour apart, it is possible you caught it in between zapping the errors and fixing the config save. Please retest with snapshot from today. I have asked our QC team to do the same.
Updated by Matthew Deets over 6 years ago
Updated to the latest snap shot this morning and retested. The traffic shaper wizard now works as expected with no errors. :)