Bug #5443
closed
Package LADVD display wrong text in gui.
100%
Description
The second bit of text is correct, but the description appears wrong. This might impact multiple packages:
Enable LLDP
Show log entries in reverse order (newest entries on top)
Enable LLDP (Link Layer Discovery Protocol).
The "Show log entries" text is display for the next three items as well.
Files
Updated by 
Updated by Anonymous about 9 years ago
After installing the LADVD package, its webpage (Services->LADVD) displays incorrect text.
The page is generated by pkg_edit.php using ladvd.xml, so the issue could be in either of those.
Updated by Kill Bill about 9 years ago
I have hard time understanding what "incorrect text" is shown where, even with the screenshot.
Updated by Chris Buechler about 9 years ago
- File Selection_024.png Selection_024.png added
The screenshot Steve posted is from 2.2.x, where it's correct. Attached is from 2.3, where it's getting "Show log entries ..." from somewhere other than the package's XML.
Updated by Chris Buechler about 9 years ago
- Status changed from New to Confirmed
Updated by Chris Buechler about 9 years ago
a slew of instances of that "Show log entries" message in pkg_edit.php that looks like a wrong copy/paste.
Updated by Kill Bill about 9 years ago
Chris Buechler wrote:
a slew of instances of that "Show log entries" message in pkg_edit.php that looks like a wrong copy/paste.
Ah yeah, this thing is screwed from the very beginning (initial "conversion" - https://github.com/pfsense/pfsense/blob/23f6cdd72757350d8c25cf9536c7d7dafc6fca16/src/usr/local/www/pkg_edit.php)
Sigh.
Updated by Anonymous about 9 years ago
- Status changed from Confirmed to Feedback
- Assignee changed from Anonymous to Jeremy Porter
Error rectified in checkbox generation func.
Updated by Anonymous about 9 years ago
- % Done changed from 0 to 100
Applied in changeset pfsense:86c772239b30e11241d11e7804da765e99ccba0b.
Updated by Jeremy Porter about 9 years ago
- Status changed from Feedback to Closed
Looks good. Fixed.