Gui - limiter (fixed)

phil.davis

Firewall->Trafic Shaper, Limiter tab.
Add a limiter.
Now edit the limiter. Press the "Add new queue" button at the bottom.
404 error.

This is due to firewall_shaper_vinterface.php line 432

$url = 'href="firewall_shaper_vinterface.php?pipe=' . $pipe . '&queue=' . $queue->GetQname() . '&action=add';

It needs href=" removed:

$url = 'firewall_shaper_vinterface.php?pipe=' . $pipe . '&queue=' . $queue->GetQname() . '&action=add';

But then I get an error:

Fatal error: Call to undefined method dnqueue_class::GetBandwidth() in /etc/inc/shaper.inc on line 4191 Call Stack: 0.0002 230400 1\. {main}() /usr/local/www/firewall_shaper_vinterface.php:0 0.0995 1192416 2\. dnqueue_class->build_form() /usr/local/www/firewall_shaper_vinterface.php:202 PHP ERROR: Type: 1, File: /etc/inc/shaper.inc, Line: 4191, Message: Call to undefined method dnqueue_class::GetBandwidth()

If I add GetBandwidth and SetBandwidth functions into the relevant class, then the form is displayed to add new queue. But when I press Save nothing is saved. I am not confident at all about why GetBandwidth should be needed here.

So now I feel like I am chasing my tail - I need to understand what this is all supposed to do, rather than just patching up the next error that is displayed!

Anyway, that might give some help to get you started.

Steve_B

Thanks Phil. I'll dive in.

Steve_B

Recorded in https://redmine.pfsense.org/issues/5733

I believe this now works correctly.

shinzo

Applied the changes and is working, thank you

shinzo

Well I thought it was done, The mask is not saving when u add the new queue. ;)

Firewall->Trafic Shaper, Limiter tab.
Add a limiter.
Now edit the limiter. Press the "Add new queue" button at the bottom.
Set mask to either source or destination and hit save and it just goes back to none

Steve_B

Thanks for testing that. Let me take another look while it is still (mostly) in my head.

Steve_B

Just pushed a change to address that.

shinzo

So it is saving now. But If u do

New limiter, add all the information and save it.
It Creates limiter, then hit (add new queue) right away and it gives me an error of

Fatal error: Call to a member function addGlobal() on null in /usr/local/www/firewall_shaper.php on line 488 Call Stack: 0.0001 228344 1. {main}() /usr/local/www/firewall_shaper.php:0 PHP ERROR: Type: 1, File: /usr/local/www/firewall_shaper.php, Line: 488, Message: Call to a member function addGlobal() on null

But if u click on the new limiter that was created then click on ( add new queue) then it works fine

Also if u create a new limiter, then click on new limiter again, It adds it to the sub of the first limiter created and just keeps adding it.

Steve_B

I think that is because you did not "Apply" the changes (creating a new limiter) before adding the new queue, so the limiter didn't really exist.

I have suppressed the PHP error message and altered the error message presented by the GUI to include "Did you remember to apply any recent changes?"

shinzo

@Steve_B:

I think that is because you did not "Apply" the changes (creating a new limiter) before adding the new queue, so the limiter didn't really exist.

I have suppressed the PHP error message and altered the error message presented by the GUI to include "Did you remember to apply any recent changes?"

Yup forgot to hit apply changes after every step