Opened 9 years ago

Closed 9 years ago

#403 closed defect (fixed)

Name bug in item enumeration (function appends '/' to some keys)

Reported by: jelte Owned by: jelte
Priority: medium Milestone:
Component: configuration Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: 0.0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

This is one of the problems noted in the first comment of #384, when getting a list of items in a config data structure, the item names can have an erroneous '/' at the end (which will end up in the key value of maps/dicts)

Subtickets

Change History (4)

comment:1 Changed 9 years ago by jelte

  • Owner changed from jelte to UnAssigned
  • Status changed from new to reviewing

r3428

updated the tests that check for the name_list result, and removed the code that added the '/'

comment:2 Changed 9 years ago by vorner

  • Owner changed from UnAssigned to vorner

comment:3 Changed 9 years ago by vorner

  • Owner changed from vorner to jelte

The code looks OK so I think it can be merged.

However, how do we track TODOs for merges? Because this will influence stuff in the b10-recurse configuration (and #389 is already closed). I saw some map specification in the stats module as well. These should be addressed at merge time I think. Where should they be written, so they are not forgotten? Or do we keep this ticket open until all the relevant branches are merged together?

comment:4 Changed 9 years ago by jelte

  • Resolution set to fixed
  • Status changed from reviewing to closed

Merged, r3682. Closing ticket.

Note: See TracTickets for help on using tickets.