Project

General

Profile

Actions

Bug #5443

closed

Package LADVD display wrong text in gui.

Added by Jeremy Porter over 8 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Jeremy Porter
Target version:
Start date:
11/13/2015
Due date:
% Done:

100%

Estimated time:

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

LADVD.jpg (72.5 KB) LADVD.jpg Anonymous, 11/13/2015 09:31 PM
Selection_024.png (38.6 KB) Selection_024.png Chris Buechler, 11/14/2015 12:32 AM
Actions #1

Updated by Jim Thompson over 8 years ago

screenshots would be nice.

Actions #2

Updated by Anonymous over 8 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.

Actions #3

Updated by Anonymous over 8 years ago

Actions #4

Updated by Kill Bill over 8 years ago

I have hard time understanding what "incorrect text" is shown where, even with the screenshot.

Actions #5

Updated by Chris Buechler over 8 years ago

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.

Actions #6

Updated by Chris Buechler over 8 years ago

  • Status changed from New to Confirmed
Actions #7

Updated by Chris Buechler over 8 years ago

a slew of instances of that "Show log entries" message in pkg_edit.php that looks like a wrong copy/paste.

Actions #8

Updated by Kill Bill over 8 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.

Actions #9

Updated by Anonymous over 8 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Anonymous to Jeremy Porter

Error rectified in checkbox generation func.

Actions #10

Updated by Anonymous over 8 years ago

  • % Done changed from 0 to 100
Actions #11

Updated by Jeremy Porter over 8 years ago

  • Status changed from Feedback to Closed

Looks good. Fixed.

Actions #12

Updated by Chris Buechler over 8 years ago

  • Status changed from Closed to Resolved

fixed, thanks

Actions

Also available in: Atom PDF