Bug #285: Dashboard widget settings not being saved - pfSense - pfSense bugtracker

Custom queries

2.4.5-p1 - Resolved/Closed
2.5.0 - Resolved/Closed
2.5.1 - Resolved/Closed
2.5.2 - Resolved/Closed
2.6.0 - Resolved/Closed
2.7.0 - Resolved/Closed
2.7.1 - All Open Issues
2.7.1 - Resolved/Closed
2.7.2 - Resolved/Closed
2.8.0 - All Open Bugs
2.8.0 - All Open Features
2.8.0 - All Open Issues
2.8.0 - All Open Regressions
2.8.0 - Feedback
2.8.0 - Needs Attention
2.8.0 - New/Confirmed
2.8.0 - Pull Requests
2.8.0 - Regressions affecting 2.7.0
2.8.0 - Resolved/Closed
21.02.2/2.5.1 - Resolved/Closed
21.05 Plus - All Closed Issues
21.05.1 Plus Target - All Closed Issues
21.05.1 Target - All Closed Issues
22.01 Plus - All Closed Issues
22.01 Target - All Closed Issues
22.05 Plus - All Closed Issues
22.05 Target - All Closed Issues
23.01 Plus - All Closed Issues
23.01 Target - All Closed Issues
23.05 Plus - All Closed Issues
23.05 Target - All Closed Issues
23.05.1 Plus - All Closed Issues
23.05.1 Target - All Closed Issues
23.09 Plus - All Closed Issues
23.09 Target - All Closed Issues
23.09.1 Plus - All Closed Issues
23.09.1 Target - All Closed Issues
24.03 Plus - All Closed Issues
24.03 Target - All Closed Issues
24.11 Plus - All Closed Issues
24.11 Plus - All Open Issues
24.11 Plus - Feedback Issues
24.11 Plus - Needs Attention/Work
24.11 Plus - New/Confirmed/In Progress Issues
24.11 Target - All Closed Issues
24.11 Target - All Open Issues
25.01 Plus - All Closed Issues
25.01 Plus - All Open Issues
25.01 Plus - Feedback Issues
25.01 Plus - Needs Attention/Work
25.01 Plus - New/Confirmed/In Progress Issues
25.01 Plus - Pull Request Review
25.01 Plus - Waiting on Merge
25.01 Target - All Closed Issues
25.01 Target - All Open Issues
All Open Issues assigned to Me
All Open Pull Requests
Any Target - All Open Regressions
Any Target - Feedback Issues
CE-Next - All Closed Issues (Move to specific target)
CE-Next - All Open Issues
CE-Next - Feedback (Likely needs target changed)
New Issues by Category - Future Target
New Issues by Category - No Target
New Issues by Category - No Target+Future
No Target - All Open Issues (Base Only)
No Target - New Issues (Base Only)
No Target - New Issues (Base and Packages)
Release Notes - Plus Target Version (DO NOT EDIT)
Release Notes - Target Version (DO NOT EDIT)

Actions

Copy link

Bug #285

closed

Dashboard widget settings not being saved

Added by Abdsalem Chaoui almost 15 years ago. Updated over 14 years ago.

Status:

Resolved

Priority:

Low

Assignee:

Category:

Dashboard

Target version:

2.0

Start date:

01/05/2010

Due date:

% Done:

100%

Estimated time:

Plus Target Version:

Release Notes:

Affected Version:

2.0

Affected Architecture:

Description

2.0-BETA1 built on Tue Jan 5 10:29:05 EST 2010

1) Firewall logs widget - changing "Number of lines to display" to a selection on the list box throws out an error
Warning: Cannot modify header information - headers already sent by (output started at /usr/local/www/niftycssCode.css:16) in /usr/local/www/widgets/widgets/log.widget.php on line 44

However the setting is saved.

2) Traffic Graph widget - changing "Refresh Interval" to a selection on the list box does not save the desired update interval.

Appologies if I was meant to list these as 2 seperate bugs.

Thanks

Slam

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Fred Taylor-Young almost 15 years ago

Regarding #1:¶

I've played around with the PHP code for the widget and watched what it does in Firebug, and have noticed that despite its action attribute set to make the form submit to itself, the form is actually submitted to index.php.

<form action="/widgets/widgets/log.widget.php" method="post" name="iforma">

The widget still tries to run its header() code though, which is causing the error message you see:

Header("Location: /");

All this behavior is caused by this line, which appears just after the <body> tag, and only ends just before the closing </body> tag:

<body link="#0000CC" vlink="#0000CC" alink="#0000CC">
<form action="index.php" method="post">
[...]
</form>
</body>

By coincidence, while playing around with the picture widget yesterday I noticed that it has a closing </form> tag before its own opening <form>, presumably to fix this problem.

I've seen three ways of fixing this issue (although it's entirely possible that there may be other things that I've overlooked):

End the first <form> tag much earlier in the page - I don't know if it needs to encompass most of the page or not, so this may not be an option
Like the picture widget, add a </form> tag
Remove the header() line - however, this will likely break the widget if the overall <form> tag is later fixed/ended before the widgets

Actions

Copy link